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

2023-11-22 Thread Jianyong Wu
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 Signed-off-by: Jianyong Wu --- .../PlatformCI/.azurepipelines/Ubuntu-GCC5.yml | 13

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

2023-11-19 Thread Jianyong Wu
Thanks Laszlo. Will fix all comments next version. Thanks Jianyong > -Original Message- > From: Laszlo Ersek > Sent: 2023年11月18日 5:54 > To: devel@edk2.groups.io; Jianyong Wu ; Sami > Mujawar > Cc: ardb+tianoc...@kernel.org > Subject: Re: [edk2-devel] [PATCH] CloudH

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

2023-11-14 Thread Jianyong Wu
Add the long lost CI for CloudHv on AArch64. Signed-off-by: Jianyong Wu --- .../.azurepipelines/Ubuntu-GCC5.yml | 13 ArmVirtPkg/PlatformCI/CloudHvBuild.py | 32 +++ 2 files changed, 45 insertions(+) create mode 100644 ArmVirtPkg/PlatformCI

[edk2-devel] [PATCH] OvmfPkg/CloudHv: update Maintainers.txt entry

2023-07-08 Thread Jianyong Wu
Add Jianyong Wu and Anatol Belski as co-reviewer for OvmfPkg/CloudHv to replace Sebastien Boeuf. Signed-off-by: Jianyong Wu Signed-off-by: Anatol Belski --- Maintainers.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Maintainers.txt b/Maintainers.txt index c4327641de

Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2023-05-22 Thread Jianyong Wu
Hello Sami, Any comments about the memory layout below? Thanks Jianyong From: Jianyong Wu Sent: 2023年4月24日 10:37 To: devel@edk2.groups.io; Jianyong Wu ; Sami Mujawar Cc: ardb+tianoc...@kernel.org; Justin He ; nd Subject: RE: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into

Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2023-04-23 Thread Jianyong Wu
eck it in edk2 by checking if there is 2MB region reserved for Cpu stack. WDYT? Thanks Jianyong From: devel@edk2.groups.io On Behalf Of Jianyong Wu via groups.io Sent: 2023年1月11日 17:28 To: Sami Mujawar ; devel@edk2.groups.io Cc: ardb+tianoc...@kernel.org; Justin He ; nd Subject: Re: [edk2-deve

[edk2-devel] [PATCH v2] ArmVirtPkg: can't find gUefiOvmfPkgTokenSpaceGuid

2023-03-28 Thread Jianyong Wu
commit f13264b34 introduces a bug for CloudHv as OvmfPkg/OvmfPkg.dec is missing in CloudHvHasAcpiDtDxe.inf which leads to gUefiOvmfPkgTokenSpaceGuid found nowhere when build. Reviewed-by: Sami Mujawar Signed-off-by: Jianyong Wu --- ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.inf

Re: [edk2-devel] [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid

2023-03-28 Thread Jianyong Wu
Hi Sami, Inline replay. > -Original Message- > From: Sami Mujawar > Sent: 2023年3月28日 16:34 > To: Jianyong Wu > Cc: devel@edk2.groups.io; Ard Biesheuvel ; > suni...@ventanamicro.com > Subject: Re: [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid > >

Re: [edk2-devel] [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid

2023-03-28 Thread Jianyong Wu
Hi Sami, I have never added CI for CloudHv. So, should I add it? Also, I will update this patch based on your comment and send the second version. Thanks, Jianyong > -Original Message- > From: Sami Mujawar > Sent: 2023年3月28日 15:52 > To: Sunil V L ; Jianyong Wu > &

Re: [edk2-devel] [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid

2023-03-27 Thread Jianyong Wu
Add Ard Biesheuvel and Sunil V L to cc list. > -Original Message- > From: Jianyong Wu > Sent: 2023年2月17日 14:48 > To: devel@edk2.groups.io; Sami Mujawar > Cc: Jianyong Wu > Subject: [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid > > commit f

[edk2-devel] [PATCH] CloudHv: can't find gUefiOvmfPkgTokenSpaceGuid

2023-02-16 Thread Jianyong Wu
commit f13264b34 introduces a bug for CloudHv as OvmfPkg/OvmfPkg.dec is missing in CloudHvHasAcpiDtDxe.inf which leads to gUefiOvmfPkgTokenSpaceGuid found nowhere. Signed-off-by: Jianyong Wu --- ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.inf | 1 + 1 file changed, 1 insertion

Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2023-01-11 Thread Jianyong Wu
the top of memory, unless the memory size is too small. WDYT? Thanks Jianyong From: Sami Mujawar Sent: Wednesday, January 11, 2023 4:35 PM To: Jianyong Wu ; devel@edk2.groups.io Cc: ardb+tianoc...@kernel.org; Justin He ; nd Subject: Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load

Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2023-01-11 Thread Jianyong Wu
Hi Sami, Happy new year and hope you are well. Following up last discussion. I try to add null lib instead of QemuBootOrderLib and it works. If there are no other comments, I will post the change in the next version. Regards, Jianyong From: devel@edk2.groups.io On Behalf Of Jianyong Wu via

Re: [edk2-devel] [PATCH 1/3] CloudHv:arm: add kernel load fs driver

2022-11-22 Thread Jianyong Wu
d You can see that the kernel load role is taken by edk2 and is faster compared to grub load. What's more we can specify kernel image without loading kernel from disk. Any problem, contact me. Thanks Jianyong > -Original Message- > From: Sami Mujawar > Sent: Tuesday,

Re: [edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2022-11-22 Thread Jianyong Wu
Hi Sami, Inline reply From: Sami Mujawar Sent: Tuesday, November 22, 2022 11:48 PM To: Jianyong Wu ; devel@edk2.groups.io Cc: ardb+tianoc...@kernel.org; Justin He ; nd Subject: Re: [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf Hi Jianyong, Please find my response inline

Re: [edk2-devel] [PATCH 2/3] CloudHv:arm: build hob for kernel image memory as read-only

2022-11-22 Thread Jianyong Wu
Hi Sami, Inline reply. > -Original Message- > From: Sami Mujawar > Sent: Tuesday, November 22, 2022 11:48 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ardb+tianoc...@kernel.org; Justin He ; nd > > Subject: Re: [PATCH 2/3] CloudHv:arm: build hob for kernel i

Re: [edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

2022-11-15 Thread Jianyong Wu
Thanks Sami! > -Original Message- > From: Sami Mujawar > Sent: Tuesday, November 15, 2022 4:52 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ardb+tianoc...@kernel.org; Justin He ; nd > > Subject: Re: [PATCH 0/3] CloudHv:arm: Enable direct kernel boot > >

Re: [edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

2022-11-14 Thread Jianyong Wu
Hello, Long time silent. Any comments? Thanks Jianyong > -Original Message- > From: Jianyong Wu > Sent: Friday, September 16, 2022 10:46 AM > To: devel@edk2.groups.io; Sami Mujawar > Cc: ardb+tianoc...@kernel.org; Justin He ; Jianyong > Wu > Subject: [PATCH 0/3

[edk2-devel] [PATCH 3/3] CloudHv:arm: add kernel load driver into dsc/fdf

2022-09-15 Thread Jianyong Wu
As CloudHv kernel load fs driver is implemented, add it into dsc/fdf. Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc | 8 +++- ArmVirtPkg/ArmVirtCloudHv.fdf | 1 + .../CloudHvKernelLoaderFsDxe/CloudHvKernelLoaderFsDxe.inf

[edk2-devel] [PATCH 2/3] CloudHv:arm: build hob for kernel image memory as read-only

2022-09-15 Thread Jianyong Wu
As we use memory to pass kernel image, the memory region where kernel image locates should be added into hob as read-only. Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoLib.c | 66 +-- 1 file changed, 62 insertions(+), 4 deletions(-) diff --git a

[edk2-devel] [PATCH 1/3] CloudHv:arm: add kernel load fs driver

2022-09-15 Thread Jianyong Wu
This is used for supporting direct kernel boot in CloudHv. CloudHv will store kernel image in system ram and pass kernel info through DT. It's firmware's responsibility to fetch the kernel data and create a file device to feed for loadImage Signed-off-by: J

[edk2-devel] [PATCH 0/3] CloudHv:arm: Enable direct kernel boot

2022-09-15 Thread Jianyong Wu
: https://github.com/tianocore/edk2/pull/3339 Signed-off-by: Jianyong Wu Jianyong Wu (3): CloudHv:arm: add kernel load fs driver CloudHv:arm: build hob for kernel image memory as read-only CloudHv:arm: add kernel loader lib dsc/fdf ArmVirtPkg/ArmVirtCloudHv.dsc | 8

Re: [edk2-devel] [PATCH v4 0/2] CloudHv/arm: Add specific mem info lib

2022-09-06 Thread Jianyong Wu
Thanks Sami, I have pushed this patch set to GitHub and all the CI passes. The link is https://github.com/tianocore/edk2/pull/3289 Thanks Jianyong > -Original Message- > From: Sami Mujawar > Sent: Tuesday, September 6, 2022 3:46 PM > To: Jianyong Wu ; devel@edk2.groups.i

[edk2-devel] [PATCH v4 2/2] CloudHv/arm: switch PeiMemLib to its own

2022-09-06 Thread Jianyong Wu
As Cloud Hypervisor has its own PeiMemLib, change it in dsc file accordingly. Reviewed-by: Sami Mujawar Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtCloudHv.dsc b/ArmVirtPkg/ArmVirtCloudHv.dsc

[edk2-devel] [PATCH v4 1/2] CloudHv/arm: add PeiMemInfoLib

2022-09-06 Thread Jianyong Wu
the memory node from DT; 2. Init page table for each memory node; 3. Add all of the memory nodes to Hob; Reviewed-by: Sami Mujawar Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoLib.c | 243 ++ .../CloudHvVirtMemInfoLib.h | 42

[edk2-devel] [PATCH v4 0/2] CloudHv/arm: Add specific mem info lib

2022-09-06 Thread Jianyong Wu
Currently, the meminfo lib for CloudHv/arm is reusing QEMU's. As the memory layout for cloud hypervisor is changed, it needs a new meminfo lib. Signed-off-by: Jianyong Wu Jianyong Wu (2): CloudHv/arm: add PeiMemInfoLib CloudHv/arm: switch PeiMemLib to its own ArmVirtPkg/ArmVirtCloudH

Re: [edk2-devel] [PATCH v3 0/2] CloudHv/arm: Add specific mem info lib

2022-09-06 Thread Jianyong Wu
Thanks, will address all of them in V4. > -Original Message- > From: devel@edk2.groups.io On Behalf Of Ard > Biesheuvel via groups.io > Sent: Monday, September 5, 2022 9:23 PM > To: devel@edk2.groups.io; Jianyong Wu > Cc: Sami Mujawar ; ardb+tianoc...@kernel.org; >

[edk2-devel] [PATCH v3 2/2] CloudHv/arm: switch PeiMemLib to its own

2022-09-01 Thread Jianyong Wu
As Cloud Hypervisor has its own PeiMemLib, change it in dsc file accordingly. Reviewed-by: Sami Mujawar Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtCloudHv.dsc b/ArmVirtPkg/ArmVirtCloudHv.dsc

[edk2-devel] [PATCH v3 1/2] CloudHv/arm: add PeiMemInfoLib

2022-09-01 Thread Jianyong Wu
memory node from DT; 2. Init page table for each memory node; 3. Add all of the memory nodes to Hob; Reviewed-by: Sami Mujawar Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoLib.c | 230 ++ .../CloudHvVirtMemInfoLib.h | 42

[edk2-devel] [PATCH v3 0/2] CloudHv/arm: Add specific mem info lib

2022-09-01 Thread Jianyong Wu
Currently, the meminfo lib for CloudHv/arm is reusing QEMU's. As the memory layout for cloud hypervisor is changed, it needs a new meminfo lib. Signed-off-by: Jianyong Wu Jianyong Wu (2): CloudHv/arm: add PeiMemInfoLib CloudHv/arm: switch PeiMemLib to its own ArmVirtPkg/ArmVirtCloudH

Re: [edk2-devel] [PATCH v2 1/2] CloudHv/arm: add PeiMemInfoLib

2022-09-01 Thread Jianyong Wu
Thanks Sami, I will address all the comments in this patch in next version. > -Original Message- > From: Sami Mujawar > Sent: Thursday, September 1, 2022 11:43 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ardb+tianoc...@kernel.org; Justin He ; nd > > Subje

Re: [edk2-devel] [PATCH v2 0/2] CloudHv/arm: Add specific mem info lib

2022-08-25 Thread Jianyong Wu
Ping ... Any comments? Thanks Jianyong > -Original Message- > From: Jianyong Wu > Sent: Friday, August 19, 2022 4:09 PM > To: devel@edk2.groups.io; Sami Mujawar > Cc: ardb+tianoc...@kernel.org; Justin He ; Jianyong > Wu > Subject: [PATCH v2 0/2] CloudHv/arm: Add

[edk2-devel] [PATCH 2/2] CloudHv/arm: switch PeiMemLib to its own

2022-08-19 Thread Jianyong Wu
As Cloud Hypervisor has its own PeiMemLib, change it in dsc file accordingly. Signed-off-by: Jianyong Wu Reviewed-by: Sami Mujawar --- ArmVirtPkg/ArmVirtCloudHv.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtCloudHv.dsc b/ArmVirtPkg/ArmVirtCloudHv.dsc

[edk2-devel] [PATCH v2 1/2] CloudHv/arm: add PeiMemInfoLib

2022-08-19 Thread Jianyong Wu
the memory node from DT; 2. Add all of the memory nodes to Hob; 3. Init page table for each memory node; Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoLib.c | 230 ++ .../CloudHvVirtMemInfoLib.h | 42

[edk2-devel] [PATCH v2 0/2] CloudHv/arm: Add specific mem info lib

2022-08-19 Thread Jianyong Wu
Currently, the meminfo lib for CloudHv/arm is reusing QEMU's. As the memory layout for cloud hypervisor is changed, it needs a new meminfo lib. Signed-off-by: Jianyong Wu Jianyong Wu (2): CloudHv/arm: add PeiMemInfoLib CloudHv/arm: switch PeiMemLib to its own ArmVirtPkg/ArmVirtCloudH

Re: [edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib

2022-08-19 Thread Jianyong Wu
Thanks Sami. I will fix them all and send the v2 later. From: Sami Mujawar Sent: Thursday, August 18, 2022 7:41 PM To: Jianyong Wu ; devel@edk2.groups.io Cc: Ard Biesheuvel ; Justin He ; nd Subject: Re: [edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib Hi Jianyong, Please find my

Re: [edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib

2022-08-17 Thread Jianyong Wu
Hi Sami, Thanks for review. All the comments are Ok for me. Just one inline reply: + +struct CloudHvMemNodeInfo CloudHvMemNode[CLOUDHV_MAX_MEM_NODE_NUM]; + +RETURN_STATUS +EFIAPI +CloudHvVirtMemInfoPeiLibConstructor ( + VOID + ) +{ + VOID *DeviceTreeBase; + EFI_RESOURCE_A

Re: [edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib

2022-08-09 Thread Jianyong Wu
Hi Sami, Thanks for review. All the comments are Ok for me. Just one inline reply: + +struct CloudHvMemNodeInfo CloudHvMemNode[CLOUDHV_MAX_MEM_NODE_NUM]; + +RETURN_STATUS +EFIAPI +CloudHvVirtMemInfoPeiLibConstructor ( + VOID + ) +{ + VOID *DeviceTreeBase; + EFI_RESOURCE_AT

Re: [edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib

2022-08-08 Thread Jianyong Wu
+cc: Ard Biesheuvel, Leif Lindholm, Gerd Hoffmann > -Original Message- > From: devel@edk2.groups.io On Behalf Of Jianyong > Wu via groups.io > Sent: Friday, July 29, 2022 3:22 PM > To: devel@edk2.groups.io; Sami Mujawar > Cc: ard.biesheu...@linaro.org; Justin

Re: [edk2-devel] [PATCH 2/2] CloudHv/arm: switch PeiMemLib to its own

2022-08-08 Thread Jianyong Wu
+cc: Ard Biesheuvel, Leif Lindholm, Gerd Hoffmann > -Original Message- > From: devel@edk2.groups.io On Behalf Of Jianyong > Wu via groups.io > Sent: Friday, July 29, 2022 3:22 PM > To: devel@edk2.groups.io; Sami Mujawar > Cc: ard.biesheu...@linaro.org; Justin

Re: [edk2-devel] [PATCH 0/2] CloudHv/arm: Add specific mem info lib

2022-08-07 Thread Jianyong Wu
+cc: Ard Biesheuvel, Leif Lindholm, Gerd Hoffmann > -Original Message- > From: devel@edk2.groups.io On Behalf Of Jianyong > Wu via groups.io > Sent: Friday, July 29, 2022 3:22 PM > To: devel@edk2.groups.io; Sami Mujawar > Cc: ard.biesheu...@linaro.org; Justin

[edk2-devel] [PATCH 2/2] CloudHv/arm: switch PeiMemLib to its own

2022-07-29 Thread Jianyong Wu
As Cloud Hypervisor has its own PeiMemLib, change it in dsc file accordingly. Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ArmVirtPkg/ArmVirtCloudHv.dsc b/ArmVirtPkg/ArmVirtCloudHv.dsc index 7559386a1d

[edk2-devel] [PATCH 0/2] CloudHv/arm: Add specific mem info lib

2022-07-29 Thread Jianyong Wu
Currently, the meminfo lib for CloudHv/arm is reusing QEMU's. As the memory layout for cloud hypervisor is changed, it needs a new meminfo lib. Signed-off-by: Jianyong Wu Jianyong Wu (2): CloudHv/arm: add PeiMemInfoLib CloudHv/arm: switch PeiMemLib to its own ArmVirtPkg/ArmVirtCloudH

[edk2-devel] [PATCH 1/2] CloudHv/arm: add PeiMemInfoLib

2022-07-29 Thread Jianyong Wu
the memory node from DT; 2. Init page table for each memory node; 3. Add all of the memory nodes to Hob; Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoLib.c | 251 ++ .../CloudHvVirtMemInfoPeiLib.inf | 46 2 files changed, 297 insertions

Re: [edk2-devel] [PATCH v4 2/3] Acpi: Install Acpi tables for Cloud hypervisor

2021-07-06 Thread Jianyong Wu
Hi Sami, Thanks for your rework on my patch. I tried the change and it works well. You can do what you like on the patch set. Thanks Jianyong > -Original Message- > From: Sami Mujawar > Sent: Tuesday, July 6, 2021 4:52 PM > To: Jianyong Wu ; devel@edk2.groups.

[edk2-devel] [PATCH v4 3/3] ArmVirtCloudHv: support Cloud Hypervisor in edk2

2021-07-05 Thread Jianyong Wu
Cloud Hypervisor is KVM based VMM and is implemented in rust. Just like other VMMs it need UEFI support to let ACPI work. That's why Cloud Hypervisor is introduced here. Cc: Laszlo Ersek Cc: Sami Mujawar Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc

[edk2-devel] [PATCH v4 2/3] Acpi: Install Acpi tables for Cloud hypervisor

2021-07-05 Thread Jianyong Wu
CPI tables following the XSDT structure and install them one by one; 4. get DSDT address from FADT and install DSDT table. Cc: Laszlo Ersek Cc: Sami Mujawar Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtPkg.dec | 6 + .../CloudHvAcpiPlatformDxe.inf

[edk2-devel] [PATCH v4 1/3] Acpi: reimplement PlatformHasAcpi for Cloud Hypervisor

2021-07-05 Thread Jianyong Wu
The current implementation of PlatformHasAcpiDt is not a common library and is on behalf of qemu. So give a specific version for Cloud Hypervisor here. Cc: Laszlo Ersek Cc: Sami Mujawar Signed-off-by: Jianyong Wu Reviewed-by: Sami Mujawar --- .../CloudHvHasAcpiDtDxe.inf

[edk2-devel] [PATCH v4 0/3] Enable Cloud Hypervisor support in edk2

2021-07-05 Thread Jianyong Wu
e. (3) remove lots of dependencies from qemu like "*Fwcfg*" lib. (4) lots of code cleanup work to let it more approach to edk2 code style. [1] https://github.com/cloud-hypervisor/cloud-hypervisor Jianyong Wu (3): Acpi: reimplement PlatformHasAcpi for Cloud Hypervisor Acpi: Ins

Re: [edk2-devel] [PATCH v3 3/3] ArmVirtCloudHv: support Cloud Hypervisor in edk2

2021-06-29 Thread Jianyong Wu
Hi Sami, Thanks, it works. Also, I will fix that path. Thanks Jianyong > -Original Message- > From: Sami Mujawar > Sent: Tuesday, June 29, 2021 4:16 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ler...@redhat.com; ardb+tianoc...@kernel.org; Justin He > ; nd >

Re: [edk2-devel] [PATCH v3 3/3] ArmVirtCloudHv: support Cloud Hypervisor in edk2

2021-06-28 Thread Jianyong Wu
Hi Sami, VariableRuntimeDxe and PlatformPeiLib depend on Tpm and I'm not sure how to address it as it can't compile without Tpm. Thanks Jianyong > -Original Message- > From: Sami Mujawar > Sent: Monday, June 28, 2021 8:43 PM > To: Jianyong Wu ; devel@ed

Re: [edk2-devel] [PATCH v3 2/3] Acpi: Install Acpi tables for Cloud hypervisor

2021-06-28 Thread Jianyong Wu
Hi Sami, All comments are accepted. Thanks for your elaborate review work! BR Jianyong > -Original Message- > From: Sami Mujawar > Sent: Monday, June 28, 2021 8:22 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ler...@redhat.com; ardb+tianoc...@kernel.org; J

Re: [edk2-devel] [PATCH v3 3/3] ArmVirtCloudHv: support Cloud Hypervisor in edk2

2021-06-28 Thread Jianyong Wu
Hi Sami, All comments are accepted. Thanks. BR Jianyong > -Original Message- > From: Sami Mujawar > Sent: Monday, June 28, 2021 8:43 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ler...@redhat.com; ardb+tianoc...@kernel.org; Justin He > ; nd > Subje

Re: [edk2-devel] [PATCH v3 1/3] Acpi: reimplement PlatformHasAcpi for Cloud Hypervisor

2021-06-28 Thread Jianyong Wu
Hi Sami, I'm glad to get this precious "Reviewed-by", thanks! BR Jianyong > -Original Message- > From: Sami Mujawar > Sent: Monday, June 28, 2021 8:45 PM > To: Jianyong Wu ; devel@edk2.groups.io > Cc: ler...@redhat.com; ardb+tianoc...@kernel.org; J

[edk2-devel] [PATCH v3 2/3] Acpi: Install Acpi tables for Cloud hypervisor

2021-06-28 Thread Jianyong Wu
CPI tables following the XSDT structure and install them one by one; 4. get DSDT address from FADT and install DSDT table. Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtPkg.dec | 6 + .../CloudHvAcpiPlatformDxe.inf| 47 ++ .../CloudHvAcpiPlatformDxe/Clo

[edk2-devel] [PATCH v3 1/3] Acpi: reimplement PlatformHasAcpi for Cloud Hypervisor

2021-06-28 Thread Jianyong Wu
The current implementation of PlatformHasAcpiDt is not a common library and is on behalf of qemu. So give a specific version for Cloud Hypervisor here. Signed-off-by: Jianyong Wu --- .../CloudHvHasAcpiDtDxe.inf | 43 .../CloudHvHasAcpiDtDxe.c

[edk2-devel] [PATCH v3 3/3] ArmVirtCloudHv: support Cloud Hypervisor in edk2

2021-06-28 Thread Jianyong Wu
Cloud Hypervisor is KVM based VMM and is implemented in rust. Just like other VMMs it needs UEFI support to let ACPI work. That is why Cloud Hypervisor is introduced here. Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc | 397 ++ ArmVirtPkg

[edk2-devel] [PATCH v3 0/3] Enable Cloud Hypervisor support in edk2

2021-06-28 Thread Jianyong Wu
pproach to edk2 code style. [1] https://github.com/cloud-hypervisor/cloud-hypervisor Jianyong Wu (3): Acpi: reimplement PlatformHasAcpi for Cloud Hypervisor Acpi: Install Acpi tables for Cloud hypervisor ArmVirtCloudHv: support Cloud Hypervisor in edk2 ArmVirtPkg/ArmVirtPkg.dec

Re: [edk2-devel] [PATCH v2 4/5] ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family

2021-06-01 Thread Jianyong Wu
Hi Sami, > -Original Message- > From: Sami Mujawar > Sent: Saturday, May 29, 2021 3:44 PM > To: Jianyong Wu ; devel@edk2.groups.io; > ler...@redhat.com; ardb+tianoc...@kernel.org > Cc: hao.a...@intel.com; Justin He ; Leif Lindholm > ; nd > Subject: Re: [

Re: [edk2-devel] [PATCH v2 1/5] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-05-26 Thread Jianyong Wu
Hi Laszlo, Based on comments from Sami, maybe I need remove this patch and reuse the qemu memory initialization lib code. Thank for your comments. Thanks Jianyong > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, May 19, 2021 2:12 PM > To: devel@edk2.groups.io;

Re: [edk2-devel] [PATCH v2 5/5] Maintainers: update Maintainers file as new files/folders created

2021-05-26 Thread Jianyong Wu
Hi Sami, Laszlo, Sorry for late reply, thanks for nice review work from both of you. Very helpful for me. Thanks Jianyong > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, May 19, 2021 2:55 PM > To: Sami Mujawar ; Jianyong Wu > ; devel@edk2.groups.io

Re: [edk2-devel] [PATCH v2 4/5] ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family

2021-05-26 Thread Jianyong Wu
Hi Laszlo, > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, May 19, 2021 2:37 PM > To: devel@edk2.groups.io; Jianyong Wu ; > ardb+tianoc...@kernel.org; Sami Mujawar > Cc: hao.a...@intel.com; Justin He ; Leif Lindholm > > Subject: Re: [edk2-devel] [

Re: [edk2-devel] [PATCH v2 4/5] ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family

2021-05-26 Thread Jianyong Wu
Hi Sami, > -Original Message- > From: Sami Mujawar > Sent: Wednesday, May 19, 2021 4:27 AM > To: Jianyong Wu ; devel@edk2.groups.io; > ler...@redhat.com; ardb+tianoc...@kernel.org > Cc: hao.a...@intel.com; Justin He ; Leif Lindholm > ; nd > Subject: Re: [

Re: [edk2-devel] [PATCH v2 3/5] ArmVirtPkg: enable ACPI for cloud hypervisor

2021-05-26 Thread Jianyong Wu
Hi Laszlo, > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, May 19, 2021 2:26 PM > To: devel@edk2.groups.io; Jianyong Wu ; > ardb+tianoc...@kernel.org; Sami Mujawar > Cc: hao.a...@intel.com; Justin He > Subject: Re: [edk2-devel] [PATCH v2 3/5] ArmVir

Re: [edk2-devel] [PATCH v2 3/5] ArmVirtPkg: enable ACPI for cloud hypervisor

2021-05-26 Thread Jianyong Wu
Hi Sami, Thanks for lots of great comments here, I will fix it one by one. Thanks Jianyong From: Sami Mujawar Sent: Wednesday, May 19, 2021 4:26 AM To: Jianyong Wu ; devel@edk2.groups.io; ler...@redhat.com; ardb+tianoc...@kernel.org Cc: hao.a...@intel.com; Justin He ; nd Subject: Re: [PATCH

Re: [edk2-devel] [PATCH v2 2/5] MdeMoudlePkg: introduce new PCD for Acpi/rsdp

2021-05-26 Thread Jianyong Wu
Hi Laszlo, > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, May 19, 2021 2:17 PM > To: devel@edk2.groups.io; Jianyong Wu ; > ardb+tianoc...@kernel.org; Sami Mujawar > Cc: hao.a...@intel.com; Justin He ; Jian J Wang > > Subject: Re: [edk2-devel] [PA

Re: [edk2-devel] [PATCH v2 2/5] MdeMoudlePkg: introduce new PCD for Acpi/rsdp

2021-05-26 Thread Jianyong Wu
Hi Sami, > -Original Message- > From: Sami Mujawar > Sent: Wednesday, May 19, 2021 4:26 AM > To: Jianyong Wu ; devel@edk2.groups.io; > ler...@redhat.com; ardb+tianoc...@kernel.org > Cc: hao.a...@intel.com; Justin He ; Jian J Wang > ; nd > Subject: Re: [PA

Re: [edk2-devel] [PATCH v2 1/5] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-05-26 Thread Jianyong Wu
Hi Sami, > -Original Message- > From: Sami Mujawar > Sent: Wednesday, May 19, 2021 4:21 AM > To: Jianyong Wu ; devel@edk2.groups.io; > ler...@redhat.com; ardb+tianoc...@kernel.org > Cc: hao.a...@intel.com; Justin He ; Leif Lindholm > ; nd > Subject: Re: [

[edk2-devel] [PATCH v2 5/5] Maintainers: update Maintainers file as new files/folders created

2021-05-16 Thread Jianyong Wu
Create new entry for Cloud Hypervisor and assign reviewer to Sami Mujawar. Cc: Sami Mujawar Signed-off-by: Jianyong Wu --- Maintainers.txt | 7 +++ 1 file changed, 7 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index cafe6b1ab85d..f8fae067c656 100644 --- a/Maintainers.txt

[edk2-devel] [PATCH v2 4/5] ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family

2021-05-16 Thread Jianyong Wu
Cloud Hypervisor is kvm based VMM and is implemented in rust. Just like other VMMs it need UEFI support to let ACPI work. That's why Cloud Hypervisor is introduced here. Cc: Laszlo Ersek Cc: Leif Lindholm Cc: Signed-off-by: Jianyong Wu --- ArmVirtPkg/ArmVirtCloudHv.dsc

[edk2-devel] [PATCH v2 3/5] ArmVirtPkg: enable ACPI for cloud hypervisor

2021-05-16 Thread Jianyong Wu
very simple way: firstly, aquire the Rsdp address from the PCD varaible in the top ".dsc"; secondly, get the Xsdp address from Rsdp structure; thirdly, get the Acpi tables following the Xsdp structrue and install it one by one. Signed-off-by: Jianyong Wu --- .../CloudHvAcpiPlatf

[edk2-devel] [PATCH v2 2/5] MdeMoudlePkg: introduce new PCD for Acpi/rsdp

2021-05-16 Thread Jianyong Wu
As there is lack of a machnism in Cloud Hypervisor to pass the base address of Rsdp in Acpi, so a PCD varialbe is introduced here to feed it. Cc: Hao A Wu Cc: Jian J Wang Signed-off-by: Jianyong Wu --- MdeModulePkg/MdeModulePkg.dec | 7 +++ 1 file changed, 7 insertions(+) diff --git a

[edk2-devel] [PATCH v2 1/5] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-05-16 Thread Jianyong Wu
Cloud Hypervisor is kvm based VMM implemented in rust. This library populates the system memory map for the Cloud Hypervisor virtual platform. Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Leif Lindholm Signed-off-by: Jianyong Wu --- .../CloudHvVirtMemInfoPeiLib.inf | 47

[edk2-devel] [PATCH v2 0/5] Enable Cloud Hypervisor support in edk2

2021-05-16 Thread Jianyong Wu
enablement bases on the implentation for qemu so some code is borrowed. [1] https://github.com/cloud-hypervisor/cloud-hypervisor Jianyong Wu (5): ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor MdeMoudlePkg: introduce new PCD for Acpi/rsdp ArmVirtPkg: enable ACPI for cloud

Re: [edk2-devel] [PATCH v1 1/4] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-04-26 Thread Jianyong Wu
Hi Laszlo, > -Original Message- > From: Laszlo Ersek > Sent: Friday, April 23, 2021 8:00 PM > To: Jianyong Wu ; edk2-devel-groups-io > ; Sami Mujawar > Cc: Justin He ; Ard Biesheuvel > ; Leif Lindholm > Subject: Re: [edk2-devel] [PATCH v1 1/4] Arm

Re: [edk2-devel] [PATCH v1 1/4] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-04-26 Thread Jianyong Wu
, April 22, 2021 9:57 PM > To: Jianyong Wu ; edk2-devel-groups-io > > Cc: Justin He ; Ard Biesheuvel > ; Leif Lindholm ; Sami > Mujawar > Subject: Re: [PATCH v1 1/4] ArmVirtPkg: Library: Memory initialization for > Cloud Hypervisor > > Hi Jianyong, > > On 04/22/21 10

Re: [edk2-devel] [PATCH v1 1/4] ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor

2021-04-26 Thread Jianyong Wu
Thanks Sami, I have registered the edk2 mail list and now I can receive the mails from it. Also, I thinks I am the member of edk2 group now. From: Sami Mujawar Sent: Thursday, April 22, 2021 10:13 PM To: Laszlo Ersek ; Jianyong Wu ; edk2-devel-groups-io ; edk2-devel-groups-io Cc: Justin He