public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py
       [not found] <cover.1652614321.git.min.m.xu@intel.com>
@ 2022-05-15 11:33 ` Min Xu
  2022-05-17  7:56   ` Gerd Hoffmann
                     ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Min Xu @ 2022-05-15 11:33 UTC (permalink / raw)
  To: devel
  Cc: Min Xu, Erdem Aktas, James Bottomley, Tom Lendacky, Gerd Hoffmann,
	Jiewen Yao, Ard Biesheuvel

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py
  2022-05-15 11:33 ` [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py Min Xu
@ 2022-05-17  7:56   ` Gerd Hoffmann
  2022-07-01  6:08   ` Yao, Jiewen
       [not found]   ` <16FD9F9CF906220C.5468@groups.io>
  2 siblings, 0 replies; 4+ messages in thread
From: Gerd Hoffmann @ 2022-05-17  7:56 UTC (permalink / raw)
  To: Min Xu
  Cc: devel, Erdem Aktas, James Bottomley, Tom Lendacky, Jiewen Yao,
	Ard Biesheuvel

On Sun, May 15, 2022 at 07:33:04PM +0800, Min Xu wrote:
> 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>

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py
  2022-05-15 11:33 ` [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py Min Xu
  2022-05-17  7:56   ` Gerd Hoffmann
@ 2022-07-01  6:08   ` Yao, Jiewen
       [not found]   ` <16FD9F9CF906220C.5468@groups.io>
  2 siblings, 0 replies; 4+ messages in thread
From: Yao, Jiewen @ 2022-07-01  6:08 UTC (permalink / raw)
  To: Xu, Min M, devel@edk2.groups.io
  Cc: Aktas, Erdem, James Bottomley, Tom Lendacky, Gerd Hoffmann,
	Ard Biesheuvel

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [edk2-devel] [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py
       [not found]   ` <16FD9F9CF906220C.5468@groups.io>
@ 2022-07-01  7:00     ` Yao, Jiewen
  0 siblings, 0 replies; 4+ messages in thread
From: Yao, Jiewen @ 2022-07-01  7:00 UTC (permalink / raw)
  To: devel@edk2.groups.io, Yao, Jiewen, Xu, Min M
  Cc: Aktas, Erdem, James Bottomley, Tom Lendacky, Gerd Hoffmann,
	Ard Biesheuvel

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
> 
> 
> 
> 
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-07-01  7:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <cover.1652614321.git.min.m.xu@intel.com>
2022-05-15 11:33 ` [PATCH 1/1] OvmfPkg/PlatformCI: add IntelTdxBuild.py Min Xu
2022-05-17  7:56   ` Gerd Hoffmann
2022-07-01  6:08   ` Yao, Jiewen
     [not found]   ` <16FD9F9CF906220C.5468@groups.io>
2022-07-01  7:00     ` [edk2-devel] " Yao, Jiewen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox