Re: [PATCH 0/3] dax: clear poison on the fly along pwrite

2021-09-17 Thread Darrick J. Wong
On Fri, Sep 17, 2021 at 01:21:25PM -0700, Dan Williams wrote: > On Fri, Sep 17, 2021 at 8:27 AM Darrick J. Wong wrote: > > > > On Fri, Sep 17, 2021 at 01:53:33PM +0100, Christoph Hellwig wrote: > > > On Thu, Sep 16, 2021 at 11:40:28AM -0700, Dan Williams wrote: > > > > > That was my gut feeling.

Re: [PATCH 0/3] dax: clear poison on the fly along pwrite

2021-09-17 Thread Dan Williams
On Fri, Sep 17, 2021 at 8:27 AM Darrick J. Wong wrote: > > On Fri, Sep 17, 2021 at 01:53:33PM +0100, Christoph Hellwig wrote: > > On Thu, Sep 16, 2021 at 11:40:28AM -0700, Dan Williams wrote: > > > > That was my gut feeling. If everyone feels 100% comfortable with > > > > zeroingas the mechanism

Re: [PATCH 0/3] dax: clear poison on the fly along pwrite

2021-09-17 Thread Dan Williams
On Fri, Sep 17, 2021 at 5:57 AM Christoph Hellwig wrote: > > On Thu, Sep 16, 2021 at 11:40:28AM -0700, Dan Williams wrote: > > > That was my gut feeling. If everyone feels 100% comfortable with > > > zeroingas the mechanism to clear poisoning I'll cave in. The most > > > important bit is that

Re: [PATCH v9 7/8] xfs: support CoW in fsdax mode

2021-09-17 Thread Darrick J. Wong
On Thu, Sep 16, 2021 at 08:32:51AM +0200, Christoph Hellwig wrote: > On Wed, Sep 15, 2021 at 05:22:27PM -0700, Darrick J. Wong wrote: > > > xfs_ilock(XFS_I(inode), XFS_MMAPLOCK_SHARED); > > > ret = dax_iomap_fault(vmf, pe_size, , NULL, > > >

Re: [PATCH 0/3] dax: clear poison on the fly along pwrite

2021-09-17 Thread Darrick J. Wong
On Fri, Sep 17, 2021 at 01:53:33PM +0100, Christoph Hellwig wrote: > On Thu, Sep 16, 2021 at 11:40:28AM -0700, Dan Williams wrote: > > > That was my gut feeling. If everyone feels 100% comfortable with > > > zeroingas the mechanism to clear poisoning I'll cave in. The most > > > important bit is

Re: [PATCH 0/3] dax: clear poison on the fly along pwrite

2021-09-17 Thread Christoph Hellwig
On Thu, Sep 16, 2021 at 11:40:28AM -0700, Dan Williams wrote: > > That was my gut feeling. If everyone feels 100% comfortable with > > zeroingas the mechanism to clear poisoning I'll cave in. The most > > important bit is that we do that through a dedicated DAX path instead > > of abusing the