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

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

Merged - https://github.com/tianocore/edk2/pull/3038

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao, Jiewen
> Sent: Friday, July 1, 2022 2:09 PM
> To: Xu, Min M <min.m.xu@intel.com>; devel@edk2.groups.io
> Cc: Aktas, Erdem <erdemaktas@google.com>; James Bottomley
> <jejb@linux.ibm.com>; Tom Lendacky <thomas.lendacky@amd.com>; Gerd
> Hoffmann <kraxel@redhat.com>; Ard Biesheuvel <ardb@kernel.org>
> Subject: Re: [edk2-devel] [PATCH 1/1] OvmfPkg/PlatformCI: add
> IntelTdxBuild.py
> 
> 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
> 
> 
> 
> 
>