Re: [NOTFORMERGE PATCH 2/2] gitlab: Add Loongarch64 KVM-only build

[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:
> > > Hi,
> > >
> > > 在 2024/1/11 下午3:10, Thomas Huth 写道:
> > > > On 02/01/2024 18.22, Philippe Mathieu-Daudé wrote:
> > > > > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > > > > ---
> > > > > Used to test 
> > > > > https://lore.kernel.org/qemu-devel/20231228084051.3235354-1-zhaotianrui@loongson.cn/
> > > >
> > > > So why is it NOTFORMERGE ? Don't we want to test KVM-only builds for 
> > > > loongarch in the long run?
> > > >
> > > >  Thomas
> > > I think we can drop this title.
> > >
> > > I tested this job by the latest loongarch kvm patches.  buf I find a 
> > > test-hmp check error.
> > Can you recreate the error manually? i.e. compile with configure 
> > --disable-tcg and then run:
> >
> >  V=2 QTEST_QEMU_BINARY=./qemu-system-loongarch64 tests/qtest/test-hmp
> >
> > That should likely provide you with a hint where it is crashing
> >
> >  Thomas
> Thank you,
>
> 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.

 Thanks
  Thomas