[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <KL1P15301MB0008BBAEBF60A852EA629A9ABF830@KL1P15301MB0008.APCP153.PROD.OUTLOOK.COM>
Date: Thu, 17 Aug 2017 18:27:30 +0000
From: Dexuan Cui <decui@...rosoft.com>
To: David Miller <davem@...emloft.net>
CC: "jhansen@...are.com" <jhansen@...are.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"devel@...uxdriverproject.org" <devel@...uxdriverproject.org>,
KY Srinivasan <kys@...rosoft.com>,
"Haiyang Zhang" <haiyangz@...rosoft.com>,
Stephen Hemminger <sthemmin@...rosoft.com>,
"georgezhang@...are.com" <georgezhang@...are.com>,
"mkubecek@...e.cz" <mkubecek@...e.cz>,
"asias@...hat.com" <asias@...hat.com>,
"stefanha@...hat.com" <stefanha@...hat.com>,
"vkuznets@...hat.com" <vkuznets@...hat.com>,
"cavery@...hat.com" <cavery@...hat.com>,
"jasowang@...hat.com" <jasowang@...hat.com>,
"rolf.neugebauer@...ker.com" <rolf.neugebauer@...ker.com>,
"dave.scott@...ker.com" <dave.scott@...ker.com>,
"marcelo.cerri@...onical.com" <marcelo.cerri@...onical.com>,
"apw@...onical.com" <apw@...onical.com>,
"olaf@...fle.de" <olaf@...fle.de>,
"joe@...ches.com" <joe@...ches.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"dan.carpenter@...cle.com" <dan.carpenter@...cle.com>
Subject: RE: [PATCH] vsock: only load vmci transport on VMware hypervisor by
default
> From: David Miller [mailto:davem@...emloft.net]
> Sent: Thursday, August 17, 2017 10:04
> I would avoid module parameters at all costs.
>
> It is the worst possible interface for users of your software.
>
> You really need to fundamentally solve the problems related to making
> sure the proper modules for the VM actually present on the system get
> loaded when necessary rather than adding hacks like this.
>
> Unlike a proper solution, these hacks are ugly but have to stay around
> forever once you put them in place.
Sorry for reminding me again, David! :-)
I'll try to figure out the correct solution.
Thanks,
-- Dexuan
Powered by blists - more mailing lists