Re: [PATCH] MIPS: KVM: Fix NULL pointer dereference

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 29, 2023 at 12:24 AM Huacai Chen <chenhuacai@xxxxxxxxxx> wrote:
>
> Hi, Zhao,
>
> On Thu, Jun 29, 2023 at 8:07 AM Yu Zhao <yuzhao@xxxxxxxxxx> wrote:
> >
> > On Mon, Jun 26, 2023 at 6:33 AM Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote:
> > >
> > > On Mon, Jun 26, 2023 at 9:59 AM Huacai Chen <chenhuacai@xxxxxxxxxxx> wrote:
> > > >
> > > > After commit 45c7e8af4a5e3f0bea4ac209 ("MIPS: Remove KVM_TE support") we
> > > > get a NULL pointer dereference when creating a KVM guest:
> > >
> > > To be honest, a bug that needed 2 years to be reproduced is probably a
> > > sign that KVM/MIPS has no users. Any objections to removing it
> > > altogether?
> >
> > ACK:
> > 1. It's still broken after this patch [1]. The most well-tested MIPS
> >    distros, i.e., Debian/OpenWrt, have CONFIG_KVM=n. (The latter doesn't
> >    even provide the QEMU package on MIPS.)
> > 2. Burden on QEMU dev. There is no guarantee that QEMU would work with
> >    KVM even if we could fix the kernel -- it actually does not until
> >    v8.0 [1], which is by luck:
> >
> >    commit a844873512400fae6bed9e87694dc96ff2f15f39
> >    Author: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> >    Date:   Sun Dec 18 01:06:45 2022 +0100
> >
> >        mips: Remove support for trap and emulate KVM
> >
> >        This support was limited to the Malta board, drop it.
> >        I do not have a machine that can run VZ KVM, so I am assuming
> >        that it works for -M malta as well.
> >
> >    (The latest Debian stable only ships v7.2.)
> >
> > [1] https://lore.kernel.org/r/ZI0R76Fx25Q2EThZ@xxxxxxxxxx/
> My testbed is Loongson-3A4000 host + Loongson-3A4000 guest + Qemu8.0,
> both TCG and KVM works.

I tried TCG too, as reported in the link above, and had no luck.

> Some thoughts:
> 1, I think your host is malta, but you cannot use a malta host to boot
> a Loongson guest, at least their kernels use different page sizes.

Good to know. Thanks.

> 2, commit a844873512400fae6bed9e87694dc96f remove the TE KVM, so if
> you are trying VZ KVM (but it seems you are using TE KVM), it can
> break nothing.

I fully understand :) I was trying to point out that QEMU/KVM had been
broken for over 2 years -- neither TE (removed from KVM) nor VZ works
with 7.2, which is probably why you used 8.0 too -- until that commit
*accidentally* fixed VZ (it was supposed to be a cleanup, not a fix).




[Index of Archives]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux