Skip to content

Commit a4d4636

Browse files
committed
ceph: disable use of dcache for readdir etc.
Ceph attempts to use the dcache to satisfy negative lookups and readdir when the entire directory contents are in cache. Disable this behavior until lingering bugs in this code are shaken out; we'll re-enable these hooks once things are fully stable. Signed-off-by: Sage Weil <[email protected]>
1 parent 9d5a09e commit a4d4636

File tree

1 file changed

+3
-26
lines changed

1 file changed

+3
-26
lines changed

fs/ceph/dir.c

Lines changed: 3 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1094,42 +1094,19 @@ static int ceph_snapdir_d_revalidate(struct dentry *dentry,
10941094
/*
10951095
* Set/clear/test dir complete flag on the dir's dentry.
10961096
*/
1097-
static struct dentry * __d_find_any_alias(struct inode *inode)
1098-
{
1099-
struct dentry *alias;
1100-
1101-
if (list_empty(&inode->i_dentry))
1102-
return NULL;
1103-
alias = list_first_entry(&inode->i_dentry, struct dentry, d_alias);
1104-
return alias;
1105-
}
1106-
11071097
void ceph_dir_set_complete(struct inode *inode)
11081098
{
1109-
struct dentry *dentry = __d_find_any_alias(inode);
1110-
1111-
if (dentry && ceph_dentry(dentry)) {
1112-
dout(" marking %p (%p) complete\n", inode, dentry);
1113-
set_bit(CEPH_D_COMPLETE, &ceph_dentry(dentry)->flags);
1114-
}
1099+
/* not yet implemented */
11151100
}
11161101

11171102
void ceph_dir_clear_complete(struct inode *inode)
11181103
{
1119-
struct dentry *dentry = __d_find_any_alias(inode);
1120-
1121-
if (dentry && ceph_dentry(dentry)) {
1122-
dout(" marking %p (%p) NOT complete\n", inode, dentry);
1123-
clear_bit(CEPH_D_COMPLETE, &ceph_dentry(dentry)->flags);
1124-
}
1104+
/* not yet implemented */
11251105
}
11261106

11271107
bool ceph_dir_test_complete(struct inode *inode)
11281108
{
1129-
struct dentry *dentry = __d_find_any_alias(inode);
1130-
1131-
if (dentry && ceph_dentry(dentry))
1132-
return test_bit(CEPH_D_COMPLETE, &ceph_dentry(dentry)->flags);
1109+
/* not yet implemented */
11331110
return false;
11341111
}
11351112

0 commit comments

Comments
 (0)