* Re: [PATCH v5 00/14] irqchip: Fix potential resource leaks
[not found] ` <02e077df-7c4e-24a7-1640-5f17894bd252@loongson.cn>
@ 2020-10-12 2:31 ` Tiezhu Yang
0 siblings, 0 replies; only message in thread
From: Tiezhu Yang @ 2020-10-12 2:31 UTC (permalink / raw)
To: Marc Zyngier
Cc: Thomas Gleixner, Jason Cooper, linux-kernel, Alban Bedel,
Guo Ren, Mark Rutland, Sekhar Nori, Bartosz Golaszewski,
David Lechner, Baruch Siach, Jisheng Zhang,
Sebastian Hesselbarth, Jiaxun Yang, Alexandre Belloni,
Arnd Bergmann, Grant Likely, u.kleine-koenig, Catalin Marinas,
Tony Lindgren, Felipe Balbi, Paul Walmsley, Palmer Dabbelt,
Albert Ou, Anup Patel, Kukjin Kim, Krzysztof Kozlowski,
Heiko Stuebner, Rob Herring, Michal Simek,
Zubair Lutfullah Kakakhel, linux-csky, linux-arm-kernel,
linux-omap, linux-riscv, linux-samsung-soc
On 09/02/2020 11:59 AM, Tiezhu Yang wrote:
> On 07/06/2020 03:30 PM, Marc Zyngier wrote:
>> On 2020-07-06 02:19, Tiezhu Yang wrote:
>>> When I test the irqchip code of Loongson, I read the related code of
>>> other
>>> chips in drivers/irqchip and I find some potential resource leaks in
>>> the
>>> error path, I think it is better to fix them.
>>>
>>> v2:
>>> - Split the first patch into a new patch series which
>>> includes small patches and add "Fixes" tag
>>> - Use "goto" label to handle error path in some patches
>>>
>>> v3:
>>> - Add missed variable "ret" in the patch #5 and #13
>>>
>>> v4:
>>> - Modify the commit message of each patch suggested by Markus Elfring
>>> - Make "irq_domain_remove(root_domain)" under CONFIG_SMP in patch #3
>>> - Add a return statement before goto label in patch #4
>>>
>>> v5:
>>> - Modify the commit messages and do some code cleanups
>>
>> Please stop replying to Markus Elfring, and give people who actually
>> care a chance to review this code. Elfring will keep asking you to make
>> absolutely pointless changes until you are blue in the face
>
> Hi Marc,
>
> Any comments?
> Could you please apply this patch series?
Hi all,
Maybe I should cc the related persons through ./scripts/get_maintainer.pl
to get Acked-by or Reviewed-by.
The cover letter link of this patch series is:
[v5,00/14] irqchip: Fix potential resource leaks
https://lore.kernel.org/patchwork/cover/1268043/
Any comments will be much appreciated.
Thanks,
Tiezhu
>
> Thanks,
> Tiezhu
>
>>
>>
>> Thanks,
>>
>> M.
>
^ permalink raw reply [flat|nested] only message in thread