linux-kernel - Re: [PATCH] selftests: default to host arch for LLVM builds

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: <b728014e-9e8f-4b65-8d9e-ace0e2f6f18c@nvidia.com>
Date: Sun, 28 Apr 2024 15:04:42 -0700
From: John Hubbard <jhubbard@...dia.com>
To: Valentin Obst <kernel@...entinobst.de>
Cc: anders.roxell@...aro.org, bpoirier@...dia.com, broonie@...nel.org,
 guillaume.tucker@...labora.com, linux-kernel@...r.kernel.org,
 linux-kselftest@...r.kernel.org, mpdesouza@...e.com, nathan@...nel.org,
 sashal@...nel.org, shuah@...nel.org
Subject: Re: [PATCH] selftests: default to host arch for LLVM builds

On 4/28/24 5:08 AM, Valentin Obst wrote:
..
>> And yes, the selftests "normal" (non-cross-compile) build is *broken*
>> right now, for clang. I didn't realize from the patch title that this is
>> actually a significant fix. Maybe we should change the subject line (patch
>> title) to something like:
>>
>>      [PATCH] selftests: fix the clang build: default to host arch for LLVM builds
> 
> Yes, I agree that the title should contain the word 'fix' somewhere. For
> me its okay if maintainers reword the title when applying the patch,
> alternatively I can send a v2. (Is it still a v2 if I change the title, or
> rather a new patch?).

It would still be a v2, although the cover letter, or the section after the
"---", would need to point to v1 so that people could make the connection.

> 
> Any thoughts on whether this also needs a 'Cc stable'? Its not quite
> clear to me if this fix meets the requirements. As above, no objections if
> maintainers should decide to add it.
> 

Maybe not, because it doesn't seem urgent. But it's a judgment call.

By the way, I've been chipping away at fixing clang selftest build
failures and warnings that are only visible after clang is working again
(due to your fix here), and I'm up to 30+ patches, and probably only a
few more to go to get all of them.

I'm expecting to post the series soon, hopefully this week. And I'm
thinking maybe I should carry your patch as the first one in the series,
in order to ensure it gets picked up. Or, I can just refer to it as a
prerequisite in the cover letter.

thanks,
-- 
John Hubbard
NVIDIA