Richard Henderson <richard.hender...@linaro.org> writes:
> On 11/3/23 12:59, Alex Bennée wrote:
>> From: Akihiko Odaki <akihiko.od...@daynix.com>
>> loongarch64-linux-user has references to XML files so include them.
>> Fixes: d32688ecdb ("default-configs: Add loongarch linux-user
>> support")
>> Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com>
>> Message-Id: <20231030054834.39145-6-akihiko.od...@daynix.com>
>> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
>> ---
>> configs/targets/loongarch64-linux-user.mak | 1 +
>> 1 file changed, 1 insertion(+)
>> diff --git a/configs/targets/loongarch64-linux-user.mak
>> b/configs/targets/loongarch64-linux-user.mak
>> index 7d1b964020..43b8a2160f 100644
>> --- a/configs/targets/loongarch64-linux-user.mak
>> +++ b/configs/targets/loongarch64-linux-user.mak
>> @@ -1,3 +1,4 @@
>> # Default configuration for loongarch64-linux-user
>> TARGET_ARCH=loongarch64
>> TARGET_BASE_ARCH=loongarch
>> +TARGET_XML_FILES=gdb-xml/loongarch-base32.xml gdb-xml/loongarch-base64.xml
>> gdb-xml/loongarch-fpu.xml
>
>
> The qemu-loongarch64 binary emulates loongarch64 only, not
> loongarch32. The inclusion of loongarch-base32.xml here is not
> relevant.
Does the system binary emulate both?
> That said, we don't exclude TYPE_LOONGARCH32_CPU or -cpu la132 from
> CONFIG_USER_ONLY, which is a separate mistake.
Where should that be done?
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
