1. Patchew
  2. EDK2
  3. View series

[edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64

Jianyong Wu posted 1 patch 5 months, 1 week ago
Failed in applying to current master (apply log)
.../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml      | 13 +++++++++++++
.../PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} |  4 ++--
2 files changed, 15 insertions(+), 2 deletions(-)
copy ArmVirtPkg/PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} (89%)
[edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64
Posted by Jianyong Wu 5 months, 1 week ago 
Add the long lost CI for CloudHv on AArch64.
As CloudHv CI works nearly the same way with other VMMs like KvmTool,
thus we can easily create its CI configuration based on KvmTool.

Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
---
 .../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml      | 13 +++++++++++++
 .../PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} |  4 ++--
 2 files changed, 15 insertions(+), 2 deletions(-)
 copy ArmVirtPkg/PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} (89%)

diff --git a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
index d1772a65fc..ab8a2db530 100644
--- a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
+++ b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
@@ -140,6 +140,19 @@ jobs:
             Build.Target: "RELEASE"
             Run: false
 
+          CLOUDHV_AARCH64_DEBUG:
+            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
+            Build.Arch: "AARCH64"
+            Build.Flags: ""
+            Build.Target: "DEBUG"
+            Run: false
+          CLOUDHV_AARCH64_RELEASE:
+            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
+            Build.Arch: "AARCH64"
+            Build.Flags: ""
+            Build.Target: "RELEASE"
+            Run: false
+
     workspace:
       clean: all
 
diff --git a/ArmVirtPkg/PlatformCI/KvmToolBuild.py b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
similarity index 89%
copy from ArmVirtPkg/PlatformCI/KvmToolBuild.py
copy to ArmVirtPkg/PlatformCI/CloudHvBuild.py
index 4d02dba124..06ada39886 100644
--- a/ArmVirtPkg/PlatformCI/KvmToolBuild.py
+++ b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
@@ -19,13 +19,13 @@ class CommonPlatform():
         for the different parts of stuart
     '''
     PackagesSupported = ("ArmVirtPkg",)
-    ArchSupported = ("AARCH64", "ARM")
+    ArchSupported = ("AARCH64")
     TargetsSupported = ("DEBUG", "RELEASE")
     Scopes = ('armvirt', 'edk2-build')
     WorkspaceRoot = os.path.realpath(os.path.join(
         os.path.dirname(os.path.abspath(__file__)), "..", ".."))
 
-    DscName = os.path.join("ArmVirtPkg", "ArmVirtKvmTool.dsc")
+    DscName = os.path.join("ArmVirtPkg", "ArmVirtCloudHv.dsc")
     FvQemuArg = "" # ignored
 
 import PlatformBuildLib
-- 
2.34.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111646): https://edk2.groups.io/g/devel/message/111646
Mute This Topic: https://groups.io/mt/102761729/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64
Posted by Laszlo Ersek 4 months, 2 weeks ago 
On 11/23/23 04:22, Jianyong Wu wrote:
> Add the long lost CI for CloudHv on AArch64.
> As CloudHv CI works nearly the same way with other VMMs like KvmTool,
> thus we can easily create its CI configuration based on KvmTool.
> 
> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
> ---
>  .../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml      | 13 +++++++++++++
>  .../PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} |  4 ++--
>  2 files changed, 15 insertions(+), 2 deletions(-)
>  copy ArmVirtPkg/PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} (89%)
> 
> diff --git a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> index d1772a65fc..ab8a2db530 100644
> --- a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> +++ b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> @@ -140,6 +140,19 @@ jobs:
>              Build.Target: "RELEASE"
>              Run: false
>  
> +          CLOUDHV_AARCH64_DEBUG:
> +            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
> +            Build.Arch: "AARCH64"
> +            Build.Flags: ""
> +            Build.Target: "DEBUG"
> +            Run: false
> +          CLOUDHV_AARCH64_RELEASE:
> +            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
> +            Build.Arch: "AARCH64"
> +            Build.Flags: ""
> +            Build.Target: "RELEASE"
> +            Run: false
> +
>      workspace:
>        clean: all
>  
> diff --git a/ArmVirtPkg/PlatformCI/KvmToolBuild.py b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
> similarity index 89%
> copy from ArmVirtPkg/PlatformCI/KvmToolBuild.py
> copy to ArmVirtPkg/PlatformCI/CloudHvBuild.py
> index 4d02dba124..06ada39886 100644
> --- a/ArmVirtPkg/PlatformCI/KvmToolBuild.py
> +++ b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
> @@ -19,13 +19,13 @@ class CommonPlatform():
>          for the different parts of stuart
>      '''
>      PackagesSupported = ("ArmVirtPkg",)
> -    ArchSupported = ("AARCH64", "ARM")
> +    ArchSupported = ("AARCH64")

