On 5/16/24 09:09, Mark Brown wrote:
On Thu, May 16, 2024 at 08:53:52AM -0600, Shuah Khan wrote:
On 5/16/24 08:02, Mark Brown wrote:
I'm seeing quite a lot of breakage in mainline as a result of
daef47b89efd0b7 ("selftests: Compile kselftest headers with
-D_GNU_SOURCE") and daef47b89efd0 ("selftests: Compile kselftest headers
with -D_GNU_SOURCE") - thus far I've found that the use of
static_assert() is triggering build breaks where testsuites aren't
picking up the addition of _GNU_SOURCE (including stopping installing
the other tests in the same directory), and there's a bunch of tests
which #define _GNU_SOURCE in their code and now trigger build warnings.
I'm looking at fixes and mitigations now.
Would it be better to revert this for now and get this for now? I wouldn't
want you to extra busy work to workaround this.
I determined the best option now is reverting the 3 patches.
commit daef47b89efd0b745e8478d69a3ad724bd8b4dc6
"selftests: Compile kselftest headers with -D_GNU_SOURCE"
"selftests/sgx: Include KHDR_INCLUDES in Makefile"
commit 2c3b8f8f37c6c0c926d584cf4158db95e62b960c
"selftests/cgroup: Drop define _GNU_SOURCE"
commit c1457d9aad5ee2feafcf85aa9a58ab50500159d2
I have this in linux-kselftest next now and will send pull
request to Linus this weekend.
Tejun, I made sure cgroup compiles after reverting the framework
patch and the cgroup test patch.
Mark,
Thank you Mark for finding the problem and sending in mitigation
fixes. I will hold off on your alsa patch:
kselftest/alsa: Ensure _GNU_SOURCE is defined
https://patchwork.kernel.org/project/linux-kselftest/patch/20240516-kselftest-fix-gnu-source-v1-1-e482ca6bfff7@xxxxxxxxxx/
There is no need to for the following patch
kselftest: Desecalate reporting of missing _GNU_SOURCE
https://patchwork.kernel.org/project/linux-kselftest/patch/20240516-kselftest-mitigate-gnu-source-v1-1-a0e814ff2874@xxxxxxxxxx/
thanks,
-- Shuah
