netdev - Re: [PATCH net v6] net: sched: cls_u32: Fix u32's systematic failure to free IDR entries for hnodes.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <67ac8505-a08b-40b1-89ec-18d5b4e255bb@orange.com>
Date: Tue, 12 Nov 2024 14:48:09 +0100
From: Alexandre Ferrieux <alexandre.ferrieux@...il.com>
To: Jamal Hadi Salim <jhs@...atatu.com>, Jakub Kicinski <kuba@...nel.org>,
 Simon Horman <horms@...nel.org>
Cc: Alexandre Ferrieux <alexandre.ferrieux@...il.com>,
 Eric Dumazet <edumazet@...gle.com>, Cong Wang <xiyou.wangcong@...il.com>,
 Jiri Pirko <jiri@...nulli.us>,
 Linux Kernel Network Developers <netdev@...r.kernel.org>
Subject: Re: [PATCH net v6] net: sched: cls_u32: Fix u32's systematic failure
 to free IDR entries for hnodes.

On 12/11/2024 13:23, Jamal Hadi Salim wrote:
> On Mon, Nov 11, 2024 at 1:26 PM Jakub Kicinski <kuba@...nel.org> wrote:
>>
>> On Sat, 9 Nov 2024 07:50:53 -0500 Jamal Hadi Salim wrote:
>> > Please split the test into a separate patch targeting net-next.
>>
>> Separate patch - okay, but why are you asking people to send the tests
>> to net-next? These sort of requests lead people to try to run
>> linux-next tests on stable trees.
> 
> AFAIK, those are the rules.
> The test case is not a fix therefore cant go to -net.
> You wait until the fix shows up in net-next then you push the test
> case into net-next.
> I should have clarified to Alexandre the "wait until the fix shows up
> in net-next" part.

OK, thank you fro the clarification, will do that :)

The v7 I posted includes all the remarks received so far:
 - static [inline]
 - test split apart
 - u32 type
 - Fixes: with proper commitid
 - Acked-by you ;)

Please tell me about any remaining issue.