[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 5 May 2009 15:38:06 -0700
From: "Ronciak, John" <john.ronciak@...el.com>
To: Jesper Dangaard Brouer <hawk@...u.dk>,
David Miller <davem@...emloft.net>
CC: "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
"hawk@...x.dk" <hawk@...x.dk>, netdev <netdev@...r.kernel.org>,
"e1000-devel@...ts.sourceforge.net"
<e1000-devel@...ts.sourceforge.net>,
"Brandeburg, Jesse" <jesse.brandeburg@...el.com>,
"Allan, Bruce W" <bruce.w.allan@...el.com>,
"Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@...el.com>
Subject: RE: [PATCH] igb: Record hardware RX overruns in net_stats
>Its really useful for diagnosing problems, and I'm betting
>that this is a
>real-life situation which people is going to experience. We
>might as well
>help our self to more easily identify this issue when people
>report drop problems.
The problem is that the RNBC aren't dropped packets as the numbers you have show. While we can agree that the MPC are the actual dropped packets and could eaily be be used in the fifo overflow count since the packets were really dropped.
>I think that both MPC and RNBC should be stored in
>rx_fifo_errors (and of
>cause still keeping them seperate to ethtool -S).
This would count RNBC packets as packets that the stack did not process, which it did. The MPC packets were not processed by the stack and should be counted as dropped. As you point out, both counts are available via ethtool -S.
>I'll post two patches with these changes tomorrow, for you evaluation.
Thanks, we look forward to see them.
Cheers,
John
-----------------------------------------------------------
"...that your people will judge you on what you can build, not what you destroy.", B. Obama, 2009
--
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