Received-SPF: pass (google.com: domain of linux-kernel+bounces-196560-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <b370276c-cbe7-4583-a906-dd0ef9f5afad@amd.com>
Date: Fri, 31 May 2024 14:49:40 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 1/1] x86/elf: Add a new .note section containing
 Xfeatures information to x86 core files
Content-Language: en-US
To: Borislav Petkov <bp@alien8.de>,
 "Balasubrmanian, Vignesh" <Vignesh.Balasubrmanian@amd.com>
Cc: Thomas Gleixner <tglx@linutronix.de>,
 "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
 "linux-toolchains@vger.kernel.org" <linux-toolchains@vger.kernel.org>,
 "mpe@ellerman.id.au" <mpe@ellerman.id.au>,
 "npiggin@gmail.com" <npiggin@gmail.com>,
 "christophe.leroy@csgroup.eu" <christophe.leroy@csgroup.eu>,
 "aneesh.kumar@kernel.org" <aneesh.kumar@kernel.org>,
 "naveen.n.rao@linux.ibm.com" <naveen.n.rao@linux.ibm.com>,
 "ebiederm@xmission.com" <ebiederm@xmission.com>,
 "keescook@chromium.org" <keescook@chromium.org>,
 "x86@kernel.org" <x86@kernel.org>,
 "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
 "linux-mm@kvack.org" <linux-mm@kvack.org>,
 "George, Jini Susan" <JiniSusan.George@amd.com>, "matz@suse.de"
 <matz@suse.de>, "binutils@sourceware.org" <binutils@sourceware.org>,
 "jhb@FreeBSD.org" <jhb@freebsd.org>,
 "felix.willgerodt@intel.com" <felix.willgerodt@intel.com>
References: <87wmo4o3r4.ffs@tglx>
 <4a090901-9705-40aa-ac3d-d67c52660f22@amd.com>
 <20240522153433.GCZk4QiX4Hf0OuI48E@fat_crate.local>
 <902b1bf0-15e6-42df-8f86-21387deef437@amd.com>
 <20240523144543.GDZk9WlwKpCKx8I3RE@fat_crate.local>
 <664e101b-3689-4876-825b-a5aa12b7978f@amd.com>
 <20240526090554.GAZlL7cpva88mMUbCK@fat_crate.local>
