| To: | Rick Jones <rick.jones2@xxxxxx> |
|---|---|
| Subject: | Re: RFC: NAPI packet weighting patch |
| From: | Andi Kleen <ak@xxxxxxx> |
| Date: | 21 Jun 2005 23:47:07 +0200 |
| Cc: | netdev@xxxxxxxxxxx, davem@xxxxxxxxxx |
| In-reply-to: | <42B87ACF.3080800@xxxxxx> |
| References: | <468F3FDA28AA87429AD807992E22D07E0450C00B@orsmsx408> <Pine.CYG.4.58.0506061647340.128@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <42A5284C.3060808@xxxxxxxx> <20050621.132044.115910664.davem@xxxxxxxxxxxxx> <42B87ACF.3080800@xxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Gnus/5.09 (Gnus v5.9.0) Emacs/21.3 |
Rick Jones <rick.jones2@xxxxxx> writes: > > Actually, it has a _HUGE_ _HUGE_ impact. If you pass the big buffer > > up, the receiving socket gets charged for the size of the huge buffer, > > not for just the size of the packet contained within. This makes > > sockets get overcharged for data reception, and it can cause all kinds > > of performance problems. > > Then copy when the socket is about to fill with overhead bytes? The stack has supported that since 2.4. Mostly because it is the only sane way to handle devices with very big MTU. But it turns off all kinds of fast paths before it happens, I guess that is what David was refering too. However I suspect the cut-off points with rx-copybreak in common driver have been often tuned before that code was introduced and it might be worth to do some retesting. -Andi |
| Previous by Date: | [patch,rfc] allow registration of multiple netpolls per interface, Jeff Moyer |
|---|---|
| Next by Date: | Re: [patch] devinet: cleanup if statements, Andi Kleen |
| Previous by Thread: | Re: RFC: NAPI packet weighting patch, David S. Miller |
| Next by Thread: | Re: RFC: NAPI packet weighting patch, Donald Becker |
| Indexes: | [Date] [Thread] [Top] [All Lists] |