Hi Anton, On Wed, 10 Aug 2011 16:49:34 +1000 Anton Blanchard <an...@samba.org> wrote: > > During memory hotplug testing, I got the following warning: > > > ERROR: Bad of_node_put() on /memory@0 > > of_find_node_by_type() loop does of_node_put for us so remove the > duplicate one inside the loop.
But does an of_node_get() on its return value .. > Signed-off-by: Anton Blanchard <an...@samba.org> > Cc: sta...@kernel.org > --- > > Index: linux-powerpc/arch/powerpc/mm/numa.c > =================================================================== > --- linux-powerpc.orig/arch/powerpc/mm/numa.c 2011-06-06 08:07:35.148708089 > +1000 > +++ linux-powerpc/arch/powerpc/mm/numa.c 2011-08-10 11:31:59.723379868 > +1000 > @@ -1214,7 +1214,6 @@ int hot_add_node_scn_to_nid(unsigned lon > break; > } > > - of_node_put(memory); > if (nid >= 0) > break; > } Won't that need an of_node_put(memory) after the loop if (nid >= 0) ? In fact you should be able to just move the of_node_put(memory) to after the loop since of_node_put(NULL) is fine. -- Cheers, Stephen Rothwell s...@canb.auug.org.au http://www.canb.auug.org.au/~sfr/
pgpCfk3jFaUpd.pgp
Description: PGP signature
_______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
