| Message ID | 20220120073911.99857-18-guoren@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | riscv: compat: Add COMPAT mode support for rv64 | expand |
On Thu, Jan 20, 2022 at 8:39 AM <guoren@kernel.org> wrote: > > From: Guo Ren <guoren@linux.alibaba.com> > > Current riscv doesn't support the 32bit KVM/arm API. Let's make it > clear by not selecting KVM_COMPAT. > > Signed-off-by: Guo Ren <guoren@linux.alibaba.com> > --- > virt/kvm/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig > index f4834c20e4a6..a8c5c9f06b3c 100644 > --- a/virt/kvm/Kconfig > +++ b/virt/kvm/Kconfig > @@ -53,7 +53,7 @@ config KVM_GENERIC_DIRTYLOG_READ_PROTECT > > config KVM_COMPAT > def_bool y > - depends on KVM && COMPAT && !(S390 || ARM64) > + depends on KVM && COMPAT && !(S390 || ARM64 || RISCV) Maybe this should be flipped around into a positive list now? The remaining architectures would be mips, powerpc and x86, but it's unclear if this actually meant to work on all of them, or any potential ones added in the future. Arnd
On Thu, Jan 20, 2022 at 6:32 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Jan 20, 2022 at 8:39 AM <guoren@kernel.org> wrote: > > > > From: Guo Ren <guoren@linux.alibaba.com> > > > > Current riscv doesn't support the 32bit KVM/arm API. Let's make it > > clear by not selecting KVM_COMPAT. > > > > Signed-off-by: Guo Ren <guoren@linux.alibaba.com> > > --- > > virt/kvm/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig > > index f4834c20e4a6..a8c5c9f06b3c 100644 > > --- a/virt/kvm/Kconfig > > +++ b/virt/kvm/Kconfig > > @@ -53,7 +53,7 @@ config KVM_GENERIC_DIRTYLOG_READ_PROTECT > > > > config KVM_COMPAT > > def_bool y > > - depends on KVM && COMPAT && !(S390 || ARM64) > > + depends on KVM && COMPAT && !(S390 || ARM64 || RISCV) > > Maybe this should be flipped around into a positive list now? I think it's another patch to do that. Not in this series. > The remaining architectures would be mips, powerpc and x86, but it's unclear > if this actually meant to work on all of them, or any potential ones > added in the Yes, it's unclear and arch maintainers need to confirm that. > future. > > Arnd
diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig index f4834c20e4a6..a8c5c9f06b3c 100644 --- a/virt/kvm/Kconfig +++ b/virt/kvm/Kconfig @@ -53,7 +53,7 @@ config KVM_GENERIC_DIRTYLOG_READ_PROTECT config KVM_COMPAT def_bool y - depends on KVM && COMPAT && !(S390 || ARM64) + depends on KVM && COMPAT && !(S390 || ARM64 || RISCV) config HAVE_KVM_IRQ_BYPASS bool