Jason Gunthorpe wrote: > On Wed, Sep 21, 2022 at 08:48:22AM -0700, Dan Williams wrote: > > > The fsdax core now manages pgmap references when servicing faults that > > install new mappings, and elevates the page reference until it is > > zapped. It coordinates with the VFS to make sure that all page > > references are dropped before the hosting inode goes out of scope > > (iput_final()). > > > > In order to delete the unnecessary pgmap reference taking in mm/gup.c > > devdax needs to move to the same model. > > I think this patch is more about making devdax and fsdax use the same > set of functions and logic so that when it gets to patch 16/17 devdax > doesn't break. That understanding matches the first paragraph, at > least. > > I would delete the remark about gup since it is really patch 17 that > allows gup to be fixed by making it so that refcount == 0 means not to > look at the pgmap (instead of refcount == 1 as is now) ? Yeah, makes sense.
