From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next] tcp: fix a lockdep issue in tcp_fastopen_reset_cipher() Date: Fri, 03 Nov 2017 15:54:44 +0900 (KST) Message-ID: <20171103.155444.2169913803825248503.davem@davemloft.net> References: <1509648784.2849.14.camel@edumazet-glaptop3.roam.corp.google.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, ycheng@google.com, cpaasch@apple.com To: eric.dumazet@gmail.com Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:36748 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751944AbdKCGyu (ORCPT ); Fri, 3 Nov 2017 02:54:50 -0400 In-Reply-To: <1509648784.2849.14.camel@edumazet-glaptop3.roam.corp.google.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Eric Dumazet Date: Thu, 02 Nov 2017 11:53:04 -0700 > From: Eric Dumazet > > icsk_accept_queue.fastopenq.lock is only fully initialized at listen() > time. > > LOCKDEP is not happy if we attempt a spin_lock_bh() on it, because > of missing annotation. (Although kernel runs just fine) > > Lets use net->ipv4.tcp_fastopen_ctx_lock to protect ctx access. > > Fixes: 1fba70e5b6be ("tcp: socket option to set TCP fast open key") > Signed-off-by: Eric Dumazet Applied.