From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754071AbcBEPC3 (ORCPT ); Fri, 5 Feb 2016 10:02:29 -0500 Received: from shell.v3.sk ([92.60.52.57]:53795 "EHLO shell.v3.sk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750853AbcBEPC1 convert rfc822-to-8bit (ORCPT ); Fri, 5 Feb 2016 10:02:27 -0500 Message-ID: <1454684532.2779.5.camel@v3.sk> Subject: Re: [PATCH 1/3] ifb: make device count build-time configurable From: Lubomir Rintel To: David Miller , stephen@networkplumber.org Cc: jhs@mojatatu.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Date: Fri, 05 Feb 2016 16:02:12 +0100 In-Reply-To: <20160112.155440.538873882796833417.davem@davemloft.net> References: <1452599804-27284-1-git-send-email-lkundrak@v3.sk> <5694F7BA.3040402@mojatatu.com> <20160112104437.0e1a841b@xeon-e3> <20160112.155440.538873882796833417.davem@davemloft.net> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.4 (3.18.4-1.fc23) Mime-Version: 1.0 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2016-01-12 at 15:54 -0500, David Miller wrote: > From: Stephen Hemminger > Date: Tue, 12 Jan 2016 10:44:37 -0800 > > > 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. > > I like this even less, it means tools behave significantly > differently > based upon what module options were passed to the kernel. > > Module options really should not change kernel behavior like this.. The module option is already there. It's defaults (creating the devices noone asked for and that potentially collide with what the user tried to create) are what we find bothersome. Lubo