RE: [PATCH v2 2/9] ext2: tell DAX the size of allocation holes

From: Matthew Wilcox
Date: Sat Sep 10 2016 - 03:53:02 EST


From: Christoph Hellwig [mailto:hch@xxxxxxxxxxxxx]
> On Sat, Sep 10, 2016 at 07:33:18AM +0000, Matthew Wilcox wrote:
> > > caller specific is unaceptable. That being said your idea doesn't
> > > sounds unreasonable, but will require a bit more work and has no
> > > real short-term need.
> >
> > So your proposal is to remove buffer_heads from ext2?
>
> No, the proposal is to remove buffer_heads from XFS first, then GFS2 and then
> maybe others like ext4. I'd like to remove buffer_heads from the DAX path for
> ext2 and ext4 entitrely for sure (and direct I/O next).

That's ... what I propose. The only use of buffer_head in my proposal is to
communicate a single extent from the filesystem to the DAX core, and that
can equally well use an iomap. Ross seems to think that converting the current
DAX code over to using iomap requires converting all of ext2 away from
buffer_head; are you saying he's wrong?