netdev - Re: [PATCH v8 net-next 2/5] cache: enforce cache groups

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 for Android: free password hash cracker in your pocket

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

Message-ID: <CANn89iLDSUKBr_trZFgkFPfCGXvTRPcaaZvh0tZd=DE4RHJ9rA@mail.gmail.com>
Date: Thu, 30 Nov 2023 11:40:43 +0100
From: Eric Dumazet <edumazet@...gle.com>
To: Coco Li <lixiaoyan@...gle.com>
Cc: Jakub Kicinski <kuba@...nel.org>, Neal Cardwell <ncardwell@...gle.com>, 
	Mubashir Adnan Qureshi <mubashirq@...gle.com>, Paolo Abeni <pabeni@...hat.com>, Andrew Lunn <andrew@...n.ch>, 
	Jonathan Corbet <corbet@....net>, David Ahern <dsahern@...nel.org>, 
	Daniel Borkmann <daniel@...earbox.net>, netdev@...r.kernel.org, Chao Wu <wwchao@...gle.com>, 
	Wei Wang <weiwan@...gle.com>, Pradeep Nemavat <pnemavat@...gle.com>
Subject: Re: [PATCH v8 net-next 2/5] cache: enforce cache groups

On Wed, Nov 29, 2023 at 8:28 AM Coco Li <lixiaoyan@...gle.com> wrote:
>
> Set up build time warnings to safeguard against future header changes of
> organized structs.
>
> Warning includes:
>
> 1) whether all variables are still in the same cache group
> 2) whether all the cache groups have the sum of the members size (in the
>    maximum condition, including all members defined in configs)
>
> The __cache_group* variables are ignored in kernel-doc check in the
> various header files they appear in to enforce the cache groups.
>
> Suggested-by: Daniel Borkmann <daniel@...earbox.net>
> Acked-by: Daniel Borkmann <daniel@...earbox.net>
> Signed-off-by: Coco Li <lixiaoyan@...gle.com>
> ---
>  include/linux/cache.h | 25 +++++++++++++++++++++++++
>  scripts/kernel-doc    |  5 +++++
>  2 files changed, 30 insertions(+)
>

Reviewed-by: Eric Dumazet <edumazet@...gle.com>