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

[PATCH v3 0/7] OvmfXen: Set PcdFSBClock at runtime


  • To: <devel@xxxxxxxxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Mon, 12 Apr 2021 14:29:56 +0100
  • Authentication-results: esa5.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Laszlo Ersek <lersek@xxxxxxxxxx>, Jordan Justen <jordan.l.justen@xxxxxxxxx>, Ard Biesheuvel <ardb+tianocore@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>
  • Delivery-date: Mon, 12 Apr 2021 13:30:23 +0000
  • Ironport-hdrordr: A9a23:vSPTFqCL8J965uDlHegktseALOonbusQ8zAX/mhLY1h8btGYm8 eynP4SyB/zj3IrVGs9nM2bUZPsfVr1zrQwxYUKJ7+tUE3duGWuJJx/9oeK+VfdMgXE3Kpm2a 9kGpIUNPTZEUV6gcHm4AOxDtYnx529/Lq1gPrFpk0CcShDd7xg4wBlCgySD01xQ018CYAkEY eHouxdpzShfnwTB/7LZEUtde7FutHNidbacQcLbiRXlDWmoDu05NfBYmGl9zgEVTcn+8ZYzU HgsyjUopquqOu6zBi07R6v071zlMH6wtVOQOyA4/JlTQnEsQqjaIR/V7DqhllczI7OmTVa9a ipnz4aM85+62zccyWOmDSF4Wjd+Q0j9mP4zhugiWbjyPaJMQ4SMdZLho5Sb3LimjAdlex7uZ g7p16xht58KCKFpyjm/t7OTR0CrDvEnVMS1cQ3vjhkUIMSZKU5l/1uwGplVLsnJQrm9ZsmG/ VyAMz56OxNeV2XBkqpylVH8ZiBeE0PEhCPRVUrp9GP0zNLgXx160cD3sQQkh47heUAYqgByO LePqtykrZSCucQcKJmHe8EBfC6E2rXXHv3QSGvCGWiMYxCAmnKoZLs+rk5jdvaM6Ag/d8Xtr yESk9SsW4pYULpDqS1rft22yGIZF/4cSXmy8lY6ZQ8kKb7XqDTKiGGSEojiY+JpOgeGNTSX7 KIMppQKfnlMGf+BO9yrl3DcqgXDUNbfNweu949VV7LiNnMMJfSreDecOzePv7EDSskcnmXOA pcYBHDYOF7qmy7UH7xhxbcH1n3fFbkxIl9FKjB8/JW4JQRN7dLrhMehT2Cl562AAwHlpZzUF p1IbvhnK/+j3Kx53zw9GlsOgBQFAJ8/KjnfHVQuwcQPk/4GIxz6um3SCR35j+qNxV/R8TZHE p0vFJs45utI5iR3yw5T/OmKH+dlGt7ngPec74s3om4oev1cJIxCZgrHIZrEx/QLgd4nQZxpH 0GThQFSE/ZHjbFjKWoi4E1PcnTesJxmh3DG78IlVvv8WGn4e0/THoSWDCjFeSNhxw1fiFZgl 1q/7VauaGHkzq0M2czxM8pdH5LdH+RCLdHCwrtXvQHppnbPCVLCUuajz2TjB8+PkDw8V8Jv3 fsKS2PdeuOOHB500oogZrCwRdRTCGwbkhwYnd1vclWDmLdoE9p3euKfKaol1aLYlwZ2+cHLQ zfaTQcIglS18m6vSTlxAqqJDED/NEDL+bdBLMsf/X4wXW2MrCFkqkAArt94ItlHMqGiJ5Sbc uvPyuuaB/oAeIg3AKY4lw/PjNvlXUimfT0nDX48WmD2mIlC/a6GiU+e5grZ/Wnq0T0TfeB15 t0yfgvu/GrD2n3YtmajY7NaTpuLQ7SvH6WQ+kkpYs8h9NoiJJDW73gFRfY3nBO2xszaO3ukl kFfah96Lfdfq9jFvZiPR5xzx4MrpCiPUErugv5DqsVZlc2lULWONuP/v7upacwBFaC4C/9I0 OW/SEY3/qtZVrF6ZcqT4YLZUhGYkk173pvuMmYcZfLNQmsf+ZfuHygMnG8d7dZYLOfGaoZqy t7593gpZ7SSwPInCTr+RdrKKNH9GiqBey1GxyFAuNJ9NKeP1KU65HaovKbvXPSc3+Wek4YjY pKeQgrYsNPhiILoac33iKxI5aH7H4Noh9723VKh1Tt0o+p7CPnBklAKxTemYgTdyJUKGK0gc PM9vW42Hzx7CNe44TKEF5dc7h1aoItZ7myCx0rBdkbvbau8aZqvz9KZw02CXUgzB/6xOFr0N 6Cqb/vctynLU2tH10P+TRIXNEp2gMqrHxNaMi449aWZB4NGusBHvs44cR3nVtS2xHOxmxZSy 8okAoikrzpBBjAGV9XEKD1h4v4gBQCvsG5mhJTjUQVvmnisU2Wvlj+ot28tHU1pVDMtwMPk9 TqeBIBxAJi8jN9yaNvMnCxQXfhbHOMPlTw1rYpMxReeCgS4tRuqBrINK8QkJ+ii0JisDQgGy A1qovbV1r3eHG3FdfqKiqtG8nwKNluY+xJMm5GC3KtLhzLltARSR3PEe0RlimEqzMrBaKUPL mhbx/YJJ4R/LYIM6iperWgv671p0Vk3kBXidil1Vq4oZtRnI+mDz85l8kye69oKQ+2/CF+89 eaUA7VJbEN7DvYtHPs5E1jVYsr6jVQ51IbqycWED6vHPWpSCha1Xs74q6sfTYfz8mZT9ZTrC PunX6N0UOmSzy8AQ2UQLsK7Gsu9AHdBKTgn3b/VYbQGnlpiY+ROJb8rDH90QDCodWHkX3lrx tolU5RWB00zh39ZqViN17mi3kANRxiDxVtlxfn5aoH/mEiukM+aDwWQ6A4jVbUNSw=
  • Ironport-sdr: J9wtclQeZNsFoPL8LnC9yUg/SiCbWXdfGqT0dR75vVERw3Aot+DbaiA/4mwF2f3y27u7UsSaEh PsMyMlxlUxjyfmcSCwICLNYWZwf6h+6ElRAZAMWbCLnR02ZCuHhQOugOGJGLcN3H/nmEz9AuL0 N5jU8AUQpbZI9K3kxmrhiok7yvaHXZDx/nUrVgoT50xMXyJCyJs1diKhrllCi5ZuInH7Ina35P 6b3uG78qYgtoshMCw0q/lhUHrYK7hX0s917sy/X47efrs4PJuqpzZOHMBhnpnk+8noMkPBQW/V oZ0=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Patch series available in this git branch:
