From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH v4] mac80211: move extra crypto data off the stack Date: Mon, 17 Oct 2016 11:35:18 +0200 Message-ID: <1476696918.19992.15.camel@sipsolutions.net> References: <1476693195-14071-1-git-send-email-johannes@sipsolutions.net> <1476696226.19992.13.camel@sipsolutions.net> (sfid-20161017_113100_410034_F49AAA79) Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: "" , Sergey Senozhatsky , "" , Herbert Xu To: Ard Biesheuvel Return-path: In-Reply-To: (sfid-20161017_113100_410034_F49AAA79) Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On Mon, 2016-10-17 at 10:30 +0100, Ard Biesheuvel wrote: > Yes. But as I replied, setting the req size is not supported atm, > although it is reasonable to demand a way to allocate additional data > in the request specifically for this issue. So let's proceed with the > aead_request_alloc/free patch, but I would like to propose something > on the API side to address this particular issue Well, if your other patch to make it OK to be on-stack would be applied instead, this wouldn't make much sense either :) In this particular patch, we could reduce the size of the struct, but I don't actually think it'll make a difference to go from 48 to 36 bytes, given alignment etc., so I think I'll just leave it as is. johannes