[PATCH 1/1] OvmfPkg/PlatformCI: Add CI coverage for RiscVVirtQemu

[PATCH 1/1] OvmfPkg/PlatformCI: Add CI coverage for RiscVVirtQemu

[Sunil V L]

Add support for building RiscVVirtQemu platform in CI.

Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
---
 .../.azurepipelines/Ubuntu-GCC5.yml           |  9 +++++
 OvmfPkg/PlatformCI/QemuBuild.py               | 37 +++++++++++++++++++
 2 files changed, 46 insertions(+)
 create mode 100644 OvmfPkg/PlatformCI/QemuBuild.py

diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index 1cf9bdf8b9c2..ad43f9fad2fd 100644
--- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -174,6 +174,15 @@ jobs:
             Run.Flags: "QEMU_SKIP=TRUE"
             Run: $(should_run)
 
+          QEMU_RISCV64_DEBUG:
+            Build.File: "$(package)/PlatformCI/QemuBuild.py"
+            Build.Arch: "RISCV64"
+            Build.Flags: ""
+            Build.Target: "DEBUG"
+            # this build is for RISC-V qemu virt not qemu-kvm
+            Run.Flags: "QEMU_SKIP=TRUE"
+            Run: $(should_run)
+
     workspace:
       clean: all
 
diff --git a/OvmfPkg/PlatformCI/QemuBuild.py b/OvmfPkg/PlatformCI/QemuBuild.py
new file mode 100644
index 000000000000..c737d98d51b4
--- /dev/null
+++ b/OvmfPkg/PlatformCI/QemuBuild.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 = ("RISCV64",)
+    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 "RiscVVirt/RiscVVirtQemu.dsc"
+
+import PlatformBuildLib
+PlatformBuildLib.CommonPlatform = CommonPlatform
-- 
2.34.1

Re: [PATCH 1/1] OvmfPkg/PlatformCI: Add CI coverage for RiscVVirtQemu

[Gerd Hoffmann]

On Wed, Mar 15, 2023 at 02:41:19PM +0530, Sunil V L wrote:
> Add support for building RiscVVirtQemu platform in CI.
> 
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>

Acked-by: Gerd Hoffmann <kraxel@redhat.com>

Re: [PATCH 1/1] OvmfPkg/PlatformCI: Add CI coverage for RiscVVirtQemu

[Sunil V L]

On Wed, Mar 15, 2023 at 10:25:25AM +0100, Gerd Hoffmann wrote:
> On Wed, Mar 15, 2023 at 02:41:19PM +0530, Sunil V L wrote:
> > Add support for building RiscVVirtQemu platform in CI.
> > 
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
> 
> Acked-by: Gerd Hoffmann <kraxel@redhat.com>
> 
Thanks!, Gerd. Merged this as commit ID 7cfe9048e3ec.

-Sunil