On Wed, May 29, 2024 at 3:58 PM Manos Pitsidianakis <manos.pitsidiana...@linaro.org> wrote: > What about TCG? Will it be available if we only build with tcg?
I will try doing this, and report back, and will make the suggested changes, and re-submit. On Wed, May 29, 2024 at 3:58 PM Manos Pitsidianakis <manos.pitsidiana...@linaro.org> wrote: > > On Tue, 28 May 2024 13:23, Will Gyda <vilhelmg...@gmail.com> wrote: > >Issue #2294: Machine microvm doesn't run under Xen accel for > >qemu-system-x86_64. > >Solution: microvm is now not build if only Xen is available. > > > >Signed-off-by: Will Gyda <vilhelmg...@gmail.com> > > > > > I suggest rewording the commit title to something like > > "i386: remove microvm from default build" > > And adding a commit message that explains that the microvm does not work > on Xen, hence if only Xen is available it should not be built. > > Also, you can add a > > Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2294 > > line before your Signed-off-by. See > https://www.qemu.org/docs/master/devel/submitting-a-patch.html > > But, seeing the issue itself, it's about the microvm being stuck under > Xen. So the commit that resolves this would either make it non-stuck or > make it impossible to start the vm to begin with. > > > >--- > > configs/devices/i386-softmmu/default.mak | 2 +- > > hw/i386/Kconfig | 2 ++ > > 2 files changed, 3 insertions(+), 1 deletion(-) > > > >diff --git a/configs/devices/i386-softmmu/default.mak > >b/configs/devices/i386-softmmu/default.mak > >index 598c6646df..6a73aee7dd 100644 > >--- a/configs/devices/i386-softmmu/default.mak > >+++ b/configs/devices/i386-softmmu/default.mak > >@@ -29,4 +29,4 @@ > > CONFIG_ISAPC=y > > CONFIG_I440FX=y > > CONFIG_Q35=y > >-CONFIG_MICROVM=y > >+#CONFIG_MICROVM=n > > Better remove this altogether since it's not a default anymore. > > >diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig > >index a6ee052f9a..f8ec8ebd7a 100644 > >--- a/hw/i386/Kconfig > >+++ b/hw/i386/Kconfig > >@@ -108,6 +108,8 @@ config Q35 > > > > config MICROVM > > bool > >+ default y > >+ depends on KVM || WHPX || NVMM || HVF > > What about TCG? Will it be available if we only build with tcg? > > > select SERIAL_ISA # for serial_hds_isa_init() > > select ISA_BUS > > select APIC > >-- > >2.25.1 > > > >