Messages in this thread | | | Subject | Re: [PATCH 1/3] net: use this_cpu_xxx replace percpu_xxx funcs | From | "Alex,Shi" <> | Date | Wed, 11 Jan 2012 17:10:18 +0800 |
| |
On Wed, 2012-01-11 at 01:03 -0800, David Miller wrote: > From: "Alex,Shi" <alex.shi@intel.com> > Date: Wed, 11 Jan 2012 16:45:33 +0800 > > >> >>>> percpu_xxx funcs are duplicated with this_cpu_xxx funcs, so replace them > >> >>>> for further code clean up. > >> >>>> > >> >>>> And in preempt safe scenario, __this_cpu_xxx funcs has a bit better > >> >>>> performance since __this_cpu_xxx has no redundant preempt_disable() > >> >>>> > >> >>>> Signed-off-by: Alex Shi <alex.shi@intel.com> > >> >>>> --- > >> >>>> net/netfilter/xt_TEE.c | 12 ++++++------ > >> >>>> net/socket.c | 4 ++-- > >> >>>> 2 files changed, 8 insertions(+), 8 deletions(-) > >> >>> > >> >>> Acked-by: Eric Dumazet <eric.dumazet@gmail.com> > >> >>> > >> >>> Thanks ! > >> >> > >> >> Anyone like to pick up this patch? or more comments for this? > >> > > >> > Kaber, David: > >> > I appreciate for your any comments on this. Could you like do me a > >> > favor? > >> > >> No objections from me. > > > > rend this patch for 3.2.0 kernel with Eric's Ack. > > > > David, do you have any concerns for this patch? I will very appreciate > > if it can met 3.3 open window. > > Please just submit it directly with the other this_cpu() patches: > > Acked-by: David S. Miller <davem@davemloft.net>
Thanks a lot! :)
| |