* [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
@ 2022-06-22 6:21 Liang He
2022-06-22 8:03 ` Marc Zyngier
2022-06-22 9:02 ` Krzysztof Kozlowski
0 siblings, 2 replies; 7+ messages in thread
From: Liang He @ 2022-06-22 6:21 UTC (permalink / raw)
To: tglx, maz, linux-kernel, windhl
We need to add missing of_node_get() for of_find_matching_node() to
keep refcount balance.
Note: of_find_matching_node() will decrease the refcount of its first
arg.
Signed-off-by: Liang He <windhl@126.com>
---
drivers/irqchip/irq-gic-v3-its.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index 5ff09de6c48f..829893cc5e10 100644
--- a/drivers/irqchip/irq-gic-v3-its.c
+++ b/drivers/irqchip/irq-gic-v3-its.c
@@ -5309,6 +5309,7 @@ static int __init its_of_probe(struct device_node *node)
* reset, don't even try to go any further, as this could
* result in something even worse.
*/
+ of_node_get(node);
for (np = of_find_matching_node(node, its_device_id); np;
np = of_find_matching_node(np, its_device_id)) {
int err;
@@ -5323,6 +5324,7 @@ static int __init its_of_probe(struct device_node *node)
return err;
}
+ of_node_get(node);
for (np = of_find_matching_node(node, its_device_id); np;
np = of_find_matching_node(np, its_device_id)) {
if (!of_device_is_available(np))
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 6:21 [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe() Liang He
@ 2022-06-22 8:03 ` Marc Zyngier
2022-06-22 8:15 ` Liang He
2022-06-22 9:02 ` Krzysztof Kozlowski
1 sibling, 1 reply; 7+ messages in thread
From: Marc Zyngier @ 2022-06-22 8:03 UTC (permalink / raw)
To: Liang He; +Cc: tglx, linux-kernel, Miaoqian Lin
+ Miaoqian Lin
On Wed, 22 Jun 2022 07:21:38 +0100,
Liang He <windhl@126.com> wrote:
>
> We need to add missing of_node_get() for of_find_matching_node() to
> keep refcount balance.
>
> Note: of_find_matching_node() will decrease the refcount of its first
> arg.
>
> Signed-off-by: Liang He <windhl@126.com>
[...]
Please send these refcount fixes as a series for all the irqchip
drivers, not one every other day.
And please synchronise with Miaoqian Lin, as you are obviously both
chasing the same bugs.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re:Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 8:03 ` Marc Zyngier
@ 2022-06-22 8:15 ` Liang He
2022-06-22 8:29 ` Marc Zyngier
0 siblings, 1 reply; 7+ messages in thread
From: Liang He @ 2022-06-22 8:15 UTC (permalink / raw)
To: Marc Zyngier; +Cc: tglx, linux-kernel, Miaoqian Lin
At 2022-06-22 16:03:26, "Marc Zyngier" <maz@kernel.org> wrote:
>+ Miaoqian Lin
>
>On Wed, 22 Jun 2022 07:21:38 +0100,
>Liang He <windhl@126.com> wrote:
>>
>> We need to add missing of_node_get() for of_find_matching_node() to
>> keep refcount balance.
>>
>> Note: of_find_matching_node() will decrease the refcount of its first
>> arg.
>>
>> Signed-off-by: Liang He <windhl@126.com>
>
>[...]
>
>Please send these refcount fixes as a series for all the irqchip
>drivers, not one every other day.
>
Sorry, I will group similar bugs in future.
>And please synchronise with Miaoqian Lin, as you are obviously both
>chasing the same bugs.
>
>Thanks,
>
> M.
Yes, before I send any new bug, I will confirm that this bug has not been reported in lore.kernel.org.
By the way, this is a bug missing |GET|, not |PUT|, which is different with Miaoqian's and my patches before.
>
>--
>Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 8:15 ` Liang He
@ 2022-06-22 8:29 ` Marc Zyngier
2022-06-22 8:34 ` Liang He
0 siblings, 1 reply; 7+ messages in thread
From: Marc Zyngier @ 2022-06-22 8:29 UTC (permalink / raw)
To: Liang He; +Cc: tglx, linux-kernel, Miaoqian Lin
On Wed, 22 Jun 2022 09:15:56 +0100,
"Liang He" <windhl@126.com> wrote:
>
>
>
> At 2022-06-22 16:03:26, "Marc Zyngier" <maz@kernel.org> wrote:
> >+ Miaoqian Lin
> >
> >On Wed, 22 Jun 2022 07:21:38 +0100,
> >Liang He <windhl@126.com> wrote:
> >>
> >> We need to add missing of_node_get() for of_find_matching_node() to
> >> keep refcount balance.
> >>
> >> Note: of_find_matching_node() will decrease the refcount of its first
> >> arg.
> >>
> >> Signed-off-by: Liang He <windhl@126.com>
> >
> >[...]
> >
> >Please send these refcount fixes as a series for all the irqchip
> >drivers, not one every other day.
> >
>
> Sorry, I will group similar bugs in future.
>
>
> >And please synchronise with Miaoqian Lin, as you are obviously both
> >chasing the same bugs.
> >
> >Thanks,
> >
> > M.
>
> Yes, before I send any new bug, I will confirm that this bug has not
> been reported in lore.kernel.org.
This is not what I meant. I am asking you to actively coordinate with
Miaoqian, working with them as a team instead of you both sending
competing series. I would like to see one series with patches from you
both.
> By the way, this is a bug missing |GET|, not |PUT|, which is
> different with Miaoqian's and my patches before.
Well, yet another reason for you both to work together, as you are
complementing each other.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re:Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 8:29 ` Marc Zyngier
@ 2022-06-22 8:34 ` Liang He
2022-06-22 9:39 ` Miaoqian Lin
0 siblings, 1 reply; 7+ messages in thread
From: Liang He @ 2022-06-22 8:34 UTC (permalink / raw)
To: Marc Zyngier, Miaoqian Lin; +Cc: tglx, linux-kernel
At 2022-06-22 16:29:49, "Marc Zyngier" <maz@kernel.org> wrote:
>On Wed, 22 Jun 2022 09:15:56 +0100,
>"Liang He" <windhl@126.com> wrote:
>>
>>
>>
>> At 2022-06-22 16:03:26, "Marc Zyngier" <maz@kernel.org> wrote:
>> >+ Miaoqian Lin
>> >
>> >On Wed, 22 Jun 2022 07:21:38 +0100,
>> >Liang He <windhl@126.com> wrote:
>> >>
>> >> We need to add missing of_node_get() for of_find_matching_node() to
>> >> keep refcount balance.
>> >>
>> >> Note: of_find_matching_node() will decrease the refcount of its first
>> >> arg.
>> >>
>> >> Signed-off-by: Liang He <windhl@126.com>
>> >
>> >[...]
>> >
>> >Please send these refcount fixes as a series for all the irqchip
>> >drivers, not one every other day.
>> >
>>
>> Sorry, I will group similar bugs in future.
>>
>>
>> >And please synchronise with Miaoqian Lin, as you are obviously both
>> >chasing the same bugs.
>> >
>> >Thanks,
>> >
>> > M.
>>
>> Yes, before I send any new bug, I will confirm that this bug has not
>> been reported in lore.kernel.org.
>
>This is not what I meant. I am asking you to actively coordinate with
>Miaoqian, working with them as a team instead of you both sending
>competing series. I would like to see one series with patches from you
>both.
OK, I get it. This depends on Miaoqian.
@linm006@gmail.com
>
>> By the way, this is a bug missing |GET|, not |PUT|, which is
>> different with Miaoqian's and my patches before.
>
>Well, yet another reason for you both to work together, as you are
>complementing each other.
>
>Thanks,
>
> M.
>
>--
>Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 6:21 [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe() Liang He
2022-06-22 8:03 ` Marc Zyngier
@ 2022-06-22 9:02 ` Krzysztof Kozlowski
1 sibling, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-22 9:02 UTC (permalink / raw)
To: Liang He, tglx, maz, linux-kernel
On 22/06/2022 08:21, Liang He wrote:
> We need to add missing of_node_get() for of_find_matching_node() to
> keep refcount balance.
>
> Note: of_find_matching_node() will decrease the refcount of its first
> arg.
>
> Signed-off-by: Liang He <windhl@126.com>
> ---
> drivers/irqchip/irq-gic-v3-its.c | 2 ++
> 1 file changed, 2 insertions(+)
Before applying the patch please check it carefully. Previous evidence
[1][2] suggests that not it was not even compiled.
[1] https://lore.kernel.org/all/202206221602.odN70SHs-lkp@intel.com/
[2]
https://lore.kernel.org/all/16f9a971.44e5.1817068ee3c.Coremail.windhl@126.com/
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe()
2022-06-22 8:34 ` Liang He
@ 2022-06-22 9:39 ` Miaoqian Lin
0 siblings, 0 replies; 7+ messages in thread
From: Miaoqian Lin @ 2022-06-22 9:39 UTC (permalink / raw)
To: Liang He, Marc Zyngier; +Cc: tglx, linux-kernel
Hi,
On 2022/6/22 16:34, Liang He wrote:
>
>
> At 2022-06-22 16:29:49, "Marc Zyngier" <maz@kernel.org> wrote:
>> On Wed, 22 Jun 2022 09:15:56 +0100,
>> "Liang He" <windhl@126.com> wrote:
>>>
>>>
>>> At 2022-06-22 16:03:26, "Marc Zyngier" <maz@kernel.org> wrote:
>>>> + Miaoqian Lin
>>>>
>>>> On Wed, 22 Jun 2022 07:21:38 +0100,
>>>> Liang He <windhl@126.com> wrote:
>>>>> We need to add missing of_node_get() for of_find_matching_node() to
>>>>> keep refcount balance.
>>>>>
>>>>> Note: of_find_matching_node() will decrease the refcount of its first
>>>>> arg.
>>>>>
>>>>> Signed-off-by: Liang He <windhl@126.com>
>>>> [...]
>>>>
>>>> Please send these refcount fixes as a series for all the irqchip
>>>> drivers, not one every other day.
>>>>
>>> Sorry, I will group similar bugs in future.
>>>
>>>
>>>> And please synchronise with Miaoqian Lin, as you are obviously both
>>>> chasing the same bugs.
>>>>
>>>> Thanks,
>>>>
>>>> M.
>>> Yes, before I send any new bug, I will confirm that this bug has not
>>> been reported in lore.kernel.org.
>> This is not what I meant. I am asking you to actively coordinate with
>> Miaoqian, working with them as a team instead of you both sending
>> competing series. I would like to see one series with patches from you
>> both.
> OK, I get it. This depends on Miaoqian.
Sure, I would like to work with Liang to fix these bugs.
>
> @linm006@gmail.com
>
>>> By the way, this is a bug missing |GET|, not |PUT|, which is
>>> different with Miaoqian's and my patches before.
It's a nice catch and I haven't noticed before.
>> Well, yet another reason for you both to work together, as you are
>> complementing each other.
>>
>> Thanks,
>>
>> M.
>>
>> --
>> Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-06-22 9:39 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-22 6:21 [PATCH] irqchip: Add missing of_ndoe_get() in its_of_probe() Liang He
2022-06-22 8:03 ` Marc Zyngier
2022-06-22 8:15 ` Liang He
2022-06-22 8:29 ` Marc Zyngier
2022-06-22 8:34 ` Liang He
2022-06-22 9:39 ` Miaoqian Lin
2022-06-22 9:02 ` Krzysztof Kozlowski
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.