Message ID | 1507554097.31614.11.camel@edumazet-glaptop3.roam.corp.google.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | [net-next] ipv6: avoid zeroing per cpu data again | expand |
On Mon, Oct 09, 2017 at 06:01:37AM -0700, Eric Dumazet wrote: > From: Eric Dumazet <edumazet@google.com> > > per cpu allocations are already zeroed, no need to clear them again. > > Fixes: d52d3997f843f ("ipv6: Create percpu rt6_info") > Signed-off-by: Eric Dumazet <edumazet@google.com> > Cc: Martin KaFai Lau <kafai@fb.com> > Cc: Tejun Heo <tj@kernel.org> Acked-by: Tejun Heo <tj@kernel.org> Thanks.
On Mon, Oct 09, 2017 at 01:01:37PM +0000, Eric Dumazet wrote: > From: Eric Dumazet <edumazet@google.com> > > per cpu allocations are already zeroed, no need to clear them again. > > Fixes: d52d3997f843f ("ipv6: Create percpu rt6_info") > Signed-off-by: Eric Dumazet <edumazet@google.com> > Cc: Martin KaFai Lau <kafai@fb.com> > Cc: Tejun Heo <tj@kernel.org> Acked-by: Martin KaFai Lau <kafai@fb.com>
From: Eric Dumazet <eric.dumazet@gmail.com> Date: Mon, 09 Oct 2017 06:01:37 -0700 > From: Eric Dumazet <edumazet@google.com> > > per cpu allocations are already zeroed, no need to clear them again. > > Fixes: d52d3997f843f ("ipv6: Create percpu rt6_info") > Signed-off-by: Eric Dumazet <edumazet@google.com> Applied.
diff --git a/net/ipv6/route.c b/net/ipv6/route.c index 606e80325b21c0e10a02e9c7d5b3fcfbfc26a003..3d7d4e09301ef4deae1985412599c6f4e973c46f 100644 --- a/net/ipv6/route.c +++ b/net/ipv6/route.c @@ -377,17 +377,7 @@ struct rt6_info *ip6_dst_alloc(struct net *net, if (rt) { rt->rt6i_pcpu = alloc_percpu_gfp(struct rt6_info *, GFP_ATOMIC); - if (rt->rt6i_pcpu) { - int cpu; - - for_each_possible_cpu(cpu) { - struct rt6_info **p; - - p = per_cpu_ptr(rt->rt6i_pcpu, cpu); - /* no one shares rt */ - *p = NULL; - } - } else { + if (!rt->rt6i_pcpu) { dst_release_immediate(&rt->dst); return NULL; }