Received-SPF: pass (google.com: domain of linux-kernel+bounces-189589-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <664e101b-3689-4876-825b-a5aa12b7978f@amd.com>
Date: Sun, 26 May 2024 10:24:41 +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>
From: "Balasubrmanian, Vignesh" <vigbalas@amd.com>
In-Reply-To: <20240523144543.GDZk9WlwKpCKx8I3RE@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?S2QvTWtOdFpXbzNWaHZpNEJvUlEzcWI1aUNvMzg0UHV1dWR2ZmRuY090NndD?=
 =?utf-8?B?Y0dwaDFpMEhndXFlMGlDbnQ1YXNaLzF5aHEveDgrUXZvbDlyMmg3cWhaRjVw?=
 =?utf-8?B?bXcyS1l6VG9uQkx0MjJySE9sRlovVGZBYmwxY0M0d3hiVzcxbFVadFYxMHd0?=
 =?utf-8?B?d2N4WURnQzhpdHNjSkZvL0RCc0g2dCtKMDQ0dDE4TmtiS1JoYmlWTnE4V1RT?=
 =?utf-8?B?MkZUK29mM3lRRU8xVnpLeXhLTDN4MTJ1Znc0NnlBMGpGdUlxME1ja1paT243?=
 =?utf-8?B?Sm1GZVdiY3g2dk9IdklIalVEOFZXbHFiUWdNY0ZlRE5YY2hwTVZuWXlzd25B?=
 =?utf-8?B?UHNiOGlzR2VqbkhBSCt4QkFrV0N2eEQxTzRwZ3hUclJEZlkzYjRTaVprVGtD?=
 =?utf-8?B?RkVKZkFnVmIya3V3TFVHZ1N2VjR6dFJTUEZwOE5vcXhoRWY5TS9QN09lT21m?=
 =?utf-8?B?d3ZlT3BwVFgybmg1RXk5eDVHOEprZk5ISVFXWUY4STVaSXc3U0tNTHM3SHhk?=
 =?utf-8?B?M2pOTjNBUVNXa0RBTmxyZ2NZTTFYdFFrZGJIbUkvbGZwUlpDK0QrVjUxeThV?=
 =?utf-8?B?UUNkakN2NWUrQkJKczdZMzNhYWhzTys0aTRvQjBEaGtlcXdEWkVwcGFIZ2ow?=
 =?utf-8?B?Q2VoNUd6OE5MZzZiZ2tJSlhzSk1zL0RZRGNoZk8wN3hrSEp1bEljV3kycW1u?=
 =?utf-8?B?T0c4Ui9abFV3TEVXa2krcmtPYnhYT1l2QWtPWlR6b1FJYUhNTURmdDNuNjVi?=
 =?utf-8?B?RFB6ZTZtUnVaRG9nZitBb0Q2V1ZIUEhUdW1yejJoQmZ2bHM5Q0h3RHRRNmRp?=
 =?utf-8?B?RzV0QlpBSExKWWp3S3p0aTFaWnRLdWduZjVZdklOcWJJVGovY2xMRCtkMXlC?=
 =?utf-8?B?L3kwbVpWREl1TUVkTk8yeGRkajlJRUg1UFN2cDlEbXpIREw4bU1mTDA0VG1i?=
 =?utf-8?B?WWNqQ1FLUjIwTzl2blRWNDFCU0UxbnlMTHZrWndwYzZ0ZWZnem9RcDBUYnh4?=
 =?utf-8?B?Qmpmbk5QNXZrYkZTZ2hEbTQwYXdUSEpKU0Q1WnUzNkVrY1VxRE5ZcW5BU1k0?=
 =?utf-8?B?akp2bFpBci9rWlNLZFhQaktLR3pjakRwckxSMllWTW91RlFvQU15ck84YUdj?=
 =?utf-8?B?aUhTNWR2L3ZVRGZSUW1JRVo0NlA1Q0llV0RMYk1PYXZQQlAzY3Noa0xBOFBj?=
 =?utf-8?B?eUVubWFiN3R0RE9LVGRaZ3piRlhWNXJNRTJwL2hyZ0phc0lxdW9DRUthQkJs?=
 =?utf-8?B?YUxSRllvOE0wNTlpeUN5SUxVMVNCcyt2L2l1RERubWUvNkZkSmMwdE5mMGh2?=
 =?utf-8?B?cmhZbUZ1bHc5cjdOWlowb2J1NzFIQWNWdnNCOENCMUFVZ1Z3dk8rQ0h0UnFT?=
 =?utf-8?B?QTBrME1SQmRoWmcwMkFzcW1BMjJUZk0xZDdITEFrVVhkWUNENFpIY003NXpz?=
 =?utf-8?B?L0x2L2FWZ0w0dkRibTdUS2pBMGxjbnZWU0NjS3l1RWVDMFdtVFJ0Tkc1ZHJ2?=
 =?utf-8?B?MU45TzArYjlEaWJMSnRuVXMrbGNGOU1OYnlNOHhIaWF3K3hDTUZHMnJvaG55?=
 =?utf-8?B?TnJJaXRleWJGbmhSMDJXdjZtRGhmMVpud3ViSDBDZk1GeXEyTXZBU04yWmRw?=
 =?utf-8?B?L2ZmdUlXTnZ4QW43S0JYYXI1MklQSTZGWWprT2lEQ1U2SnRlQmpOdkJicisv?=
 =?utf-8?B?OWpxdmlvM3lrTEh6Y3F0S1RwN1BHTFBMRWZVclpkT2g0ajF0U0lJcExZZnAr?=
 =?utf-8?B?MnVHTFk3R3A0YlNRV1kxc1hHUldnZFFrTndPSk1wc3BuKy9LcG1YZVRhSktD?=
 =?utf-8?B?OS9JendKb0hlUEhIajRzMDJaQnE5Sll1dm1wUmwyQjlnRkpwN2JvS3o4QXEx?=
 =?utf-8?B?Q3dHdlRpbzJ6ellrTmh3VW05MTNLWjMxRWFyUFJ0RU5pc0xxRlNlaEpCejh2?=
 =?utf-8?B?K2Qxak8vbjFwRmtIS1Ivd0FXVmZzV1U2bzVoT1VjcWFBRmF4ZEhEeWlRaCto?=
 =?utf-8?B?Qk5TUmh5K1hsVk1HM2t1Z1cwZS91d0Q2a09WbndNYW9nQ0taMytqWGcydDJX?=
 =?utf-8?B?UWFXNlRhUkVHZ25RVDMzM3BlZGhoNXk1S1ZyVlNZQ01jSTA5enRlcWJZOVBR?=
 =?utf-8?Q?VhFZawi4V7AtTSijl3rB6f9kq?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bcf36095-be1a-457e-d7ce-08dc7d400088
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR12MB6124.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 May 2024 04:55:02.3741
 (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: +z5/G3jBIEbDDl6NVnChv7J1/uYCVQx9l6z6DGv5+qbuPVIHuLDwCbra4ZZk70xxZyQeplLQ+mFowc2n2wQDow==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB7411


On 5/23/2024 8:15 PM, Borislav Petkov wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
>
>
> On Thu, May 23, 2024 at 11:57:00AM +0530, Balasubrmanian, Vignesh wrote:
>> Currently, this enum is the same as XSAVE, but when we add other features, this
>> enum might have a different value of the XSAVE features and can be modified
>> without disturbing the existing kernel code.
> We will do that when we cross that bridge, right?

I am struggling to interpret.
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?

thanks,
vigneshbalu.

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