Thread (21 messages) 21 messages, 3 authors, 2012-05-10

Re: [PATCH 7/9] net: add skb_orphan_frags to copy aside frags with destructors

From: David Miller <davem@davemloft.net>
Date: 2012-05-03 17:55:39

From: "Michael S. Tsirkin" <mst@redhat.com>
Date: Thu, 3 May 2012 18:41:43 +0300
On Thu, May 03, 2012 at 03:56:09PM +0100, Ian Campbell wrote:
quoted
This should be used by drivers which need to hold on to an skb for an extended
(perhaps unbounded) period of time. e.g. the tun driver which relies on
userspace consuming the skb.

Signed-off-by: Ian Campbell <redacted>
Cc: mst@redhat.com

Right. But local sockets queue at socket forever as well.
I think this should be called in skb_set_owner_r?

This might somewhat penalize speed for local clients in the name
of correctness but these are rare so being correct is
more important I think.
But, on the other hand, putting the check into skb_set_owner_r() is a
not so nice test to have in the fast path of every socket receive.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help