Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-11 Thread Amir Goldstein
On Thu, Apr 11, 2019 at 1:16 PM Jan Kara wrote: > > On Wed 10-04-19 13:42:23, Amir Goldstein wrote: > > On Wed, Apr 10, 2019 at 12:10 PM Jan Kara wrote: > > > > > > On Tue 09-04-19 21:01:54, Amir Goldstein wrote: > > > > On Tue, Apr 9, 2019 at 6:43 PM Jan Kara wrote: > > > > > On Tue 09-04-19

Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-11 Thread Jan Kara
On Wed 10-04-19 13:42:23, Amir Goldstein wrote: > On Wed, Apr 10, 2019 at 12:10 PM Jan Kara wrote: > > > > On Tue 09-04-19 21:01:54, Amir Goldstein wrote: > > > On Tue, Apr 9, 2019 at 6:43 PM Jan Kara wrote: > > > > On Tue 09-04-19 14:49:22, Amir Goldstein wrote: > > > > > Commit 23d0127096cb

Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-10 Thread Amir Goldstein
On Wed, Apr 10, 2019 at 12:10 PM Jan Kara wrote: > > On Tue 09-04-19 21:01:54, Amir Goldstein wrote: > > On Tue, Apr 9, 2019 at 6:43 PM Jan Kara wrote: > > > On Tue 09-04-19 14:49:22, Amir Goldstein wrote: > > > > Commit 23d0127096cb ("fs/sync.c: make sync_file_range(2) use > > > > WB_SYNC_NONE

Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-10 Thread Jan Kara
On Tue 09-04-19 21:01:54, Amir Goldstein wrote: > On Tue, Apr 9, 2019 at 6:43 PM Jan Kara wrote: > > On Tue 09-04-19 14:49:22, Amir Goldstein wrote: > > > Commit 23d0127096cb ("fs/sync.c: make sync_file_range(2) use WB_SYNC_NONE > > > writeback") claims that sync_file_range(2) syscall was

Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-09 Thread Amir Goldstein
On Tue, Apr 9, 2019 at 6:43 PM Jan Kara wrote: > > On Tue 09-04-19 14:49:22, Amir Goldstein wrote: > > Commit 23d0127096cb ("fs/sync.c: make sync_file_range(2) use WB_SYNC_NONE > > writeback") claims that sync_file_range(2) syscall was "created for > > userspace to be able to issue background

Re: [PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-09 Thread Jan Kara
On Tue 09-04-19 14:49:22, Amir Goldstein wrote: > Commit 23d0127096cb ("fs/sync.c: make sync_file_range(2) use WB_SYNC_NONE > writeback") claims that sync_file_range(2) syscall was "created for > userspace to be able to issue background writeout and so waiting for > in-flight IO is undesirable

[PATCH] fs/sync.c: sync_file_range(2) may use WB_SYNC_ALL writeback

2019-04-09 Thread Amir Goldstein
Commit 23d0127096cb ("fs/sync.c: make sync_file_range(2) use WB_SYNC_NONE writeback") claims that sync_file_range(2) syscall was "created for userspace to be able to issue background writeout and so waiting for in-flight IO is undesirable there" and changes the writeback (back) to WB_SYNC_NONE.