[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080603123520.GF18161@solarflare.com>
Date: Tue, 3 Jun 2008 13:35:22 +0100
From: Ben Hutchings <bhutchings@...arflare.com>
To: James Cammarata <jimi@...x.net>
Cc: Bill Fink <billfink@...dspring.com>, linux-kernel@...r.kernel.org,
Linux Netdev List <netdev@...r.kernel.org>
Subject: Re: [PATCH] net: add ability to clear stats via ethtool - e1000/pcnet32
James Cammarata wrote:
> >First of all, when assisting a user, they typically aren't even
> >running an snmp daemon (and there might be firewall issues to
> >access it if they are). And I don't think the "ethtool -S" driver
> >stats are even accessible via SNMP (although they may contribute
> >to more generic interface stats which are), and it is the specific
> >driver stats which are often key to help diagnosing the problem.
>
> Ok, just to jump back in and add my $0.02, I plan to send a patch to
> the ethtool project to do what was suggested (snapshot to /var, and
> diff stats against that for future reference). I would just like to
> point out that this will introduce inconsistencies between _every_
> other source of interface stats (ifconfig/procfs/etc). I can deal
> with that, it just means one more thing to keep track of when you're
> troubleshooting.
You could perhaps add a third column to the statistics when a snapshot
exists, with the second column still showing absolute values and the
third showing the difference from the snapshot.
Ben.
--
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
--
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