[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <OF5ECADA10.ABB9C3B0-ON65257321.000FEF47-65257321.001004EC@in.ibm.com>
Date: Mon, 23 Jul 2007 08:24:58 +0530
From: Krishna Kumar2 <krkumar2@...ibm.com>
To: Patrick McHardy <kaber@...sh.net>
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, 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 06/12 -Rev2] rtnetlink changes.
Hi Patrick,
Patrick McHardy <kaber@...sh.net> wrote on 07/22/2007 10:40:37 PM:
> Krishna Kumar wrote:
> > diff -ruNp org/include/linux/if_link.h rev2/include/linux/if_link.h
> > --- org/include/linux/if_link.h 2007-07-20 16:33:35.000000000 +0530
> > +++ rev2/include/linux/if_link.h 2007-07-20 16:35:08.000000000 +0530
> > @@ -78,6 +78,8 @@ enum
> > IFLA_LINKMODE,
> > IFLA_LINKINFO,
> > #define IFLA_LINKINFO IFLA_LINKINFO
> > + IFLA_TXBTHSKB, /* Driver support for Batch'd skbs */
> > +#define IFLA_TXBTHSKB IFLA_TXBTHSKB
>
>
> Ughh what a name :) I prefer pronouncable names since they are
> much easier to remember and don't need comments explaining
> what they mean.
>
> But I actually think offering just an ethtool interface would
> be better, at least for now.
Great, I will remove /sys and rtnetlink and keep the Ethtool i/f.
Thanks,
- KK
-
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