Re: [PATCH] xfrm: don't call xfrm_policy_cache_flush under xfrm_state_lock

2017-09-28 Thread Steffen Klassert
On Wed, Sep 27, 2017 at 02:31:03PM +0200, Florian Westphal wrote: > Artem Savkov wrote: > > I might be wrong but it doesn't look like xfrm_state_lock is required > > for xfrm_policy_cache_flush and calling it under this lock triggers both > > "sleeping function called from invalid context" and "po

Re: [PATCH] xfrm: don't call xfrm_policy_cache_flush under xfrm_state_lock

2017-09-27 Thread Florian Westphal
Artem Savkov wrote: > I might be wrong but it doesn't look like xfrm_state_lock is required > for xfrm_policy_cache_flush and calling it under this lock triggers both > "sleeping function called from invalid context" and "possible circular > locking dependency detected" warnings on flush. > > Fix

[PATCH] xfrm: don't call xfrm_policy_cache_flush under xfrm_state_lock

2017-09-27 Thread Artem Savkov
I might be wrong but it doesn't look like xfrm_state_lock is required for xfrm_policy_cache_flush and calling it under this lock triggers both "sleeping function called from invalid context" and "possible circular locking dependency detected" warnings on flush. Fixes: ec30d78c14a8 xfrm: add xdst p