* [PATCH] of: overlay: fix warning being reported as error in add_changeset_property [not found] <1671195782-23078-1-git-send-email-ankit.16.kumar@nokia.com> @ 2022-12-30 8:40 ` Ankit 16. Kumar (Nokia) 2023-01-02 14:35 ` Frank Rowand 0 siblings, 1 reply; 4+ messages in thread From: Ankit 16. Kumar (Nokia) @ 2022-12-30 8:40 UTC (permalink / raw) To: robh+dt@kernel.org, frowand.list@gmail.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org The print causes false reporting of the issue which actually is a warning Fixes: 2fe0e8769df9 ("of: overlay: check prevents multiple fragments touching same property") Signed-off-by: Ankit Kumar <ankit.16.kumar@nokia.com> --- drivers/of/overlay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c index ed4e6c144a68..0da39b8461e7 100644 --- a/drivers/of/overlay.c +++ b/drivers/of/overlay.c @@ -358,7 +358,7 @@ static int add_changeset_property(struct overlay_changeset *ovcs, } if (!of_node_check_flag(target->np, OF_OVERLAY)) - pr_err("WARNING: memory leak will occur if overlay removed, property: %pOF/%s\n", + pr_warn("WARNING: memory leak will occur if overlay removed, +property: %pOF/%s\n", target->np, new_prop->name); if (ret) { -- 2.30.1 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] of: overlay: fix warning being reported as error in add_changeset_property 2022-12-30 8:40 ` [PATCH] of: overlay: fix warning being reported as error in add_changeset_property Ankit 16. Kumar (Nokia) @ 2023-01-02 14:35 ` Frank Rowand 2023-01-10 6:23 ` Frank Rowand 0 siblings, 1 reply; 4+ messages in thread From: Frank Rowand @ 2023-01-02 14:35 UTC (permalink / raw) To: Ankit 16. Kumar (Nokia), robh+dt@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org On 12/30/22 02:40, Ankit 16. Kumar (Nokia) wrote: > > The print causes false reporting of the issue which actually is a warning How did you select the commit in this Fixes tag? > Fixes: 2fe0e8769df9 ("of: overlay: check prevents multiple fragments touching same property") > > Signed-off-by: Ankit Kumar <ankit.16.kumar@nokia.com> > --- > drivers/of/overlay.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c index ed4e6c144a68..0da39b8461e7 100644 > --- a/drivers/of/overlay.c > +++ b/drivers/of/overlay.c > @@ -358,7 +358,7 @@ static int add_changeset_property(struct overlay_changeset *ovcs, > } > > if (!of_node_check_flag(target->np, OF_OVERLAY)) > - pr_err("WARNING: memory leak will occur if overlay removed, property: %pOF/%s\n", > + pr_warn("WARNING: memory leak will occur if overlay removed, > +property: %pOF/%s\n", > target->np, new_prop->name); > > if (ret) { > -- > 2.30.1 > NACK. This patch is incorrect. The reported memory leak is a bug, not a warning. I'll write up some information about why the memory leak occurs, then reply to this email with the additional info. -Frank ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] of: overlay: fix warning being reported as error in add_changeset_property 2023-01-02 14:35 ` Frank Rowand @ 2023-01-10 6:23 ` Frank Rowand 2023-01-10 18:00 ` Frank Rowand 0 siblings, 1 reply; 4+ messages in thread From: Frank Rowand @ 2023-01-10 6:23 UTC (permalink / raw) To: Ankit 16. Kumar (Nokia), robh+dt@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org On 1/2/23 08:35, Frank Rowand wrote: > On 12/30/22 02:40, Ankit 16. Kumar (Nokia) wrote: >> >> The print causes false reporting of the issue which actually is a warning > > How did you select the commit in this Fixes tag? > >> Fixes: 2fe0e8769df9 ("of: overlay: check prevents multiple fragments touching same property") >> >> Signed-off-by: Ankit Kumar <ankit.16.kumar@nokia.com> >> --- >> drivers/of/overlay.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c index ed4e6c144a68..0da39b8461e7 100644 >> --- a/drivers/of/overlay.c >> +++ b/drivers/of/overlay.c >> @@ -358,7 +358,7 @@ static int add_changeset_property(struct overlay_changeset *ovcs, >> } >> >> if (!of_node_check_flag(target->np, OF_OVERLAY)) >> - pr_err("WARNING: memory leak will occur if overlay removed, property: %pOF/%s\n", >> + pr_warn("WARNING: memory leak will occur if overlay removed, >> +property: %pOF/%s\n", >> target->np, new_prop->name); >> >> if (ret) { >> -- >> 2.30.1 >> > > NACK. This patch is incorrect. The reported memory leak is a bug, not a warning. > > I'll write up some information about why the memory leak occurs, then reply to this > email with the additional info. The additional information is now available at: https://elinux.org/Device_Tree_Linux#Object_Lifetime > > -Frank ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] of: overlay: fix warning being reported as error in add_changeset_property 2023-01-10 6:23 ` Frank Rowand @ 2023-01-10 18:00 ` Frank Rowand 0 siblings, 0 replies; 4+ messages in thread From: Frank Rowand @ 2023-01-10 18:00 UTC (permalink / raw) To: Ankit 16. Kumar (Nokia), robh+dt@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org On 1/10/23 00:23, Frank Rowand wrote: > On 1/2/23 08:35, Frank Rowand wrote: >> On 12/30/22 02:40, Ankit 16. Kumar (Nokia) wrote: >>> >>> The print causes false reporting of the issue which actually is a warning >> >> How did you select the commit in this Fixes tag? >> >>> Fixes: 2fe0e8769df9 ("of: overlay: check prevents multiple fragments touching same property") >>> >>> Signed-off-by: Ankit Kumar <ankit.16.kumar@nokia.com> >>> --- >>> drivers/of/overlay.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c index ed4e6c144a68..0da39b8461e7 100644 >>> --- a/drivers/of/overlay.c >>> +++ b/drivers/of/overlay.c >>> @@ -358,7 +358,7 @@ static int add_changeset_property(struct overlay_changeset *ovcs, >>> } >>> >>> if (!of_node_check_flag(target->np, OF_OVERLAY)) >>> - pr_err("WARNING: memory leak will occur if overlay removed, property: %pOF/%s\n", >>> + pr_warn("WARNING: memory leak will occur if overlay removed, >>> +property: %pOF/%s\n", >>> target->np, new_prop->name); >>> >>> if (ret) { >>> -- >>> 2.30.1 >>> >> >> NACK. This patch is incorrect. The reported memory leak is a bug, not a warning. >> >> I'll write up some information about why the memory leak occurs, then reply to this >> email with the additional info. > > The additional information is now available at: > > https://elinux.org/Device_Tree_Linux#Object_Lifetime I have now expanded the information at that link to content that is beyond the original topic. Those interested in devicetree memory object may find the additional info useful. > >> >> -Frank > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-01-10 18:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1671195782-23078-1-git-send-email-ankit.16.kumar@nokia.com>
2022-12-30 8:40 ` [PATCH] of: overlay: fix warning being reported as error in add_changeset_property Ankit 16. Kumar (Nokia)
2023-01-02 14:35 ` Frank Rowand
2023-01-10 6:23 ` Frank Rowand
2023-01-10 18:00 ` Frank Rowand
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).