From: Herbert Xu Subject: Re: [PATCH V2] Fix a sleep-in-atomic bug in shash_setkey_unaligned Date: Thu, 5 Oct 2017 11:40:54 +0800 Message-ID: <20171005034054.GB31996@gondor.apana.org.au> References: <1506997522-26684-1-git-send-email-baijiaju1990@163.com> <20171003052643.GB22750@gondor.apana.org.au> <20171003224505.GE19750@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andy Lutomirski , Jia-Ju Bai , "David S. Miller" , Neil Horman , vyasevich@gmail.com, Kalle Valo , Linux Crypto Mailing List , Network Development , linux-sctp@vger.kernel.org, Linux Wireless List To: Marcelo Ricardo Leitner Return-path: Content-Disposition: inline In-Reply-To: <20171003224505.GE19750@localhost.localdomain> Sender: netdev-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org On Tue, Oct 03, 2017 at 07:45:06PM -0300, Marcelo Ricardo Leitner wrote: > > > Usually if you're invoking setkey from a non-sleeping code-path > > you're probably doing something wrong. > > Usually but not always. There are 3 calls to that function on SCTP > code: > - pack a cookie, which is sent on an INIT_ACK packet to the client > - unpack the cookie above, after it is sent back by the client on a > COOKIE_ECHO packet > - send a chunk authenticated by a hash I'm not talking about the code-path in question. I'm talking about the function which generates the secret key in the first place. AFAICS that's only called in GFP_KERNEL context. What am I missing? Cheers, -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt