Re: [PATCH 1/3] mm: replace memmap_context by memplug_context
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Laurent Dufour <ldufour@xxxxxxxxxxxxx>
- Subject: Re: [PATCH 1/3] mm: replace memmap_context by memplug_context
- From: Michal Hocko <mhocko@xxxxxxxx>
- Date: Mon, 14 Sep 2020 10:59:53 +0200
- Cc: akpm@xxxxxxxxxxxxxxxxxxxx, David Hildenbrand <david@xxxxxxxxxx>, Oscar Salvador <osalvador@xxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, linux-mm@xxxxxxxxx, "Rafael J . Wysocki" <rafael@xxxxxxxxxx>, nathanl@xxxxxxxxxxxxx, cheloha@xxxxxxxxxxxxx, Tony Luck <tony.luck@xxxxxxxxx>, Fenghua Yu <fenghua.yu@xxxxxxxxx>, linux-ia64@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
- In-reply-to: <d3843235-2e02-a4ad-494c-20222a7e3b45@linux.ibm.com>
- References: <20200911134831.53258-1-ldufour@linux.ibm.com> <20200911134831.53258-2-ldufour@linux.ibm.com> <20200914084904.GA16999@dhcp22.suse.cz> <d3843235-2e02-a4ad-494c-20222a7e3b45@linux.ibm.com>
On Mon 14-09-20 10:51:06, Laurent Dufour wrote:
> Le 14/09/2020 à 10:49, Michal Hocko a écrit :
[...]
> > /*
> > * Memory initialization context, use to differentiate memory added by
> > * the platform statically or via memory hotplug interface.
> > */
> > enum meminit_context {
> > MEMINIT_EARLY,
> > MEMINIT_HOTPLUG
> > }
> >
>
> Sounds good too.
> What about its definition's place, in include/mm/zone.h as David suggested ?
Yes. This shouldn't be in the hotplug proper.
--
Michal Hocko
SUSE Labs
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]