On 09/17/2007 06:29 PM, Kok, Auke wrote: > Jiri Slaby wrote: >> On 12/23/-28158 08:59 PM, Auke Kok wrote: >>> This incorporates the new napi_struct changes into e1000e. Included >>> bugfix for ifdown hang from Krishna Kumar for e1000. >>> >>> Disabling polling is no longer needed at init time, so remove >>> napi_disable() call from _probe(). >>> >>> This also fixes an endless polling loop where the driver signalled >>> "polling done" improperly back to the stack. >>> >>> Signed-off-by: Auke Kok <[EMAIL PROTECTED]>
Tested-by: Jiri Slaby <[EMAIL PROTECTED]> >>> --- >>> >>> drivers/net/e1000e/e1000.h | 2 ++ >>> drivers/net/e1000e/netdev.c | 40 ++++++++++++++++------------------------ >>> 2 files changed, 18 insertions(+), 24 deletions(-) [...] > I have no lockups here as well anymore, so I'm wondering if you accidentally > used > V3 of the patch. And so am I, it works, I ported one change to e1000 (non-e) driver and missed the return 0; removal. Fixed both of them now, so one of them caused it. thanks a lot, -- Jiri Slaby ([EMAIL PROTECTED]) Faculty of Informatics, Masaryk University - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html