login
Header Space

 
 

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

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jeff Garzik <jeff@...>
Cc: <netdev@...>, Ayaz Abdulla <aabdulla@...>, LKML <linux-kernel@...>, Andrew Morton <akpm@...>
Date: Sunday, October 7, 2007 - 5:08 am

* Jeff Garzik <jeff@garzik.org> wrote:


/me agrees violently

btw., when i played with this tunable under -rt:

 enum {
         NV_OPTIMIZATION_MODE_THROUGHPUT,
         NV_OPTIMIZATION_MODE_CPU
 };
 static int optimization_mode = NV_OPTIMIZATION_MODE_THROUGHPUT;

the MODE_CPU one gave (much) _higher_ bandwidth. The queueing model in 
forcedeth seemed to be not that robust and i think a single queueing 
model should be adopted instead of this tunable. (which i think just hid 
some bug/dependency) But i never got to the bottom of it so it's just 
the impression i got.

	Ingo
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 6/n] forcedeth: protect slow path with mutex, Jeff Garzik, (Sun Oct 7, 7:23 am)
Re: [PATCH 0/5] forcedeth: several proposed updates for test..., Ingo Molnar, (Sun Oct 7, 5:08 am)
[PATCH 5/5] forcedeth: timer overhaul, Jeff Garzik, (Sat Oct 6, 11:15 am)
[PATCH 2/5] forcedeth: interrupt handling cleanup, Jeff Garzik, (Sat Oct 6, 11:14 am)
[PATCH 1/5] forcedeth: make NAPI unconditional, Jeff Garzik, (Sat Oct 6, 11:13 am)
speck-geostationary