From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 1/3] ifb: make device count build-time configurable Date: Tue, 12 Jan 2016 10:44:37 -0800 Message-ID: <20160112104437.0e1a841b@xeon-e3> References: <1452599804-27284-1-git-send-email-lkundrak@v3.sk> <5694F7BA.3040402@mojatatu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Lubomir Rintel , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "David S. Miller" To: Jamal Hadi Salim Return-path: In-Reply-To: <5694F7BA.3040402@mojatatu.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, 12 Jan 2016 07:55:22 -0500 Jamal Hadi Salim wrote: > On 16-01-12 06:56 AM, Lubomir Rintel wrote: > > The devices can be created at run-time for quite some time already and the > > load-time device creation collides with attempts to create the device of > > the same name: > > > > # rmmod ifb > > # ip link add ifb0 type ifb > > RTNETLINK answers: File exists > > > > This is pretty much the same situation as was with the block loop devices > > which was solved by adding a build-time configuration that the > > distributions could use as they deem fit while keeping the default for > > compatibility. > > > > Let's do that here as well. > > > > Signed-off-by: Lubomir Rintel > > I guess module options are frowned upon. so: I would prefer that this were done with a module parameter, the same as dummy. Only developers build their own configured kernels. Having the value set later at module load time is preferable.