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

Re: [PATCH] tools/guest: Fix comment regarding CPUID compatibility


  • To: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Fri, 4 Feb 2022 14:09:44 +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=BNAzDebrtrUti65iUmF01QOluoTqxRHy34ABZA/n8L8=; b=F3cWA/10Nduk6fVsYv6jWZ8uOr/7raveWSSFz3P3PlW9BUbTAc+ZhjuHGuaL2bQKZpjBnOLDtEl3mNGjA1BhjbPkugw5vLVoezVXhxik5LMmZC6n/NjJxrwoBTKcx+u8ZZagP+6FUT/XfFTAzYfbJE3XxIrkTn8jEYFtPYk2f8jRLFOlcNaRW82Tez0cs8z2XjHOqWCnHVjbEPQrQLJha7uz1gA0Hxn7A78g115tCFPktZmncOq5AILVDlKOVKkExU1xiuY6PmdZkOS12aD0zZ7C6pN+Uz5A4stGtTKfrfORj97pIilKlJkaYSvxfnD7mtkzkXtifIWA0O6J7vyj6w==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=O117QMWMloPIRVcFD8x+r8OCu75G8Xp3eSuVP6P2K72NLShhbMldIkTJ5H4sp7kxbB5AS+u+F5JQKZsxPCfSN82dPclHyVXVipRsjYZ/gtY41M2MxKtR7g82sa5nBYGiGZlL17Utn+R/h5KBlzXrW/mNUXgTwrxtnqAMc8t5F9NUPNlPlnWT+ehVOBJFJlHz0Wu5+UGjlltAAExeGp4fSlfB/uB7ps4FrJIKpYz0g9PzvvqrxVQAz1S5C+liSQCsgtcHMThpGDyDUDV/X4c+/8aTdgEqmgrUOLlCG7W/2Z0jXuvd0TEXNqkCKKWXuhHexg84APLA9CZbz54q8gAMZg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Wei Liu <wl@xxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>
  • Delivery-date: Fri, 04 Feb 2022 13:09:59 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 04.02.2022 13:12, Andrew Cooper wrote:
> On 04/02/2022 08:31, Jan Beulich wrote:
>> On 03.02.2022 19:10, Andrew Cooper wrote:
>>> It was Xen 4.14 where CPUID data was added to the migration stream, and 4.13
>>> that we need to worry about with regards to compatibility.  Xen 4.12 isn't
>>> relevant.
>>>
>>> Expand and correct the commentary.
>>>
>>> Fixes: 111c8c33a8a1 ("x86/cpuid: do not expand max leaves on restore")
>> But doesn't this commit amend 685e922d6f30 ("tools/libxc: Rework
>> xc_cpuid_apply_policy() to use {get,set}_cpu_policy()"), which is
>> where DEF_MAX_* disappeared?
> 
> No. All that happened in that change was that we switched to using
> 
> cpuid.h:89:#define CPUID_GUEST_NR_EXTD_AMD
> 
> instead, which remained the same size until Xen 4.15 when e9b4fe26364
> bumped it.

Oh, right. I did try to look for a replacement, but managed to miss
this. But then, as much as 4.12 isn't relevant, isn't it the case
that the fact that CPUID data was added to the stream in 4.14 isn't
relevant here either, and it's instead the bumping in 4.15 which is?
IOW while I've been fine with the comment adjustment anyway, there
would still want to be an adjustment to the description.

>> While looking at this, wasn't Roger's change incomplete, in that
>> for Intel the extended leaf upper bound was 0x80000008 in 4.12?
> 
> CPUID_GUEST_NR_EXTD_INTEL is still 8, so this is all fine.

Again, somehow I did manage to miss the replacement defines.

Jan




 


Rackspace

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