On Fri, Apr 23, 2021 at 9:40 AM Christophe JAILLET
<christophe.jaillet@xxxxxxxxxx> wrote:
>
> If we exit the for_each_of_cpu_node loop early, the reference on the
> current node must be decremented, otherwise there is a leak.
>
> Fixes: a94fe366340a ("powerpc: use for_each_of_cpu_node iterator")
> Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
> ---
> Strangely, the commit above added the needed of_node_put in one place but
> missed 2 other places!
Well, maintained it in one place and forgot to add in the other two.
> This is strange, so maybe I misunderstand something. Review carefully
> ---
> arch/powerpc/platforms/powermac/feature.c | 2 ++
> 1 file changed, 2 insertions(+)
Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
I don't really think patch 2 is worthwhile but that's up to the
powerpc maintainers.
Rob
