From: Rick Jones <rick.jones2@hp.com>
To: Florian Fainelli <florian@openwrt.org>
Cc: Ben Greear <greearb@candelatech.com>,
"John W. Linville" <linville@tuxdriver.com>,
Johannes Berg <johannes@sipsolutions.net>,
linux-wireless@vger.kernel.org
Subject: Re: [RFC 2/2] mac80211: Support getting sta_info stats via ethtool.
Date: Fri, 16 Mar 2012 10:14:18 -0700 [thread overview]
Message-ID: <4F6374EA.6020605@hp.com> (raw)
In-Reply-To: <4F635562.6090505@openwrt.org>
On 03/16/2012 07:59 AM, Florian Fainelli wrote:
> Le 03/16/12 15:54, Ben Greear a écrit :
>> On 03/16/2012 07:18 AM, Florian Fainelli wrote:
>>> Hello,
>>>
>>> Le 03/16/12 14:49, John W. Linville a écrit :
>>>> On Thu, Mar 15, 2012 at 11:56:24AM -0700, Ben Greear wrote:
>>>>> On 03/15/2012 11:52 AM, Johannes Berg wrote:
>>>>>> On Thu, 2012-03-15 at 10:39 -0700, greearb@candelatech.com wrote:
>>>>>>> From: Ben Greear<greearb@candelatech.com>
>>>>>>>
>>>>>>> This lets ethtool print out stats related to station
>>>>>>> interfaces. Does not yet get stats from the underlying
>>>>>>> driver.
>>>>>>
>>>>>> Hmm. What's the advantage of using ethtool over iw, which already
>>>>>> has a bunch of these numbers?
>>>>>
>>>>> Well, ethtool api might be easier for some apps to use,
>>>>> and perhaps easier for users to read if all they want
>>>>> are stats.
>>>>
>>>> And they can use the same tool for both wired and wireless interfaces
>>>> -- could be handy.
>>>
>>> iw already provides statistics which are relevant for wireless
>>> interfaces. I really don't see the point in also reporting them via
>>> ethtool, also it is going to
>>> be error prone if someone updates the netlink interface and forgets
>>> about the ethtool one.
>>
>> Ethtool provides what it provides. If it's missing a stat, I or
>> someone else can add it.
>
> What if we don't want to bloat ethtool with new stats? I mean, someone
> else one day will see wireless stats in, and say, hey why don't I add
> atm, x25, or any protocol of the moment to ethtool, then what do we do?
Rejoice and perhaps add an alias to ethtool called nictool or linktool
or somesuch? As already pointed-out there is a non-trivial niceness to
not having to have umpteen different tools to get stats from a network
interface.
"What's in a name" may apply, but these are wireless *ethernet*
interfaces right?
rick jones
next prev parent reply other threads:[~2012-03-16 17:14 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-15 17:39 [RFC 1/2] cfg80211: Add framework to support ethtool stats greearb
2012-03-15 17:39 ` [RFC 2/2] mac80211: Support getting sta_info stats via ethtool greearb
2012-03-15 18:52 ` Johannes Berg
2012-03-15 18:56 ` Ben Greear
2012-03-16 13:49 ` John W. Linville
2012-03-16 14:18 ` Florian Fainelli
2012-03-16 14:54 ` Ben Greear
2012-03-16 14:59 ` Florian Fainelli
2012-03-16 16:39 ` Ben Greear
2012-03-16 17:14 ` Rick Jones [this message]
2012-03-15 18:55 ` Johannes Berg
2012-03-15 19:03 ` Ben Greear
2012-03-15 19:05 ` Johannes Berg
2012-03-15 18:50 ` [RFC 1/2] cfg80211: Add framework to support ethtool stats Johannes Berg
2012-03-15 18:52 ` Ben Greear
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=4F6374EA.6020605@hp.com \
--to=rick.jones2@hp.com \
--cc=florian@openwrt.org \
--cc=greearb@candelatech.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
/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).