diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2017-02-27 11:57:11 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-12 06:44:19 +0100 |
| commit | e8917cd860db436fdb5abcad6ae579f420c6c9df (patch) | |
| tree | baae9c9fdf59802f3ee2f50593a711a46d3c0109 /fs | |
| parent | c85fc3f1d59f48d6f885b965ad61110a6af8e184 (diff) | |
| download | linux-stable-e8917cd860db436fdb5abcad6ae579f420c6c9df.tar.gz linux-stable-e8917cd860db436fdb5abcad6ae579f420c6c9df.tar.bz2 linux-stable-e8917cd860db436fdb5abcad6ae579f420c6c9df.zip | |
f2fs: avoid to issue redundant discard commands
commit 8b107f5b97772c7c0c218302e9a4d15b4edf50b4 upstream.
If segs_per_sec is over 1 like under SMR, previously f2fs issues discard
commands redundantly on the same section, since we didn't move end position
for the previous discard command.
E.g.,
start end
| |
prefree_bitmap = [01111100111100]
And, after issue discard for this section,
end start
| |
prefree_bitmap = [01111100111100]
Select this section again by searching from (end + 1),
start end
| |
prefree_bitmap = [01111100111100]
Fixes: 36abef4e796d38 ("f2fs: introduce mode=lfs mount option")
Cc: Damien Le Moal <damien.lemoal@wdc.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/f2fs/segment.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 0d8802453758..983bf0caf2df 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -935,6 +935,8 @@ next: start = start_segno + sbi->segs_per_sec; if (start < end) goto next; + else + end = start - 1; } mutex_unlock(&dirty_i->seglist_lock); |