Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread Christoph Hellwig
On Thu, Apr 21, 2022 at 02:35:02PM +1000, Dave Chinner wrote: > Sure, I'm not a maintainer and just the stand-in patch shepherd for > a single release. However, being unable to cleanly merge code we > need integrated into our local subsystem tree for integration > testing because a patch

Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread 堀口 直也
Hi everyone, On Thu, Apr 21, 2022 at 02:35:02PM +1000, Dave Chinner wrote: > On Wed, Apr 20, 2022 at 07:20:07PM -0700, Dan Williams wrote: > > [ add Andrew and Naoya ] > > > > On Wed, Apr 20, 2022 at 6:48 PM Shiyang Ruan > > wrote: > > > > > > Hi Dave, > > > > > > 在 2022/4/21 9:20, Dave

Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread Dave Chinner
On Wed, Apr 20, 2022 at 07:20:07PM -0700, Dan Williams wrote: > [ add Andrew and Naoya ] > > On Wed, Apr 20, 2022 at 6:48 PM Shiyang Ruan wrote: > > > > Hi Dave, > > > > 在 2022/4/21 9:20, Dave Chinner 写道: > > > Hi Ruan, > > > > > > On Tue, Apr 19, 2022 at 12:50:38PM +0800, Shiyang Ruan wrote: >

Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread Dan Williams
[ add Andrew and Naoya ] On Wed, Apr 20, 2022 at 6:48 PM Shiyang Ruan wrote: > > Hi Dave, > > 在 2022/4/21 9:20, Dave Chinner 写道: > > Hi Ruan, > > > > On Tue, Apr 19, 2022 at 12:50:38PM +0800, Shiyang Ruan wrote: > >> This patchset is aimed to support shared pages tracking for fsdax. > > > > Now

Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread Shiyang Ruan
Hi Dave, 在 2022/4/21 9:20, Dave Chinner 写道: Hi Ruan, On Tue, Apr 19, 2022 at 12:50:38PM +0800, Shiyang Ruan wrote: This patchset is aimed to support shared pages tracking for fsdax. Now that this is largely reviewed, it's time to work out the logistics of merging it. Thanks! Changes

Re: [PATCH v13 0/7] fsdax: introduce fs query to support reflink

2022-04-20 Thread Dave Chinner
Hi Ruan, On Tue, Apr 19, 2022 at 12:50:38PM +0800, Shiyang Ruan wrote: > This patchset is aimed to support shared pages tracking for fsdax. Now that this is largely reviewed, it's time to work out the logistics of merging it. > Changes since V12: > - Rebased onto next-20220414 What does this

Re: [PATCH v13 5/7] mm: Introduce mf_dax_kill_procs() for fsdax case

2022-04-20 Thread Darrick J. Wong
On Tue, Apr 19, 2022 at 12:50:43PM +0800, Shiyang Ruan wrote: > This new function is a variant of mf_generic_kill_procs that accepts a > file, offset pair instead of a struct to support multiple files sharing > a DAX mapping. It is intended to be called by the file systems as part > of the

Re: [PATCH v13 4/7] fsdax: Introduce dax_lock_mapping_entry()

2022-04-20 Thread Darrick J. Wong
On Tue, Apr 19, 2022 at 12:50:42PM +0800, Shiyang Ruan wrote: > The current dax_lock_page() locks dax entry by obtaining mapping and > index in page. To support 1-to-N RMAP in NVDIMM, we need a new function > to lock a specific dax entry corresponding to this file's mapping,index. > And output

Re: [PATCH v13 3/7] pagemap,pmem: Introduce ->memory_failure()

2022-04-20 Thread Darrick J. Wong
On Tue, Apr 19, 2022 at 12:50:41PM +0800, Shiyang Ruan wrote: > When memory-failure occurs, we call this function which is implemented > by each kind of devices. For the fsdax case, pmem device driver > implements it. Pmem device driver will find out the filesystem in which > the corrupted page

Re: [PATCH v13 1/7] dax: Introduce holder for dax_device

2022-04-20 Thread Darrick J. Wong
On Tue, Apr 19, 2022 at 12:50:39PM +0800, Shiyang Ruan wrote: > To easily track filesystem from a pmem device, we introduce a holder for > dax_device structure, and also its operation. This holder is used to > remember who is using this dax_device: > - When it is the backend of a filesystem, the

Re: [PATCH v13 7/7] fsdax: set a CoW flag when associate reflink mappings

2022-04-20 Thread Darrick J. Wong
On Tue, Apr 19, 2022 at 12:50:45PM +0800, Shiyang Ruan wrote: > Introduce a PAGE_MAPPING_DAX_COW flag to support association with CoW file > mappings. In this case, since the dax-rmap has already took the > responsibility to look up for shared files by given dax page, > the page->mapping is no

Re: [PATCH v13.1 6/7] xfs: Implement ->notify_failure() for XFS

2022-04-20 Thread Darrick J. Wong
On Wed, Apr 20, 2022 at 03:33:42PM +0800, Shiyang Ruan wrote: > Introduce xfs_notify_failure.c to handle failure related works, such as > implement ->notify_failure(), register/unregister dax holder in xfs, and > so on. > > If the rmap feature of XFS enabled, we can query it to find files and >

[PATCH v13.1 6/7] xfs: Implement ->notify_failure() for XFS

2022-04-20 Thread Shiyang Ruan
Introduce xfs_notify_failure.c to handle failure related works, such as implement ->notify_failure(), register/unregister dax holder in xfs, and so on. If the rmap feature of XFS enabled, we can query it to find files and metadata which are associated with the corrupt data. For now all we do is