From patchwork Mon Jul 29 15:39:09 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anthony PERARD X-Patchwork-Id: 11064077 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 443B614DB for ; Mon, 29 Jul 2019 15:41:55 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 338F928613 for ; Mon, 29 Jul 2019 15:41:55 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 27B952875C; Mon, 29 Jul 2019 15:41:55 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id CA84628751 for ; Mon, 29 Jul 2019 15:41:50 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hs7kc-0007zN-TY; Mon, 29 Jul 2019 15:39:50 +0000 Received: from us1-rack-dfw2.inumbo.com ([104.130.134.6]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hs7kb-0007z6-Em for xen-devel@lists.xenproject.org; Mon, 29 Jul 2019 15:39:49 +0000 X-Inumbo-ID: 19010282-b217-11e9-8980-bc764e045a96 Received: from esa4.hc3370-68.iphmx.com (unknown [216.71.155.144]) by us1-rack-dfw2.inumbo.com (Halon) with ESMTPS id 19010282-b217-11e9-8980-bc764e045a96; Mon, 29 Jul 2019 15:39:48 +0000 (UTC) Authentication-Results: esa4.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=anthony.perard@citrix.com; spf=Pass smtp.mailfrom=anthony.perard@citrix.com; spf=None smtp.helo=postmaster@mail.citrix.com Received-SPF: None (esa4.hc3370-68.iphmx.com: no sender authenticity information available from domain of anthony.perard@citrix.com) identity=pra; client-ip=162.221.158.21; receiver=esa4.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="anthony.perard@citrix.com"; x-conformance=sidf_compatible Received-SPF: Pass (esa4.hc3370-68.iphmx.com: domain of anthony.perard@citrix.com designates 162.221.158.21 as permitted sender) identity=mailfrom; client-ip=162.221.158.21; receiver=esa4.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="anthony.perard@citrix.com"; x-conformance=sidf_compatible; x-record-type="v=spf1"; x-record-text="v=spf1 ip4:209.167.231.154 ip4:178.63.86.133 ip4:195.66.111.40/30 ip4:85.115.9.32/28 ip4:199.102.83.4 ip4:192.28.146.160 ip4:192.28.146.107 ip4:216.52.6.88 ip4:216.52.6.188 ip4:162.221.158.21 ip4:162.221.156.83 ~all" Received-SPF: None (esa4.hc3370-68.iphmx.com: no sender authenticity information available from domain of postmaster@mail.citrix.com) identity=helo; client-ip=162.221.158.21; receiver=esa4.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="postmaster@mail.citrix.com"; x-conformance=sidf_compatible IronPort-SDR: qMPg8t0G8QThSS16Mam/oS5nELdWPesEqgKf0xjY2qy/jEwJZVH4u/nqO0DTamcMowWVPNWd/L WiQpOONAai3OjzkGm1wMTe8TKVSzyCKyKnWSb6P4eOgcbqXpQzCVFKAo+qc9PGBm33rPaTtD96 lwfxNDFZ4A9nkG66HebvEp69QIzbGuZ2pBckW9UZ1piwTATsPyiDukD0L+z9wb+0qv8E0lfHF1 xvfJYjvKuPaVLvGVT35GQCynJxrCoa7Jy4HsAh2/vHEV4W6ccaMyukDU0KIJ3MuLmkS8v2N8vH Ddk= X-SBRS: 2.7 X-MesageID: 3709373 X-Ironport-Server: esa4.hc3370-68.iphmx.com X-Remote-IP: 162.221.158.21 X-Policy: $RELAYED X-IronPort-AV: E=Sophos;i="5.64,323,1559534400"; d="scan'208";a="3709373" From: Anthony PERARD To: Date: Mon, 29 Jul 2019 16:39:09 +0100 Message-ID: <20190729153944.24239-1-anthony.perard@citrix.com> X-Mailer: git-send-email 2.22.0 MIME-Version: 1.0 Subject: [Xen-devel] [PATCH v4 00/35] Specific platform to run OVMF in Xen PVH and HVM guests X-BeenThere: xen-devel@lists.xenproject.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Cc: Ard Biesheuvel , Jordan Justen , Julien Grall , Andrew Cooper , Anthony PERARD , xen-devel@lists.xenproject.org, Laszlo Ersek , =?utf-8?q?Roger_Pau_Monn=C3=A9?= Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP Patch series available in this git branch: https://xenbits.xen.org/git-http/people/aperard/ovmf.git br.platform-xen-pvh-v4 Changes in v4: - patch "OvmfPkg/XenPlatformPei: Reserve hvmloader's memory only when it has run" was removed, and instead a different change is done in "OvmfPkg/XenPlatformPei: Rework memory detection" - other changes detailed in the notes of each patch Hi, I've started to create a Xen specific platform, in OvmfPkg/XenOvmf.dsc with the goal to make it work on both Xen HVM and Xen PVH. The first few patches only create the platform and duplicate some code from OvmfPkg and the later patches makes OVMF boot in a Xen PVH guest and can boot a Linux guest. After this patch series, I'd like to wait a bit before removing Xen support from the OvmfPkg*.dsc, to allow time to switch to the new Xen only platform, maybe 1 year. To build and boot: To build, simply run OvmfPkg/build.sh -p OvmfPkg/OvmfXen.dsc Then use OVMF.fd as a kernel of a pvh guest config file (with xl/libxl). Patch series available in this git branch: https://xenbits.xen.org/git-http/people/aperard/ovmf.git br.platform-xen-pvh-v4 Anthony PERARD (35): OvmfPkg/ResetSystemLib: Add missing dependency on PciLib OvmfPkg: Create platform OvmfXen OvmfPkg: Introduce XenResetVector OvmfPkg: Introduce XenPlatformPei OvmfPkg/OvmfXen: Creating an ELF header OvmfPkg/XenResetVector: Add new entry point for Xen PVH OvmfPkg/XenResetVector: Saving start of day pointer for PVH guests OvmfPkg/XenResetVector: Allow jumpstart from either hvmloader or PVH OvmfPkg/OvmfXen: use a TimerLib instance that depends only on the CPU OvmfPkg/XenPlatformPei: Detect OVMF_INFO from hvmloader OvmfPkg/XenPlatformPei: Use mXenHvmloaderInfo to get E820 OvmfPkg/XenPlatformPei: Grab RSDP from PVH guest start of day struct OvmfPkg/Library/XenPlatformLib: New library OvmfPkg/AcpiPlatformDxe: Use XenPlatformLib OvmfPkg/AcpiPlatformDxe: Use Xen PVH RSDP if it exist OvmfPkg/XenHypercallLib: Enable it in PEIM OvmfPkg/XenPlatformPei: Reinit XenHypercallLib OvmfPkg/XenPlatformPei: Introduce XenHvmloaderDetected OvmfPkg/XenPlatformPei: Setup HyperPages earlier OvmfPkg/XenPlatformPei: Introduce XenPvhDetected OvmfPkg: Import XENMEM_memory_map hypercall to Xen/memory.h OvmfPkg/XenPlatformPei: no hvmloader: get the E820 table via hypercall OvmfPkg/XenPlatformPei: Rework memory detection OvmfPkg/XenPlatformPei: Reserve VGA memory region, to boot Linux OvmfPkg/XenPlatformPei: Ignore missing PCI Host Bridge on Xen PVH OvmfPkg/XenPlatformLib: Cache result for XenDetected OvmfPkg/PlatformBootManagerLib: Use XenDetected from XenPlatformLib OvmfPkg/PlatformBootManagerLib: Handle the absence of PCI bus on Xen PVH OvmfPkg/OvmfXen: Override PcdFSBClock to Xen vLAPIC timer frequency OvmfPkg/OvmfXen: Introduce XenTimerDxe OvmfPkg/PlatformBootManagerLib: Use a Xen console for ConOut/ConIn OvmfPkg: Introduce PcdXenGrantFrames OvmfPkg: Introduce XenIoPvhDxe to initialize Grant Tables OvmfPkg: Move XenRealTimeClockLib from ArmVirtPkg OvmfPkg/OvmfXen: use RealTimeClockRuntimeDxe from EmbeddedPkg OvmfPkg/OvmfPkg.dec | 10 + ArmVirtPkg/ArmVirtXen.dsc | 2 +- OvmfPkg/OvmfPkgIa32.dsc | 1 + OvmfPkg/OvmfPkgIa32X64.dsc | 1 + OvmfPkg/OvmfPkgX64.dsc | 1 + OvmfPkg/{OvmfPkgX64.dsc => OvmfXen.dsc} | 238 +------- OvmfPkg/OvmfXen.fdf | 539 ++++++++++++++++++ OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 3 +- .../PlatformBootManagerLib.inf | 6 +- .../Library/ResetSystemLib/ResetSystemLib.inf | 1 + .../XenHypercallLib/XenHypercallLib.inf | 4 +- .../Library/XenPlatformLib/XenPlatformLib.inf | 33 ++ .../XenRealTimeClockLib.inf | 0 OvmfPkg/XenBusDxe/XenBusDxe.inf | 2 + OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.inf | 35 ++ OvmfPkg/XenPlatformPei/XenPlatformPei.inf | 100 ++++ OvmfPkg/XenResetVector/XenResetVector.inf | 41 ++ OvmfPkg/XenTimerDxe/XenTimerDxe.inf | 42 ++ OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h | 6 +- OvmfPkg/Include/Guid/XenInfo.h | 8 +- .../Xen/arch-x86/hvm/start_info.h | 143 +++++ OvmfPkg/Include/IndustryStandard/Xen/memory.h | 23 + OvmfPkg/Include/Library/XenHypercallLib.h | 12 + OvmfPkg/Include/Library/XenPlatformLib.h | 53 ++ .../PlatformBootManagerLib/BdsPlatform.h | 1 + OvmfPkg/XenBusDxe/XenBusDxe.h | 1 + OvmfPkg/XenPlatformPei/Cmos.h | 52 ++ OvmfPkg/XenPlatformPei/Platform.h | 136 +++++ OvmfPkg/XenPlatformPei/Xen.h | 39 ++ OvmfPkg/XenTimerDxe/XenTimerDxe.h | 177 ++++++ OvmfPkg/AcpiPlatformDxe/Xen.c | 41 +- .../PlatformBootManagerLib/BdsPlatform.c | 43 +- .../PlatformBootManagerLib/PlatformData.c | 49 +- .../Library/ResetSystemLib/ResetSystemLib.c | 3 +- .../Library/XenHypercallLib/X86XenHypercall.c | 8 +- .../Library/XenHypercallLib/XenHypercall.c | 16 + .../Library/XenPlatformLib/XenPlatformLib.c | 81 +++ .../XenRealTimeClockLib/XenRealTimeClockLib.c | 0 OvmfPkg/OvmfXenElfHeaderGenerator.c | 140 +++++ OvmfPkg/PlatformPei/Xen.c | 3 - OvmfPkg/XenBusDxe/GrantTable.c | 3 +- OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.c | 53 ++ OvmfPkg/XenPlatformPei/AmdSev.c | 64 +++ OvmfPkg/XenPlatformPei/ClearCache.c | 112 ++++ OvmfPkg/XenPlatformPei/Cmos.c | 60 ++ OvmfPkg/XenPlatformPei/Fv.c | 76 +++ OvmfPkg/XenPlatformPei/MemDetect.c | 490 ++++++++++++++++ OvmfPkg/XenPlatformPei/Platform.c | 463 +++++++++++++++ OvmfPkg/XenPlatformPei/Xen.c | 373 ++++++++++++ OvmfPkg/XenTimerDxe/XenTimerDxe.c | 355 ++++++++++++ Maintainers.txt | 10 +- .../XenResetVector/Ia16/Real16ToFlat32.asm | 137 +++++ .../XenResetVector/Ia16/ResetVectorVtf0.asm | 79 +++ .../XenResetVector/Ia32/Flat32ToFlat64.asm | 68 +++ OvmfPkg/XenResetVector/Ia32/PageTables64.asm | 149 +++++ .../XenResetVector/Ia32/SearchForBfvBase.asm | 87 +++ OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm | 84 +++ OvmfPkg/XenResetVector/XenResetVector.nasmb | 71 +++ 58 files changed, 4523 insertions(+), 305 deletions(-) copy OvmfPkg/{OvmfPkgX64.dsc => OvmfXen.dsc} (76%) create mode 100644 OvmfPkg/OvmfXen.fdf create mode 100644 OvmfPkg/Library/XenPlatformLib/XenPlatformLib.inf rename {ArmVirtPkg => OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.inf (100%) create mode 100644 OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.inf create mode 100644 OvmfPkg/XenPlatformPei/XenPlatformPei.inf create mode 100644 OvmfPkg/XenResetVector/XenResetVector.inf create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.inf create mode 100644 OvmfPkg/Include/IndustryStandard/Xen/arch-x86/hvm/start_info.h create mode 100644 OvmfPkg/Include/Library/XenPlatformLib.h create mode 100644 OvmfPkg/XenPlatformPei/Cmos.h create mode 100644 OvmfPkg/XenPlatformPei/Platform.h create mode 100644 OvmfPkg/XenPlatformPei/Xen.h create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.h create mode 100644 OvmfPkg/Library/XenPlatformLib/XenPlatformLib.c rename {ArmVirtPkg => OvmfPkg}/Library/XenRealTimeClockLib/XenRealTimeClockLib.c (100%) create mode 100644 OvmfPkg/OvmfXenElfHeaderGenerator.c create mode 100644 OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.c create mode 100644 OvmfPkg/XenPlatformPei/AmdSev.c create mode 100644 OvmfPkg/XenPlatformPei/ClearCache.c create mode 100644 OvmfPkg/XenPlatformPei/Cmos.c create mode 100644 OvmfPkg/XenPlatformPei/Fv.c create mode 100644 OvmfPkg/XenPlatformPei/MemDetect.c create mode 100644 OvmfPkg/XenPlatformPei/Platform.c create mode 100644 OvmfPkg/XenPlatformPei/Xen.c create mode 100644 OvmfPkg/XenTimerDxe/XenTimerDxe.c create mode 100644 OvmfPkg/XenResetVector/Ia16/Real16ToFlat32.asm create mode 100644 OvmfPkg/XenResetVector/Ia16/ResetVectorVtf0.asm create mode 100644 OvmfPkg/XenResetVector/Ia32/Flat32ToFlat64.asm create mode 100644 OvmfPkg/XenResetVector/Ia32/PageTables64.asm create mode 100644 OvmfPkg/XenResetVector/Ia32/SearchForBfvBase.asm create mode 100644 OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm create mode 100644 OvmfPkg/XenResetVector/XenResetVector.nasmb