[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] revert "hvmloader: PA range 0xfc000000-0xffffffff should be UC"
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Date: Thu, 16 Dec 2021 10:54:08 +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=Ifj/s1+DTiS4db5Jgx1vQe5DmXzeizJkdHFj9VeABKc=; b=aA+BiQJsLPBZLvYDFZ7DUh0FYTPkM+yAzCV48hmGWyzH0uf/GDPiwHhqKwmWaJQC6cERyxpjjgR2KwPtZLfQxZYtzthCrs5D1dQgLtpL1VAGOKpoYUsaKgQxncC6lhk83an+FFOF2UpoT6OE2ijo6YyDw3nVzw+sVKUqMIM6m/XcZofciJWB0SgyHSFieuphvTmUeL4bmF67I53MXEme3uHFhGf9leZiomCuo20QfIR4aL0GF+fxNT/Et9v2XHGUb0xwb6ZkyKCTlajAw1Qv9ysebv+T8DA4AMuS7G4FirFyis8VCmXGHEzKN2yHS6tVH06Kjpl6t6yVeRvAQhX/oA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eEQJIdOer2jQA/vN2sblRFwPC7/F/0Z1Ql/0Gbck0R07gDc2MkEdvg3YgOu8eP/3uZDSjzE+8nvNKFEwHTwLREYUSw5BZoVLOwz84CLrcZ7e+6Srtv6dywUhHzkVcC4DszP+uvuOF3Xe+48eokk6U9g6iUQ0Xwh0BU5ylOhGd6LHuP1g2o9h4EMzvztbeMIx5bYbWXqckdz5lRitIDAPtfhRqiAQ79oInKMolFNzudZXgYffzR+7WR4a6tJHMaksKMLR5buXgllCVZYEovG4Cjz2c7TL0x1FzDKt5AAc3nk1Gl4MCQJSCBDQ/znj6P1y/5XxCmHxPDyN8+ZODP8FTg==
- 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>
- Delivery-date: Thu, 16 Dec 2021 09:54:33 +0000
- Ironport-data: A9a23:Tma9hq4MIkrWborL5kQcVAxRtOfAchMFZxGqfqrLsTDasY5as4F+v mVOXW2Db66MZ2uhfIwkYI2/8kIP6J/Vn4diTARvryo1Hi5G8cbLO4+Ufxz6V8+wwmwvb67FA +E2MISowBUcFyeEzvuV3zyIQUBUjclkfJKlYAL/En03FV8MpBsJ00o5wrdj29Aw27BVPivW0 T/Mi5yHULOa82Yc3lI8s8pvfzs24ZweEBtB1rAPTagjUG32zhH5P7pGTU2FFFPqQ5E8IwKPb 72rIIdVXI/u10xF5tuNyt4Xe6CRK1LYFVDmZnF+A8BOjvXez8CbP2lS2Pc0MC9qZzu1c99Z7 9FO6o2PTAoTMfP+o8EHfiVHMDk9IvgTkFPHCSDXXc27ykTHdz3nwul0DVFwNoodkgp1KTgQr 7pCcmlLN03dwbLtqF64YrAEasALNs7kMZlZonh95TrYEewnUdbIRKCiCdpwgm9p25ERRa62i 8wxZxhpLzfGUk11JmwlC5AiuMHvvXCkSmgNwL6SjfVuuDWCpOBr65DyNPLFd9rMQt9a9m6Iq 2SD82nnDxUyMN2E1SHD4n+qnvXIny7wRMQVDrLQ3vxgjUCXx2cTIAYLTlb9qv684nNSQPoGd RZSoHB36/Fvqgr7FbERQiFUvlajuh1DSuVVHtc1tjO277D53xm6HisbG2sphMMdiOc6Qjkj1 1msltzvBCByvLD9dU9x5ot4vhvpZ3FLcDZqiTssCFJcvoK9+N1bYgfnF447SMaIYsvJ9SYcK txghAw3nP0tgMECzM1XFniX0mv39vAlouPYjzg7v15JDCslNOZJhKTysDA3CMqsyq7DFDFtW 1BexKCjABgmV83lqcB0aLxl8EuVz/iEKibAplVkAoMs8T+gk1b6I9sBvWAlexc0aJ5fEdMMX KM1kVkLjHO0FCH1BZKbnqrrU5h6pUQePYqNug/ogipmPcEqKV7vENBGbk+MxWH9+HXAYolkU ap3hf2EVC5AYYw+lWLeb75EjdcDm3BvrUuOFMuT50n2jtKjiIu9FO5t3K2mNbtisstpYWz9r r5iCid940kFDbClPHCIqdV7wJJjBSFTOK0aYvd/L4arCgFnBHsgG7nWx7YgcJZihKNbiqHD+ XTVZ6OS4AOXaaTvJVrYZ3Z9RqnoWJoj/3s3MTZ1ZQSj2mQ5YJbp56AaLsNlcb4i/e1l7Ph1U /haJJnQXqUREmzKq2YHcJ3wjI1+bxD31wiACDWoPWokdJl6Sg2XptK9Jlnz9DMDBzacvNclp +HyzRvSRJcOHlwwDMvfZP+14Um2uHwRxLB7U0fSe4EBc0Tw6ol6bSf2i6Zvcc0LLBzCwBqc1 hqXXkhE9bWc/ddt/YCQ166eroqvH+9vJWZgHjHWveSsKC3X3mu/2oscAuyGSi/QCTHv86K4a OQLk/ylaK8bnExHupZXGqpwyf5s/MPmorJXw1g2HHjPaFj3WLpsLmPfgJtKv6xJgLRYpRG3S gSE/dwDYeeFP8bsEVgwIgs5b7vciaFIy2eKtfllcl/n4CJX/aacVRQANhaBvyVRMb9pPd532 uwmosMXt1SyhxdC3gxqVcyIG7Bg9kA9bpg=
- Ironport-hdrordr: A9a23:2RBusKEtWV+WEPDvpLqFcJHXdLJyesId70hD6qkvc3Nom52j+/ xGws536faVslcssHFJo6HmBEClewKnyXcT2/htAV7CZnichILMFu9fBOTZsl/d8kHFh4tgPO JbAtRD4b7LfClHZKTBkXCF+r8bqbHtmsDY5ts2jU0dNT2CA5sQkTuRYTzrdHGeKjM2YabQQ/ Gnl7V6TnebCD8qR/X+IkNAc/nIptXNmp6jSRkaByQ/4A3LqT+z8rb1HzWRwx9bClp0sPof2F mAtza8yrSosvm9xBOZ/2jP765OkN+k7tdYHsSDhuUcNz2poAe1Y4ZKXaGEoVkO0auSwWdvtO OJjwYrPsx15X+UVmapoSH10w2l6zoq42+K8y7vvVLT5ejCAB4qActIgoxUNjHD7VA7gd162K VXm0qEqpt+F3r77WrAzumNcysvulu/oHIkn+JWpWdYS5EiZLhYqpFa1F9JEa0HADnx5OkcYa ZT5fnnlbZrmG6hHjPkVjEF+q3vYp1zJGbLfqE6gL3V79AM90oJinfxx6Qk7wA9HdwGOt15Dt //Q9dVfYd1P7srhJJGdZc8qPSMex7wqDL3QSuvyAfcZek600ykke+D3Fxy3pDsRKA1
- Ironport-sdr: Tzt7NAjVVgNBoVIYh6zSQtyS4gQ3eikQCezd7CyUkZ9NOhJYtbsajZO8Xn3Fu6CtCxVg88cLfY ONOFai215HYN1ji+HtCJfkaoyNeJIYSGKnmisAoNMrR1EdrX7x7mkXBEdy4TZ/4ZxwtMK8UOSj /YgcaGV6sY+kIYTcgqVQyX0t6jx9sDKx6zV6zQlzxbI0eFPO7F3IlGsvoi7v0RXNI+rWYwfXmk rOQdOW66ORQx942/fEepFDh+0NsN2lBMuzZGhj2wGF2ziV6mNr+lZeWtb8qmjMeu2m9nDTCcuH 7h5n3C7JTGN6NI2DNAWQgbeO
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
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?
> While this is benign to epte_get_entry_emt() as long as the IOMMU isn't
> enabled for a guest, it becomes a very noticable problem otherwise: It
> takes about half a minute for OVMF to decompress itself into its
> designated address range.
>
> And even beyond OVMF there's no reason to have e.g. the ACPI memory
> range marked UC.
>
> Fixes: c22bd567ce22 ("hvmloader: PA range 0xfc000000-0xffffffff should be UC")
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
This is just a hole from hvmloader PoV, so it's fine to not apply any
specific MTRR attributes.
Thanks, Roger.
|
