[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/2] x86/cpuid: Infrastructure to support pseudo feature identifiers


  • To: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Wed, 5 Oct 2022 15:53:19 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.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=inVV5Ie3QYghmXNDUIr67TzbUUcoGvEHtnoUDnuNlpo=; b=R1hHuQ3VjO75RiOFb2SJG2VPoEaRGyEcyDbxk6R1WEqvKesO4CD92hgWgZB7f5zW0TNhV47TZZV18pdsVQMs/gselZtrj15W8nhd8bcsRIlc1HFNQi9OQ0i3AndLa28EY9ROM/OfQ1QzGiMCdh278uCf/ARycBd6kXnCOukXMlvYtAmbTsZ41yROPxiJ0mdUzhLAcBeW7JSM2mMhbOgUrJPyPLl+9JKYkACuEY8tuT0Ar2aY97NVmqZWA+OT90ohyCOIRcUCnycn24cH6ryKk4S6FusLtaBwaEVws4+qKQEF2IjhKqLQS1E+pvb42vLRnYUvPVuX1Hfcp7HsK0IQ7w==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=n4CsF6bYBWZ6u9L7SRiIeNVfTA5u5ZyxgBTNHf2AyXe2t0TJyTkFG2VQ3dS4Mfmx7DrsdddNSWB1S8bl/TPprFGiS1kGOKTt0U4Zpsyc52zOUyUttnrcJdEkScdo9kbfhv7LewV3Nnt9J3nMP6eFlLLn87f+AfqmUJlglVi5bbupi1nZo69EkrAC30jM+5uqxavaQ/f0FRv55TcYvxVbnKyGCuowzw5usojBhdyjb7UKK54s/FzKyiDlK5ZQk7EckHSgpJQsjIXFp3EBtgqBHiXFLQUSBFNoUOW/wzoeiYsGHu3oEsAmbVh6vmL6Wi6lDKnNfLa/502anCWagyqy2A==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Wei Liu <wl@xxxxxxx>, Henry Wang <Henry.Wang@xxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>, Demi Marie Obenour <demi@xxxxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Wed, 05 Oct 2022 13:53:41 +0000
  • Ironport-data: A9a23:aXdY2qiWUQBSpWl1tSSKyM3jX161ihEKZh0ujC45NGQN5FlHY01je htvCjvQMqnfZzP3KIpyO4SwpEhUv5HTytY3TgVo/i9hQS8b9cadCdqndUqhZCn6wu8v7q5Ex 55HNoSfdpBcolv0/ErF3m3J9CEkvU2wbuOgTrSCYkidfCc8IA85kxVvhuUltYBhhNm9Emult Mj75sbSIzdJ4RYtWo4vw//F+U4HUMja4mtC5AVmPa0T5TcyqlFOZH4hDfDpR5fHatE88t6SH 47r0Ly/92XFyBYhYvvNfmHTKxBirhb6ZGBiu1IOM0SQqkEqSh8ai87XAME0e0ZP4whlqvgqo Dl7WT5cfi9yVkHEsLx1vxC1iEiSN4UekFPMCSDXXcB+UyQq2pYjqhljJBheAGEWxgp4KUdM7 NYiJB1XVyGeiNmO8oi9afRz3f12eaEHPKtH0p1h5RfwKK98BLX8GeDN79Ie2yosjMdTG/qYf 9AedTdkcBXHZVtIJ0sTD5U92uyvgxETcRUB8A7T+fVxvTaVkFMZPLvFabI5fvSQQspYhACAr 3/u9GXlGBAKcteYzFJp91r83LeWw3yjAer+EpWfp9xOvVG27FcMIw85BUu/uf6YoVWXDoc3x 0s8v3BGQbIJ3FymSJzxUgO1pFaAvwUAQJxAHusi8gaPx6HIpQGDCQAsbBRMddgnv88eXiEx2 xmCmNaBLSNrmK2YTzSa7Lj8hRO/PzIEa1ALYyAsRBEApdLkpekbixvVS995Ha2dj9vrGCrxy TSHsCg/gboIiccBka68+Djvjz+pp7DISBA0/QTdWm+57gJ/a5Whbobu4l/ehcusN66cR1iF+ X0bwc6X6blUCYnXzHDSBuIQALuu+vCJdiXGhkJiFIUg8DLr/GO/eYdX43d1I0IB3ts4RAIFq XT74Wt5jKK/9lPwBUOrS+pd0/gX8JU=
  • Ironport-hdrordr: A9a23:t/DmeK1xp3m1a6QivB2A9AqjBdJxeYIsimQD101hICG9Lfb0qy n+pp4mPEHP4wr5OEtOpTlPAtjkfZr5z+8M3WB3B8bYYOCGghrQEGgG1+ffKlLbexEWmtQttp uINpIOcuEYbmIK8voSgjPIdOrIqePvmM7IuQ6d9QYKcegDUdAd0+4TMHf+LqQZfnglOXJvf6 Dsm/av6gDQD0g/X4CePD0oTuLDr9rEmNbPZgMHPQcu7E2jnC6l87nzFjmfx1M7XylUybkv3G DZm0ihj5/T+c2T+1v57Sv+/p5WkNzuxp9qA9GNsNEcLnHBmxulf4NoXpyFpXQQrPu04Fgnvd HQq1MLPth16VnWYmapyCGdkDXI4XIL0TvP2FWYiXzsrYjQQy87MdNIgcZ8fgHC40Qtkdlg2O YTtljp/6Z/PFflpmDQ9tLIXxZlmg6dpmcjq/caizh6XZEFYLFcgIQD9Ad+EYsGHgj99Ic7ed MeRf301bJzSxe3fnrZtm5gzJiFWWkyJA6PRgw4tsmcw1Ft7QVE5npd4PZasmYL9Zo7RZUBzf /DKL5UmLZHSdJTRb5hBc8aKPHHRFDlcFbpCia/MF7nHKYINzbmsJjs+og44+msZdgh0IYyop LcS1lV3FRCNH4GMff+nKGjzyq9A1lUBV/Wu4NjDtlCy/HBrYPQQGy+oAtEqbrknx0daverKc pbdqgmR8MLFlGeabqh7zeOJaW6FkNuIfH9muxLL25m8fi7XbHCh6j8TMv5AobLPHINZl7fa0 FzLwQbYv8wo3yWZg==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Wed, Oct 05, 2022 at 01:34:06PM +0000, Andrew Cooper wrote:
> On 05/10/2022 10:55, Roger Pau Monné wrote:
> > On Tue, Oct 04, 2022 at 05:08:09PM +0100, Andrew Cooper wrote:
> >> A future change will want a cpuid-like identifier which doesn't have a 
> >> mapping
> >> to a feature bit.
> > Why we prefer this logic over just giving such pseudo features a
> > synthetic bit or akin?
> 
> Synthetic bits are (intentionally) not available to cmdline parsing.  We
> need something that is available for parsing.

I think Jan expressed my view better, in that it would be nicer to
just have the MSR_ARCH_CAPS bits in the featureset, and listed in
cpufeatureset.h like we handle CPUID features.

Maybe we want to go your proposed route now if that's easier however.

Thanks, Roger.



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.