Re: [PATCH v2 1/2] ext4: Close race between direct IO and ext4_break_layouts()

2018-09-11 Thread Jan Kara
On Mon 10-09-18 11:23:56, Eric Sandeen wrote: > On 8/8/18 12:25 PM, Dave Jiang wrote: > > From: Ross Zwisler > > > > If the refcount of a page is lowered between the time that it is returned > > by dax_busy_page() and when the refcount is again checked in > > ext4_break_layouts() =>

Re: [PATCH v2 1/2] ext4: Close race between direct IO and ext4_break_layouts()

2018-09-10 Thread Eric Sandeen
On 8/8/18 12:25 PM, Dave Jiang wrote: > From: Ross Zwisler > > If the refcount of a page is lowered between the time that it is returned > by dax_busy_page() and when the refcount is again checked in > ext4_break_layouts() => ___wait_var_event(), the waiting function > ext4_wait_dax_page() will

[PATCH v2 1/2] ext4: Close race between direct IO and ext4_break_layouts()

2018-08-08 Thread Dave Jiang
From: Ross Zwisler If the refcount of a page is lowered between the time that it is returned by dax_busy_page() and when the refcount is again checked in ext4_break_layouts() => ___wait_var_event(), the waiting function ext4_wait_dax_page() will never be called. This means that