From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Alex,Shi" Subject: Re: [PATCH 1/3] net: use this_cpu_xxx replace percpu_xxx funcs Date: Fri, 02 Dec 2011 10:33:20 +0800 Message-ID: <1322793200.11530.33.camel@debian> References: <1318237851.27949.190.camel@debian> <1318324774.27949.693.camel@debian> <1318382964.27949.782.camel@debian> <1318428673.29699.13.camel@debian> <1318908091.23426.52.camel@debian> <1319016218.23426.104.camel@debian> <1319078687.23426.146.camel@debian> <1319087281.8416.45.camel@edumazet-laptop> <1319095929.23426.180.camel@debian> <1319099893.3781.0.camel@edumazet-HP-Compaq-6005-Pro-SFF-PC> <1321866051.30341.565.camel@debian> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Christoph Lameter , "tj@kernel.org" , "linux-kernel@vger.kernel.org" , "Huang, Ying" , Thomas Gleixner , "mingo@redhat.com" , "avi@redhat.com" , "akpm@linux-foundation.org" , David Miller , "kaber@trash.net" , "a.p.zijlstra@chello.nl" , "kvm@vger.kernel.org" , "jeremy@xensource.com" To: Eric Dumazet Return-path: In-Reply-To: <1321866051.30341.565.camel@debian> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Mon, 2011-11-21 at 17:00 +0800, Alex,Shi wrote: > On Thu, 2011-10-20 at 16:38 +0800, Eric Dumazet wrote: > > Le jeudi 20 octobre 2011 =C3=A0 15:32 +0800, Alex,Shi a =C3=A9crit = : > > > percpu_xxx funcs are duplicated with this_cpu_xxx funcs, so repla= ce them > > > for further code clean up. > > >=20 > > > And in preempt safe scenario, __this_cpu_xxx funcs has a bit bett= er > > > performance since __this_cpu_xxx has no redundant preempt_disable= () > > >=20 > > > Signed-off-by: Alex Shi > > > --- > > > net/netfilter/xt_TEE.c | 12 ++++++------ > > > net/socket.c | 4 ++-- > > > 2 files changed, 8 insertions(+), 8 deletions(-) > >=20 > > Acked-by: Eric Dumazet > >=20 > > Thanks ! >=20 > Anyone like to pick up this patch? or more comments for this?=20 Kaber, David:=20 I appreciate for your any comments on this. Could you like do me a favor?=20