Thread (36 messages) 36 messages, 10 authors, 2011-09-28

Re: [net-next 11/13] igb: Make Tx budget for NAPI user adjustable

flat view

From: David Miller <davem@davemloft.net>
Date: 2011-09-19 21:00:42

From: Alexander Duyck <redacted>
Date: Mon, 19 Sep 2011 09:32:18 -0700
The fact is ixgbe has been using this parameter this way for over 2
years now and the main goal of this patch was just to synchronize how
things work on igb and ixgbe.

Our hardware doesn't have a mechanism for firing an interrupt after X
number of frames so instead we simply have modified things so that we
will only process X number of frames and then fire another
interrupt/poll if needed.  As such we aren't that far out of
compliance with the meaning of how this parameter is supposed to be
used.
All I can say is this was a huge mistake you therefore need to revert
the IXGBE change, these ethtool settings are not for changing NAPI or
software interrupt behavior.

And if you guys plan to be difficult on this and refuse to remove the
IXGBE bits, I'm letting you guys know ahead of time that I'll do it
for you.

If the hardware can't support this facility, neither should these
ethtool hooks, because the whole point is to avoid hardware interrupts
from firing using these parameters.

Propose new mechanisms to control NAPI behavior if you want.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help