From: "Balasubrmanian, Vignesh" <vigbalas@amd.com>
In-Reply-To: <20240526090554.GAZlL7cpva88mMUbCK@fat_crate.local>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?L2Zoc0lkZUZ3bmdxUk0wVTVzTVptd1dNUnBIN2VUREVSa3UzYjNBRmFGTGFB?=
 =?utf-8?B?ODE5NnhqbGo2ditGMFIwU0dqYmdzL2JuYjZaMlJUQysrUFVFdEVEL01rRC85?=
 =?utf-8?B?ekRDeGxoUWplakNkUVR6ZlVJZ2NIZ1NmTDZnUWg3Y1dkNFNTY2dQQytWRENy?=
 =?utf-8?B?SDZGMHExK2RuTnFMbUpsdUlJQ2dpTTdmMmZ5dFVGT2YzYXgrSGtZNnFDbFV0?=
 =?utf-8?B?Rlk0NCs0SG82QVJ4RDFLQVdEK0kwa2pNZFJjNXpJMmRYTlR2dkFwWWw5d0po?=
 =?utf-8?B?c1JHeVgxOW14aGtPSWsrbmtrUUFKOTVLNlkvTEUzSXlGSFppOXR0WGExNy9m?=
 =?utf-8?B?QjlxZEtpYWRoem9vbVlnMDdHbXB1RXc0bElyNTJVV0FnL1J3V3ZYK1JqWnhv?=
 =?utf-8?B?a3FQRmhtVTF3V3llRXRmR3hOd0VMekRaalprSFc1ZmZYNm81M21CQ0hva2Vj?=
 =?utf-8?B?REFRWWlwT3VLMnFlSDNXSm1XeWVwZU4yZzBleEdCcUwwZzdHQVFCUFM2WUF1?=
 =?utf-8?B?aDBpMXNaZlh3dW5BSWZkdEFVVEJzbG5KaGl2RHpTMm43WFJnOTRzbVBtYStW?=
 =?utf-8?B?bVoxWHFMeGVrUEUycmRVWVlKbmhZcHZPblZnaExyMVJCMCtteWJDcmFpZ0Y2?=
 =?utf-8?B?Mml1WDVrR0h4MTFOeWZOQllaWXIrbDQ2LzFleWRHb2tBcGFWQlhOWCtNWm81?=
 =?utf-8?B?Q1NCMlZrN2ZNVGtGL0VDUEpmOVE5LzdHam5zRjZiK3F1WDcwK240Z3o5b3VN?=
 =?utf-8?B?M0dRS1ZHZ2JaSXVacTgrRTZnR1RHUG1FRSswS1NsOGRGdUFUY2x1RUtXS2dH?=
 =?utf-8?B?SXpranBQR2Zwa1l6YXIyV0dMRHRHUHA0Q3Q3NUc0UjVMQUZFazk1cVB3L2xW?=
 =?utf-8?B?UGFHelZNWkthTlVlbVpEaHV4Y3g2UUdzZXNNR2lYN09oWnA1aGNJTkpMNWQ1?=
 =?utf-8?B?NVZDcDhmYXczTDBmSkQ1Q1lXQXc5SG9LQ2JPRnBETlQvVnVjbjJVd1d6RVRx?=
 =?utf-8?B?aTJLNS9WSUhWd0g1UkRGaE5kT3JaWHNHL0dZOUFaSzdQb0FXL3ZkOEtsRDlK?=
 =?utf-8?B?OWhBRG9DOXRDVDRRNW5vS3VQcU0yd252dkppR3FFdzBUc09mSUVwQ0ZwNjFj?=
 =?utf-8?B?MUFkSm9aT0ZTWDFQSzN2MFVNTWZ4QzFDRy94YzAwOVpFTU4wcEcwZGczUWhq?=
 =?utf-8?B?SE85NS81dlNoOXE3ZDhhMkZzUTZ5U0Njeng0bzR1NTYya0Z0UVo4ZlNadC80?=
 =?utf-8?B?QlhoeXNWdURTS21VOUNSQy9kVncyM29HUHE1alJjZzlxcDJSUUJ4RERTNlFt?=
 =?utf-8?B?RlV2eWZBWDhDTzhWU3h0R2ljKzh0c0RoOGZ0c3ZUN2VVOUNDT2lHalVlZTk3?=
 =?utf-8?B?UU9EWlY3cGFyL0NWWS9UNi9LTVlHZzQvL0EybnIyRVR1akVudU8vVGVHR3R5?=
 =?utf-8?B?dUtYRmlpWkhBS1pRMFE4WW5lcUZYQVozd2ozSVhTclE3TWJmWW1qOEc5MXRX?=
 =?utf-8?B?OTkxSW1yekp5dk9SS1RUQURqaVNnTFh4YWZiclBxMXdSVTIxV0VRNTFWeWQ4?=
 =?utf-8?B?eFFrOVdTdXZEbjBvTW8rRUNqWlhaN1hFRi8yejRuS0YvL083RVBWMHl3Rm9j?=
 =?utf-8?B?WmU2ODB0WklFV3F2WnBZQU1MYlF2SVNQZjBkUWlQMC9XU29aYjdFaHNkTUho?=
 =?utf-8?B?YUdMMDhEOEQ4SWZMZHRVSXdSdVJXdmtWdWxsdzR4SXkvU2xaUGFYWlBVTVEv?=
 =?utf-8?B?UnJzSnFHUVBMV28zd1h2UFZIVXJ6S0dGb3VuUVk1azAzUjRHanBIcFNJeUNH?=
 =?utf-8?B?Nm1Sc05COHNBc2ZRN1FXa2w2azZTWSsybW1uQmR5WlVXMXF2eFB5SWEzVC9h?=
 =?utf-8?B?UCs1a3hyV20zT0prZEw4OGhBS25SL0Z0UEdOUWFPYm5CMGtLbHNWZFIwTnhy?=
 =?utf-8?B?U1hrNk03NWRVdGhUUGhQcVMvd2FBam9XUnc4N25xZzViaTRyK005NkdXenVa?=
 =?utf-8?B?dDZ4N05GMFpaelNrN2tmcE9QdlpNa055NU1LLzB2ZGxHZ3dsUUUrY3VuWWEy?=
 =?utf-8?B?SW9pTVlJcmE0cU9maDBKbW91T3A3R3ZPQ1c2N2k3YThhOXkxd0oyaFgySTl6?=
 =?utf-8?Q?ejWDowKxTkWiPy5IOPmETIEA5?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7698cde1-e339-42d0-683a-08dc8152d3fe
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR12MB6124.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 May 2024 09:19:52.7177
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +SnwaE6Ytpq5hT/oVKFH3VwjV1eU/fLZDWj8vU1HhkQ/Ap7vwlFW/VOs7ylZo4j1EwZmZBytd+99bjIvfzZOoQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6369


On 5/26/2024 2:35 PM, Borislav Petkov wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
>
>
> On Sun, May 26, 2024 at 10:24:41AM +0530, Balasubrmanian, Vignesh wrote:
>> If we can add a new enum only when we extend, then as Thomas suggested can
>> we use other kernel variables as in the first version of the patch until we
>> extend for other/new features?
> I assume by "other kernel variables" you mean CPUID?
>
> If so, can you change the layout of your buffer once you export it to
> userspace?
In a couple of the previous review mails
(https://lore.kernel.org/lkml/24f71d52-0891-4cfc-8dec-9f13ed618eee@intel.com/
and
https://lore.kernel.org/lkml/20240314162954.GAZfMmAnYQoRjRbRzc@fat_crate.local/),
it was suggested that the new .note should not use any internal definitions
like "xstate_sizes", "xstate_offsets" and ""xstate_flags" which are also the
direct output of cpuid instruction.

Also, the feature ID in .note records should be independent of the existing
XSAVE feature IDs (this was the comment as I understood). I defined the 
new enum
and mapping function to ensure that these remain independent of each other.

Thomas' comments on this version are that we should use existing variables
instead of re-evaluating cpuid. Also, to avoid the new enum and mapping
function which will make not any sense unless it is extended for
newer/different features.

That will be like our first version of the patch
https://lore.kernel.org/lkml/20240314112359.50713-2-vigbalas@amd.com/

So other than with the new enum (custom_feature) and the new mapping 
function
(get_sub_leaf), we are unsure as to how to maintain the layout to be
independent of x86' cpuid.

In the current version of the patch, the fields -- type, size, and offset
are derived from the cpuid instruction currently (and could be derived from
existing kernel variables in the future). The xsave flags are not used
currently, it can be zero(reserved) for now and its layout can be modified
(as per the need at that time) when the need arises.

If there are other ways to maintain the independence of the layout of a 
record
in .note section from the cpuid instruction other than depending on a 
new enum
and a new mapping function, we would be glad to follow it.

thanks
vigneshbalu
> --
> Regards/Gruss,
>      Boris.
>
> https://people.kernel.org/tglx/notes-about-netiquette