Re: [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>

> -----Original Message-----
> From: Xu, Min M <min.m.xu@intel.com>
> Sent: Sunday, May 15, 2022 7:33 PM
> To: devel@edk2.groups.io
> Cc: Xu, Min M <min.m.xu@intel.com>; Aktas, Erdem
> <erdemaktas@google.com>; James Bottomley <jejb@linux.ibm.com>; Tom
> Lendacky <thomas.lendacky@amd.com>; Gerd Hoffmann <kraxel@redhat.com>;
> Yao, Jiewen <jiewen.yao@intel.com>; Ard Biesheuvel <ardb@kernel.org>
> Subject: [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py
> 
> Add build test for OvmfPkg/IntelTdx
> 
> Cc: Erdem Aktas <erdemaktas@google.com>
> Cc: James Bottomley <jejb@linux.ibm.com>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Jiewen Yao <Jiewen.yao@intel.com>
> Cc: Ard Biesheuvel <ardb@kernel.org>
> Signed-off-by: Min Xu <min.m.xu@intel.com>
> ---
>  .../.azurepipelines/Ubuntu-GCC5.yml           | 11 ++++++
>  OvmfPkg/PlatformCI/IntelTdxBuild.py           | 37 +++++++++++++++++++
>  2 files changed, 48 insertions(+)
>  create mode 100644 OvmfPkg/PlatformCI/IntelTdxBuild.py
> 
> diff --git a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> index 6dec90d9fcaf..7160d95f7e04 100644
> --- a/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> +++ b/OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> @@ -160,6 +160,17 @@ jobs:
>              # this build is for xen not qemu-kvm
>              Run.Flags: "QEMU_SKIP=TRUE"
>              Run: $(should_run)
> +
> +          INTELTDX_X64_DEBUG:
> +            Build.File: "$(package)/PlatformCI/IntelTdxBuild.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/IntelTdxBuild.py
> b/OvmfPkg/PlatformCI/IntelTdxBuild.py
> new file mode 100644
> index 000000000000..4b19630ce3c7
> --- /dev/null
> +++ b/OvmfPkg/PlatformCI/IntelTdxBuild.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 "IntelTdx/IntelTdxX64.dsc"
> +
> +import PlatformBuildLib
> +PlatformBuildLib.CommonPlatform = CommonPlatform
> --
> 2.29.2.windows.2