[Nouveau] [PATCH 08/25] memremap: move dev_pagemap callbacks into a separate structure

Christoph Hellwig hch at lst.de
Mon Jun 17 20:09:57 UTC 2019


On Mon, Jun 17, 2019 at 02:08:14PM -0600, Logan Gunthorpe wrote:
> I just noticed this is missing a line to set pgmap->ops to
> pci_p2pdma_pagemap_ops. I must have gotten confused by the other users
> in my original review. Though I'm not sure how this compiles as the new
> struct is static and unused. However, it is rendered moot in Patch 16
> when this is all removed.

It probably was there in the original and got lost in the merge conflicts
from the rebase.  I should have dropped all the reviewed-bys for patches
with non-trivial merge resolution, sorry.


More information about the Nouveau mailing list