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

Re: [PATCH] libx86: Introduce x86_cpu_policy_calculate_compatible() with MSR_ARCH_CAPS handling


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Wed, 5 May 2021 16:18:10 +0100
  • 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-SenderADCheck; bh=BhXJwv1YbiixRQMPu9WCfwdzGSKbSp4J4vdDjkuUw2E=; b=j2K6+YcTIRolZmEHzfoOvOAxq2cTv/ezS8peR8HXIbSp+5SIbNBns4wpzkESMfaYcF56K/KK69IlMG9rQwglvjrvcNaJc5R1K/TH4x/RumJx5ynFrVS9lbVd6T5oH68y/9xqfeuzooo03kDz2FUMMX9q4CLoO9GH6s71oQgTzHQrBKlbXI7tVcDeZolbHdnDXX7ADb8c3zXen48Ss6QAUZ8ZNLja7xqEmCe0/xNVneXhbjc74zzYaVzOrd701+hg0BnYecDlI4AitmwbQaqEdMc9lWpQ9+HwDnRjJlMQBqTfnfxGIkYSMqn7DC1O4SdhUyabv3UltXC4Q7pxzE85/g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jUJn913lFvv354V/PP2Vpd5UCxTpk6omQEohxdfBSjT/Av7YsuUxbG9AKiv9hM1w3QFdFgWDHAp3I0HApikXDzN1MzdNQW0taYsuSqmoAYBIsEgdQFMBy9itN8YbYPgqK7VvtaTFybChpBQaRzPmLnnNn+lM8HaMk/xQqwB7yCwMaJG4b5FgTh2avAIktd4oDyitk+YF/idWjLOJ3OYh1fY8/vjTmSe/1Ns4pRuPa5MZiemaM0D/89YYM/9JhkLaBmRzqO0KSDAKD6h1YjWrkLWGEjqDKDHJQvJ8Pn9QO0jQC7GvAGtyxavyTcQCsXQ5EhbJ99pnZOi6C5DTUd1JMw==
  • Authentication-results: esa5.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Wed, 05 May 2021 15:18:27 +0000
  • Ironport-hdrordr: A9a23:GFwEHa7ns+vb3MaJTAPXwRKBI+orLtY04lQ7vn1ZYQBJc8Ceis CllOka0xixszoKRHQ8g7m7VZWoa3Xa6JJz/M0tLa6vNTOW3VeAAaNDyc/ZwzPmEzDj7eI178 1dWoV3FdGYNzdHpOng5g3QKbgd6f2m1IztuuvE1XdqSmhRGsNdxiN0EBySHEEzZCQuP/oEPa GR7MZGuDasEE5/BqiGL0IIQvTZoJnznI/mCCRsOzcc9AKMgTm0gYSKcCSw4xF2aVJy6IZny0 fpuUjT5qKvs/a0oyWsrVP73tBtt/bKjvdGDMyIoMAJJjvrkRbAXvUdZ5Sy+AobjcvqxFE2kM TdgxpIBbUO11rhOlubjDGo+w783C0g43XvoGXo/kfLkIjCax8RT+9i7LgpFifx2g4bk/xXlJ 9v5Sa/saFaCBvR9R6Nn+TgZlVRuWef5UY5nfV7tQ05baIuLIV/gKY4501vHJIJDEvBmfsaOd grNs3a6fpMGGnqCUzxjy1qyNyoaHw5Ag2LdEgEotCUyDhbhhlCvjIl+PA=
  • Ironport-sdr: d1I0m5LSx96YFbIfhV7YaHnv8xSX4B8oZHP9+ifGwT6DRF5tIT/jKfkV/tQfsPHwj5ceVQKF9g 4WkhleZRoYsSULJTrakBBBMatK3wGcYHnQCii0X0JGjwMxYKe8hiyNO7px3kuau3qrTBwajIqh TwLckxlwOPmYgXMw2VGLvtKqcZOgP0/KqRzzyGNGseVI5hjSam56XE01d8dE6Wj4oX96AtomQZ 44qazqBczz8B+tQ2FaQMlAxd9Hq1TAHvS91ueUo42jmDQ/o06jcJY2rn/43AF5PWeCUUSbkzTn 160=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 05/05/2021 16:00, Jan Beulich wrote:
> On 05.05.2021 16:50, Andrew Cooper wrote:
>> On 05/05/2021 15:48, Jan Beulich wrote:
>>> On 05.05.2021 16:29, Andrew Cooper wrote:
>>>> Technically, MCXSR_MASK is also a hard blocker to migration, but we
>>>> don't even have that data in a consumable form, and we just might be
>>>> extremely lucky and discover that it is restricted to non-64-bit CPUs.
>>> "it" being what here? The value's presence / absence in an {F,}XSAVE
>>> image? Or the precise value of it?
>> The precise value of it.
> Not sure whether DAZ is new enough, but relatively sure MM isn't.
>
>>   Migrating across the boundary where the
>> default changed will cause {F,}RSTOR instructions to #GP.
> That's understood. Is there actually anything standing in the way
> of treating MXCSR_MASK like yet another feature flag group?

Well - we'd need to find somewhere to put it.  It doesn't fit in
architectural CPUID or MSR information.

We could add a 3rd category of information in "a cpu policy", and that
is always an option.

However, this issue hasn't been a problem so far, is only for very
legacy CPUs at this point, and isn't high on my list of worries.

~Andrew




 


Rackspace

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