Hi Andrea:

I'm very sorry for the late reply.

I just had the Spring Festival holiday,

so I replied the email as soon as I saw it.


On Tue, Feb 06, 2024 at 10:10:02AM +0800, Xianglai Li wrote:
The UEFI loading mode in loongarch is very different
from that in other architectures:loongarch's UEFI code
is in rom, while other architectures' UEFI code is in flash.

loongarch UEFI can be loaded as follows:
-machine virt,pflash=pflash0-format
-bios ./QEMU_EFI.fd

Other architectures load UEFI using the following methods:
-machine virt,pflash0=pflash0-format,pflash1=pflash1-format

loongarch's UEFI loading method makes qemu and libvirt incompatible
when using NVRAM, and the cost of loongarch's current loading method
far outweighs the benefits, so we decided to use the same UEFI loading
scheme as other architectures.

Cc: Andrea Bolognani <abolo...@redhat.com>
Cc: maob...@loongson.cn
Cc: Philippe Mathieu-Daudé <phi...@linaro.org>
Cc: Song Gao <gaos...@loongson.cn>
Cc: zhaotian...@loongson.cn
Signed-off-by: Xianglai Li <lixiang...@loongson.cn>
---
  hw/loongarch/acpi-build.c   |  29 +++++++++--
  hw/loongarch/virt.c         | 101 ++++++++++++++++++++++++++----------
  include/hw/loongarch/virt.h |  10 ++--
  3 files changed, 107 insertions(+), 33 deletions(-)
For future reference, it's usually good practice to keep track of
changes between subsequent versions of the same patchset.

OK! Since there is only one patch, I did not add a cover letter.

I saw that you replied test-by later,

so I plan to resend patch V2 and bring the cover letter with change log and your test-by label.


Can you please confirm that the build of edk2 added with [1] is
intended to work with a version of QEMU that contains these changes?
I'd like to test things out as soon as I get a moment.

Yes! [1] Works well with qemu with the current patch.

Thanks.

Xianglai.


Thanks.


[1] 
https://github.com/lixianglai/LoongarchVirtFirmware/commit/985ce19438d9544968c7e921c6acf2c74fd4713e


Reply via email to