[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAGVrzcbW3A5RPibKO=i92dj0815eYV6aFRWhOfXA-KgNAgk9kg@mail.gmail.com>
Date: Thu, 27 Mar 2014 22:56:10 -0700
From: Florian Fainelli <f.fainelli@...il.com>
To: Stephen Rothwell <sfr@...b.auug.org.au>
Cc: David Decotigny <decot@...glers.com>,
"David S. Miller" <davem@...emloft.net>,
Jamal Hadi Salim <jhs@...atatu.com>,
netdev <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Eric Dumazet <edumazet@...gle.com>,
"Eric W. Biederman" <ebiederm@...ssion.com>,
Weilong Chen <chenweilong@...wei.com>,
Jiri Pirko <jiri@...nulli.us>, Amir Vadai <amirv@...lanox.com>,
Michael Dalton <mwdalton@...gle.com>,
Al Viro <viro@...iv.linux.org.uk>, Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH v1] net-sysfs: expose number of link up/down transitions
2014-03-27 22:17 GMT-07:00 Stephen Rothwell <sfr@...b.auug.org.au>:
> Hi David,
>
> On Thu, 27 Mar 2014 21:35:00 -0700 David Decotigny <decot@...glers.com> wrote:
>>
>> Tested:
>> grep . /sys/class/net/*/count_link_*
>> + ip link set dev X down/up
>>
>> Signed-off-by: David Decotigny <decot@...glers.com>
>
> It is not a very enlightening commit message. You should tell us why we
> should add this to the kernel.
This might be useful to detect unstable links for instance, by
measuring how many link transitions you get, which could indicate that
either your NIC or your link partner is at fault.
If possible this should come with a Documentation/ABI/testing/ entry,
quite a lot of network sysfs attributes are lacking one at the moment,
but there is no reason why this should continue.
--
Florian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists