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

Re: [PATCH][4.15] x86: mirror compat argument translation area for 32-bit PV


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Mon, 22 Feb 2021 12:35:21 +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=jWSnlA+PQt1lgyX9O3AGqFCKiGlVGdo+mKIDMnFHSlo=; b=npS7tCfXG33kP/HgPPLYcX5iVkWy7zL53WlD4CrO7BGpxMwJz2GxR4S/7h3Xxol071hzlO7hvy67m++ikfcNnGdJ/4I7ajFk/1gTx7NQDuhFIgYrgTuyM2Vh/FnfXLHiR+fYWF5s/yR3Rmw8bfp80XGJKejQua3u5mzdGQvvvMv/GeF4dgkkEOUq45/lnlWBD8Oy4PaR2y61FFI/JOpp3pr3NzWYB7ds9+WTPn7t8GkgBX5n7kqosHW4ok++zV5kENzEWiAQP3e3izTJdalW8o1WcwHQt1GUr13QYefXoJwxV8Dc2DTJlMwmKK5z7J2+PVVFi8ih81OpCamubMAb2g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cIP0BWXwi6AHjWyRGPMSWgE5KwlLWwzCprB+os+cXMnQWQpvH3RHtjsbleG2eX8+PiBrAIL4uNWFYvNgxeYDGfwI3lLp+ZeYJKck5ZffXfPNVx6oBrhISrnnqYzdzNM0sX/h3g2nEjjDXbXR1F2KGRDUBol+SU4FSnWVfS9swvZDJNkfN6NJSPt6Tb9dUasPLwk849ldFXeJhaPCvL2jYuelfzDQ/Dft5s2U6kMx1x0KGNEK4yPnHJltU8N7xrtS8FE9QwqD9uUx4OjPZCXRv0LerTZ5u0sEXhblZrENvKz6v/9KU0XBebJZantSUf0FrxJWKE2ZhpkEXQ9NotjTXA==
  • Authentication-results: esa2.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>, Ian Jackson <iwj@xxxxxxxxxxxxxx>
  • Delivery-date: Mon, 22 Feb 2021 11:35:39 +0000
  • Ironport-sdr: ShrAMV53vK/XWFH9buubr0ojDKOgVtUeP5g7MDQsP4urFBzUqOQXT4UoiXX9I/53luwInBVZx/ Q1BdBEWc+mT2YSSNi62g/11ZlssC4zqlwB+DCvIKVd/iQByUpBBoa+BsBtHcfYMQNg0NfbyPXo mWmQj4QY/3ba8vk+s5mkj1bML4CWHThODMKCyR2vppIuN10nitT2tNhK3Ogr4+/n2548dWTUQW eDwnBzYnafLzscDPCusByH/BVW5dZCp9kbR+rSnTFRUY+PpQb8FZTwLbYTbslu58yj4nCH4ykF XFA=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Feb 22, 2021 at 11:27:07AM +0100, Jan Beulich wrote:
> Now that we guard the entire Xen VA space against speculative abuse
> through hypervisor accesses to guest memory, the argument translation
> area's VA also needs to live outside this range, at least for 32-bit PV
> guests. To avoid extra is_hvm_*() conditionals, use the alternative VA
> uniformly.

Since you are double mapping the per-domain virtual area, won't it
make more sense to map it just once outside of the Xen virtual space
area? (so it's always using PML4_ADDR(511))

Is there anything concerning in the per-domain area that should be
protected against speculative accesses?

Thanks, Roger.



 


Rackspace

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