All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] kvm: fix KVM_MAX_VCPU_ID handling
@ 2021-09-13 13:57 ` Juergen Gross
  0 siblings, 0 replies; 27+ messages in thread
From: Juergen Gross @ 2021-09-13 13:57 UTC (permalink / raw)
  To: kvm, x86, linux-kernel, linux-doc, linux-mips, kvm-ppc,
	linuxppc-dev, linux-kselftest
  Cc: Juergen Gross, Paolo Bonzini, Sean Christopherson,
	Vitaly Kuznetsov, Wanpeng Li, Jim Mattson, Joerg Roedel,
	Thomas Gleixner, Ingo Molnar, Borislav Petkov, H. Peter Anvin,
	Jonathan Corbet, Huacai Chen, Aleksandar Markovic,
	Thomas Bogendoerfer, Paul Mackerras, Michael Ellerman,
	Benjamin Herrenschmidt, Paul Mackerras, Shuah Khan, Shuah Khan

Revert commit 76b4f357d0e7d8f6f00 which was based on wrong reasoning
and rename KVM_MAX_VCPU_ID to KVM_MAX_VCPU_IDS in order to avoid the
same issue in future.

Juergen Gross (2):
  x86/kvm: revert commit 76b4f357d0e7d8f6f00
  kvm: rename KVM_MAX_VCPU_ID to KVM_MAX_VCPU_IDS

 Documentation/virt/kvm/devices/xics.rst            | 2 +-
 Documentation/virt/kvm/devices/xive.rst            | 2 +-
 arch/mips/kvm/mips.c                               | 2 +-
 arch/powerpc/include/asm/kvm_book3s.h              | 2 +-
 arch/powerpc/include/asm/kvm_host.h                | 4 ++--
 arch/powerpc/kvm/book3s_xive.c                     | 2 +-
 arch/powerpc/kvm/powerpc.c                         | 2 +-
 arch/x86/include/asm/kvm_host.h                    | 2 +-
 arch/x86/kvm/ioapic.c                              | 2 +-
 arch/x86/kvm/ioapic.h                              | 4 ++--
 arch/x86/kvm/x86.c                                 | 2 +-
 include/linux/kvm_host.h                           | 4 ++--
 tools/testing/selftests/kvm/kvm_create_max_vcpus.c | 2 +-
 virt/kvm/kvm_main.c                                | 2 +-
 14 files changed, 17 insertions(+), 17 deletions(-)

-- 
2.26.2


^ permalink raw reply	[flat|nested] 27+ messages in thread
* Re: [PATCH 2/2] kvm: rename KVM_MAX_VCPU_ID to, KVM_MAX_VCPU_IDS
@ 2021-09-14 15:59 ` Christian Zigotzky
  0 siblings, 0 replies; 27+ messages in thread
From: Christian Zigotzky @ 2021-09-14 15:59 UTC (permalink / raw)
  To: Juergen Gross
  Cc: x86, Wanpeng Li, mad skateman, kvm, linux-doc, linux-kernel,
	Darren Stevens, Paul Mackerras, linux-kselftest, H. Peter Anvin,
	Shuah Khan, Jonathan Corbet, Joerg Roedel, Huacai Chen, ,
	Aleksandar Markovic, ,
	Ingo Molnar, R.T.Dickinson, kvm-ppc, Borislav Petkov, Shuah Khan,
	Thomas Gleixner, Jim Mattson, Thomas Bogendoerfer, ,
	linux-mips, Paolo Bonzini, Vitaly, Kuznetsov, linuxppc-dev

Hello Juergen,
Hello All,

Since the RC1 of kernel 5.13, -smp 2 and -smp 4 don't work with a 
virtual e5500 QEMU KVM-HV machine anymore. [1]
I see in the serial console, that the uImage doesn't load. I use the 
following QEMU command for booting:

qemu-system-ppc64 -M ppce500 -cpu e5500 -enable-kvm -m 1024 -kernel 
uImage -drive format=raw,file=MintPPC32-X5000.img,index=0,if=virtio 
-netdev user,id=mynet0 -device virtio-net,netdev=mynet0 -append "rw 
root=/dev/vda" -device virtio-vga -device virtio-mouse-pci -device 
virtio-keyboard-pci -device pci-ohci,id=newusb -device 
usb-audio,bus=newusb.0 -smp 4

The kernels boot without KVM-HV.

Summary for KVM-HV:

-smp 1 -> works
-smp 2 -> doesn't work
-smp 3 -> works
-smp 4 -> doesn't work

I used -smp 4 before the RC1 of kernel 5.13 because my FSL P5040 BookE 
machine [2] has 4 cores.

Does this patch solve this issue? [3]

Thanks,
Christian

[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2021-May/229103.html
[2] http://wiki.amiga.org/index.php?title=X5000
[3] 
https://lists.ozlabs.org/pipermail/linuxppc-dev/2021-September/234152.html

^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2022-07-05 16:11 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-13 13:57 [PATCH 0/2] kvm: fix KVM_MAX_VCPU_ID handling Juergen Gross
2021-09-13 13:57 ` Juergen Gross
2021-09-13 13:57 ` Juergen Gross
2021-09-13 13:57 ` [PATCH 1/2] x86/kvm: revert commit 76b4f357d0e7d8f6f00 Juergen Gross
2021-11-08 20:14   ` Ben Gardon
2021-11-08 20:15     ` Ben Gardon
2021-11-09  8:46       ` Juergen Gross
2021-11-09 17:15         ` Ben Gardon
2021-09-13 13:57 ` [PATCH 2/2] kvm: rename KVM_MAX_VCPU_ID to KVM_MAX_VCPU_IDS Juergen Gross
2021-09-13 13:57   ` Juergen Gross
2021-09-13 13:57   ` Juergen Gross
2021-09-13 16:24   ` Sean Christopherson
2021-09-13 16:24     ` Sean Christopherson
2021-09-13 16:56     ` Eduardo Habkost
2021-09-13 16:56       ` Eduardo Habkost
2021-09-13 16:56       ` Eduardo Habkost
2021-09-13 18:21       ` Sean Christopherson
2021-09-13 18:21         ` Sean Christopherson
2021-09-23 16:21 ` [PATCH 0/2] kvm: fix KVM_MAX_VCPU_ID handling Paolo Bonzini
2021-09-23 16:21   ` Paolo Bonzini
2021-09-23 16:21   ` Paolo Bonzini
2021-09-14 15:59 [PATCH 2/2] kvm: rename KVM_MAX_VCPU_ID to, KVM_MAX_VCPU_IDS Christian Zigotzky
2021-09-14 15:59 ` Christian Zigotzky
2021-09-14 15:59 ` Christian Zigotzky
2022-07-05 16:07 ` Christian Zigotzky
2022-07-05 16:07   ` Christian Zigotzky
2022-07-05 16:07   ` Christian Zigotzky

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.