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

From: Florian Westphal <[email-masked]>
Date: Wed, 27 Sep 2017 14:31:03 +0200

Artem Savkov <[email-masked]> 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.
> 
> Fixes: ec30d78c14a8 xfrm: add xdst pcpu cache
> Signed-off-by: Artem Savkov <[email-masked]>

You're right, its not needed (and wrong).

Acked-by: Florian Westphal <[email-masked]>