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

Re: [PATCH 1/3] x86/Intel: skip PLATFORM_INFO reads on family 0xf


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Fri, 11 Feb 2022 10:40:08 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; 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=A1f9r5ncU9SUlNKvw4uKoNawn8Lzqtw1MJjsoLb4tXk=; b=OY+EKWmYomycA+EqWNWOhonDuLdMj32BQHodAgK9Sm7M97DjeY8lS8IelR5WdlNNn7a9Ogplp2hJ95EwtwuG7ReMWrknQLHXbX8RuGbd0nV3jVfNlgz8nGS6BNetgrQs8x+eaLIL71xiV62lJrq3l30sBZV+CeQ8Ng136OLroNuLzGUvhVX1hHjl9sGRmc/dx8HGP4KpquwIFLKOd2BdZkpganeo1tGsOO2uK/i7iNPNMbIMl8m9T1L/FYMspYIz8/ttxZluWtAgM7ibpt5imwrN1fHXTVgVrQp8hTL7Bq75nlYWOgROwsbC2XOJN1mqr2SbrUEOUx/E5OQ7WN8+IQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hmUcBRtpTzjGre9rfiEVcFeOjNmf0L9UC5DizgAYlQ1jNG9ElxCI88q3R8797zqpXxoj1EkfNSJm3oBwcfMHwFDj3LWs2/4uYQJNI9e7HMRC5DH9RqmlFhvU7nW9yYE5gzdiSfR+XAGQXLaTimFt/tFWGQmO8cCgvJHJUmSgobSgiPgPC52tDPxhoLwNbis2kwwNRL3JgRWAWpW64Moi/gcdyapAMTJZaLu+Faub+87jqFZuGVlJ5omVl+Zbq+TjLblR65VNOnbLrJyWpJpr1CdTjiguMyxgt61PGcDSNm34R04aJq78gGAXGn06pqQ+MWG8MKkwvi/vMqpajJL2IQ==
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 11 Feb 2022 09:40:50 +0000
  • Ironport-data: A9a23:eQDfMKJUvBnrxoPGFE+RA5IlxSXFcZb7ZxGr2PjKsXjdYENShjUDm zcdWT2ObvbYamXyc9ElaYW18ExSvJWEmIQwHgZlqX01Q3x08seUXt7xwmUcns+xwm8vaGo9s q3yv/GZdJhcokcxIn5BC5C5xZVG/fjgqoHUVaiUakideSc+EH170Ug6wrZg6mJVqYPR7z2l6 IuaT/L3YDdJ6xYsWo7Dw/vewP/HlK2aVAIw5jTSV9gS1LPtvyB94KYkDbOwNxPFrrx8RYZWc QphIIaRpQs19z91Yj+sfy2SnkciGtY+NiDW4pZatjTLbrGvaUXe345iXMfwZ3u7hB2Jlepa+ utriqeREzt2Janogv44U0lHRnQW0a1uoNcrIFC6uM2XiUbHb2Ht07NlC0Re0Y8wo7gtRzsUr LpBdW5LPkvra+GemdpXTsF2gcsuNo/zNZ43sXB81zDJS/0hRPgvRo2UuYMAg2lp2aiiG97FY foieR1ONSjkRAdgExAsU4sRmcC30yyXnzpw9wvO+PtfD3Lo5BN1+KjgNpzSYNPibdVYmAOUq 3zL+0z9AwoGL5qPxDyd6HWui+TT2yThV+o6Fre16/pri1273XEIBVsdUl7TnBWiohfgAZQFc RVSo3dw6/hpnKC2cjXjdx2ymlW2gDgjYPsKL7Ih7CWX0I/UsxnMUwDoUQV9QNAhscY3Qxkj2 VmIg87lCFRTjVGFdZ6O3uzK9G3vYED5OUdHPHZZFlVdv7EPtalu1kqnczp1LEKiYjQZ8xnUy ivCkiUxjq57YSUjh/TipgCvb95BS/H0ou8JCuf/AzrNAuBRPtfNi2mUBb7zt6cowGGxFAfpg ZT8s5LChN3i9LnU/MB3fM0DHauy+9GOOyDGjFhkEvEJrmrxpyP5IN8PumwnfC+F1/ronxezM Sc/XisLuvdu0IaCN/crM+pd9ex2pUQfKTgVfq+NNYcfCnSAXASG4DtvdSatM5PFyyARfVUEE c7DK66EVC9CYYw+lWbeb7pNgNcDm3FlrUuOFM+T8vhS+efHDJJjYexeawXmgyFQxP7snTg5B P4Ba5XUm08CDIUToED/qOYuELzDFlBibbjeoM1LbO+TZA1gHWAqEfjKxr09PYdimsxoei3gp xlRg2dUlwjyg2PpMwKPZiwxYb/jR88n/3k6ITYtLRCj3H16OdSj66IWdp0We7g79bM8ka4oH qddI8jQUO5STjnn+igGacWvpoJVaxn21xmFODCoYWZjcsc4FRDJ4NLtYiDm6DIKUnisrcI7r rD5jlHbTJMPSh5MFsHTbP7znVq9sWJEwLB5XlfSI8kVc0LpqdA4Jyv0h/4xAscNNRScmWfKi 1fIWU8V/LCfrZU0/d/FgbG/g72oS+YuTFBHG2T77KqtMXWI9GSU3oIdAv2DeirQVT2o9fz6N /lV1fz1LNYOgE1O79hnC79uwK8zu4nvqrtdwlg2FXnHdQ32WLZpI33A1shTrKxdgLRevFLuC E6I/9BbP5SPOd/kTwFNdFZ0MLzb2KFGgCTW4NQ0PF7+tX1+87ewWElPOwWB1X5GJ7xvPYJ5m eostab6MeBkZsbG5jpesh1pyg==
  • Ironport-hdrordr: A9a23:KWYzl6OAIOR8nMBcTyP155DYdb4zR+YMi2TDiHofdfUFSKClfp 6V8cjztSWUtN4QMEtQ/uxoHJPwO080lKQFmrX5WI3NYOCIghrLEGgP1/qG/9SkIVyCygc/79 YfT0EdMqyIMbESt6+Ti2PZYrUdKZu8gdiVbI/lvghQpGpRGsddBmlCe2Km+hocfng7OXN1Lu vV2uN34x6bPVgHZMWyAXcIG8DFut3wjZrjJToLHQQu5gWihS6hrOeSKWnT4j4uFxd0hZsy+2 nMlAL0oo2lrvGA0xfZk0ve9Y5fltfNwsZKQOaMls8WADPxjRvAXvUrZ5Sy+BQO5M2/4lcjl9 fB5z8mIsRI8nvUOlq4pBP8sjOQpwoG2jvH8xu1kHHjqcv2SHYREMxan79UdRPf9g4JoMx8+L gj5RPXi7NnSTf72Ajt7dnBUB9n0mCup2A5rOIVh3tDFaMDdb5qq5AF9k89KuZNIMvD0vFnLA BSNrCd2B4PGmnqLEwx/1MfjeBEZ05DUCtvGSM5y46oOzs/pgEM86JX/r1bop46zuNMd3Bz3Z WwDk1ZrsA+ciYoV9MPOA4ge7rANoWfe2OEDIqtSW6XZp3vfUi976LK3A==
  • Ironport-sdr: nrYU3ccSSfdgGjEryl8uVvPQvkmDcp1AauUgeOk1nT6h9nmrPLjoAtVU766Pq7kV6r3I6cUlon 6E6uWwNX/CwBPideYwDF6pEfXUdLBbB8TTKPwDugZl5QQDh2ylo82nyHPWRZH8VPkSpO+cwSyo hf7l6qMkiM9kTZkz+2K6DQ46444STvQCE4uWpz3kbN2QsWshHsDtK5TDQUq3zyQAyrW4ucGdj+ AZyj2qWF2sY78rIiAC2DXwvbUXNLMOkv3YJtTKqRhMBhlN9BlH/Xc8TPzfmWc/zyYiuh3vhvwV LLM+QGHACKUkCtcDpuaQ/WKZ
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Thu, Feb 10, 2022 at 03:55:52PM +0100, Jan Beulich wrote:
> This avoids unnecessary (and always somewhat scary) log messages for the
> recovered from #GP(0).

Could we maybe get rid of the #GP messages for cases like this where we
are explicitly probing for MSR presence? (ie: it's expected that we
can get a #GP)

> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> ---
> Perhaps even use "!= 6" in at least the CPUID-faulting family check?

Likely, or else you would also need to check for family 11 (Knights
Corner?) which doesn't seem to support PLATFORM_INFO either.

Thanks, Roger.



 


Rackspace

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