From: Stephen Hemminger <stephen@networkplumber.org>
To: Randy Dunlap <rdunlap@infradead.org>
Cc: Greg KH <greg@kroah.com>,
Vincent Legoll <vincent.legoll@gmail.com>,
devel@linuxdriverproject.org, haiyangz@microsoft.com,
sthemmin@microsoft.com,
Linux Kernel ML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] hyperv: make HYPERV a menuconfig to ease disabling it all
Date: Sat, 16 Dec 2017 09:51:20 -0800 [thread overview]
Message-ID: <20171216095120.568fad34@xeon-e3> (raw)
In-Reply-To: <b8d654d1-67c9-b6bc-7aad-e5c7a4beec48@infradead.org>
On Thu, 14 Dec 2017 09:43:48 -0800
Randy Dunlap <rdunlap@infradead.org> wrote:
> On 12/13/2017 11:51 PM, Greg KH wrote:
> > On Wed, Dec 13, 2017 at 01:23:38PM -0800, Stephen Hemminger wrote:
> >> On Wed, 13 Dec 2017 09:54:19 +0100
> >> Vincent Legoll <vincent.legoll@gmail.com> wrote:
> >>
> >>> Hello,
> >>>
> >>> On Sun, Dec 10, 2017 at 6:50 AM, Stephen Hemminger
> >>> <stephen@networkplumber.org> wrote:
> >>>> Will this break existing configs?
> >>>
> >>> I don't think so. Last time I did some similar changes, the kbuild
> >>> test robot found some warnings on some configurations, I hope
> >>> it will find problems (if any) for that series too (this one is not alone,
> >>> I've got a bunch of other similar patches in-flight)
> >>>
> >>> Thanks
> >>
> >> NAK
> >>
> >> Let me give a concrete example of how this will break users.
> >>
> >> 1. Assume user has a working .config file in their kernel build directory
> >> which builds a kernel that works on Hyper-V.
> >>
> >> 2. Add your patch (or assume it makes into a later version).
> >>
> >> 3. User then does
> >>
> >> $ make oldconfig
> >> scripts/kconfig/conf --oldconfig Kconfig
> >> *
> >> * Restart config...
> >> *
> >> *
> >> * Microsoft Hyper-V guest support
> >> *
> >> Microsoft Hyper-V guest support (HYPERV_MENU) [N/y] (NEW)
> >>
> >> If they hit return, the default value is not enabling HyperV and they
> >> will then go on to build a kernel that will not boot on your system.
> >>
> >> The default MUST be set to Yes.
>
> That should work.
>
> > Or you can just not take these types of odd and silly changes to the
> > Kconfig files, and leave it as-is. I have yet to see the good reason
> > why these are needed at all.
>
> Some of us would like to be able to disable many like drivers at one time
> instead of having to go down a list of say 20-30 drivers and disable them
> one at a time.
>
It makes sense to organize the config if you dont break old configs.
It would be more logical to group and treat all para-virtualized guest
support in same way. Hyper-V should be next to KVM and Xen.
next prev parent reply other threads:[~2017-12-16 17:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-09 15:21 [PATCH] hyperv: make HYPERV a menuconfig to ease disabling it all Vincent Legoll
2017-12-10 5:50 ` Stephen Hemminger
2017-12-13 8:54 ` Vincent Legoll
2017-12-13 21:23 ` Stephen Hemminger
2017-12-14 7:51 ` Greg KH
2017-12-14 17:43 ` Randy Dunlap
2017-12-16 10:07 ` [PATCH,v2] hyperv: make HYPERV a menuconfig to ease disabling it al Vincent Legoll
2017-12-16 10:07 ` [PATCH] hyperv: make HYPERV a menuconfig to ease disabling it all Vincent Legoll
2017-12-16 17:51 ` Stephen Hemminger [this message]
2017-12-17 14:02 ` Vincent Legoll
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171216095120.568fad34@xeon-e3 \
--to=stephen@networkplumber.org \
--cc=devel@linuxdriverproject.org \
--cc=greg@kroah.com \
--cc=haiyangz@microsoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=sthemmin@microsoft.com \
--cc=vincent.legoll@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).