Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread Michael S. Tsirkin
On Mon, Jul 25, 2011 at 03:02:29AM -0700, David Miller wrote: > From: Herbert Xu > Date: Mon, 25 Jul 2011 17:57:11 +0800 > > > However, I think we should add a WARN_ON to the splice skb path > > so that should a packet find its way through a path that we haven't > > thought of then at least we'll

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread David Miller
From: Herbert Xu Date: Mon, 25 Jul 2011 17:57:11 +0800 > However, I think we should add a WARN_ON to the splice skb path > so that should a packet find its way through a path that we haven't > thought of then at least we'll know about it. Good idea. -- To unsubscribe from this list: send the lin

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread Herbert Xu
On Mon, Jul 25, 2011 at 12:44:14PM +0300, Michael S. Tsirkin wrote: > > if yes that seems to always clone an skb, which in turn > does the copy so we are fine? Yes you're right, it should be safe. However, I think we should add a WARN_ON to the splice skb path so that should a packet find its way

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread Michael S. Tsirkin
On Mon, Jul 25, 2011 at 04:40:57PM +0800, Herbert Xu wrote: > On Mon, Jul 25, 2011 at 11:07:43AM +0300, Michael S. Tsirkin wrote: > > > > However macvtap passes an skb directly to the > > lower device, so as long as macvtap is the only user > > of that interface, we are fine I think - there's > > n

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread Herbert Xu
On Mon, Jul 25, 2011 at 11:07:43AM +0300, Michael S. Tsirkin wrote: > > However macvtap passes an skb directly to the > lower device, so as long as macvtap is the only user > of that interface, we are fine I think - there's > no way for an skb to get from macvtap to splice > read path I think. > >

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-25 Thread Michael S. Tsirkin
On Mon, Jul 25, 2011 at 08:42:00AM +0800, Herbert Xu wrote: > Shirley Ma wrote: > > > > This patch clears tx zero-copy flag as needed. > > > > Sign-off-by: Shirley Ma > > I think we also need to copy and clear this flag on the splice > read path as that takes a direct page reference. > > I ho

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-24 Thread Herbert Xu
Shirley Ma wrote: > > This patch clears tx zero-copy flag as needed. > > Sign-off-by: Shirley Ma I think we also need to copy and clear this flag on the splice read path as that takes a direct page reference. I hope there isn't any other path that does this. Cheers, -- Email: Herbert Xu Ho

Re: [PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-09 Thread David Miller
From: Shirley Ma Date: Sat, 09 Jul 2011 00:12:46 -0700 > This patch clears tx zero-copy flag as needed. > > Signed-off-by: Shirley Ma Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at

[PATCH net-next] skbuff: clear tx zero-copy flag

2011-07-09 Thread Shirley Ma
Hello Dave, This patch clears tx zero-copy flag as needed. Sign-off-by: Shirley Ma --- net/core/skbuff.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index a9577a2..d220119 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.