Hi,
在 2024/1/11 下午7:26, Philippe Mathieu-Daudé 写道:
On 11/1/24 10:51, gaosong wrote:
在 2024/1/11 下午5:04, Thomas Huth 写道:
On 11/01/2024 09.50, gaosong wrote:
在 2024/1/11 下午4:20, Thomas Huth 写道:
On 11/01/2024 08.37, gaosong wrote:
LoongArch no support these cmds or some problems .
- "gva2gpa 0",
- "memsave 0 4096 \"/dev/null\"",
- "x /8i 0x100",
- "xp /16x 0",
Could we disable these 4 cmds or the test_temp check?
After we fix the cmds problems, we can enable them.
Even if loongarch does not support one of these commands, it should
not crash QEMU. So please fix the crashes first before considering
to enable the KVM-only test in the CI.
Sure, we will fix the cmds problems first.
The issue might be missing get_phys_page_attrs_debug() implementation.
We use '--enable-kvm --disable-tcg ',
the get_phys_page_debug() is NULL, so the test-hmp failed.
target/loongarch/cpu.c
...
#ifndef CONFIG_USER_ONLY
#include "hw/core/sysemu-cpu-ops.h"
static const struct SysemuCPUOps loongarch_sysemu_ops = {
#ifdef CONFIG_TCG
.get_phys_page_debug = loongarch_cpu_get_phys_page_debug,
#endif
};
...
I see the other architectures only implement
get_phys_page_attrs_debug() or get_phys_page_debug()
and not only build these functions on tcg mode.
Should we need implement get_phys_page_attrs_debug() ? or just use
get_phys_page_debug()
Thanks.
Song Gao
