<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, 5 Mar, 2019, 4:08 AM Linus Walleij <<a href="mailto:linus.walleij@linaro.org">linus.walleij@linaro.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Mar 4, 2019 at 3:48 PM Manivannan Sadhasivam<br>
<<a href="mailto:manivannan.sadhasivam@linaro.org" target="_blank" rel="noreferrer">manivannan.sadhasivam@linaro.org</a>> wrote:<br>
> On Mon, Mar 04, 2019 at 01:29:33PM +0100, Linus Walleij wrote:<br>
> > On Fri, Mar 1, 2019 at 4:55 PM Manivannan Sadhasivam<br>
> > <<a href="mailto:manivannan.sadhasivam@linaro.org" target="_blank" rel="noreferrer">manivannan.sadhasivam@linaro.org</a>> wrote:<br>
> > > Hi Wen,<br>
> > ><br>
> > > On Fri, Mar 01, 2019 at 04:56:42PM +0800, Wen Yang wrote:<br>
> > > > The call to of_get_next_child returns a node pointer with refcount<br>
> > > > incremented thus it must be explicitly decremented after the last<br>
> > > > usage.<br>
> > > ><br>
> > > > Detected by coccinelle with the following warnings:<br>
> > > > ./arch/arm/mach-actions/platsmp.c:112:2-8: ERROR: missing of_node_put; acquired a node pointer with refcount incremented on line 103, but without a corresponding object release within this function.<br>
> > > > ./arch/arm/mach-actions/platsmp.c:124:2-8: ERROR: missing of_node_put; acquired a node pointer with refcount incremented on line 115, but without a corresponding object release within this function.<br>
> > > > ./arch/arm/mach-actions/platsmp.c:137:3-9: ERROR: missing of_node_put; acquired a node pointer with refcount incremented on line 128, but without a corresponding object release within this function.<br>
> > > ><br>
> > ><br>
> > > We have a floating patch for this:<br>
> > > <a href="https://www.spinics.net/lists/arm-kernel/msg694544.html" rel="noreferrer noreferrer" target="_blank">https://www.spinics.net/lists/arm-kernel/msg694544.html</a><br>
> > ><br>
> > > Andreas: Can you please take a second look at the patchset submitted by Linus<br>
> > > Walleij and Russel for simplifying the Actions startup code?<br>
> ><br>
> > Andreas wrote a version of simplifying secondary startup in the<br>
> > same spirit as Russell's patches, and it's merged and all<br>
> > is fine I think.<br>
> ><br>
><br>
> Oops. I think I missed that! Can you please point me to that patch? And how it<br>
> got merged? I did the PR for actions stuff this time and haven't included any<br>
> mach-actions patches.<br>
<br>
I just did git log arch/arm/mach-actions but I think it came in quite some time<br>
ago, not last merge window:<br>
<br>
But you see:<br>
commit 6c2eb3e76fb84e2eb46d484f71fab469c0d9532c<br>
"ARM: owl: smp: Drop owl_secondary_boot()"<br>
commit bad29933fef76fb6ee577f4a0b6d145c1f52f663<br>
"ARM: owl: smp: Use __pa_symbol()"<br>
commit 18cfd9429d8a82c49add8f3ca9d366599bfcac45<br>
"ARM: owl: smp: Drop bogus holding pen"<br>
<br>
platsmp.c looks just fine these days. Except for what Wei's patch is fixing,<br>
of_node_put().<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Nope. platsmp.c still requires some cleanup like removing the redundant bootlock and pen_release flag as pointed out by Russel. Andreas just replied to your cleanup patches but there was no follow up since that. So, I guess we can just apply Russell's patches and this patch once Andreas is fine with it (it looks good to me though). </div><div dir="auto"><br></div><div dir="auto">Thanks, </div><div dir="auto">Mani</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Yours,<br>
Linus Walleij<br>
</blockquote></div></div></div>