Home » Mailing list archives » linux-kernel » 2010 » April » 23

Re: [PATCH] e100: expose broadcast_disabled as a module option

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Stephen Hemminger
Subject: Re: [PATCH] e100: expose broadcast_disabled as a module option
Date: Friday, April 23, 2010 - 3:02 pm

On Fri, 23 Apr 2010 23:03:59 +0200
Erwan Velu <erwanaliasr1@gmail.com> wrote:


The point is that the driver can look at IFF_BROADCAST rather than having
module parameter. Module parameters are device driver specific and should
be avoid as much as possible in favor of general mechanism. This is a repeated
problem where users and vendors make special hooks that only work with their
driver, which makes life hard for other users and distribution providers.
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH] e100: expose broadcast_disabled as a module option, Erwan Velu, (Fri Apr 23, 1:14 pm)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Jeff Kirsher, (Fri Apr 23, 1:22 pm)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Stephen Hemminger, (Fri Apr 23, 1:58 pm)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Erwan Velu, (Fri Apr 23, 2:03 pm)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Stephen Hemminger, (Fri Apr 23, 3:02 pm)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Erwan Velu, (Mon Apr 26, 1:49 am)
Re: [PATCH] e100: expose broadcast_disabled as a module option, Erwan Velu, (Mon Apr 26, 7:45 am)