Re: [PATCH 6/8] mm: don't include <linux/memremap.h> in <linux/mm.h>

From: Christoph Hellwig
Date: Tue Feb 08 2022 - 01:47:00 EST


On Mon, Feb 07, 2022 at 04:19:29PM -0500, Felix Kuehling wrote:
>
> Am 2022-02-07 um 01:32 schrieb Christoph Hellwig:
>> Move the check for the actual pgmap types that need the free at refcount
>> one behavior into the out of line helper, and thus avoid the need to
>> pull memremap.h into mm.h.
>>
>> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
>
> The amdkfd part looks good to me.
>
> It looks like this patch is not based on Alex Sierra's coherent memory
> series. He added two new helpers is_device_coherent_page and
> is_dev_private_or_coherent_page that would need to be moved along with
> is_device_private_page and is_pci_p2pdma_page.

Yes. I Naked that series because it spreads te mess with the refcount
further in this latest version. My intent is that it gets rebased
on top of this to avoid that spread. Same for the p2p series form Logan.