From mboxrd@z Thu Jan  1 00:00:00 1970
From: Shirley Ma <mashirle@us.ibm.com>
Subject: Re: [RFC PATCH V2 5/5] Add TX zero copy in macvtap
Date: Fri, 10 Dec 2010 08:25:55 -0800
Message-ID: <1291998355.2167.53.camel@localhost.localdomain>
References: <1291976026.2167.49.camel@localhost.localdomain>
	 <1291976864.3580.7.camel@edumazet-laptop>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Cc: Avi Kivity <avi@redhat.com>, Arnd Bergmann <arnd@arndb.de>,
	mst@redhat.com, xiaohui.xin@intel.com, netdev@vger.kernel.org,
	kvm@vger.kernel.org, linux-kernel@vger.kernel.org
To: Eric Dumazet <eric.dumazet@gmail.com>
Return-path: <linux-kernel-owner@vger.kernel.org>
In-Reply-To: <1291976864.3580.7.camel@edumazet-laptop>
Sender: linux-kernel-owner@vger.kernel.org
List-Id: netdev.vger.kernel.org

On Fri, 2010-12-10 at 11:27 +0100, Eric Dumazet wrote:
> You could make one atomic_add() outside of the loop, and factorize
> many
> things...
> 
> atomic_add(len, &skb->sk->sk_wmem_alloc);
> skb->data_len += len;
> skb->len += len;
> skb->truesize += len;
> while (len) {
>         ...
> } 

Yep, thanks, will update it!

Shirley