[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20180816172026.s2v3ytqnkiboo72s@xakep.localdomain>
Date: Thu, 16 Aug 2018 17:20:27 +0000
From: Pasha Tatashin <Pavel.Tatashin@...rosoft.com>
To: "osalvador@...hadventures.net" <osalvador@...hadventures.net>
CC: "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"mhocko@...e.com" <mhocko@...e.com>,
"vbabka@...e.cz" <vbabka@...e.cz>,
Pasha Tatashin <Pavel.Tatashin@...rosoft.com>,
"Jonathan.Cameron@...wei.com" <Jonathan.Cameron@...wei.com>,
"arbab@...ux.vnet.ibm.com" <arbab@...ux.vnet.ibm.com>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Oscar Salvador <osalvador@...e.de>
Subject: Re: [PATCH v2 3/4] mm/memory_hotplug: Make
register_mem_sect_under_node a cb of walk_memory_range
On 18-06-22 13:18:38, osalvador@...hadventures.net wrote:
> From: Oscar Salvador <osalvador@...e.de>
>
> link_mem_sections() and walk_memory_range() share most of the code,
> so we can use convert link_mem_sections() into a dummy function that calls
> walk_memory_range() with a callback to register_mem_sect_under_node().
>
> This patch converts register_mem_sect_under_node() in order to
> match a walk_memory_range's callback, getting rid of the
> check_nid argument and checking instead if the system is still
> boothing, since we only have to check for the nid if the system
> is in such state.
>
> Signed-off-by: Oscar Salvador <osalvador@...e.de>
> Suggested-by: Pavel Tatashin <pasha.tatashin@...cle.com>
Reviewed-by: Pavel Tatashin <pavel.tatashin@...rosoft.com>
Powered by blists - more mailing lists