[PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

Dexuan Cui decui at microsoft.com
Tue Jul 7 10:20:41 UTC 2015


> -----Original Message-----
> From: Olaf Hering [mailto:olaf at aepfle.de]
> Sent: Tuesday, July 7, 2015 18:10
> To: Dexuan Cui; Paul Bolle
> Cc: gregkh at linuxfoundation.org; davem at davemloft.net;
> netdev at vger.kernel.org; linux-kernel at vger.kernel.org; driverdev-
> devel at linuxdriverproject.org; apw at canonical.com; jasowang at redhat.com; KY
> Srinivasan; Haiyang Zhang
> Subject: Re: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature
> 
> On Tue, Jul 07, Paul Bolle wrote:
> 
> > On ma, 2015-07-06 at 07:47 -0700, Dexuan Cui wrote:
> > > --- /dev/null
> > > +++ b/net/hv_sock/Kconfig
> >
> > > +config HYPERV_SOCK
> > > +	tristate "Microsoft Hyper-V Socket (EXPERIMENTAL)"
> > > +	depends on HYPERV
> > > +	default m
> 
> > It's a bit odd to advise to say N if one is unsure and set the default
> > to 'm' at the same time.
> 
> The 'default' line has to be removed IMO.
> 
> Olaf

OK, removing the line seems better than 'default n', though both reproduce
the same "# CONFIG_HYPERV_SOCK is not set".

-- Dexuan


More information about the devel mailing list