[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070723105629.278fcce3@oldman.hamilton.local>
Date: Mon, 23 Jul 2007 10:56:29 +0100
From: Stephen Hemminger <shemminger@...ux-foundation.org>
To: Krishna Kumar2 <krkumar2@...ibm.com>
Cc: davem@...emloft.net, gaagaan@...il.com,
general@...ts.openfabrics.org, hadi@...erus.ca,
herbert@...dor.apana.org.au, jagana@...ibm.com, jeff@...zik.org,
johnpol@....mipt.ru, Patrick McHardy <kaber@...sh.net>,
kumarkr@...ux.ibm.com, mcarlson@...adcom.com, mchan@...adcom.com,
netdev@...r.kernel.org, peter.p.waskiewicz.jr@...el.com,
rdreier@...co.com, rick.jones2@...com, Robert.Olsson@...a.slu.se,
sri@...ibm.com, tgraf@...g.ch, xma@...ibm.com
Subject: Re: [PATCH 04/10] net-sysfs.c changes.
On Sat, 21 Jul 2007 12:16:30 +0530
Krishna Kumar2 <krkumar2@...ibm.com> wrote:
> Stephen Hemminger <shemminger@...ux-foundation.org> wrote on 07/20/2007
> 09:52:03 PM:
> > Patrick McHardy <kaber@...sh.net> wrote:
> >
> > > Krishna Kumar2 wrote:
> > > > Patrick McHardy <kaber@...sh.net> wrote on 07/20/2007 03:37:20 PM:
> > > >
> > > >
> > > >
> > > >> rtnetlink support seems more important than sysfs to me.
> > > >>
> > > >
> > > > Thanks, I will add that as a patch. The reason to add to sysfs is
> that
> > > > it is easier to change for a user (and similar to tx_queue_len).
> > > >
> > >
> >
> > But since batching is so similar to TSO, i really should be part of the
> > flags and controlled by ethtool like other offload flags.
>
> So should I add all three interfaces (or which ones) :
>
> 1. /sys (like for tx_queue_len)
> 2. netlink
> 3. ethtool.
>
> Or only 2 & 3 are enough ?
>
Yes, please do #3 and maybe #2.
Sysfs api's are a long term ABI problem.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists