RE: [PATCH v3 00/10] fsdax,xfs: Add reflink support for fsdax

2021-04-02 Thread ruansy.f...@fujitsu.com


> -Original Message-
> From: Christoph Hellwig 
> Sent: Friday, April 2, 2021 3:50 PM
> Subject: Re: [PATCH v3 00/10] fsdax,xfs: Add reflink support for fsdax
> 
> Shiyang, Dan:
> 
> given that the whole reflink+dax thing is going to take a while and thus not 
> going
> to happen for this merge window, what about queueing up the cleanup patches
> 1,2 and 3 so that we can reduce the patch load a little?

OK.  I'll send a new version of these 3 patches based on latest comment.


--
Thanks,
Ruan Shiyang.


Re: [PATCH v3 00/10] fsdax,xfs: Add reflink support for fsdax

2021-04-02 Thread Christoph Hellwig
Shiyang, Dan:

given that the whole reflink+dax thing is going to take a while and thus
not going to happen for this merge window, what about queueing up the
cleanup patches 1,2 and 3 so that we can reduce the patch load a little?


Re: [PATCH v3 00/10] fsdax,xfs: Add reflink support for fsdax

2021-03-23 Thread Ritesh Harjani




On 3/19/21 7:22 AM, Shiyang Ruan wrote:

From: Shiyang Ruan 

This patchset is attempt to add CoW support for fsdax, and take XFS,
which has both reflink and fsdax feature, as an example.



Thanks for the patchset. I have tried reviewing the series from logical
correctness and to some extent functional correctness.
Since I am not well versed with the core functionality of COW operation,
so I may have requested for some clarifications where I could not get
the code 100% on what it is doing.




(Rebased on v5.11)
==


Thanks. Yes, I see some conflicts when tried to apply on latest kernel.

-ritesh