git://xenbits.xen.org/people/aperard/ovmf.git br.apic-timer-freq-v3

Changes in v3:
- typos and codying style

Changes in v2:
- main change is to allow mapping of Xen pages outside of the RAM
  see patch: "OvmfPkg/XenPlatformPei: Map extra physical address"
- that new function allows to map the Xen shared info page (where we can find
  information about tsc frequency) at the highest physical address allowed.

Hi,

OvmfXen uses the APIC timer, but with an hard-coded frequency that may change
as pointed out here:
  https://edk2.groups.io/g/devel/message/45185
  <20190808134423.ybqg3qkpw5ucfzk4@Air-de-Roger>

This series changes that so the frequency is calculated at runtime.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2490

There is also one cleanup patch that has nothing to do with the rest.

Cheers,

Anthony PERARD (7):
  OvmfPkg/XenResetVector: Silent a warning from nasm
  MdePkg: Allow PcdFSBClock to by Dynamic
  OvmfPkg/IndustryStandard/Xen: Apply EDK2 coding style to
    XEN_VCPU_TIME_INFO
  OvmfPkg/IndustryStandard: Introduce PageTable.h
  OvmfPkg/XenPlatformPei: Map extra physical address
  OvmfPkg/XenPlatformPei: Calibrate APIC timer frequency
  OvmfPkg/OvmfXen: Set PcdFSBClock

 MdePkg/MdePkg.dec                             |   8 +-
 OvmfPkg/OvmfXen.dsc                           |   4 +-
 OvmfPkg/XenPlatformPei/XenPlatformPei.inf     |   4 +
 .../IndustryStandard/PageTable.h}             | 117 +-------
 OvmfPkg/Include/IndustryStandard/Xen/xen.h    |  17 +-
 .../BaseMemEncryptSevLib/X64/VirtualMemory.h  | 143 +---------
 OvmfPkg/XenPlatformPei/Platform.h             |  10 +
 OvmfPkg/XenPlatformPei/Platform.c             |   1 +
 OvmfPkg/XenPlatformPei/Xen.c                  | 252 ++++++++++++++++++
 OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm    |   2 +-
 10 files changed, 287 insertions(+), 271 deletions(-)
 copy OvmfPkg/{Library/BaseMemEncryptSevLib/X64/VirtualMemory.h => 
Include/IndustryStandard/PageTable.h} (60%)

-- 
Anthony PERARD


 


Rackspace

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