Re: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages

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

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

On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig <hch@xxxxxx> wrote:
>
> memremap.c is only built when CONFIG_ZONE_DEVICE is set, so remove
> the superflous extra check.

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 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 6/8] mm: don't include <linux/memremap.h> in <linux/mm.h>
Next by Date: [PATCH 05/23] drm/amd/display: Fix color encoding mismatch
Previous by thread: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages
Next by thread: Re: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages
Index(es):
- Date
- Thread

[Index of Archives] [Linux USB Devel] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]