netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Robert Olsson <Robert.Olsson@data.slu.se>
To: Ben Greear <greearb@candelatech.com>
Cc: Robert Olsson <Robert.Olsson@data.slu.se>,
	"'netdev@oss.sgi.com'" <netdev@oss.sgi.com>,
	Jeff Garzik <jgarzik@mandrakesoft.com>
Subject: Re: NAPI-ized tulip patch against 2.4.20-rc1
Date: Wed, 6 Nov 2002 20:47:01 +0100	[thread overview]
Message-ID: <15817.29109.859144.565330@robur.slu.se> (raw)
In-Reply-To: <3DC95631.6030001@candelatech.com>


Ben Greear writes:

 > >  I see you increased the RX-ring to 1024 pkts. 
 > >  Did you really see any improvement with this?
 > 
 > It helped drop fewer packets when running 4 ports at 92Mbps+
 > However, the difference between that and 512 is not large.
 > I would really like to make that size adjustable at module load
 > time and/or runtime, but I'm not sure how easy that would be.
 > 
 > Imagine being able to crank up your receive buffers when running at
 > very high speeds (and/or when you start dropping packets).  At lower speeds,
 > shrink things down and free up resources....

 I still doubt ;-) 

 With e1000 I played with various settings for RX-buffers rather recently 
 when the 82544 increased the number of available buffers from 256 to 4096.

 And I guess my test looks a bit like yours... Injecting an "overload" of 
 packets. I found was 256 buffers was the optimum. Approximative of course.
 
 
 > Were you testing SMP or uni-processor?  I've been doing the tulip testing
 > on a single processor P-IV system.

 We tested both at that time but it's rewritten now... 

 And as you saw for SMP with recycle it is easy to feed the recycled skb 
 back to CPU were it was created/processed.

 So kfree/skb_headerinit "should" cause less cache bouncing. Still we need
 to pollute one cache line to find the CPU "owner". I posted some of 
 numbers here time ago but it would be nice to verify with CPU's performance 
 counters.


 Cheers.

						--ro

  parent reply	other threads:[~2002-11-06 19:47 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-06  6:36 NAPI-ized tulip patch against 2.4.20-rc1 Ben Greear
2002-11-06  6:42 ` Ben Greear
2002-11-06 17:34 ` Robert Olsson
2002-11-06 17:49   ` Ben Greear
2002-11-06 18:31     ` Donald Becker
2002-11-06 18:44       ` Ben Greear
2002-11-06 20:47         ` Donald Becker
2002-11-07  7:08           ` Ben Greear
2002-11-07 13:24             ` jamal
2002-11-07 18:16               ` greear
2002-11-07 21:26                 ` Robert Olsson
2002-11-07 21:25                   ` Ben Greear
2002-11-07 23:29             ` Ben Greear
2002-11-08 11:30               ` jamal
2002-11-08 17:40                 ` Ben Greear
2002-11-07 12:57           ` jamal
2002-11-06 19:47     ` Robert Olsson [this message]
2002-11-06 21:30       ` Ben Greear
2002-11-07 12:48   ` jamal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=15817.29109.859144.565330@robur.slu.se \
    --to=robert.olsson@data.slu.se \
    --cc=greearb@candelatech.com \
    --cc=jgarzik@mandrakesoft.com \
    --cc=netdev@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).