* [PATCH v4 1/7] OvmfPkg/PlatformCI: factor out PlatformBuildLib.py
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 2/7] OvmfPkg/PlatformCI: add QEMU_SKIP Gerd Hoffmann
` (6 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Move SettingsManager and PlatformBuilder classes to PlatformBuildLib.py
file, keep only CommonPlatform class in PlatformBuild.py. Allows
reusing these classes for other builds. Pure code motion, no functional
change.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
---
OvmfPkg/PlatformCI/PlatformBuild.py | 223 +-----------------
.../{PlatformBuild.py => PlatformBuildLib.py} | 32 ---
2 files changed, 6 insertions(+), 249 deletions(-)
copy OvmfPkg/PlatformCI/{PlatformBuild.py => PlatformBuildLib.py} (87%)
diff --git a/OvmfPkg/PlatformCI/PlatformBuild.py b/OvmfPkg/PlatformCI/PlatformBuild.py
index 627bb7b992db..6c541cdea4a5 100644
--- a/OvmfPkg/PlatformCI/PlatformBuild.py
+++ b/OvmfPkg/PlatformCI/PlatformBuild.py
@@ -5,17 +5,11 @@
# SPDX-License-Identifier: BSD-2-Clause-Patent
##
import os
-import logging
-import io
-
-from edk2toolext.environment import shell_environment
-from edk2toolext.environment.uefi_build import UefiBuilder
-from edk2toolext.invocables.edk2_platform_build import BuildSettingsManager
-from edk2toolext.invocables.edk2_setup import SetupSettingsManager, RequiredSubmodule
-from edk2toolext.invocables.edk2_update import UpdateSettingsManager
-from edk2toolext.invocables.edk2_pr_eval import PrEvalSettingsManager
-from edk2toollib.utility_functions import RunCmd
+import sys
+sys.path.append(os.path.dirname(os.path.abspath(__file__)))
+from PlatformBuildLib import SettingsManager
+from PlatformBuildLib import PlatformBuilder
# ####################################################################################### #
# Common Configuration #
@@ -45,210 +39,5 @@ class CommonPlatform():
dsc += ".dsc"
return dsc
-
- # ####################################################################################### #
- # Configuration for Update & Setup #
- # ####################################################################################### #
-class SettingsManager(UpdateSettingsManager, SetupSettingsManager, PrEvalSettingsManager):
-
- def GetPackagesSupported(self):
- ''' return iterable of edk2 packages supported by this build.
- These should be edk2 workspace relative paths '''
- return CommonPlatform.PackagesSupported
-
- def GetArchitecturesSupported(self):
- ''' return iterable of edk2 architectures supported by this build '''
- return CommonPlatform.ArchSupported
-
- def GetTargetsSupported(self):
- ''' return iterable of edk2 target tags supported by this build '''
- return CommonPlatform.TargetsSupported
-
- def GetRequiredSubmodules(self):
- ''' return iterable containing RequiredSubmodule objects.
- If no RequiredSubmodules return an empty iterable
- '''
- rs = []
-
- # intentionally declare this one with recursive false to avoid overhead
- rs.append(RequiredSubmodule(
- "CryptoPkg/Library/OpensslLib/openssl", False))
-
- # To avoid maintenance of this file for every new submodule
- # lets just parse the .gitmodules and add each if not already in list.
- # The GetRequiredSubmodules is designed to allow a build to optimize
- # the desired submodules but it isn't necessary for this repository.
- result = io.StringIO()
- ret = RunCmd("git", "config --file .gitmodules --get-regexp path", workingdir=self.GetWorkspaceRoot(), outstream=result)
- # Cmd output is expected to look like:
- # submodule.CryptoPkg/Library/OpensslLib/openssl.path CryptoPkg/Library/OpensslLib/openssl
- # submodule.SoftFloat.path ArmPkg/Library/ArmSoftFloatLib/berkeley-softfloat-3
- if ret == 0:
- for line in result.getvalue().splitlines():
- _, _, path = line.partition(" ")
- if path is not None:
- if path not in [x.path for x in rs]:
- rs.append(RequiredSubmodule(path, True)) # add it with recursive since we don't know
- return rs
-
- def SetArchitectures(self, list_of_requested_architectures):
- ''' Confirm the requests architecture list is valid and configure SettingsManager
- to run only the requested architectures.
-
- Raise Exception if a list_of_requested_architectures is not supported
- '''
- unsupported = set(list_of_requested_architectures) - set(self.GetArchitecturesSupported())
- if(len(unsupported) > 0):
- errorString = ( "Unsupported Architecture Requested: " + " ".join(unsupported))
- logging.critical( errorString )
- raise Exception( errorString )
- self.ActualArchitectures = list_of_requested_architectures
-
- def GetWorkspaceRoot(self):
- ''' get WorkspacePath '''
- return CommonPlatform.WorkspaceRoot
-
- def GetActiveScopes(self):
- ''' return tuple containing scopes that should be active for this process '''
- return CommonPlatform.Scopes
-
- def FilterPackagesToTest(self, changedFilesList: list, potentialPackagesList: list) -> list:
- ''' Filter other cases that this package should be built
- based on changed files. This should cover things that can't
- be detected as dependencies. '''
- build_these_packages = []
- possible_packages = potentialPackagesList.copy()
- for f in changedFilesList:
- # BaseTools files that might change the build
- if "BaseTools" in f:
- if os.path.splitext(f) not in [".txt", ".md"]:
- build_these_packages = possible_packages
- break
-
- # if the azure pipeline platform template file changed
- if "platform-build-run-steps.yml" in f:
- build_these_packages = possible_packages
- break
-
- return build_these_packages
-
- def GetPlatformDscAndConfig(self) -> tuple:
- ''' If a platform desires to provide its DSC then Policy 4 will evaluate if
- any of the changes will be built in the dsc.
-
- The tuple should be (<workspace relative path to dsc file>, <input dictionary of dsc key value pairs>)
- '''
- dsc = CommonPlatform.GetDscName(",".join(self.ActualArchitectures))
- return (f"OvmfPkg/{dsc}", {})
-
-
- # ####################################################################################### #
- # Actual Configuration for Platform Build #
- # ####################################################################################### #
-class PlatformBuilder( UefiBuilder, BuildSettingsManager):
- def __init__(self):
- UefiBuilder.__init__(self)
-
- def AddCommandLineOptions(self, parserObj):
- ''' Add command line options to the argparser '''
- parserObj.add_argument('-a', "--arch", dest="build_arch", type=str, default="IA32,X64",
- help="Optional - CSV of architecture to build. IA32 will use IA32 for Pei & Dxe. "
- "X64 will use X64 for both PEI and DXE. IA32,X64 will use IA32 for PEI and "
- "X64 for DXE. default is IA32,X64")
-
- def RetrieveCommandLineOptions(self, args):
- ''' Retrieve command line options from the argparser '''
-
- shell_environment.GetBuildVars().SetValue("TARGET_ARCH"," ".join(args.build_arch.upper().split(",")), "From CmdLine")
- dsc = CommonPlatform.GetDscName(args.build_arch)
- shell_environment.GetBuildVars().SetValue("ACTIVE_PLATFORM", f"OvmfPkg/{dsc}", "From CmdLine")
-
- def GetWorkspaceRoot(self):
- ''' get WorkspacePath '''
- return CommonPlatform.WorkspaceRoot
-
- def GetPackagesPath(self):
- ''' Return a list of workspace relative paths that should be mapped as edk2 PackagesPath '''
- return ()
-
- def GetActiveScopes(self):
- ''' return tuple containing scopes that should be active for this process '''
- return CommonPlatform.Scopes
-
- def GetName(self):
- ''' Get the name of the repo, platform, or product being build '''
- ''' Used for naming the log file, among others '''
- # check the startup nsh flag and if set then rename the log file.
- # this helps in CI so we don't overwrite the build log since running
- # uses the stuart_build command.
- if(shell_environment.GetBuildVars().GetValue("MAKE_STARTUP_NSH", "FALSE") == "TRUE"):
- return "OvmfPkg_With_Run"
- return "OvmfPkg"
-
- def GetLoggingLevel(self, loggerType):
- ''' Get the logging level for a given type
- base == lowest logging level supported
- con == Screen logging
- txt == plain text file logging
- md == markdown file logging
- '''
- return logging.DEBUG
-
- def SetPlatformEnv(self):
- logging.debug("PlatformBuilder SetPlatformEnv")
- self.env.SetValue("PRODUCT_NAME", "OVMF", "Platform Hardcoded")
- self.env.SetValue("MAKE_STARTUP_NSH", "FALSE", "Default to false")
- self.env.SetValue("QEMU_HEADLESS", "FALSE", "Default to false")
- return 0
-
- def PlatformPreBuild(self):
- return 0
-
- def PlatformPostBuild(self):
- return 0
-
- def FlashRomImage(self):
- VirtualDrive = os.path.join(self.env.GetValue("BUILD_OUTPUT_BASE"), "VirtualDrive")
- os.makedirs(VirtualDrive, exist_ok=True)
- OutputPath_FV = os.path.join(self.env.GetValue("BUILD_OUTPUT_BASE"), "FV")
-
- #
- # QEMU must be on the path
- #
- cmd = "qemu-system-x86_64"
- args = "-debugcon stdio" # write messages to stdio
- args += " -global isa-debugcon.iobase=0x402" # debug messages out thru virtual io port
- args += " -net none" # turn off network
- args += f" -drive file=fat:rw:{VirtualDrive},format=raw,media=disk" # Mount disk with startup.nsh
-
- if (self.env.GetValue("QEMU_HEADLESS").upper() == "TRUE"):
- args += " -display none" # no graphics
-
- if (self.env.GetBuildValue("SMM_REQUIRE") == "1"):
- args += " -machine q35,smm=on" #,accel=(tcg|kvm)"
- #args += " -m ..."
- #args += " -smp ..."
- args += " -global driver=cfi.pflash01,property=secure,value=on"
- args += " -drive if=pflash,format=raw,unit=0,file=" + os.path.join(OutputPath_FV, "OVMF_CODE.fd") + ",readonly=on"
- args += " -drive if=pflash,format=raw,unit=1,file=" + os.path.join(OutputPath_FV, "OVMF_VARS.fd")
- else:
- args += " -pflash " + os.path.join(OutputPath_FV, "OVMF.fd") # path to firmware
-
-
- if (self.env.GetValue("MAKE_STARTUP_NSH").upper() == "TRUE"):
- f = open(os.path.join(VirtualDrive, "startup.nsh"), "w")
- f.write("BOOT SUCCESS !!! \n")
- ## add commands here
- f.write("reset -s\n")
- f.close()
-
- ret = RunCmd(cmd, args)
-
- if ret == 0xc0000005:
- #for some reason getting a c0000005 on successful return
- return 0
-
- return ret
-
-
-
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
diff --git a/OvmfPkg/PlatformCI/PlatformBuild.py b/OvmfPkg/PlatformCI/PlatformBuildLib.py
similarity index 87%
copy from OvmfPkg/PlatformCI/PlatformBuild.py
copy to OvmfPkg/PlatformCI/PlatformBuildLib.py
index 627bb7b992db..90ac0b29a892 100644
--- a/OvmfPkg/PlatformCI/PlatformBuild.py
+++ b/OvmfPkg/PlatformCI/PlatformBuildLib.py
@@ -17,35 +17,6 @@ from edk2toolext.invocables.edk2_pr_eval import PrEvalSettingsManager
from edk2toollib.utility_functions import RunCmd
- # ####################################################################################### #
- # Common Configuration #
- # ####################################################################################### #
-class CommonPlatform():
- ''' Common settings for this platform. Define static data here and use
- for the different parts of stuart
- '''
- PackagesSupported = ("OvmfPkg",)
- ArchSupported = ("IA32", "X64")
- TargetsSupported = ("DEBUG", "RELEASE", "NOOPT")
- Scopes = ('ovmf', 'edk2-build')
- WorkspaceRoot = os.path.realpath(os.path.join(
- os.path.dirname(os.path.abspath(__file__)), "..", ".."))
-
- @classmethod
- def GetDscName(cls, ArchCsv: str) -> str:
- ''' return the DSC given the architectures requested.
-
- ArchCsv: csv string containing all architectures to build
- '''
- dsc = "OvmfPkg"
- if "IA32" in ArchCsv.upper().split(","):
- dsc += "Ia32"
- if "X64" in ArchCsv.upper().split(","):
- dsc += "X64"
- dsc += ".dsc"
- return dsc
-
-
# ####################################################################################### #
# Configuration for Update & Setup #
# ####################################################################################### #
@@ -249,6 +220,3 @@ class PlatformBuilder( UefiBuilder, BuildSettingsManager):
return 0
return ret
-
-
-
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PATCH v4 2/7] OvmfPkg/PlatformCI: add QEMU_SKIP
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 1/7] OvmfPkg/PlatformCI: factor out PlatformBuildLib.py Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 3/7] OvmfPkg/PlatformCI: add BhyveBuild.py Gerd Hoffmann
` (5 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Skip the qemu boot test in case QEMU_SKIP is set to true.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
OvmfPkg/PlatformCI/PlatformBuildLib.py | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/OvmfPkg/PlatformCI/PlatformBuildLib.py b/OvmfPkg/PlatformCI/PlatformBuildLib.py
index 90ac0b29a892..bfef9849c749 100644
--- a/OvmfPkg/PlatformCI/PlatformBuildLib.py
+++ b/OvmfPkg/PlatformCI/PlatformBuildLib.py
@@ -183,6 +183,11 @@ class PlatformBuilder( UefiBuilder, BuildSettingsManager):
os.makedirs(VirtualDrive, exist_ok=True)
OutputPath_FV = os.path.join(self.env.GetValue("BUILD_OUTPUT_BASE"), "FV")
+ if (self.env.GetValue("QEMU_SKIP") and
+ self.env.GetValue("QEMU_SKIP").upper() == "TRUE"):
+ logging.info("skipping qemu boot test")
+ return 0
+
#
# QEMU must be on the path
#
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PATCH v4 3/7] OvmfPkg/PlatformCI: add BhyveBuild.py
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 1/7] OvmfPkg/PlatformCI: factor out PlatformBuildLib.py Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 2/7] OvmfPkg/PlatformCI: add QEMU_SKIP Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 9:26 ` Philippe Mathieu-Daudé
2021-12-13 8:14 ` [PATCH v4 4/7] OvmfPkg/PlatformCI: add MicrovmBuild.py Gerd Hoffmann
` (4 subsequent siblings)
7 siblings, 1 reply; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Add build test for OvmfPkg/Bhyve.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
---
.../.azurepipelines/Ubuntu-GCC5.yml | 9 +++++
OvmfPkg/PlatformCI/BhyveBuild.py | 37 +++++++++++++++++++
2 files changed, 46 insertions(+)
create mode 100644 OvmfPkg/PlatformCI/BhyveBuild.py
diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index 7117b86b8177..cc37a2be8f93 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -113,6 +113,15 @@ jobs:
Build.Target: "NOOPT"
Run.Flags: $(run_flags)
Run: $(should_run)
+
+ BHYVE_X64_DEBUG:
+ Build.File: "$(package)/PlatformCI/BhyveBuild.py"
+ Build.Arch: "X64"
+ Build.Flags: ""
+ Build.Target: "DEBUG"
+ # this build is for freebsd bhyve not qemu-kvm
+ Run.Flags: "QEMU_SKIP=TRUE"
+ Run: $(should_run)
workspace:
clean: all
diff --git a/OvmfPkg/PlatformCI/BhyveBuild.py b/OvmfPkg/PlatformCI/BhyveBuild.py
new file mode 100644
index 000000000000..c2de96d23673
--- /dev/null
+++ b/OvmfPkg/PlatformCI/BhyveBuild.py
@@ -0,0 +1,37 @@
+# @file
+# Script to Build OVMF UEFI firmware
+#
+# Copyright (c) Microsoft Corporation.
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+##
+import os
+import sys
+
+sys.path.append(os.path.dirname(os.path.abspath(__file__)))
+from PlatformBuildLib import SettingsManager
+from PlatformBuildLib import PlatformBuilder
+
+ # ####################################################################################### #
+ # Common Configuration #
+ # ####################################################################################### #
+class CommonPlatform():
+ ''' Common settings for this platform. Define static data here and use
+ for the different parts of stuart
+ '''
+ PackagesSupported = ("OvmfPkg",)
+ ArchSupported = ("X64",)
+ TargetsSupported = ("DEBUG", "RELEASE", "NOOPT")
+ Scopes = ('ovmf', 'edk2-build')
+ WorkspaceRoot = os.path.realpath(os.path.join(
+ os.path.dirname(os.path.abspath(__file__)), "..", ".."))
+
+ @classmethod
+ def GetDscName(cls, ArchCsv: str) -> str:
+ ''' return the DSC given the architectures requested.
+
+ ArchCsv: csv string containing all architectures to build
+ '''
+ return "Bhyve/BhyveX64.dsc"
+
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH v4 3/7] OvmfPkg/PlatformCI: add BhyveBuild.py
2021-12-13 8:14 ` [PATCH v4 3/7] OvmfPkg/PlatformCI: add BhyveBuild.py Gerd Hoffmann
@ 2021-12-13 9:26 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-12-13 9:26 UTC (permalink / raw)
To: Gerd Hoffmann, devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Ard Biesheuvel, Sean Brogan, Jordan Justen, Pawel Polawski,
Bret Barkelew, Ard Biesheuvel
On 12/13/21 09:14, Gerd Hoffmann wrote:
> Add build test for OvmfPkg/Bhyve.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
> Acked-by: Ard Biesheuvel <ardb@kernel.org>
> ---
> .../.azurepipelines/Ubuntu-GCC5.yml | 9 +++++
> OvmfPkg/PlatformCI/BhyveBuild.py | 37 +++++++++++++++++++
> 2 files changed, 46 insertions(+)
> create mode 100644 OvmfPkg/PlatformCI/BhyveBuild.py
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v4 4/7] OvmfPkg/PlatformCI: add MicrovmBuild.py
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
` (2 preceding siblings ...)
2021-12-13 8:14 ` [PATCH v4 3/7] OvmfPkg/PlatformCI: add BhyveBuild.py Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 5/7] OvmfPkg/PlatformCI: add AmdSevBuild.py Gerd Hoffmann
` (3 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Add build test for OvmfPkg/Microvm.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
.../.azurepipelines/Ubuntu-GCC5.yml | 10 +++++
OvmfPkg/PlatformCI/MicrovmBuild.py | 37 +++++++++++++++++++
2 files changed, 47 insertions(+)
create mode 100644 OvmfPkg/PlatformCI/MicrovmBuild.py
diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index cc37a2be8f93..cd7eaae54ec7 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -122,6 +122,16 @@ jobs:
# this build is for freebsd bhyve not qemu-kvm
Run.Flags: "QEMU_SKIP=TRUE"
Run: $(should_run)
+
+ MICROVM_X64_DEBUG:
+ Build.File: "$(package)/PlatformCI/MicrovmBuild.py"
+ Build.Arch: "X64"
+ Build.Flags: ""
+ Build.Target: "DEBUG"
+ # no storage supported by microvm build
+ # can't boot to uefi shell -> skip test
+ Run.Flags: "QEMU_SKIP=TRUE"
+ Run: $(should_run)
workspace:
clean: all
diff --git a/OvmfPkg/PlatformCI/MicrovmBuild.py b/OvmfPkg/PlatformCI/MicrovmBuild.py
new file mode 100644
index 000000000000..7ad114969611
--- /dev/null
+++ b/OvmfPkg/PlatformCI/MicrovmBuild.py
@@ -0,0 +1,37 @@
+# @file
+# Script to Build OVMF UEFI firmware
+#
+# Copyright (c) Microsoft Corporation.
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+##
+import os
+import sys
+
+sys.path.append(os.path.dirname(os.path.abspath(__file__)))
+from PlatformBuildLib import SettingsManager
+from PlatformBuildLib import PlatformBuilder
+
+ # ####################################################################################### #
+ # Common Configuration #
+ # ####################################################################################### #
+class CommonPlatform():
+ ''' Common settings for this platform. Define static data here and use
+ for the different parts of stuart
+ '''
+ PackagesSupported = ("OvmfPkg",)
+ ArchSupported = ("X64",)
+ TargetsSupported = ("DEBUG", "RELEASE", "NOOPT")
+ Scopes = ('ovmf', 'edk2-build')
+ WorkspaceRoot = os.path.realpath(os.path.join(
+ os.path.dirname(os.path.abspath(__file__)), "..", ".."))
+
+ @classmethod
+ def GetDscName(cls, ArchCsv: str) -> str:
+ ''' return the DSC given the architectures requested.
+
+ ArchCsv: csv string containing all architectures to build
+ '''
+ return "Microvm/MicrovmX64.dsc"
+
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PATCH v4 5/7] OvmfPkg/PlatformCI: add AmdSevBuild.py
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
` (3 preceding siblings ...)
2021-12-13 8:14 ` [PATCH v4 4/7] OvmfPkg/PlatformCI: add MicrovmBuild.py Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 8:14 ` [PATCH v4 6/7] OvmfPkg/PlatformCI: dummy grub.efi for AmdSev Gerd Hoffmann
` (2 subsequent siblings)
7 siblings, 0 replies; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Add build test for OvmfPkg/AmdSev.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
.../.azurepipelines/Ubuntu-GCC5.yml | 10 +++++
OvmfPkg/PlatformCI/AmdSevBuild.py | 37 +++++++++++++++++++
2 files changed, 47 insertions(+)
create mode 100644 OvmfPkg/PlatformCI/AmdSevBuild.py
diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index cd7eaae54ec7..803bc8bbba6a 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -114,6 +114,16 @@ jobs:
Run.Flags: $(run_flags)
Run: $(should_run)
+ AMDSEV_X64_DEBUG:
+ Build.File: "$(package)/PlatformCI/AmdSevBuild.py"
+ Build.Arch: "X64"
+ Build.Flags: ""
+ Build.Target: "DEBUG"
+ # restricted + stripped down build
+ # can't boot to uefi shell -> skip test
+ Run.Flags: "QEMU_SKIP=TRUE"
+ Run: $(should_run)
+
BHYVE_X64_DEBUG:
Build.File: "$(package)/PlatformCI/BhyveBuild.py"
Build.Arch: "X64"
diff --git a/OvmfPkg/PlatformCI/AmdSevBuild.py b/OvmfPkg/PlatformCI/AmdSevBuild.py
new file mode 100644
index 000000000000..2dd72cfe80d9
--- /dev/null
+++ b/OvmfPkg/PlatformCI/AmdSevBuild.py
@@ -0,0 +1,37 @@
+# @file
+# Script to Build OVMF UEFI firmware
+#
+# Copyright (c) Microsoft Corporation.
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+##
+import os
+import sys
+
+sys.path.append(os.path.dirname(os.path.abspath(__file__)))
+from PlatformBuildLib import SettingsManager
+from PlatformBuildLib import PlatformBuilder
+
+ # ####################################################################################### #
+ # Common Configuration #
+ # ####################################################################################### #
+class CommonPlatform():
+ ''' Common settings for this platform. Define static data here and use
+ for the different parts of stuart
+ '''
+ PackagesSupported = ("OvmfPkg",)
+ ArchSupported = ("X64",)
+ TargetsSupported = ("DEBUG", "RELEASE", "NOOPT")
+ Scopes = ('ovmf', 'edk2-build')
+ WorkspaceRoot = os.path.realpath(os.path.join(
+ os.path.dirname(os.path.abspath(__file__)), "..", ".."))
+
+ @classmethod
+ def GetDscName(cls, ArchCsv: str) -> str:
+ ''' return the DSC given the architectures requested.
+
+ ArchCsv: csv string containing all architectures to build
+ '''
+ return "AmdSev/AmdSevX64.dsc"
+
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* [PATCH v4 6/7] OvmfPkg/PlatformCI: dummy grub.efi for AmdSev
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
` (4 preceding siblings ...)
2021-12-13 8:14 ` [PATCH v4 5/7] OvmfPkg/PlatformCI: add AmdSevBuild.py Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 9:42 ` Philippe Mathieu-Daudé
2021-12-13 8:14 ` [PATCH v4 7/7] OvmfPkg/PlatformCI: add XenBuild.py Gerd Hoffmann
2021-12-13 11:37 ` [edk2-devel] [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Ard Biesheuvel
7 siblings, 1 reply; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew,
Jiewen Yao, Ard Biesheuvel
Building grub.efi for AmdSev is difficult because it depends on patches
not yet merged to upstream grub. So shortcut the grub build by simply
creating an empty grub.efi file. That allows to at least build-test the
AmdSev variant.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Dov Murik <dovmurik@linux.ibm.com>
---
OvmfPkg/PlatformCI/AmdSevBuild.py | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/OvmfPkg/PlatformCI/AmdSevBuild.py b/OvmfPkg/PlatformCI/AmdSevBuild.py
index 2dd72cfe80d9..816caafb0084 100644
--- a/OvmfPkg/PlatformCI/AmdSevBuild.py
+++ b/OvmfPkg/PlatformCI/AmdSevBuild.py
@@ -6,6 +6,7 @@
##
import os
import sys
+import subprocess
sys.path.append(os.path.dirname(os.path.abspath(__file__)))
from PlatformBuildLib import SettingsManager
@@ -35,3 +36,7 @@ class CommonPlatform():
import PlatformBuildLib
PlatformBuildLib.CommonPlatform = CommonPlatform
+
+# hack alert -- create dummy grub.efi
+subprocess.run(['touch', 'OvmfPkg/AmdSev/Grub/grub.efi'])
+subprocess.run(['ls', '-l', '--sort=time', 'OvmfPkg/AmdSev/Grub'])
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH v4 6/7] OvmfPkg/PlatformCI: dummy grub.efi for AmdSev
2021-12-13 8:14 ` [PATCH v4 6/7] OvmfPkg/PlatformCI: dummy grub.efi for AmdSev Gerd Hoffmann
@ 2021-12-13 9:42 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-12-13 9:42 UTC (permalink / raw)
To: Gerd Hoffmann, devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Ard Biesheuvel, Sean Brogan, Jordan Justen, Pawel Polawski,
Bret Barkelew, Ard Biesheuvel
On 12/13/21 09:14, Gerd Hoffmann wrote:
> Building grub.efi for AmdSev is difficult because it depends on patches
> not yet merged to upstream grub. So shortcut the grub build by simply
> creating an empty grub.efi file. That allows to at least build-test the
> AmdSev variant.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
> Acked-by: Ard Biesheuvel <ardb@kernel.org>
> Reviewed-by: Dov Murik <dovmurik@linux.ibm.com>
> ---
> OvmfPkg/PlatformCI/AmdSevBuild.py | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/OvmfPkg/PlatformCI/AmdSevBuild.py b/OvmfPkg/PlatformCI/AmdSevBuild.py
> index 2dd72cfe80d9..816caafb0084 100644
> --- a/OvmfPkg/PlatformCI/AmdSevBuild.py
> +++ b/OvmfPkg/PlatformCI/AmdSevBuild.py
> @@ -6,6 +6,7 @@
> ##
> import os
> import sys
> +import subprocess
>
> sys.path.append(os.path.dirname(os.path.abspath(__file__)))
> from PlatformBuildLib import SettingsManager
> @@ -35,3 +36,7 @@ class CommonPlatform():
>
> import PlatformBuildLib
> PlatformBuildLib.CommonPlatform = CommonPlatform
> +
> +# hack alert -- create dummy grub.efi
> +subprocess.run(['touch', 'OvmfPkg/AmdSev/Grub/grub.efi'])
> +subprocess.run(['ls', '-l', '--sort=time', 'OvmfPkg/AmdSev/Grub'])
Why run 'ls'?
What about replacing subprocess by open?
open('OvmfPkg/AmdSev/Grub/grub.efi', 'a').close()
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v4 7/7] OvmfPkg/PlatformCI: add XenBuild.py
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
` (5 preceding siblings ...)
2021-12-13 8:14 ` [PATCH v4 6/7] OvmfPkg/PlatformCI: dummy grub.efi for AmdSev Gerd Hoffmann
@ 2021-12-13 8:14 ` Gerd Hoffmann
2021-12-13 9:26 ` Philippe Mathieu-Daudé
2021-12-13 11:37 ` [edk2-devel] [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Ard Biesheuvel
7 siblings, 1 reply; 12+ messages in thread
From: Gerd Hoffmann @ 2021-12-13 8:14 UTC (permalink / raw)
To: devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Gerd Hoffmann, Ard Biesheuvel,
Sean Brogan, Jordan Justen, Pawel Polawski, Bret Barkelew
Add build test for OvmfXen.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
.../.azurepipelines/Ubuntu-GCC5.yml | 9 +++++
OvmfPkg/PlatformCI/XenBuild.py | 37 +++++++++++++++++++
2 files changed, 46 insertions(+)
create mode 100644 OvmfPkg/PlatformCI/XenBuild.py
diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index 803bc8bbba6a..2de0688cfeb1 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -142,6 +142,15 @@ jobs:
# can't boot to uefi shell -> skip test
Run.Flags: "QEMU_SKIP=TRUE"
Run: $(should_run)
+
+ OVMF_XEN_X64_DEBUG:
+ Build.File: "$(package)/PlatformCI/XenBuild.py"
+ Build.Arch: "X64"
+ Build.Flags: ""
+ Build.Target: "DEBUG"
+ # this build is for xen not qemu-kvm
+ Run.Flags: "QEMU_SKIP=TRUE"
+ Run: $(should_run)
workspace:
clean: all
diff --git a/OvmfPkg/PlatformCI/XenBuild.py b/OvmfPkg/PlatformCI/XenBuild.py
new file mode 100644
index 000000000000..25a70a177d65
--- /dev/null
+++ b/OvmfPkg/PlatformCI/XenBuild.py
@@ -0,0 +1,37 @@
+# @file
+# Script to Build OVMF UEFI firmware
+#
+# Copyright (c) Microsoft Corporation.
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+##
+import os
+import sys
+
+sys.path.append(os.path.dirname(os.path.abspath(__file__)))
+from PlatformBuildLib import SettingsManager
+from PlatformBuildLib import PlatformBuilder
+
+ # ####################################################################################### #
+ # Common Configuration #
+ # ####################################################################################### #
+class CommonPlatform():
+ ''' Common settings for this platform. Define static data here and use
+ for the different parts of stuart
+ '''
+ PackagesSupported = ("OvmfPkg",)
+ ArchSupported = ("X64",)
+ TargetsSupported = ("DEBUG", "RELEASE", "NOOPT")
+ Scopes = ('ovmf', 'edk2-build')
+ WorkspaceRoot = os.path.realpath(os.path.join(
+ os.path.dirname(os.path.abspath(__file__)), "..", ".."))
+
+ @classmethod
+ def GetDscName(cls, ArchCsv: str) -> str:
+ ''' return the DSC given the architectures requested.
+
+ ArchCsv: csv string containing all architectures to build
+ '''
+ return "OvmfXen.dsc"
+
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
--
2.33.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH v4 7/7] OvmfPkg/PlatformCI: add XenBuild.py
2021-12-13 8:14 ` [PATCH v4 7/7] OvmfPkg/PlatformCI: add XenBuild.py Gerd Hoffmann
@ 2021-12-13 9:26 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-12-13 9:26 UTC (permalink / raw)
To: Gerd Hoffmann, devel
Cc: Michael D Kinney, dovmurik, Jiewen Yao, Liming Gao,
Ard Biesheuvel, Sean Brogan, Jordan Justen, Pawel Polawski,
Bret Barkelew
On 12/13/21 09:14, Gerd Hoffmann wrote:
> Add build test for OvmfXen.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
> .../.azurepipelines/Ubuntu-GCC5.yml | 9 +++++
> OvmfPkg/PlatformCI/XenBuild.py | 37 +++++++++++++++++++
> 2 files changed, 46 insertions(+)
> create mode 100644 OvmfPkg/PlatformCI/XenBuild.py
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [edk2-devel] [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm
2021-12-13 8:14 [PATCH v4 0/7] OvmfPkg/PlatformCI: hook up AmdSev, Bhyve and Microvm Gerd Hoffmann
` (6 preceding siblings ...)
2021-12-13 8:14 ` [PATCH v4 7/7] OvmfPkg/PlatformCI: add XenBuild.py Gerd Hoffmann
@ 2021-12-13 11:37 ` Ard Biesheuvel
7 siblings, 0 replies; 12+ messages in thread
From: Ard Biesheuvel @ 2021-12-13 11:37 UTC (permalink / raw)
To: edk2-devel-groups-io, Gerd Hoffmann
Cc: Michael D Kinney, Dov Murik, Jiewen Yao, Liming Gao,
Philippe Mathieu-Daudé, Ard Biesheuvel, Sean Brogan,
Jordan Justen, Pawel Polawski, Bret Barkelew
On Mon, 13 Dec 2021 at 09:15, Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> v4:
> - pick up review tags.
> - rebase to latest master.
>
> v3:
> - pick up Acked-by:
> - use __file__ to extend path
> v2:
> - hook up OvmfXen too.
> - add CI maintainers to Cc.
>
> Gerd Hoffmann (7):
> OvmfPkg/PlatformCI: factor out PlatformBuildLib.py
> OvmfPkg/PlatformCI: add QEMU_SKIP
> OvmfPkg/PlatformCI: add BhyveBuild.py
> OvmfPkg/PlatformCI: add MicrovmBuild.py
> OvmfPkg/PlatformCI: add AmdSevBuild.py
> OvmfPkg/PlatformCI: dummy grub.efi for AmdSev
> OvmfPkg/PlatformCI: add XenBuild.py
>
Merged as #2294
Thanks,
> .../.azurepipelines/Ubuntu-GCC5.yml | 38 +++
> OvmfPkg/PlatformCI/AmdSevBuild.py | 42 ++++
> OvmfPkg/PlatformCI/BhyveBuild.py | 37 +++
> OvmfPkg/PlatformCI/MicrovmBuild.py | 37 +++
> OvmfPkg/PlatformCI/PlatformBuild.py | 223 +-----------------
> .../{PlatformBuild.py => PlatformBuildLib.py} | 37 +--
> OvmfPkg/PlatformCI/XenBuild.py | 37 +++
> 7 files changed, 202 insertions(+), 249 deletions(-)
> create mode 100644 OvmfPkg/PlatformCI/AmdSevBuild.py
> create mode 100644 OvmfPkg/PlatformCI/BhyveBuild.py
> create mode 100644 OvmfPkg/PlatformCI/MicrovmBuild.py
> copy OvmfPkg/PlatformCI/{PlatformBuild.py => PlatformBuildLib.py} (87%)
> create mode 100644 OvmfPkg/PlatformCI/XenBuild.py
>
> --
> 2.33.1
>
>
>
>
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread