[PATCH 10/12] xfs: use iomap to implement DAX

2016-09-16 Thread Christoph Hellwig
Another users of buffer_heads bytes the dust. Signed-off-by: Christoph Hellwig Reviewed-by: Ross Zwisler --- fs/xfs/xfs_file.c | 61 +++--- fs/xfs/xfs_iomap.c | 11 ++ 2 files changed, 24 insertions(+), 48 deletions(-) diff --git a/fs/xf

Re: [PATCH 10/12] xfs: use iomap to implement DAX

2016-09-14 Thread Christoph Hellwig
On Wed, Sep 14, 2016 at 10:29:33PM -0700, Darrick J. Wong wrote: > I'm a little confused about xfs_file_iomap_begin() and IOMAP_WRITE -- > prior to this patchset, it was only called via page_mkwrite and what > used to be write_begin, and all it did was create a delalloc reservation > to back the wr

Re: [PATCH 10/12] xfs: use iomap to implement DAX

2016-09-14 Thread Darrick J. Wong
On Wed, Sep 14, 2016 at 12:01:29PM +0200, Christoph Hellwig wrote: > Another users of buffer_heads bytes the dust. Yay! Babbling below. :) > Signed-off-by: Christoph Hellwig > --- > fs/xfs/xfs_file.c | 61 > +++--- > fs/xfs/xfs_iomap.c | 11 +++

Re: [PATCH 10/12] xfs: use iomap to implement DAX

2016-09-14 Thread Christoph Hellwig
On Wed, Sep 14, 2016 at 11:32:47AM -0600, Ross Zwisler wrote: > I think this should be xfs_setfilesize(ip, pos, ret)? 'count' and 'ret' are > the same in non-error cases, but in error cases where iomap_dax_rw() does some > work and then encounters an error, 'ret' could be smaller. In error cases

Re: [PATCH 10/12] xfs: use iomap to implement DAX

2016-09-14 Thread Ross Zwisler
On Wed, Sep 14, 2016 at 12:01:29PM +0200, Christoph Hellwig wrote: > Another users of buffer_heads bytes the dust. > > Signed-off-by: Christoph Hellwig > --- > fs/xfs/xfs_file.c | 61 > +++--- > fs/xfs/xfs_iomap.c | 11 ++ > 2 files chang

[PATCH 10/12] xfs: use iomap to implement DAX

2016-09-14 Thread Christoph Hellwig
Another users of buffer_heads bytes the dust. Signed-off-by: Christoph Hellwig --- fs/xfs/xfs_file.c | 61 +++--- fs/xfs/xfs_iomap.c | 11 ++ 2 files changed, 24 insertions(+), 48 deletions(-) diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_f