Re: [PATCH v2] mm: Drop useless local parameters of __register_one_node()

From: Michal Hocko
Date: Thu Jun 29 2017 - 08:12:21 EST


On Thu 29-06-17 19:58:52, Dou Liyang wrote:
> Hi Michal,
>
> At 06/29/2017 07:12 PM, Michal Hocko wrote:
> >On Wed 21-06-17 10:57:26, Dou Liyang wrote:
> >>... initializes local parameters "p_node" & "parent" for
> >>register_node().
> >>
> >>But, register_node() does not use them.
> >>
> >>Remove the related code of "parent" node, cleanup __register_one_node()
> >>and register_node().
> >>
> >>Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> >>Cc: David Rientjes <rientjes@xxxxxxxxxx>
> >>Cc: Michal Hocko <mhocko@xxxxxxxxxx>
> >>Cc: isimatu.yasuaki@xxxxxxxxxxxxxx
> >>Signed-off-by: Dou Liyang <douly.fnst@xxxxxxxxxxxxxx>
> >>Acked-by: David Rientjes <rientjes@xxxxxxxxxx>
> >
> >I am sorry, this slipped through cracks.
> >Acked-by: Michal Hocko <mhocko@xxxxxxxx>
>
> Thanks for your Acked-by, but this patch has been added to the -mm tree.
> Its filename is
> mm-drop-useless-local-parameters-of-__register_one_node.patch

Yeah, don't worry. Andrew will add the acked-by in his tree.

> This patch should soon appear at
>
> http://ozlabs.org/~akpm/mmots/broken-out/mm-drop-useless-local-parameters-of-__register_one_node.patch
> and later at
>
> http://ozlabs.org/~akpm/mmotm/broken-out/mm-drop-useless-local-parameters-of-__register_one_node.patch
>
> I don't know what should I do next ? :)

Wait for Andrew to send this to Linus during the next merge window.
--
Michal Hocko
SUSE Labs