[f2fs-dev] [PATCH] f2fs: enlarge block plug coverage

2018-04-04 Thread Chao Yu
This patch enlarges block plug coverage in __issue_discard_cmd, in order to collect more pending bios before issuing them, to avoid being disturbed by previous discard I/O in IO aware discard mode. Signed-off-by: Chao Yu --- fs/f2fs/segment.c | 7 +-- 1 file changed, 5 insertions(+), 2 delet

[f2fs-dev] [PATCH v2] f2fs: enlarge block plug coverage

2018-04-04 Thread Chao Yu
This patch enlarges block plug coverage in __issue_discard_cmd, in order to collect more pending bios before issuing them, to avoid being disturbed by previous discard I/O in IO aware discard mode. Signed-off-by: Chao Yu --- v2: - fix to move plug to correct place. fs/f2fs/segment.c | 6 --

[f2fs-dev] [PATCH] f2fs: remove redundant block plug

2018-04-04 Thread Chao Yu
For buffered IO, we don't need to use block plug to cache bio, for direct IO, generic f2fs_direct_IO has already added block plug, so let's remove redundant one in .write_iter. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/f2fs

[f2fs-dev] [PATCH] f2fs: issue all big range discards in umount process

2018-04-04 Thread Yunlei He
This patch modify max_requests to UINT_MAX, to issue all big range discards in umount. Signed-off-by: Yunlei He --- fs/f2fs/segment.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 5854cc4..a4b8e3e2 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/seg

Re: [f2fs-dev] [PATCH] f2fs: issue all big range discards in umount process

2018-04-04 Thread Chao Yu
On 2018/4/4 17:29, Yunlei He wrote: > This patch modify max_requests to UINT_MAX, to issue > all big range discards in umount. > > Signed-off-by: Yunlei He Reviewed-by: Chao Yu Thanks, -- Check out the vibrant tech c

Re: [f2fs-dev] [PATCH v10 00/62] Convert page cache to XArray

2018-04-04 Thread Mike Kravetz
On 03/29/2018 08:41 PM, Matthew Wilcox wrote: > From: Matthew Wilcox > > I'd like to thank Andrew for taking the first eight XArray patches > into -next. He's understandably nervous about taking the rest of the > patches into -next given how few of the remaining patches have review > tags on the

Re: [f2fs-dev] [PATCH] f2fs: fix to show encrypt flag in FS_IOC_GETFLAGS

2018-04-04 Thread Sasha Levin via Linux-f2fs-devel
Hi Chao Yu. [This is an automated email] This commit has been processed by the -stable helper bot and determined to be a high probability candidate for -stable trees. (score: 11.4088) The bot has tested the following trees: v4.15.15, v4.14.32, v4.9.92, v4.4.126, v4.15.15: Build OK! v4.14.32: B

[f2fs-dev] [PATCH] f2fs: no need to take the address of the array of sb->s_uuid

2018-04-04 Thread Gao Xiang
Keep in line with the common case since it is some weird to take the address of an array again. Signed-off-by: Gao Xiang --- fs/f2fs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 42d564c5ccd0..f864ab702fa2 100644 --- a/fs/f2f

[f2fs-dev] [GIT PULL] f2fs update for 4.17-rc1

2018-04-04 Thread Jaegeuk Kim
Hi Linus, Could you please consider this pull request? Thanks, The following changes since commit 3664ce2d930983966d2aac0e167f1332988c4e25: Merge tag 'powerpc-4.16-4' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux (2018-02-24 16:05:50 -0800) are available in the Git reposit

Re: [f2fs-dev] [PATCH] f2fs: fix to show encrypt flag in FS_IOC_GETFLAGS

2018-04-04 Thread Jaegeuk Kim
On 04/03, Chao Yu wrote: > On 2018/4/3 4:21, Jaegeuk Kim wrote: > > On 04/02, Chao Yu wrote: > >> This patch fixes to show encrypt flag in FS_IOC_GETFLAGS like ext4 does. > > > > Actually, we have to show internal flags owned by f2fs, not generic ones. > > We may need to define all of them separat

Re: [f2fs-dev] [PATCH] mkfs.f2fs: fix incorrect cold data location

2018-04-04 Thread Jaegeuk Kim
On 04/04, Chao Yu wrote: > If last_zone((total_zones >> 2)) is equal or less than > next_zone(CURSEG_COLD_NODE), cold data area will be located in the > same position with hot data, fixes it. verify_cur_segs() will rearrage this? > > Signed-off-by: Chao Yu > --- > mkfs/f2fs_format.c | 2 +- >

Re: [f2fs-dev] [PATCH] f2fs: enlarge block plug coverage

2018-04-04 Thread Jaegeuk Kim
On 04/04, Chao Yu wrote: > This patch enlarges block plug coverage in __issue_discard_cmd, in > order to collect more pending bios before issuing them, to avoid > being disturbed by previous discard I/O in IO aware discard mode. Hmm, then we need to wait for huge discard IO for over 10 secs, which

Re: [f2fs-dev] [PATCH v10 00/62] Convert page cache to XArray

2018-04-04 Thread Matthew Wilcox
On Wed, Apr 04, 2018 at 09:35:46AM -0700, Mike Kravetz wrote: > Running with this XArray series on top of next-20180329 consistently 'hangs' > on shutdown looping (?forever?) in tag_pages_for_writeback/xas_for_each_tag. > All I have to do is make sure there is some activity on the ext4 fs before >

Re: [f2fs-dev] [PATCH] f2fs: no need to take the address of the array of sb->s_uuid

2018-04-04 Thread Jaegeuk Kim
Hi Gao, Could you please check your email settings? It's broken. Thanks, On 04/05, Gao Xiang wrote: > Keep in line with the common case since it is some weird > to take the address of an array again. > > Signed-off-by: Gao Xiang > --- > fs/f2fs/super.c | 2 +- > 1 file changed, 1 insertion(+)

[f2fs-dev] [RESEND PATCH] f2fs: no need to take the address of the array of sb->s_uuid

2018-04-04 Thread Gao Xiang
Keep in line with the common case since it is some weird to take the address of an array again. Signed-off-by: Gao Xiang --- fix auto-wrapping of email client fs/f2fs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c index 9587ca0..4d4

Re: [f2fs-dev] [PATCH] f2fs: no need to take the address of the array of sb->s_uuid

2018-04-04 Thread Gao Xiang
Hi Jaegeuk, On 2018/4/5 11:54, Jaegeuk Kim wrote: > Hi Gao, > > Could you please check your email settings? > It's broken. > > Thanks,Sorry to bother for the pervious email.. My business email client was just in a mess. :( Thanks, > > On 04/05, Gao Xiang wrote: >> Keep in line with the common