Only one bit is read in check_valid_map, holding a lock to do that doesn't help anything except decreasing performance.
Signed-off-by: Huang, Ying <ying.hu...@intel.com> --- v2: Fixed a build warning. --- fs/f2fs/gc.c | 3 --- 1 file changed, 3 deletions(-) --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -378,14 +378,11 @@ static void put_gc_inode(struct list_hea static int check_valid_map(struct f2fs_sb_info *sbi, unsigned int segno, int offset) { - struct sit_info *sit_i = SIT_I(sbi); struct seg_entry *sentry; int ret; - mutex_lock(&sit_i->sentry_lock); sentry = get_seg_entry(sbi, segno); ret = f2fs_test_bit(offset, sentry->cur_valid_map); - mutex_unlock(&sit_i->sentry_lock); return ret; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/