netdev - Re: [PATCH 8/9] mlx5: stop warning for 64KB pages

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: <24a4f667-6eef-484d-8f9c-129d3ddf70a1@app.fastmail.com>
Date: Thu, 28 Mar 2024 23:21:14 +0100
From: "Arnd Bergmann" <arnd@...db.de>
To: "Justin Stitt" <justinstitt@...gle.com>, "Arnd Bergmann" <arnd@...nel.org>
Cc: linux-kernel@...r.kernel.org, "Saeed Mahameed" <saeedm@...dia.com>,
 "Leon Romanovsky" <leon@...nel.org>,
 "David S . Miller" <davem@...emloft.net>,
 "Eric Dumazet" <edumazet@...gle.com>, "Jakub Kicinski" <kuba@...nel.org>,
 "Paolo Abeni" <pabeni@...hat.com>, "Nathan Chancellor" <nathan@...nel.org>,
 "Jonathan Lemon" <jonathan.lemon@...il.com>,
 "Maxim Mikityanskiy" <maxtram95@...il.com>,
 "Daniel Borkmann" <daniel@...earbox.net>,
 "Nick Desaulniers" <ndesaulniers@...gle.com>,
 "Bill Wendling" <morbo@...gle.com>, "Tariq Toukan" <tariqt@...dia.com>,
 "Gal Pressman" <gal@...dia.com>, Netdev <netdev@...r.kernel.org>,
 linux-rdma@...r.kernel.org, llvm@...ts.linux.dev
Subject: Re: [PATCH 8/9] mlx5: stop warning for 64KB pages

On Thu, Mar 28, 2024, at 23:09, Justin Stitt wrote:
> On Thu, Mar 28, 2024 at 7:32 AM Arnd Bergmann <arnd@...nel.org> wrote:
>>
>> From: Arnd Bergmann <arnd@...db.de>
>>
>> When building with 64KB pages, clang points out that xsk->chunk_size
>> can never be PAGE_SIZE:
>
> This is under W=1 right? Otherwise this is a mighty annoying warning.

At the moment yes. I'm fairly sure that I've covered all the
common cases with thousands of randconfig builds, so we should
be able to make it the default when this series is fully merged.

     Arnd