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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.