netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
To: David Miller <davem@davemloft.net>
Cc: "netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: RFC: how to control device specific features. sysfs or mod param?
Date: Sat, 05 Dec 2009 02:22:53 -0800	[thread overview]
Message-ID: <1260008573.3565.17.camel@localhost> (raw)
In-Reply-To: <20091205.021321.73648278.davem@davemloft.net>

On Sat, 2009-12-05 at 02:13 -0800, David Miller wrote:
> From: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
> Date: Sat, 05 Dec 2009 01:55:55 -0800
> 
> > My question is I want to toggle the underlying hardware mode to use one
> > or the other filter mechanism.  I can either do this using a module
> > parameter, which I'd like to avoid, or I can do this in a sysfs
> > parameter.  What do people think is the best approach?  Is there another
> > option I'm not considering here?
> 
> Other than sysfs another option to consider is trying to fit
> this into the ethtool framework somehow.

Yes, I looked at either ethtool or extending any of the existing
rtnetlink interfaces.  But my problem with those is this setting is very
specific to my hardware, and I'm not sure anyone else could benefit from
this.

I could try and make it as generic as possible, say pass a 32 or 64-bit
generic bitfield to a driver through an ethtool operation, and each
driver would apply some secret sauce based on the bitfield?

The sysfs option seems somewhat natural though.  It's part of the driver
itself, and would be a toggle switch for that specific driver.

I have no attachment to any of these options.  I just want whatever is
maintainable, makes sense, and if others can benefit, that's a great
bonus.

-PJ


  reply	other threads:[~2009-12-05 10:22 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-05  9:55 RFC: how to control device specific features. sysfs or mod param? Peter P Waskiewicz Jr
2009-12-05 10:13 ` David Miller
2009-12-05 10:22   ` Peter P Waskiewicz Jr [this message]
2009-12-05 10:25     ` David Miller
2009-12-05 10:29       ` Peter P Waskiewicz Jr

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1260008573.3565.17.camel@localhost \
    --to=peter.p.waskiewicz.jr@intel.com \
    --cc=davem@davemloft.net \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).