Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-19 Thread Yang , Xiao/杨 晓
Hi Darrick, Brian and Christoph Ping. I hope to get your feedback. 1) I have confirmed that the following patch set did not change the test result of generic/470 with thin-volume. Besides, I didn't see any failure when running generic/470 based on normal PMEM device instaed of thin-volume.

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-15 Thread Yang , Xiao/杨 晓
On 2022/9/15 18:14, Yang, Xiao/杨 晓 wrote: On 2022/9/15 0:28, Darrick J. Wong wrote: On Wed, Sep 14, 2022 at 08:34:26AM -0400, Brian Foster wrote: On Wed, Sep 14, 2022 at 05:38:02PM +0800, Yang, Xiao/杨 晓 wrote: On 2022/9/14 14:44, Yang, Xiao/杨 晓 wrote: On 2022/9/9 21:01, Brian Foster wrote:

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-15 Thread Yang , Xiao/杨 晓
On 2022/9/15 0:28, Darrick J. Wong wrote: On Wed, Sep 14, 2022 at 08:34:26AM -0400, Brian Foster wrote: On Wed, Sep 14, 2022 at 05:38:02PM +0800, Yang, Xiao/杨 晓 wrote: On 2022/9/14 14:44, Yang, Xiao/杨 晓 wrote: On 2022/9/9 21:01, Brian Foster wrote: Yes.. I don't recall all the internals of

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-14 Thread Darrick J. Wong
On Wed, Sep 14, 2022 at 08:34:26AM -0400, Brian Foster wrote: > On Wed, Sep 14, 2022 at 05:38:02PM +0800, Yang, Xiao/杨 晓 wrote: > > On 2022/9/14 14:44, Yang, Xiao/杨 晓 wrote: > > > On 2022/9/9 21:01, Brian Foster wrote: > > > > Yes.. I don't recall all the internals of the tools and test, but IIRC

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-14 Thread Brian Foster
On Wed, Sep 14, 2022 at 05:38:02PM +0800, Yang, Xiao/杨 晓 wrote: > On 2022/9/14 14:44, Yang, Xiao/杨 晓 wrote: > > On 2022/9/9 21:01, Brian Foster wrote: > > > Yes.. I don't recall all the internals of the tools and test, but IIRC > > > it relied on discard to perform zeroing between checkpoints or

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-14 Thread Yang , Xiao/杨 晓
On 2022/9/14 14:44, Yang, Xiao/杨 晓 wrote: On 2022/9/9 21:01, Brian Foster wrote: Yes.. I don't recall all the internals of the tools and test, but IIRC it relied on discard to perform zeroing between checkpoints or some such and avoid spurious failures. The purpose of running on dm-thin was

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-14 Thread Yang , Xiao/杨 晓
On 2022/9/9 21:01, Brian Foster wrote: Yes.. I don't recall all the internals of the tools and test, but IIRC it relied on discard to perform zeroing between checkpoints or some such and avoid spurious failures. The purpose of running on dm-thin was merely to provide reliable discard zeroing

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-09 Thread Brian Foster
On Thu, Sep 08, 2022 at 09:46:04PM +0800, Shiyang Ruan wrote: > > > 在 2022/8/4 8:51, Darrick J. Wong 写道: > > On Wed, Aug 03, 2022 at 06:47:24AM +, ruansy.f...@fujitsu.com wrote: > > ... > > > > > > > > > > > > > > > BTW, since these patches (dax + THIS + pmem-unbind) > > > > > > > are >

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-09-08 Thread Shiyang Ruan
在 2022/8/4 8:51, Darrick J. Wong 写道: On Wed, Aug 03, 2022 at 06:47:24AM +, ruansy.f...@fujitsu.com wrote: ... BTW, since these patches (dax + THIS + pmem-unbind) are waiting to be merged, is it time to think about "removing the experimental tag" again? :) It's probably time to

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-08-03 Thread Shiyang Ruan
在 2022/8/4 8:51, Darrick J. Wong 写道: On Wed, Aug 03, 2022 at 06:47:24AM +, ruansy.f...@fujitsu.com wrote: 在 2022/7/29 12:54, Darrick J. Wong 写道: On Fri, Jul 29, 2022 at 03:55:24AM +, ruansy.f...@fujitsu.com wrote: 在 2022/7/22 0:16, Darrick J. Wong 写道: On Thu, Jul 21, 2022 at

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-08-03 Thread Darrick J. Wong
On Wed, Aug 03, 2022 at 06:47:24AM +, ruansy.f...@fujitsu.com wrote: > > > 在 2022/7/29 12:54, Darrick J. Wong 写道: > > On Fri, Jul 29, 2022 at 03:55:24AM +, ruansy.f...@fujitsu.com wrote: > >> > >> > >> 在 2022/7/22 0:16, Darrick J. Wong 写道: > >>> On Thu, Jul 21, 2022 at 02:06:10PM +,

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-08-03 Thread ruansy.f...@fujitsu.com
在 2022/7/29 12:54, Darrick J. Wong 写道: > On Fri, Jul 29, 2022 at 03:55:24AM +, ruansy.f...@fujitsu.com wrote: >> >> >> 在 2022/7/22 0:16, Darrick J. Wong 写道: >>> On Thu, Jul 21, 2022 at 02:06:10PM +, ruansy.f...@fujitsu.com wrote: 在 2022/7/1 8:31, Darrick J. Wong 写道: > On Thu,

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-07-28 Thread Darrick J. Wong
On Fri, Jul 29, 2022 at 03:55:24AM +, ruansy.f...@fujitsu.com wrote: > > > 在 2022/7/22 0:16, Darrick J. Wong 写道: > > On Thu, Jul 21, 2022 at 02:06:10PM +, ruansy.f...@fujitsu.com wrote: > >> 在 2022/7/1 8:31, Darrick J. Wong 写道: > >>> On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-07-28 Thread ruansy.f...@fujitsu.com
在 2022/7/22 0:16, Darrick J. Wong 写道: > On Thu, Jul 21, 2022 at 02:06:10PM +, ruansy.f...@fujitsu.com wrote: >> 在 2022/7/1 8:31, Darrick J. Wong 写道: >>> On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: Failure notification is not supported on partitions. So, when we mount

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-07-21 Thread Darrick J. Wong
On Thu, Jul 21, 2022 at 02:06:10PM +, ruansy.f...@fujitsu.com wrote: > 在 2022/7/1 8:31, Darrick J. Wong 写道: > > On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: > >> Failure notification is not supported on partitions. So, when we mount > >> a reflink enabled xfs on a partition

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-07-21 Thread ruansy.f...@fujitsu.com
在 2022/7/1 8:31, Darrick J. Wong 写道: > On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: >> Failure notification is not supported on partitions. So, when we mount >> a reflink enabled xfs on a partition with dax option, let it fail with >> -EINVAL code. >> >> Signed-off-by: Shiyang

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-07-21 Thread ruansy.f...@fujitsu.com
Hi Andrew, I am not sure if you had seen this. So make a ping. 在 2022/7/1 13:14, Shiyang Ruan 写道: > > > 在 2022/7/1 8:31, Darrick J. Wong 写道: >> On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: >>> Failure notification is not supported on partitions.  So, when we mount >>> a

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-06-30 Thread Shiyang Ruan
在 2022/7/1 8:31, Darrick J. Wong 写道: On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: Failure notification is not supported on partitions. So, when we mount a reflink enabled xfs on a partition with dax option, let it fail with -EINVAL code. Signed-off-by: Shiyang Ruan

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-06-30 Thread Darrick J. Wong
On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: > Failure notification is not supported on partitions. So, when we mount > a reflink enabled xfs on a partition with dax option, let it fail with > -EINVAL code. > > Signed-off-by: Shiyang Ruan Looks good to me, though I think this

Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-06-09 Thread Christoph Hellwig
On Thu, Jun 09, 2022 at 10:34:35PM +0800, Shiyang Ruan wrote: > Failure notification is not supported on partitions. So, when we mount > a reflink enabled xfs on a partition with dax option, let it fail with > -EINVAL code. Looks good: Reviewed-by: Christoph Hellwig

[PATCH] xfs: fail dax mount if reflink is enabled on a partition

2022-06-09 Thread Shiyang Ruan
Failure notification is not supported on partitions. So, when we mount a reflink enabled xfs on a partition with dax option, let it fail with -EINVAL code. Signed-off-by: Shiyang Ruan --- fs/xfs/xfs_super.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git