Re: [PATCH] memory hotplug: fix comments when add section

From: Michal Hocko
Date: Wed Nov 15 2017 - 04:19:56 EST


On Wed 15-11-17 16:40:05, Fan Du wrote:
> Here, pfn_to_node should be page_to_nid.
>
> Signed-off-by: Fan Du <fan.du@xxxxxxxxx>

Yes, it is a typo. And the one worth fixing because this will make git grep
great again

Acked-by: Michal Hocko <mhocko@xxxxxxxx>

> ---
> mm/memory_hotplug.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c
> index d4b5f29..c3c1c57 100644
> --- a/mm/memory_hotplug.c
> +++ b/mm/memory_hotplug.c
> @@ -265,7 +265,7 @@ static int __meminit __add_section(int nid, unsigned long phys_start_pfn,
> /*
> * Make all the pages reserved so that nobody will stumble over half
> * initialized state.
> - * FIXME: We also have to associate it with a node because pfn_to_node
> + * FIXME: We also have to associate it with a node because page_to_nid
> * relies on having page with the proper node.
> */
> for (i = 0; i < PAGES_PER_SECTION; i++) {
> --
> 1.8.3.1

--
Michal Hocko
SUSE Labs