Re: [PULL v2 1/1] loongarch: Change the UEFI loading mode to loongarch

From: lixianglai <lixianglai@loongson.cn>
To: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Song Gao" <gaosong@loongson.cn>,
	qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, Andrea Bolognani <abologna@redhat.com>,
	maobibo@loongson.cn, zhaotianrui@loongson.cn
Subject: Re: [PULL v2 1/1] loongarch: Change the UEFI loading mode to loongarch
Date: Sat, 2 Mar 2024 10:31:50 +0800	[thread overview]
Message-ID: <5e2419ec-6c1f-e21c-2105-fadb5971239c@loongson.cn> (raw)
In-Reply-To: <168d29d9-b82e-4dd5-abdd-93759037033c@linaro.org>

[-- Attachment #1: Type: text/plain, Size: 1871 bytes --]

Hi Philippe:
> On 29/2/24 12:38, Song Gao wrote:
>> From: Xianglai Li <lixianglai@loongson.cn>
>>
>> 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.
>
> FYI I'm still trying to find a way to avoid that, planning to discuss
> more with libvirt folks. Well, maybe it is a waste of my time and I
> should just stop worrying / caring about this long standing issue.
>
Thank you so much for your attention to this issue

and your long-standing contributions to the community!!:-)

Best regards,

Xianglai.

>> Cc: Andrea Bolognani <abologna@redhat.com>
>> Cc: maobibo@loongson.cn
>> Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
>> Cc: Song Gao <gaosong@loongson.cn>
>> Cc: zhaotianrui@loongson.cn
>> Signed-off-by: Xianglai Li <lixianglai@loongson.cn>
>> Tested-by: Andrea Bolognani <abologna@redhat.com>
>> Reviewed-by: Song Gao <gaosong@loongson.cn>
>> Message-Id: 
>> <0bd892aa9b88e0f4cc904cb70efd0251fc1cde29.1708336919.git.lixianglai@loongson.cn>
>> Signed-off-by: Song Gao <gaosong@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(-)
>

[-- Attachment #2: Type: text/html, Size: 5722 bytes --]