diff options
| author | Eric Dumazet <dada1@cosmosbay.com> | 2008-11-24 15:52:46 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-11-24 15:52:46 -0800 |
| commit | 2e77d89b2fa8e3f8325b8ce7893ec3645f41aff5 (patch) | |
| tree | ae40aa75449f705bd166630f9bcb5f41373d8248 /net/ipv4/raw.c | |
| parent | 4db0acf3c0afbbbb2ae35a65f8896ca6655a47ec (diff) | |
| download | kernel_goldelico_gta04-2e77d89b2fa8e3f8325b8ce7893ec3645f41aff5.zip kernel_goldelico_gta04-2e77d89b2fa8e3f8325b8ce7893ec3645f41aff5.tar.gz kernel_goldelico_gta04-2e77d89b2fa8e3f8325b8ce7893ec3645f41aff5.tar.bz2 | |
net: avoid a pair of dst_hold()/dst_release() in ip_append_data()
We can reduce pressure on dst entry refcount that slowdown UDP transmit
path on SMP machines. This pressure is visible on RTP servers when
delivering content to mediagateways, especially big ones, handling
thousand of streams. Several cpus send UDP frames to the same
destination, hence use the same dst entry.
This patch makes ip_append_data() eventually steal the refcount its
callers had to take on the dst entry.
This doesnt avoid all refcounting, but still gives speedups on SMP,
on UDP/RAW transmit path
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/raw.c')
| -rw-r--r-- | net/ipv4/raw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/raw.c b/net/ipv4/raw.c index 998fcff..dff8bc4 100644 --- a/net/ipv4/raw.c +++ b/net/ipv4/raw.c @@ -572,7 +572,7 @@ back_from_confirm: ipc.addr = rt->rt_dst; lock_sock(sk); err = ip_append_data(sk, ip_generic_getfrag, msg->msg_iov, len, 0, - &ipc, rt, msg->msg_flags); + &ipc, &rt, msg->msg_flags); if (err) ip_flush_pending_frames(sk); else if (!(msg->msg_flags & MSG_MORE)) |