Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages
From: Dan Williams <dan.j.williams@xxxxxxxxx>
Date: Mon, 7 Feb 2022 15:42:39 -0800
Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Felix Kuehling <Felix.Kuehling@xxxxxxx>, Alex Deucher <alexander.deucher@xxxxxxx>, Christian König <christian.koenig@xxxxxxx>, "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>, Ben Skeggs <bskeggs@xxxxxxxxxx>, Karol Herbst <kherbst@xxxxxxxxxx>, Lyude Paul <lyude@xxxxxxxxxx>, Jason Gunthorpe <jgg@xxxxxxxx>, Alistair Popple <apopple@xxxxxxxxxx>, Logan Gunthorpe <logang@xxxxxxxxxxxx>, Ralph Campbell <rcampbell@xxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>, Maling list - DRI developers <dri-devel@xxxxxxxxxxxxxxxxxxxxx>, nouveau@xxxxxxxxxxxxxxxxxxxxx, Linux NVDIMM <nvdimm@xxxxxxxxxxxxxxx>, Linux MM <linux-mm@xxxxxxxxx>
In-reply-to: <20220207063249.1833066-6-hch@lst.de>
References: <20220207063249.1833066-1-hch@lst.de> <20220207063249.1833066-6-hch@lst.de>

On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig <hch@xxxxxx> wrote:
>
> Make put_devmap_managed_page return if it took charge of the page
> or not and remove the separate page_is_devmap_managed helper.

Looks good to me:

Reviewed-by: Dan Williams <dan.j.williams@xxxxxxxxx>

References:
- start sorting out the ZONE_DEVICE refcount mess
  - From: Christoph Hellwig
- [PATCH 5/8] mm: simplify freeing of devmap managed pages
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 05/35] x86/fpu/xstate: Introduce CET MSR and XSAVES supervisor states
Next by Date: Re: [PATCH v2 1/3] mm: enable MADV_DONTNEED for hugetlb mappings
Previous by thread: Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages
Next by thread: Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]