From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Kalle Valo To: Johannes Berg Cc: Kees Cook , Herbert Xu , Arnd Bergmann , Eric Biggers , "Gustavo A. R. Silva" , Alasdair Kergon , Rabin Vincent , Tim Chen , "Rafael J. Wysocki" , Pavel Machek , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Philipp Reisner , Lars Ellenberg , Jens Axboe , Giovanni Cabiddu , Mike Snitzer , Paul Mackerras , Greg Kroah-Hartman , David Howells , Tudor-Dan Ambarus , Jia-Ju Bai , Andrew Morton , Geert Uytterhoeven , Josh Poimboeuf , David Woodhouse , Will Deacon , dm-devel@redhat.com, linux-pm@vger.kernel.org, linux-crypto@vger.kernel.org, drbd-dev@lists.linbit.com, linux-block@vger.kernel.org, qat-linux@intel.com, linux-ppp@vger.kernel.org, netdev@vger.kernel.org, devel@driverdev.osuosl.org, linux-afs@lists.infradead.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 13/18] wireless/lib80211: Convert from ahash to shash References: <20180724164936.37477-1-keescook@chromium.org> <20180724164936.37477-14-keescook@chromium.org> <1532503493.3223.0.camel@sipsolutions.net> Date: Wed, 25 Jul 2018 11:05:42 +0300 In-Reply-To: <1532503493.3223.0.camel@sipsolutions.net> (Johannes Berg's message of "Wed, 25 Jul 2018 09:24:53 +0200") Message-ID: <87tvong2p5.fsf@purkki.adurom.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii List-ID: Johannes Berg writes: > On Tue, 2018-07-24 at 09:49 -0700, Kees Cook wrote: >> In preparing to remove all stack VLA usage from the kernel[1], this >> removes the discouraged use of AHASH_REQUEST_ON_STACK in favor of >> the smaller SHASH_DESC_ON_STACK by converting from ahash-wrapped-shash >> to direct shash. By removing a layer of indirection this both improves >> performance and reduces stack usage. The stack allocation will be made >> a fixed size in a later patch to the crypto subsystem. > > I think you sent this before - it should be in net-next now. Yup, I see it in net-next: d17504b16ea2 wireless/lib80211: Convert from ahash to shash -- Kalle Valo