* [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-18 10:43 Sudeep Holla
2018-01-18 14:57 ` Rob Herring
2018-01-19 15:37 ` Sudeep Holla
0 siblings, 2 replies; 8+ messages in thread
From: Sudeep Holla @ 2018-01-18 10:43 UTC (permalink / raw)
To: devicetree, linux-kernel
Cc: Sudeep Holla, Loys Ollivier, Andy Gross, Bjorn Andersson,
Rob Herring, Frank Rowand
We need to call of_node_put() for device nodes obtained with
of_find_node_by_path().
Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
Reported-by: Loys Ollivier <lollivier@baylibre.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Frank Rowand <frowand.list@gmail.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
drivers/of/platform.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
Hi Rob,
You had Acked the original patch and it's queued via ARM SoC. So can you
please provide ack on this if you are OK with the change.
Regards,
Sudeep
diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index 6a01dab597c7..c00d81dfac0b 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -514,8 +514,10 @@ static int __init of_platform_default_populate_init(void)
of_platform_device_create(node, NULL, NULL);
node = of_find_node_by_path("/firmware");
- if (node)
+ if (node) {
of_platform_populate(node, NULL, NULL, NULL);
+ of_node_put(node);
+ }
/* Populate everything else. */
of_platform_default_populate(NULL, NULL, NULL);
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
2018-01-18 10:43 [PATCH -next] of: platform: fix OF node refcount leak Sudeep Holla
@ 2018-01-18 14:57 ` Rob Herring
2018-01-19 15:37 ` Sudeep Holla
1 sibling, 0 replies; 8+ messages in thread
From: Rob Herring @ 2018-01-18 14:57 UTC (permalink / raw)
To: Sudeep Holla
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
linux-kernel, Loys Ollivier, Andy Gross, Bjorn Andersson,
Frank Rowand
On Thu, Jan 18, 2018 at 4:43 AM, Sudeep Holla <sudeep.holla@arm.com> wrote:
> We need to call of_node_put() for device nodes obtained with
> of_find_node_by_path().
>
> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
> Reported-by: Loys Ollivier <lollivier@baylibre.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Frank Rowand <frowand.list@gmail.com>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> ---
> drivers/of/platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> Hi Rob,
>
> You had Acked the original patch and it's queued via ARM SoC. So can you
> please provide ack on this if you are OK with the change.
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-19 15:37 ` Sudeep Holla
0 siblings, 0 replies; 8+ messages in thread
From: Sudeep Holla @ 2018-01-19 15:37 UTC (permalink / raw)
To: devicetree, linux-kernel, Andy Gross
Cc: Sudeep Holla, Loys Ollivier, Bjorn Andersson, Rob Herring, Frank Rowand
Hi Andy,
On 18/01/18 10:43, Sudeep Holla wrote:
> We need to call of_node_put() for device nodes obtained with
> of_find_node_by_path().
>
> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
> Reported-by: Loys Ollivier <lollivier@baylibre.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Frank Rowand <frowand.list@gmail.com>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> ---
> drivers/of/platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
Can you pick up this with other fixes you would send after v4.16-rc1 or
do you prefer me to route through ARM-SoC directly ? Let me know.
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-19 15:37 ` Sudeep Holla
0 siblings, 0 replies; 8+ messages in thread
From: Sudeep Holla @ 2018-01-19 15:37 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, Andy Gross
Cc: Sudeep Holla, Loys Ollivier, Bjorn Andersson, Rob Herring, Frank Rowand
Hi Andy,
On 18/01/18 10:43, Sudeep Holla wrote:
> We need to call of_node_put() for device nodes obtained with
> of_find_node_by_path().
>
> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
> Reported-by: Loys Ollivier <lollivier-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Cc: Frank Rowand <frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> Signed-off-by: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>
> ---
> drivers/of/platform.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
Can you pick up this with other fixes you would send after v4.16-rc1 or
do you prefer me to route through ARM-SoC directly ? Let me know.
--
Regards,
Sudeep
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-19 23:34 ` Andy Gross
0 siblings, 0 replies; 8+ messages in thread
From: Andy Gross @ 2018-01-19 23:34 UTC (permalink / raw)
To: Sudeep Holla
Cc: devicetree, Linux Kernel list, Loys Ollivier, Bjorn Andersson,
Rob Herring, Frank Rowand
On 19 January 2018 at 09:37, Sudeep Holla <sudeep.holla@arm.com> wrote:
> Hi Andy,
>
> On 18/01/18 10:43, Sudeep Holla wrote:
>> We need to call of_node_put() for device nodes obtained with
>> of_find_node_by_path().
>>
>> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
>> Reported-by: Loys Ollivier <lollivier@baylibre.com>
>> Cc: Rob Herring <robh+dt@kernel.org>
>> Cc: Frank Rowand <frowand.list@gmail.com>
>> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
>> ---
>> drivers/of/platform.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>
> Can you pick up this with other fixes you would send after v4.16-rc1 or
> do you prefer me to route through ARM-SoC directly ? Let me know.
I can pick this up for fixes, no problem. I can send it along right
now if you like. This is a pretty low risk patch.
Regards,
Andy
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-19 23:34 ` Andy Gross
0 siblings, 0 replies; 8+ messages in thread
From: Andy Gross @ 2018-01-19 23:34 UTC (permalink / raw)
To: Sudeep Holla
Cc: devicetree, Linux Kernel list, Loys Ollivier, Bjorn Andersson,
Rob Herring, Frank Rowand
On 19 January 2018 at 09:37, Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> wrote:
> Hi Andy,
>
> On 18/01/18 10:43, Sudeep Holla wrote:
>> We need to call of_node_put() for device nodes obtained with
>> of_find_node_by_path().
>>
>> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
>> Reported-by: Loys Ollivier <lollivier-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>> Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>> Cc: Frank Rowand <frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> Signed-off-by: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>
>> ---
>> drivers/of/platform.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>
> Can you pick up this with other fixes you would send after v4.16-rc1 or
> do you prefer me to route through ARM-SoC directly ? Let me know.
I can pick this up for fixes, no problem. I can send it along right
now if you like. This is a pretty low risk patch.
Regards,
Andy
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-22 11:13 ` Sudeep Holla
0 siblings, 0 replies; 8+ messages in thread
From: Sudeep Holla @ 2018-01-22 11:13 UTC (permalink / raw)
To: Andy Gross
Cc: Sudeep Holla, devicetree, Linux Kernel list, Loys Ollivier,
Bjorn Andersson, Rob Herring, Frank Rowand
On 19/01/18 23:34, Andy Gross wrote:
> On 19 January 2018 at 09:37, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> Hi Andy,
>>
>> On 18/01/18 10:43, Sudeep Holla wrote:
>>> We need to call of_node_put() for device nodes obtained with
>>> of_find_node_by_path().
>>>
>>> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
>>> Reported-by: Loys Ollivier <lollivier@baylibre.com>
>>> Cc: Rob Herring <robh+dt@kernel.org>
>>> Cc: Frank Rowand <frowand.list@gmail.com>
>>> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
>>> ---
>>> drivers/of/platform.c | 4 +++-
>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>
>> Can you pick up this with other fixes you would send after v4.16-rc1 or
>> do you prefer me to route through ARM-SoC directly ? Let me know.
>
> I can pick this up for fixes, no problem. I can send it along right
> now if you like. This is a pretty low risk patch.
>
Thanks Andy.
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] of: platform: fix OF node refcount leak
@ 2018-01-22 11:13 ` Sudeep Holla
0 siblings, 0 replies; 8+ messages in thread
From: Sudeep Holla @ 2018-01-22 11:13 UTC (permalink / raw)
To: Andy Gross
Cc: Sudeep Holla, devicetree, Linux Kernel list, Loys Ollivier,
Bjorn Andersson, Rob Herring, Frank Rowand
On 19/01/18 23:34, Andy Gross wrote:
> On 19 January 2018 at 09:37, Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> wrote:
>> Hi Andy,
>>
>> On 18/01/18 10:43, Sudeep Holla wrote:
>>> We need to call of_node_put() for device nodes obtained with
>>> of_find_node_by_path().
>>>
>>> Fixes: 3aa0582fdb82 ("of: platform: populate /firmware/ node from of_platform_default_populate_init()")
>>> Reported-by: Loys Ollivier <lollivier-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>>> Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>>> Cc: Frank Rowand <frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>> Signed-off-by: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>
>>> ---
>>> drivers/of/platform.c | 4 +++-
>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>
>> Can you pick up this with other fixes you would send after v4.16-rc1 or
>> do you prefer me to route through ARM-SoC directly ? Let me know.
>
> I can pick this up for fixes, no problem. I can send it along right
> now if you like. This is a pretty low risk patch.
>
Thanks Andy.
--
Regards,
Sudeep
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-01-22 11:13 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-18 10:43 [PATCH -next] of: platform: fix OF node refcount leak Sudeep Holla
2018-01-18 14:57 ` Rob Herring
2018-01-19 15:37 ` Sudeep Holla
2018-01-19 15:37 ` Sudeep Holla
2018-01-19 23:34 ` Andy Gross
2018-01-19 23:34 ` Andy Gross
2018-01-22 11:13 ` Sudeep Holla
2018-01-22 11:13 ` Sudeep Holla
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.