[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] revert "hvmloader: PA range 0xfc000000-0xffffffff should be UC"
- To: Andrew Cooper <amc96@xxxxxxxx>
- From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Date: Thu, 16 Dec 2021 12:09:17 +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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=hji2z3QqydBhhpIzlLRwVcOrjCHh9Pg5U8XVhBYmkmE=; b=oN2ZjhZ1lUmscfPw6tj3mr3T/JszyJz9WK1R7MPX1mLh6p9UY49NmCc2ab6viZ3a0uqbK0YSnBbdL+pHiYhV3N4m/UTxu/InD3TBAhkDS5soJZ8zeaAq9crBeAzU5lxua1ChFSRoCLqjgkO50CHvSydv6deC3BBvVFFG35mPNjKSkxrczca9ywxYk62nBeDpfoZxnPJPKIQTCh4Rm/KFerSB7Gw2aFYtAX3YXb5fj0C7setvQEcnQRySa9EjQBecy/BgXzSF1rJYhlZQKqc/M4w7qGPRo4n3rKmHxJ1DYQ68gjJt2XpR03e9LsHP3aa6W0M8GfkWqbug4DWxFyWKpQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ItFlQyXQO1Hwh9NlVh2Rat+nTCEDoQPEyQk1aix/Ah82NfT+1694xtHkk39t2/9flp3kQODdGomqfJiCGiAQQ2afNL/wACzsjubmxKLx6OCOixGh2+8OXvpLkEzozmwXzn4bIfviM8Iq34kyUIM2YEZV3679p0QcCJi8rME9GRDsOlK2v/oV4Ud2B+n4UyHmqxLFfUHgqiEdcifwgs/ZVqseneUGsGp6Py+HQg4AC7K3Jw6Ymgd8NA0/ZHFM4qky/2guYRvCdLfL4TqFmzDefEZp6dz2OjZ3CT5YkMDk3rr1dnXdY0c1kO+Vw/ZXS0PoQrbbeC9WUT66n1enwsKpaw==
- Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
- Cc: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
- Delivery-date: Thu, 16 Dec 2021 11:09:37 +0000
- Ironport-data: A9a23:TAT09anxaX6RznRnHXxZCUjo5gxuIURdPkR7XQ2eYbSJt1+Wr1Gzt xIZWz3UPqyOZmD3Kop+aNmwoE9VvMfTxoIyGVQ5qX8xEyMWpZLJC+rCIxarNUt+DCFioGGLT Sk6QoOdRCzhZiaE/n9BClVlxJVF/fngqoDUUYYoAQgsA180IMsdoUg7wbdg2Nc22YLR7z6l4 rseneWOYDdJ5BYsWo4kw/rrRMRH5amaVJsw5zTSVNgT1LPsvyB94KE3fMldG0DQUIhMdtNWc s6YpF2PEsE1yD92Yj+tuu6TnkTn2dc+NyDW4pZdc/DKbhSvOkXee0v0XRYRQR4/ttmHozx+4 PpQiMa7TQwbBa73tu8WEBdBGTElBrITrdcrIVDn2SCS50jPcn+qyPRyFkAme4Yf/46bA0kXq 6ZecmpUKEne2aTmm9pXScE17ignBNPsM44F/Glp0BnSDOo8QICFSKLPjTNd9Glo354SRKqPD yYfQTZuNw/lTwVUAFlUAao+oumX22v9fQQN/Tp5ooJoujOOnWSdyoPFINfTP9CHW8hRtkKZv X7duXT0BAkAM96SwibD9Wij7sfQmQvrVYRUE6e3ntZjhVbCmEQIEhYYE122vZGRiESzRtZeI Ew84Tc1oO4580nDczXmd0Tm+jje5EdaAocOVb1hgO2Q9kbKyymEOzIPSjVzU98dmsQ9BjIz3 E/SpMy8UFSDr4apYX6a876Vqxa7Ni4UMXIOaEc4cOcV3zXwiNpt10ySF76PBIbw14SoQm+on 1hmuQBn3+1L5fPnwZlX6rwub9iEgpHSBjA46QzMNo5OxlMoPdX1D2BEBLWy0BqhEGp7ZgXR1 JTns5LHhAzrMX1rvHbXKAnqNOv2j8tpyBWG3TZS82AJrlxBAUKLc4FK+y1ZL0x0KMsCcjKBS BaN4lIMtcQMbCbwNfQfj2eN5yICl/KI+TPNDK+8UzazSsIpKF/vEN9GOyZ8IFwBYGBzyPpia P93gO6nDGoACLQP8dZFb7x17FPf/QhnnTm7bcmil3yPiOPCDFbIGeZtGAbfNYgRsfLbyDg5B v4CbqNmPT0EC7agCsQWmKZORW03wY8TWcqr9pcJL7HbeWKL2ggJUpfs/F/oQKQ894x9nebU5 HCtHEhezVv0n3rcLguWLHtkbdvSsVxX9hrX5AQgYgSl3WYNe4Gq4PtNfpc7Z+B/pudi0eR1X 78OfMDZWqZDTTHO+jI8a5jhrdM9KET31FzWZyf1MiIie5NARhDS/oO2dAXY6yRTXDG8stEzo uP821qDE4YDXQlrEO3fdOmrkwGqpXEYle8rBxnIL9BfdV/C6o9vLyCt3PY7L9tVcUfIxyeA1 hbQChAd/LGfr4gw+djPpKaFs4b2TLcuQhsERzHWtO/kOzPb82yvxZ57fNyJJT2NBnnp/KiCZ PlOy62uOvMwg1sX4ZF3FKxmzPxi6oK39aNa1AltAF7Cc0+vVuF7OnCD0MRC6v9Ny7tetVfkU 06D4IAHa7CAOcejG18NPgs1KO+E0KhMyDXV6P00JmT85TN2o+XbARkDYUHUhXwPNqZxPaMk3 fwl6Zwf5AGIgxY3NsqL03JP/GOWI31cC6gqu/n22mMwZtbHHr2aXaHhNw==
- Ironport-hdrordr: A9a23:WujjvarQ8l1FiOmi3NVauiYaV5oveYIsimQD101hICG9Ffbo8P xG/c5rsSMc7Qx7ZJhOo7y90cW7Lk80lqQU3WByB9mftWDd0QPDQb2KhrGC/xTQXwH46+5Bxe NBXsFFebjN5IFB/KXHCd+DYrQd/OU=
- Ironport-sdr: Ln9HIEBabr0E0ZSky9HoJW/u+CqLCGGcG4OsvMtWj5uo51dfqWaWOmGrHB0i/3hiQ/lPMUfr9P lMkNCJLUeRUdznIerK1i0xJ82HFNMyQjpC7y5dBD+UN6PKliVVM1XW/HkcHJMETF+pnL3BZBue E7wPJWqifj/Er09ehnHQa+obsAYS85GdwWBm3h3bG/YYwWSk5nK9rOuUDq2QyBVGdWTfMN/4Ln pamlkHUW0X0+satxafCBfgr0/cIm25M8dirrbPXCMKXlkzscHmqdDWu8/8X4LdZQZPuzLa7qVM fT8KWpQ+GyIaiUrOdTTD2Efh
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Thu, Dec 16, 2021 at 11:00:17AM +0000, Andrew Cooper wrote:
> On 16/12/2021 09:54, Roger Pau Monné wrote:
> > On Tue, Dec 14, 2021 at 09:21:22AM +0100, Jan Beulich wrote:
> >> This reverts commit c22bd567ce22f6ad9bd93318ad0d7fd1c2eadb0d.
> >>
> >> While its description is correct from an abstract or real hardware pov,
> >> the range is special inside HVM guests. The range being UC in particular
> >> gets in the way of OVMF, which places itself at [FFE00000,FFFFFFFF].
> > I would assume this range to be unpopulated? Does hvmloader populate
> > it in order to place ovmf?
>
> It's generally not unpopulated. The video RAM lives there until the VGA
> BAR is reprogrammed.
Right, but that's an MMIO area from guests PoV, even if in our
implementation is backed by RAM pages.
> The reason OVMF places itself there is because it is where the real SPI
> ROM is mapped in address space on a real system.
Just to clarify my understanding, this is not reported as a RAM region
to guests? So hvmloader or the domain builder populates this with RAM
to place OVMF, even if not reported as a RAM region in the memory map
(much like with ACPI tables for example).
I wonder whether we should have some kind of document or code comment
about the guest memory layout (maybe there's one and I'm missing it).
Thanks, Roger.
|