Re: [PATCH 2/2] [PATCH] xfs: Close race between direct IO and xfs_break_layouts()

2018-08-08 Thread Dave Jiang
On 08/08/2018 01:53 AM, Jan Kara wrote: > On Tue 07-08-18 15:11:43, Dave Jiang wrote: >> This patch is the duplicate of ross's fix for ext4 for xfs. >> >> 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 >>

Re: [PATCH 2/2] [PATCH] xfs: Close race between direct IO and xfs_break_layouts()

2018-08-08 Thread Jan Kara
On Tue 07-08-18 15:11:43, Dave Jiang wrote: > This patch is the duplicate of ross's fix for ext4 for xfs. > > 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 > xfs_break_layouts() => ___wait_var_event(), the

[PATCH 2/2] [PATCH] xfs: Close race between direct IO and xfs_break_layouts()

2018-08-07 Thread Dave Jiang
This patch is the duplicate of ross's fix for ext4 for xfs. 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 xfs_break_layouts() => ___wait_var_event(), the waiting function xfs_wait_dax_page() will never be