Re: [PATCH 1/1] x86/elf: Add a new .note section containing Xfeatures information to x86 core files

To: Michael Ellerman <mpe@xxxxxxxxxxxxxx>, "Balasubrmanian, Vignesh" <Vignesh.Balasubrmanian@xxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "linux-toolchains@xxxxxxxxxxxxxxx" <linux-toolchains@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH 1/1] x86/elf: Add a new .note section containing Xfeatures information to x86 core files
From: "Balasubrmanian, Vignesh" <vigbalas@xxxxxxx>
Date: Tue, 26 Mar 2024 15:39:30 +0530
Cc: "npiggin@xxxxxxxxx" <npiggin@xxxxxxxxx>, "christophe.leroy@xxxxxxxxxx" <christophe.leroy@xxxxxxxxxx>, "aneesh.kumar@xxxxxxxxxx" <aneesh.kumar@xxxxxxxxxx>, "naveen.n.rao@xxxxxxxxxxxxx" <naveen.n.rao@xxxxxxxxxxxxx>, "ebiederm@xxxxxxxxxxxx" <ebiederm@xxxxxxxxxxxx>, "keescook@xxxxxxxxxxxx" <keescook@xxxxxxxxxxxx>, "x86@xxxxxxxxxx" <x86@xxxxxxxxxx>, "linuxppc-dev@xxxxxxxxxxxxxxxx" <linuxppc-dev@xxxxxxxxxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, "Petkov, Borislav" <Borislav.Petkov@xxxxxxx>, "George, Jini Susan" <JiniSusan.George@xxxxxxx>, "matz@xxxxxxx" <matz@xxxxxxx>, "binutils@xxxxxxxxxxxxxx" <binutils@xxxxxxxxxxxxxx>, "jhb@xxxxxxxxxxx" <jhb@xxxxxxxxxxx>, "felix.willgerodt@xxxxxxxxx" <felix.willgerodt@xxxxxxxxx>
In-reply-to: <87o7bg31jd.fsf@mail.lhotse>
References: <20240314112359.50713-1-vigbalas@amd.com> <20240314112359.50713-2-vigbalas@amd.com> <87o7bg31jd.fsf@mail.lhotse>
User-agent: Mozilla Thunderbird

IMHO you should split the changes to replace ARCH_HAVE_EXTRA_ELF_NOTES
with CONFIG_ARCH_HAVE_EXTRA_ELF_NOTES into a lead-up patch.

cheers

Thanks for the input and i will take care in next version.

regards,
vigneshbalu.

References:
- [PATCH 0/1] Add XSAVE layout description to Core files for debuggers to support varying XSAVE layouts
  - From: Vignesh Balasubramanian
- [PATCH 1/1] x86/elf: Add a new .note section containing Xfeatures information to x86 core files
  - From: Vignesh Balasubramanian
- Re: [PATCH 1/1] x86/elf: Add a new .note section containing Xfeatures information to x86 core files
  - From: Michael Ellerman