[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/4] x86: Introduce MSR_UNHANDLED
- To: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- From: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
- Date: Fri, 19 Feb 2021 09:56:32 -0500
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.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=bRxSbWplBavRs1LQ1gzX3mYcuKe7una/4Vqo3LzU7p0=; b=Xzg7ubfP0IDg8rQxMz+fIOPkLQPc0scxX9DnmkbPFq/aQYV12BAkLZjPi9zyhH5OClQZ+CYsA76a5V/lrf0UIOLXc7fzsKJ12syfuvdSAHMOlv3tLmhdtdD3kvmBc6eTQKLqSw9bBL7jypvvRqg9fH9oRYK4xHE7bgzkbOIi+uIWLVEETKdSFCpxE2Eur9vxVkgYA8UyS/kcxkt2RXs3JioQL7juKuyGjIyFVC4FCsOOvFzfoYa7x2Odn8QyPWreN7KBT0zkR87SQLFoYPJo1EFcRTDnrmgmXVBNaE7IE/QwihVwi+44u74FTKN6asMxgTjQ61VwJ+nweQIidm0CrA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=C6qprb7UXLoQkqSLLU/7JdgOKi/RKegWj7v+YnD465LNgBS+CvQKWuiLj6+t5DXOO4IfS7YGkH4wCVbb8TPTRlPt0+XZ0MDeBUoLUEZs6wuJ1lVDjIPIcypmJJ6wtYc0/0/sgLyWRzfODdAK1VVXR7newOfyV9C5hiW3I5EVjiqQFmACL8ejS9HB8ipPjxSUkN49iDfTxofzzO8f612ulJ9uOhC13oKQBx1pYXEpm/q9e5fC4N5M8xpHR2cQSTjvbHjN2pqAOUYOEK2kUGPZozyKGaBIaajKZ1zerTekouqdgxoRBJA3KpAdOCXcXAftuYWWhEXx9u4bCw9Q5GKmkA==
- Authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=oracle.com;
- Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, iwj@xxxxxxxxxxxxxx, wl@xxxxxxx, anthony.perard@xxxxxxxxxx, jbeulich@xxxxxxxx, andrew.cooper3@xxxxxxxxxx, jun.nakajima@xxxxxxxxx, kevin.tian@xxxxxxxxx
- Delivery-date: Fri, 19 Feb 2021 14:56:51 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 2/18/21 5:51 AM, Roger Pau Monné wrote:
> On Wed, Jan 20, 2021 at 05:49:10PM -0500, Boris Ostrovsky wrote:
>> When toolstack updates MSR policy, this MSR offset (which is the last
>> index in the hypervisor MSR range) is used to indicate hypervisor
>> behavior when guest accesses an MSR which is not explicitly emulated.
> It's kind of weird to use an MSR to store this. I assume this is done
> for migration reasons?
Not really. It just seemed to me that MSR policy is the logical place to do
that. Because it *is* a policy of how to deal with such accesses, isn't it?
> Isn't is possible to convey this data in the xl migration stream
> instead of having to pack it with MSRs?
I haven't looked at this but again --- the feature itself has nothing to do
with migration. The fact that folding it into policy makes migration of this
information "just work" is just a nice side benefit of this implementation.
-boris
|