Re: [RFT] pcnet32 NAPI changes

2006-06-20 Thread Lennart Sorensen
On Tue, Jun 20, 2006 at 11:05:04AM -0500, Jon Mason wrote: > The point of my comment was CPU utilization. > > It appears that a bug is trying to be fixed by adding NAPI. This > sounds a bit hackish to me, and could hide the root cause of the > problem. So I'm not sure that is the best idea, but I

Re: [RFT] pcnet32 NAPI changes

2006-06-20 Thread Jon Mason
On Tue, Jun 20, 2006 at 10:48:07AM -0400, Lennart Sorensen wrote: > On Tue, Jun 20, 2006 at 08:53:55AM -0500, Jon Mason wrote: > > The amount of polls per received packet is very low, thus removing the > > benefit of NAPI. A compile time option would allow those users who know > > better to DTRT.

Re: [RFT] pcnet32 NAPI changes

2006-06-20 Thread Lennart Sorensen
On Tue, Jun 20, 2006 at 08:53:55AM -0500, Jon Mason wrote: > The amount of polls per received packet is very low, thus removing the > benefit of NAPI. A compile time option would allow those users who know > better to DTRT. Well I know on the slow poke system I run on, with the napi polling, the

Re: [RFT] pcnet32 NAPI changes

2006-06-20 Thread Jon Mason
On Mon, Jun 19, 2006 at 04:49:33PM -0400, Lennart Sorensen wrote: > On Mon, Jun 19, 2006 at 03:41:40PM -0500, Jon Mason wrote: > > I believe it is preferred to be a compile option for non-gigabit > > drivers, given that it will be eating a lot of cycles for infrequent > > packets (especially for th

Re: [RFT] pcnet32 NAPI changes

2006-06-19 Thread Lennart Sorensen
On Mon, Jun 19, 2006 at 03:41:40PM -0500, Jon Mason wrote: > I believe it is preferred to be a compile option for non-gigabit > drivers, given that it will be eating a lot of cycles for infrequent > packets (especially for the 10Mb). I believe there was a thread about > this last year when e100 wa

Re: [RFT] pcnet32 NAPI changes

2006-06-19 Thread Jon Mason
On Fri, Jun 16, 2006 at 12:11:54PM -0700, Don Fry wrote: > This patch is a collection of changes to pcnet32 which does the > following: > > - Fix section mismatch warning. > - fix set_ringparam to correctly handle memory allocation failures > - fix off-by-one in get_ringparam. > - cleanup at end

Re: [RFT] pcnet32 NAPI changes

2006-06-19 Thread Lennart Sorensen
On Fri, Jun 16, 2006 at 12:11:54PM -0700, Don Fry wrote: > This patch is a collection of changes to pcnet32 which does the > following: > > - Fix section mismatch warning. > - fix set_ringparam to correctly handle memory allocation failures > - fix off-by-one in get_ringparam. > - cleanup at end

[RFT] pcnet32 NAPI changes

2006-06-16 Thread Don Fry
This patch is a collection of changes to pcnet32 which does the following: - Fix section mismatch warning. - fix set_ringparam to correctly handle memory allocation failures - fix off-by-one in get_ringparam. - cleanup at end of loopback_test when not up. - Add NAPI to driver, fixing set_ringpara