Re: [PATCH net] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-10-25 Thread Jonathan Basseri
On Tue, Oct 24, 2017 at 9:25 PM, Steffen Klassert wrote: > On Tue, Oct 24, 2017 at 09:58:48AM -0700, Jonathan Basseri 😶 wrote: >> On Tue, Oct 24, 2017 at 12:04 AM, Steffen Klassert >> wrote: >> > >> > On Mon, Oct 23, 2017 at 06:18:55PM -0700, Jonathan Basser

[PATCH net v2] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-10-25 Thread Jonathan Basseri
://android-review.googlesource.com/418659 Signed-off-by: Jonathan Basseri --- net/xfrm/xfrm_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c index 12213477cd3a..1f5cee2269af 100644 --- a/net/xfrm/xfrm_state.c +++ b/net/xfrm/xfrm_state.c @@ -2045,3

Re: [PATCH net] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-10-24 Thread Jonathan Basseri 😶
On Tue, Oct 24, 2017 at 12:04 AM, Steffen Klassert wrote: > > On Mon, Oct 23, 2017 at 06:18:55PM -0700, Jonathan Basseri wrote: > > If a socket has a valid dst cache, then xfrm_lookup_route will get > > skipped. However, the cache is not invalidated when applying policy to

Re: [PATCH net] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-10-23 Thread Jonathan Basseri 😶
On Mon, Oct 23, 2017 at 6:18 PM, Jonathan Basseri wrote: > If a socket has a valid dst cache, then xfrm_lookup_route will get > skipped. However, the cache is not invalidated when applying policy to a > socket (i.e. IPV6_XFRM_POLICY). The result is that new policies are > sometime

[PATCH net] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-10-23 Thread Jonathan Basseri
XFRM lookup if dst_entry in socket cache is valid") Tested: https://android-review.googlesource.com/517555 Tested: https://android-review.googlesource.com/418659 Signed-off-by: Jonathan Basseri diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c index 12213477cd3a..1f5cee2269af 100644

[PATCH net] xfrm: Clear sk_dst_cache when applying per-socket policy.

2017-08-15 Thread Jonathan Basseri
XFRM policies should have a similar effect on sk_dst_cache entries that match the policy, but that is not fixed in this patch. Fixes: 00bc0ef5880d ("ipv6: Skip XFRM lookup if dst_entry in socket cache is valid") Tested: https://android-review.googlesource.com/418659 Signed-off-by: Jonath