Home » Mailing list archives » linux-kernel » 2007 » October » 7

Re: [PATCH 0/5] forcedeth: several proposed updates for testing

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Jeff Garzik <jeff@...>
To: <netdev@...>, Ayaz Abdulla <aabdulla@...>
Cc: LKML <linux-kernel@...>, Andrew Morton <akpm@...>, Yinghai Lu <yhlu.kernel@...>, Ingo Molnar <mingo@...>
Subject: Re: [PATCH 0/5] forcedeth: several proposed updates for testing
Date: Sunday, October 7, 2007 - 10:47 am

Jeff Garzik wrote:

OK, I've successfully tested patches 1-5 on an AMD64 system with

00:0a.0 Bridge: nVidia Corporation CK804 Ethernet Controller (rev a3)

A trivial s/napi/tx_napi/ fix must be applied to patch #3 (sent in 
separate email).  Once that is done, the patchset works flawlessly here, 
passing simple RX, TX, RX+TX TCP stress tests.

I never ran into any TX problems, of the type hinted at by the "Known 
bugs" section at the top of forcedeth.c.  Either (a) my chip does not 
have that bug, (b) my chip needs DEV_NEED_TIMERIRQ for other reasons, or 
(c) the issue is not a hardware issue but a driver bug that is now fixed.

I'm going to hope its (c), <grin>  But only testing will tell.

	Jeff


P.S.  Depending on when this gets fixed, you might have to revert 
net-2.6.24.git commit 5f5dace1ce001b24fb8286e09ffd3c4d2b547e09 ("[NET]: 
Various dst_ifdown routines to catch refcounting bugs").
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/5] forcedeth: several proposed updates for testing, Jeff Garzik, (Sat Oct 6, 11:12 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Jeff Garzik, (Sun Oct 7, 10:47 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Yinghai Lu, (Sun Oct 7, 3:39 pm)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Jeff Garzik, (Sun Oct 7, 4:05 pm)
[PATCH 6/n] forcedeth: protect slow path with mutex, Jeff Garzik, (Sun Oct 7, 7:23 am)
Re: [PATCH 6/n] forcedeth: protect slow path with mutex, Jeff Garzik, (Sun Oct 7, 10:40 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Ingo Molnar, (Sun Oct 7, 5:08 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Jeff Garzik, (Sun Oct 7, 7:34 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Jeff Garzik, (Sat Oct 6, 11:24 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Jeff Garzik, (Sat Oct 6, 11:17 am)
[PATCH 5/5] forcedeth: timer overhaul, Jeff Garzik, (Sat Oct 6, 11:15 am)
[PATCH 4/5] forcedeth: internal simplification and cleanups, Jeff Garzik, (Sat Oct 6, 11:14 am)
[PATCH 3/5] forcedeth: process TX completions using NAPI, Jeff Garzik, (Sat Oct 6, 11:14 am)
Re: [PATCH 3/5] forcedeth: process TX completions using NAPI, Jeff Garzik, (Sun Oct 7, 10:39 am)
[PATCH 2/5] forcedeth: interrupt handling cleanup, Jeff Garzik, (Sat Oct 6, 11:14 am)
Re: [PATCH 2/5] forcedeth: interrupt handling cleanup, Ingo Molnar, (Sun Oct 7, 5:03 am)
Re: [PATCH 2/5] forcedeth: interrupt handling cleanup, Yinghai Lu, (Sun Oct 7, 12:43 am)
Re: [PATCH 2/5] forcedeth: interrupt handling cleanup, Jeff Garzik, (Sun Oct 7, 7:40 am)
Re: [PATCH 2/5] forcedeth: interrupt handling cleanup, Yinghai Lu, (Sun Oct 7, 3:36 pm)
Re: [PATCH 2/5] forcedeth: interrupt handling cleanup, Jeff Garzik, (Sun Oct 7, 4:07 pm)
[PATCH 1/5] forcedeth: make NAPI unconditional, Jeff Garzik, (Sat Oct 6, 11:13 am)