From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-2.mimecast.com ([205.139.110.61]:56291 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1732759AbgDOCoO (ORCPT ); Tue, 14 Apr 2020 22:44:14 -0400 From: Jason Wang Subject: [PATCH V2] vhost: do not enable VHOST_MENU by default Date: Wed, 15 Apr 2020 10:43:56 +0800 Message-Id: <20200415024356.23751-1-jasowang@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Sender: linux-s390-owner@vger.kernel.org List-ID: To: jasowang@redhat.com, mst@redhat.com Cc: linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, kvm@vger.kernel.org, virtualization@lists.linux-foundation.org, netdev@vger.kernel.org, geert@linux-m68k.org, tsbogend@alpha.franken.de, benh@kernel.crashing.org, paulus@samba.org, heiko.carstens@de.ibm.com, gor@linux.ibm.com, borntraeger@de.ibm.com, Michael Ellerman We try to keep the defconfig untouched after decoupling CONFIG_VHOST out of CONFIG_VIRTUALIZATION in commit 20c384f1ea1a ("vhost: refine vhost and vringh kconfig") by enabling VHOST_MENU by default. Then the defconfigs can keep enabling CONFIG_VHOST_NET without the caring of CONFIG_VHOST. But this will leave a "CONFIG_VHOST_MENU=3Dy" in all defconfigs and even for the ones that doesn't want vhost. So it actually shifts the burdens to the maintainers of all other to add "CONFIG_VHOST_MENU is not set". So this patch tries to enable CONFIG_VHOST explicitly in defconfigs that enables CONFIG_VHOST_NET and CONFIG_VHOST_VSOCK. Acked-by: Christian Borntraeger (s390) Acked-by: Michael Ellerman (powerpc) Cc: Thomas Bogendoerfer Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: Heiko Carstens Cc: Vasily Gorbik Cc: Christian Borntraeger Reported-by: Geert Uytterhoeven Signed-off-by: Jason Wang --- Change since V1: - depends on EVENTFD for VHOST --- arch/mips/configs/malta_kvm_defconfig | 1 + arch/powerpc/configs/powernv_defconfig | 1 + arch/powerpc/configs/ppc64_defconfig | 1 + arch/powerpc/configs/pseries_defconfig | 1 + arch/s390/configs/debug_defconfig | 1 + arch/s390/configs/defconfig | 1 + drivers/vhost/Kconfig | 26 +++++++++----------------- 7 files changed, 15 insertions(+), 17 deletions(-) diff --git a/arch/mips/configs/malta_kvm_defconfig b/arch/mips/configs/ma= lta_kvm_defconfig index 8ef612552a19..06f0c7a0ca87 100644 --- a/arch/mips/configs/malta_kvm_defconfig +++ b/arch/mips/configs/malta_kvm_defconfig @@ -18,6 +18,7 @@ CONFIG_PCI=3Dy CONFIG_VIRTUALIZATION=3Dy CONFIG_KVM=3Dm CONFIG_KVM_MIPS_DEBUG_COP0_COUNTERS=3Dy +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_MODULES=3Dy CONFIG_MODULE_UNLOAD=3Dy diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/config= s/powernv_defconfig index 71749377d164..404245b4594d 100644 --- a/arch/powerpc/configs/powernv_defconfig +++ b/arch/powerpc/configs/powernv_defconfig @@ -346,5 +346,6 @@ CONFIG_CRYPTO_DEV_VMX=3Dy CONFIG_VIRTUALIZATION=3Dy CONFIG_KVM_BOOK3S_64=3Dm CONFIG_KVM_BOOK3S_64_HV=3Dm +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_PRINTK_TIME=3Dy diff --git a/arch/powerpc/configs/ppc64_defconfig b/arch/powerpc/configs/= ppc64_defconfig index 7e68cb222c7b..4599fc7be285 100644 --- a/arch/powerpc/configs/ppc64_defconfig +++ b/arch/powerpc/configs/ppc64_defconfig @@ -61,6 +61,7 @@ CONFIG_ELECTRA_CF=3Dy CONFIG_VIRTUALIZATION=3Dy CONFIG_KVM_BOOK3S_64=3Dm CONFIG_KVM_BOOK3S_64_HV=3Dm +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_OPROFILE=3Dm CONFIG_KPROBES=3Dy diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/config= s/pseries_defconfig index 6b68109e248f..4cad3901b5de 100644 --- a/arch/powerpc/configs/pseries_defconfig +++ b/arch/powerpc/configs/pseries_defconfig @@ -321,5 +321,6 @@ CONFIG_CRYPTO_DEV_VMX=3Dy CONFIG_VIRTUALIZATION=3Dy CONFIG_KVM_BOOK3S_64=3Dm CONFIG_KVM_BOOK3S_64_HV=3Dm +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_PRINTK_TIME=3Dy diff --git a/arch/s390/configs/debug_defconfig b/arch/s390/configs/debug_= defconfig index 0c86ba19fa2b..6ec6e69630d1 100644 --- a/arch/s390/configs/debug_defconfig +++ b/arch/s390/configs/debug_defconfig @@ -57,6 +57,7 @@ CONFIG_PROTECTED_VIRTUALIZATION_GUEST=3Dy CONFIG_CMM=3Dm CONFIG_APPLDATA_BASE=3Dy CONFIG_KVM=3Dm +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_VHOST_VSOCK=3Dm CONFIG_OPROFILE=3Dm diff --git a/arch/s390/configs/defconfig b/arch/s390/configs/defconfig index 6b27d861a9a3..d1b3bf83d687 100644 --- a/arch/s390/configs/defconfig +++ b/arch/s390/configs/defconfig @@ -57,6 +57,7 @@ CONFIG_PROTECTED_VIRTUALIZATION_GUEST=3Dy CONFIG_CMM=3Dm CONFIG_APPLDATA_BASE=3Dy CONFIG_KVM=3Dm +CONFIG_VHOST=3Dm CONFIG_VHOST_NET=3Dm CONFIG_VHOST_VSOCK=3Dm CONFIG_OPROFILE=3Dm diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig index e79cbbdfea45..29f171a53d8a 100644 --- a/drivers/vhost/Kconfig +++ b/drivers/vhost/Kconfig @@ -12,23 +12,19 @@ config VHOST_RING This option is selected by any driver which needs to access the host side of a virtio ring. =20 -config VHOST - tristate +menuconfig VHOST + tristate "Vhost Devices" + depends on EVENTFD select VHOST_IOTLB help - This option is selected by any driver which needs to access - the core of vhost. + Enable option to support host kernel or hardware accelerator + for virtio device. =20 -menuconfig VHOST_MENU - bool "VHOST drivers" - default y - -if VHOST_MENU +if VHOST =20 config VHOST_NET tristate "Host kernel accelerator for virtio net" - depends on NET && EVENTFD && (TUN || !TUN) && (TAP || !TAP) - select VHOST + depends on NET && (TUN || !TUN) && (TAP || !TAP) ---help--- This kernel module can be loaded in host kernel to accelerate guest networking with virtio_net. Not to be confused with virtio_net @@ -39,8 +35,7 @@ config VHOST_NET =20 config VHOST_SCSI tristate "VHOST_SCSI TCM fabric driver" - depends on TARGET_CORE && EVENTFD - select VHOST + depends on TARGET_CORE default n ---help--- Say M here to enable the vhost_scsi TCM fabric module @@ -48,8 +43,7 @@ config VHOST_SCSI =20 config VHOST_VSOCK tristate "vhost virtio-vsock driver" - depends on VSOCKETS && EVENTFD - select VHOST + depends on VSOCKETS select VIRTIO_VSOCKETS_COMMON default n ---help--- @@ -62,8 +56,6 @@ config VHOST_VSOCK =20 config VHOST_VDPA tristate "Vhost driver for vDPA-based backend" - depends on EVENTFD - select VHOST depends on VDPA help This kernel module can be loaded in host kernel to accelerate --=20 2.20.1