* [PATCH] virtio: make VIRTIO a menuconfig to ease disabling it all [not found] <459e2097-f20a-18f8-17d9-2d9ed011211d@infradead.org> @ 2018-01-07 11:33 ` Vincent Legoll [not found] ` <20180107113356.2806-1-vincent.legoll@gmail.com> 1 sibling, 0 replies; 4+ messages in thread From: Vincent Legoll @ 2018-01-07 11:33 UTC (permalink / raw) To: Michael S. Tsirkin, Jason Wang, virtualization, linux-kernel Cc: Vincent Legoll, Randy Dunlap No need to get into the submenu to disable all VIRTIO-related config entries. This makes it easier to disable all VIRTIO config options without entering the submenu. It will also enable one to see that en/dis-abled state from the outside menu. This is only intended to change menuconfig UI, not change the config dependencies. v2: Added "default y" to avoid breaking existing configs v3: Fixed wrong indentation, added *-by from Randy Signed-off-by: Vincent Legoll <vincent.legoll@gmail.com> Reviewed-by: Randy Dunlap <rdunlap@infradead.org> Tested-by: Randy Dunlap <rdunlap@infradead.org> # works for me --- drivers/virtio/Kconfig | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig index cff773f15b7e..35897649c24f 100644 --- a/drivers/virtio/Kconfig +++ b/drivers/virtio/Kconfig @@ -5,7 +5,11 @@ config VIRTIO bus, such as CONFIG_VIRTIO_PCI, CONFIG_VIRTIO_MMIO, CONFIG_RPMSG or CONFIG_S390_GUEST. -menu "Virtio drivers" +menuconfig VIRTIO_MENU + bool "Virtio drivers" + default y + +if VIRTIO_MENU config VIRTIO_PCI tristate "PCI driver for virtio devices" @@ -79,4 +83,4 @@ config VIRTIO_MMIO_CMDLINE_DEVICES If unsure, say 'N'. -endmenu +endif # VIRTIO_MENU -- 2.14.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
[parent not found: <20180107113356.2806-1-vincent.legoll@gmail.com>]
* Ping Re: [PATCH] virtio: make VIRTIO a menuconfig to ease disabling it all [not found] ` <20180107113356.2806-1-vincent.legoll@gmail.com> @ 2018-01-23 4:31 ` Michael Ellerman [not found] ` <87h8rdp55n.fsf@concordia.ellerman.id.au> 1 sibling, 0 replies; 4+ messages in thread From: Michael Ellerman @ 2018-01-23 4:31 UTC (permalink / raw) To: Michael S. Tsirkin, Jason Wang, virtualization, linux-kernel Cc: Vincent Legoll, Randy Dunlap Vincent Legoll <vincent.legoll@gmail.com> writes: > No need to get into the submenu to disable all VIRTIO-related > config entries. > > This makes it easier to disable all VIRTIO config options > without entering the submenu. It will also enable one > to see that en/dis-abled state from the outside menu. > > This is only intended to change menuconfig UI, not change > the config dependencies. > > v2: Added "default y" to avoid breaking existing configs > v3: Fixed wrong indentation, added *-by from Randy > > Signed-off-by: Vincent Legoll <vincent.legoll@gmail.com> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org> > Tested-by: Randy Dunlap <rdunlap@infradead.org> # works for me > --- > drivers/virtio/Kconfig | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) This has been broken in linux-next for ~6 weeks now, can we please merge this and get it fixed. cheers ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <87h8rdp55n.fsf@concordia.ellerman.id.au>]
* Re: Ping Re: [PATCH] virtio: make VIRTIO a menuconfig to ease disabling it all [not found] ` <87h8rdp55n.fsf@concordia.ellerman.id.au> @ 2018-01-23 8:08 ` Vincent Legoll [not found] ` <CAEwRq=oJ0apphUFDheoX5d-BB5yjG3rZD8m4yhK75eZcm6eqVg@mail.gmail.com> 1 sibling, 0 replies; 4+ messages in thread From: Vincent Legoll @ 2018-01-23 8:08 UTC (permalink / raw) To: Michael Ellerman, sfr Cc: virtualization, Randy Dunlap, linux-kernel, Michael S. Tsirkin On 1/23/18, Michael Ellerman <mpe@ellerman.id.au> wrote: > This has been broken in linux-next for ~6 weeks now, can we please merge > this and get it fixed. Added Stephen Rothwell to cc -- Vincent Legoll ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <CAEwRq=oJ0apphUFDheoX5d-BB5yjG3rZD8m4yhK75eZcm6eqVg@mail.gmail.com>]
* Re: Ping Re: [PATCH] virtio: make VIRTIO a menuconfig to ease disabling it all [not found] ` <CAEwRq=oJ0apphUFDheoX5d-BB5yjG3rZD8m4yhK75eZcm6eqVg@mail.gmail.com> @ 2018-01-28 10:14 ` Michael Ellerman 0 siblings, 0 replies; 4+ messages in thread From: Michael Ellerman @ 2018-01-28 10:14 UTC (permalink / raw) To: Michael S. Tsirkin, Jason Wang, Vincent Legoll, sfr Cc: Randy Dunlap, linux-kernel, virtualization Vincent Legoll <vincent.legoll@gmail.com> writes: > On 1/23/18, Michael Ellerman <mpe@ellerman.id.au> wrote: >> This has been broken in linux-next for ~6 weeks now, can we please merge >> this and get it fixed. > > Added Stephen Rothwell to cc It should be fixed in the virtio tree, which the other Michael and Jason maintain AFAIK. cheers ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-01-28 10:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <459e2097-f20a-18f8-17d9-2d9ed011211d@infradead.org>
2018-01-07 11:33 ` [PATCH] virtio: make VIRTIO a menuconfig to ease disabling it all Vincent Legoll
[not found] ` <20180107113356.2806-1-vincent.legoll@gmail.com>
2018-01-23 4:31 ` Ping " Michael Ellerman
[not found] ` <87h8rdp55n.fsf@concordia.ellerman.id.au>
2018-01-23 8:08 ` Vincent Legoll
[not found] ` <CAEwRq=oJ0apphUFDheoX5d-BB5yjG3rZD8m4yhK75eZcm6eqVg@mail.gmail.com>
2018-01-28 10:14 ` Michael Ellerman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox