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

Re: [PATCH 0/3] x86/ept: force WB to foreign and grant mappings


  • To: Jun Nakajima <jun.nakajima@xxxxxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 15 Jun 2021 15:37:55 +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-SenderADCheck; bh=Ry2vwoqwKaS725viSjbrqwk+RpNR3ZoO/f0y2jFHWLc=; b=CTn1R1WvJm0nVEdUPZnFmcFtuWrNgYURXGah6r1P0jV6UXEGIKJH0wCGs1mbJI64vSCEbaYWbNDiCLSDiXD+ilfz5xOkD6liKs6Qo33Jgwlymwy6n0U5OPjWiv0YuygXVkUqD5VLezwLFvd0GooH9uh6vhbDT/G9Ij442f3bB1Q0D+PFwd15JcKPxkGLTj3jL5Z6VCk+zFWc/SM42DVj9R1QHaf1Vzk417l8flgRA5U4Z1/il5a7k93jsH0XxahXPVikaNc4bMMVsmt4Fgj0GgQI2dA0XmidaRPH8YPS6lMI+AUGHONxPMpX3Bnd3lz8CnreGpMi4CIrvn7c6qvV2A==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hsjMqnBpE3VSt0i1CBcnJ5zAmGwcvJhauTibAfo+mxSzamlQZliOizjH8PjGE+QZh+CVx5GsYIygI8XbQZgrk3Onk9mPzroXzhbHs1v0pSNmsjWcO0m52biC3GLNEkN8328ZNGNAXHfHw+Zl09gdtq4ua0QD3yKbMxU60s++Kh5nuWiviGu0e3t+fW3/A5dCNxqXffoGPNmy4BhKLQsyKpON4vhgVEU9Tz0xznchhxwNmg+UbuN9Aoq4VkS1vPGUegJNw5ak9bgw7TBaEGEDAWiZms2SsVYIllnN1PizlzWxkIu9vNgkqPmqhlVtnfx4WfDThL/pAzYFxg0deE6H/w==
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>
  • Delivery-date: Tue, 15 Jun 2021 13:50:18 +0000
  • Ironport-hdrordr: A9a23:08ITlKCY1cabRfnlHejKsseALOsnbusQ8zAXPh9KJSC9I/b0qy nxpp4mPEfP+U0ssQIb6Kq90ci7MD7hHPtOjrX5Uo3SOzUO1FHJEGgm1/qb/9SCIVyzygc+79 YHT0EWMrSZZjYasS+52njALz9K+qjkzEncv5a7854bd3AOV0gP1XYbNu/IKDwseOHmaKBJS6 Z0s/A3/gaISDA9Qv6AQl8YQuPEu9rH0Kj+aRpDPBI/7hKS5AnYtYLSIlyy8i0lFxVS3Lwl7W /J1zbh7qHmifu80RO07R6I071m3PDM8OEGItCQgc4ILTipsB+hYMBaRrWeoCpdmpDy1L9/qq iVnz4Qe+BIr1/BdGC8phXgnyP61iw11nPkwViExVP+vM3QXlsBeoR8rLMcViGcx1srvdl63q 4O9XmerYBrARTJmzm4z8TUVittilG/rRMZ4K8uZj1kIMsjgY1q3M4iFXBuYdI99eXBmcQa+d xVfYLhDTBtABSnhj7izy9SKZeXLwkO91+9MzQ/U4quondrdTlCvjQl7d1akXEa+J0nTZ5Yo+ zCL6RzjblLCtQbdKRnGY46MIKK40H2MGXx2VipUCLa/WA8SgDwgo+y5K9w6PCheZQOwpd3kJ PdUElAvWp3f071E8WB0JBC7xiIGQyGLH/Q49Ab44I8tqz3RbLtPyHGQFcyk9G4q/FaBsHAQf 68NJ9fHvemJ2rzHoRC2RH4RvBpWDEjuQ0uy6IGsne104j2w6HRx5zmmcfoVc/Q+GwfKxLC61 M4LUvOzet7nz6WZkM=
  • Ironport-sdr: vsqxLTZPmXluUffK1IiDGVP/48xF0QmS24uqOrIlZkSO6pj83jvHU4oQSBhW8KWO9GKLnfhDOQ cQ7tagr0snYZjOuYFABeg+KaybqnMabjwUNAPicnrx6WU2dRbH4C/gPo56Fgc7Dk+T2uWRxpIT mkS0Hh3Qgq2av2PSNxJpp+ea8olmJRwaOXmC2eA6CxoxVLomFIrFwrOipFXWtFRdPi3cpP889U 6s7zHoZrDO4ZURr/76ujpHkOuC0prAWxofhHN7tpu1jAoq9CB/QMEHTN0uJtNFKckT5xHrkxVm F0U=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Ping?

This is missing an Ack or otherwise from the Intel maintainers.

Thanks, Roger.

On Fri, May 28, 2021 at 07:39:32PM +0200, Roger Pau Monne wrote:
> 
> Hello,
> 
> The aim of this series is to force the cache attribute of foreign and
> grant mappings to WB for HVM/PVH guests. This is required because those
> mappings will be likely be using unpopulated memory ranges in the p2m,
> and those are usually UC in the MTRR state.
> 
> Having the guest set the correct MTRR attributes is also unlikely,
> because the number of MTRR ranges is finite.
> 
> Roger Pau Monne (3):
>   x86/mtrr: remove stale function prototype
>   x86/mtrr: move epte_get_entry_emt to p2m-ept.c
>   x86/ept: force WB cache attributes for grant and foreign maps
> 
>  xen/arch/x86/hvm/mtrr.c           | 107 +---------------------
>  xen/arch/x86/hvm/vmx/vmx.c        |   6 +-
>  xen/arch/x86/mm/p2m-ept.c         | 145 ++++++++++++++++++++++++++++--
>  xen/include/asm-x86/hvm/vmx/vmx.h |   2 +
>  xen/include/asm-x86/mtrr.h        |   7 +-
>  5 files changed, 147 insertions(+), 120 deletions(-)
> 
> -- 
> 2.31.1
> 



 


Rackspace

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