diff options
author | Joel Becker <joel.becker@oracle.com> | 2008-10-09 20:20:31 -0400 |
---|---|---|
committer | Mark Fasheh <mfasheh@suse.com> | 2008-10-14 14:51:57 -0400 |
commit | 0fcaa56a2a020dd6f90c202b7084e6f4cbedb6c2 (patch) | |
tree | 057204016523ce0274d4a23ec02944075f084e8c /fs/ocfs2/dir.c | |
parent | 31d33073ca38603dea705dae45e094a64ca062d6 (diff) |
ocfs2: Simplify ocfs2_read_block()
More than 30 callers of ocfs2_read_block() pass exactly OCFS2_BH_CACHED.
Only six pass a different flag set. Rather than have every caller care,
let's make ocfs2_read_block() take no flags and always do a cached read.
The remaining six places can call ocfs2_read_blocks() directly.
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
Diffstat (limited to 'fs/ocfs2/dir.c')
-rw-r--r-- | fs/ocfs2/dir.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/fs/ocfs2/dir.c b/fs/ocfs2/dir.c index 828437ca91ba..459e6b8467dc 100644 --- a/fs/ocfs2/dir.c +++ b/fs/ocfs2/dir.c | |||
@@ -188,8 +188,7 @@ static struct buffer_head *ocfs2_find_entry_id(const char *name, | |||
188 | struct ocfs2_dinode *di; | 188 | struct ocfs2_dinode *di; |
189 | struct ocfs2_inline_data *data; | 189 | struct ocfs2_inline_data *data; |
190 | 190 | ||
191 | ret = ocfs2_read_block(dir, OCFS2_I(dir)->ip_blkno, &di_bh, | 191 | ret = ocfs2_read_block(dir, OCFS2_I(dir)->ip_blkno, &di_bh); |
192 | OCFS2_BH_CACHED); | ||
193 | if (ret) { | 192 | if (ret) { |
194 | mlog_errno(ret); | 193 | mlog_errno(ret); |
195 | goto out; | 194 | goto out; |
@@ -417,8 +416,7 @@ static inline int ocfs2_delete_entry_id(handle_t *handle, | |||
417 | struct ocfs2_dinode *di; | 416 | struct ocfs2_dinode *di; |
418 | struct ocfs2_inline_data *data; | 417 | struct ocfs2_inline_data *data; |
419 | 418 | ||
420 | ret = ocfs2_read_block(dir, OCFS2_I(dir)->ip_blkno, | 419 | ret = ocfs2_read_block(dir, OCFS2_I(dir)->ip_blkno, &di_bh); |
421 | &di_bh, OCFS2_BH_CACHED); | ||
422 | if (ret) { | 420 | if (ret) { |
423 | mlog_errno(ret); | 421 | mlog_errno(ret); |
424 | goto out; | 422 | goto out; |
@@ -596,8 +594,7 @@ static int ocfs2_dir_foreach_blk_id(struct inode *inode, | |||
596 | struct ocfs2_inline_data *data; | 594 | struct ocfs2_inline_data *data; |
597 | struct ocfs2_dir_entry *de; | 595 | struct ocfs2_dir_entry *de; |
598 | 596 | ||
599 | ret = ocfs2_read_block(inode, OCFS2_I(inode)->ip_blkno, | 597 | ret = ocfs2_read_block(inode, OCFS2_I(inode)->ip_blkno, &di_bh); |
600 | &di_bh, OCFS2_BH_CACHED); | ||
601 | if (ret) { | 598 | if (ret) { |
602 | mlog(ML_ERROR, "Unable to read inode block for dir %llu\n", | 599 | mlog(ML_ERROR, "Unable to read inode block for dir %llu\n", |
603 | (unsigned long long)OCFS2_I(inode)->ip_blkno); | 600 | (unsigned long long)OCFS2_I(inode)->ip_blkno); |