[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] libxl/CPUID: drop two more feature flag table entries
- To: Anthony PERARD <anthony.perard@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Wed, 23 Aug 2023 15:53:18 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=1U38fg8lBM8azfpzS7TxC4bRSqxC3VNBO7YVE0qdvwU=; b=E0w1PHLQ5LLu81GYr1cPcrmpvSvBGmcey+JNg0kerBHsCBh+yp6f7xnhdbjBRJ42KCNoBzmHHk7eyNA9BDqZCbk0VKdSsjkpltf69eHDQ2yKSIMEIXNrW90FwG0JupiWyC6BBssM9775SIRonfpE8nQ8xmYt0JA4plCBXgq2L3b1ozfoSaIME4Vl1PSagko+fxnsnguZr4nwAP5kMRq4jfz19q7WIjuwXcCvXIpb0u/YpcusJIlhb8BqMcYDxwjaG2WYj8NKISHhHgyyo7dhWkQ0Ye92DKGJlsdeLQOC+0dXsc7I8hL/LXRHjVV9Szdnuhwb5mPFXq/RXyU05QikAg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HqDOD82FTeBW8qDageJ4VX71Dgr46X043stpng7/C4wKOmlAaP0iH4qASfhxCRyLe64UGW5ngoMtWpcKj5tvriSixYqMP61fHNeZ7rwnPYPqbcNS6nxcEM1HV4n1IXFsrGaI6e51QT+QTiTWW34BuvWur/IKZpEOcA1ek6TNQ9J1ZmbnwDfCXQSxWnVyD6CmpdauznTuQ0QPxyrwIfZp1M3eOxiGXwkwkvvZiRs5cbV/GOwLfVNmuybfJPMriPrZNP/c+VzyMZDhYskqYSI8qGnKIKqAsAoptgIsUGLg5/rDSX1CgoNu3ESD05i6kWJNsrT06OyUDMc/MucZBLcgvA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Delivery-date: Wed, 23 Aug 2023 13:53:27 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 23.08.2023 15:45, Anthony PERARD wrote:
> On Wed, Aug 23, 2023 at 09:21:26AM +0200, Jan Beulich wrote:
>> Two entries were left in place by d638fe233cb3 ("libxl: use the cpuid
>> feature names from cpufeatureset.h"), despite matching the generated
>> names.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>
> Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
Thanks.
>> ---
>> Permitting "psn", "ia64, "cntxid", and "perfctr_*" when the hypervisor
>> doesn't even know of the bits (perhaps wrongly so) is kind of odd as
>> well. Permitting bits like "est", which the hypervisor knows of but
>> doesn't expose to guests, is also questionable.
>
> I think those are just aliases, to a specific bit in a bitmap. Even if
> we remove those aliases, it is still possible to instruct libxl to do
> something with those bits. So there presence isn't permission, I don't
> think.
It's not permission, but recognizing the field name when its use then
(in the best case) doesn't do anything is at least misleading.
> Looks like "est" is just an alias for "eist", so it doesn't seems useful
> to remove it either.
Well, I'm effectively also questioning the exposure of "eist". Using
INIT_FEATURE_NAMES here was likely okay as a quick first approach, but
I think we want to limit what is recognized to what actually is useful
in at least some cases (yet better would of course be to also not
recognize e.g. HVM-only options when we're dealing with a PV guest).
Jan
|