Right, and this one change is new in version 2 of the patch.

My R-b stands.

I'm picking this up now.

Laszlo

>      TargetsSupported = ("DEBUG", "RELEASE")
>      Scopes = ('armvirt', 'edk2-build')
>      WorkspaceRoot = os.path.realpath(os.path.join(
>          os.path.dirname(os.path.abspath(__file__)), "..", ".."))
>  
> -    DscName = os.path.join("ArmVirtPkg", "ArmVirtKvmTool.dsc")
> +    DscName = os.path.join("ArmVirtPkg", "ArmVirtCloudHv.dsc")
>      FvQemuArg = "" # ignored
>  
>  import PlatformBuildLib



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112317): https://edk2.groups.io/g/devel/message/112317
Mute This Topic: https://groups.io/mt/102761729/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64
Posted by Laszlo Ersek 4 months, 2 weeks ago 
Hi Jianyong,

On 12/11/23 17:31, Laszlo Ersek wrote:
> On 11/23/23 04:22, Jianyong Wu wrote:
>> Add the long lost CI for CloudHv on AArch64.
>> As CloudHv CI works nearly the same way with other VMMs like KvmTool,
>> thus we can easily create its CI configuration based on KvmTool.
>>
>> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
>> Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
>> ---
>>  .../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml      | 13 +++++++++++++
>>  .../PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} |  4 ++--
>>  2 files changed, 15 insertions(+), 2 deletions(-)
>>  copy ArmVirtPkg/PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} (89%)
>>
>> diff --git a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> index d1772a65fc..ab8a2db530 100644
>> --- a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> +++ b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> @@ -140,6 +140,19 @@ jobs:
>>              Build.Target: "RELEASE"
>>              Run: false
>>  
>> +          CLOUDHV_AARCH64_DEBUG:
>> +            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
>> +            Build.Arch: "AARCH64"
>> +            Build.Flags: ""
>> +            Build.Target: "DEBUG"
>> +            Run: false
>> +          CLOUDHV_AARCH64_RELEASE:
>> +            Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
>> +            Build.Arch: "AARCH64"
>> +            Build.Flags: ""
>> +            Build.Target: "RELEASE"
>> +            Run: false
>> +
>>      workspace:
>>        clean: all
>>  
>> diff --git a/ArmVirtPkg/PlatformCI/KvmToolBuild.py b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> similarity index 89%
>> copy from ArmVirtPkg/PlatformCI/KvmToolBuild.py
>> copy to ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> index 4d02dba124..06ada39886 100644
>> --- a/ArmVirtPkg/PlatformCI/KvmToolBuild.py
>> +++ b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> @@ -19,13 +19,13 @@ class CommonPlatform():
>>          for the different parts of stuart
>>      '''
>>      PackagesSupported = ("ArmVirtPkg",)
>> -    ArchSupported = ("AARCH64", "ARM")
>> +    ArchSupported = ("AARCH64")
> 
> Right, and this one change is new in version 2 of the patch.
> 
> My R-b stands.
> 
> I'm picking this up now.
> 

The CI run failed for this patch when I tried to merge it; can you
please review <https://github.com/tianocore/edk2/pull/5137>?

Thanks,
Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112321): https://edk2.groups.io/g/devel/message/112321
Mute This Topic: https://groups.io/mt/102761729/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64
Posted by Sami Mujawar 4 months, 2 weeks ago 
Hi Laszlo,

I will look into this and provide a fix shortly.

Regards,

Sami Mujawar

On 11/12/2023, 17:21, "Laszlo Ersek" <lersek@redhat.com <mailto:lersek@redhat.com>> wrote:


Hi Jianyong,


On 12/11/23 17:31, Laszlo Ersek wrote:
> On 11/23/23 04:22, Jianyong Wu wrote:
>> Add the long lost CI for CloudHv on AArch64.
>> As CloudHv CI works nearly the same way with other VMMs like KvmTool,
>> thus we can easily create its CI configuration based on KvmTool.
>>
>> Reviewed-by: Laszlo Ersek <lersek@redhat.com <mailto:lersek@redhat.com>>
>> Signed-off-by: Jianyong Wu <jianyong.wu@arm.com <mailto:jianyong.wu@arm.com>>
>> ---
>> .../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 13 +++++++++++++
>> .../PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} | 4 ++--
>> 2 files changed, 15 insertions(+), 2 deletions(-)
>> copy ArmVirtPkg/PlatformCI/{KvmToolBuild.py => CloudHvBuild.py} (89%)
>>
>> diff --git a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> index d1772a65fc..ab8a2db530 100644
>> --- a/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> +++ b/ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
>> @@ -140,6 +140,19 @@ jobs:
>> Build.Target: "RELEASE"
>> Run: false
>> 
>> + CLOUDHV_AARCH64_DEBUG:
>> + Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
>> + Build.Arch: "AARCH64"
>> + Build.Flags: ""
>> + Build.Target: "DEBUG"
>> + Run: false
>> + CLOUDHV_AARCH64_RELEASE:
>> + Build.File: "$(package)/PlatformCI/CloudHvBuild.py"
>> + Build.Arch: "AARCH64"
>> + Build.Flags: ""
>> + Build.Target: "RELEASE"
>> + Run: false
>> +
>> workspace:
>> clean: all
>> 
>> diff --git a/ArmVirtPkg/PlatformCI/KvmToolBuild.py b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> similarity index 89%
>> copy from ArmVirtPkg/PlatformCI/KvmToolBuild.py
>> copy to ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> index 4d02dba124..06ada39886 100644
>> --- a/ArmVirtPkg/PlatformCI/KvmToolBuild.py
>> +++ b/ArmVirtPkg/PlatformCI/CloudHvBuild.py
>> @@ -19,13 +19,13 @@ class CommonPlatform():
>> for the different parts of stuart
>> '''
>> PackagesSupported = ("ArmVirtPkg",)
>> - ArchSupported = ("AARCH64", "ARM")
>> + ArchSupported = ("AARCH64")
> 
> Right, and this one change is new in version 2 of the patch.
> 
> My R-b stands.
> 
> I'm picking this up now.
> 


The CI run failed for this patch when I tried to merge it; can you
please review <https://github.com/tianocore/edk2/pull/5137>? <https://github.com/tianocore/edk2/pull/5137&gt;?>


Thanks,
Laszlo







-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112469): https://edk2.groups.io/g/devel/message/112469
Mute This Topic: https://groups.io/mt/102761729/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v2] CloudHv: Add CI for CloudHv on AArch64
Posted by Sami Mujawar 4 months, 2 weeks ago 
Hi Laszlo,

I have posted a v3 patch to fix the issue at https://edk2.groups.io/g/devel/message/112485
I have done a draft pull request to check the v3 patch passes CI and this can be seen at https://github.com/tianocore/edk2/pull/5145

Regards,

Sami Mujawar


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112486): https://edk2.groups.io/g/devel/message/112486
Mute This Topic: https://groups.io/mt/102761729/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.