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

2021-07-08 Thread Sami Mujawar
Merged as 4c051c2c65a8..0e3b6bd0ee75 Thanks. Regards, Sami Mujawar On 07/07/2021, 02:43, "Jianyong Wu" wrote: 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

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.io > Cc: ler...@redhat.com

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

2021-07-06 Thread Sami Mujawar
Hi Jianyong, I should have caught this earlier in my review. However, if you agree, I will do the following changes before pushing the patch. 1. The subject line of the commit message does not confirm to the edk2 coding standard. It should have ‘ArmVirtPkg: ’ 2. The ACPI table signat

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

2021-07-05 Thread Jianyong Wu
There is no device like Fw-cfg in Qemu in Cloud Hypervisor, so a specific Acpi handler is introduced here. The handler implemented here is in a very simple way: 1. acquire the RSDP from the PCD variable in the top ".dsc"; 2. get the XSDT address from RSDP structure; 3. get the ACPI tables followin