Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-27 Thread Darrick J. Wong
On Tue, Feb 27, 2018 at 09:46:54AM -0700, Dave Jiang wrote: > > > On 02/20/2018 04:23 PM, Darrick J. Wong wrote: > > On Wed, Feb 21, 2018 at 10:15:24AM +1100, Dave Chinner wrote: > >> On Tue, Feb 20, 2018 at 03:01:09PM -0800, Darrick J. Wong wrote: > >>> On Sun, Feb 18, 2018 at 11:23:17AM +1100,

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-27 Thread Dave Jiang
On 02/20/2018 04:23 PM, Darrick J. Wong wrote: > On Wed, Feb 21, 2018 at 10:15:24AM +1100, Dave Chinner wrote: >> On Tue, Feb 20, 2018 at 03:01:09PM -0800, Darrick J. Wong wrote: >>> On Sun, Feb 18, 2018 at 11:23:17AM +1100, Dave Chinner wrote: On Fri, Feb 16, 2018 at 09:22:47AM -0800,

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-20 Thread Darrick J. Wong
On Wed, Feb 21, 2018 at 10:15:24AM +1100, Dave Chinner wrote: > On Tue, Feb 20, 2018 at 03:01:09PM -0800, Darrick J. Wong wrote: > > On Sun, Feb 18, 2018 at 11:23:17AM +1100, Dave Chinner wrote: > > > On Fri, Feb 16, 2018 at 09:22:47AM -0800, Darrick J. Wong wrote: > > > > On Fri, Feb 16, 2018 at

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-20 Thread Dave Chinner
On Tue, Feb 20, 2018 at 03:01:09PM -0800, Darrick J. Wong wrote: > On Sun, Feb 18, 2018 at 11:23:17AM +1100, Dave Chinner wrote: > > On Fri, Feb 16, 2018 at 09:22:47AM -0800, Darrick J. Wong wrote: > > > On Fri, Feb 16, 2018 at 10:04:26AM -0700, Dave Jiang wrote: > > > > In a situation where the

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-20 Thread Darrick J. Wong
On Sun, Feb 18, 2018 at 11:23:17AM +1100, Dave Chinner wrote: > On Fri, Feb 16, 2018 at 09:22:47AM -0800, Darrick J. Wong wrote: > > On Fri, Feb 16, 2018 at 10:04:26AM -0700, Dave Jiang wrote: > > > In a situation where the rt_dev is DAX and data_dev is not DAX, if the > > > user > > > requests

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-17 Thread Dave Chinner
On Fri, Feb 16, 2018 at 09:22:47AM -0800, Darrick J. Wong wrote: > On Fri, Feb 16, 2018 at 10:04:26AM -0700, Dave Jiang wrote: > > In a situation where the rt_dev is DAX and data_dev is not DAX, if the user > > requests to remove the realtime flag via ioctl we can no longer support DAX > > for

Re: [PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-16 Thread Darrick J. Wong
On Fri, Feb 16, 2018 at 10:04:26AM -0700, Dave Jiang wrote: > In a situation where the rt_dev is DAX and data_dev is not DAX, if the user > requests to remove the realtime flag via ioctl we can no longer support DAX > for that file. Dynamic changing of S_DAX on the inode is not supported due > to

[PATCH v6 3/3] xfs: reject removal of realtime flag when datadev doesn't support DAX

2018-02-16 Thread Dave Jiang
In a situation where the rt_dev is DAX and data_dev is not DAX, if the user requests to remove the realtime flag via ioctl we can no longer support DAX for that file. Dynamic changing of S_DAX on the inode is not supported due to various complications in the existing implementation. Therefore