Re: [PATCH] mpls: Add missing RCU-bh read side critical section locking in output path

2016-06-28 Thread Lennert Buytenhek
On Thu, Jun 23, 2016 at 12:00:55PM -0400, David Miller wrote: > > From: David Barroso > > > > When locally originated IP traffic hits a route that says to push > > MPLS labels, we'll get a call chain dst_output() -> lwtunnel_output() > > -> mpls_output() -> neigh_xmit() -> ___neigh_lookup_noref(

Re: [PATCH] mpls: Add missing RCU-bh read side critical section locking in output path

2016-06-23 Thread David Miller
From: Lennert Buytenhek Date: Mon, 20 Jun 2016 21:05:27 +0300 > From: David Barroso > > When locally originated IP traffic hits a route that says to push > MPLS labels, we'll get a call chain dst_output() -> lwtunnel_output() > -> mpls_output() -> neigh_xmit() -> ___neigh_lookup_noref() where t

Re: [PATCH] mpls: Add missing RCU-bh read side critical section locking in output path

2016-06-21 Thread Robert Shearman
On 20/06/16 19:05, Lennert Buytenhek wrote: From: David Barroso When locally originated IP traffic hits a route that says to push MPLS labels, we'll get a call chain dst_output() -> lwtunnel_output() -> mpls_output() -> neigh_xmit() -> ___neigh_lookup_noref() where the last function in this cha

[PATCH] mpls: Add missing RCU-bh read side critical section locking in output path

2016-06-20 Thread Lennert Buytenhek
From: David Barroso When locally originated IP traffic hits a route that says to push MPLS labels, we'll get a call chain dst_output() -> lwtunnel_output() -> mpls_output() -> neigh_xmit() -> ___neigh_lookup_noref() where the last function in this chain accesses a RCU-bh protected struct neigh_ta

Re: [PATCH] mpls: Add missing RCU-bh read side critical section locking in output path

2016-06-20 Thread David Ahern
On 6/20/16 12:05 PM, Lennert Buytenhek wrote: From: David Barroso When locally originated IP traffic hits a route that says to push MPLS labels, we'll get a call chain dst_output() -> lwtunnel_output() -> mpls_output() -> neigh_xmit() -> ___neigh_lookup_noref() where the last function in this c