From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from multi.imgtec.com ([194.200.65.239]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1UwpyR-0005by-G5 for linux-mtd@lists.infradead.org; Wed, 10 Jul 2013 08:42:08 +0000 Message-ID: <51DD1E0F.80101@imgtec.com> Date: Wed, 10 Jul 2013 09:40:47 +0100 From: James Hogan MIME-Version: 1.0 To: Rusty Russell Subject: Re: [RESEND][PATCH] mtd: refactor call to request_module References: <20130708130512.GA24315@www.outflux.net> <87hag3cdp0.fsf@rustcorp.com.au> In-Reply-To: <87hag3cdp0.fsf@rustcorp.com.au> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: linux-mtd@lists.infradead.org, akpm@linux-foundation.org, David Woodhouse , Kees Cook , linux-kernel@vger.kernel.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 10/07/13 04:10, Rusty Russell wrote: > Kees Cook writes: >> This reduces the size of the stack frame when calling request_module(). >> Performing the sprintf before the call is not needed. >> >> Signed-off-by: Kees Cook > > Acked-by: Rusty Russell I remember reviewing this before, so feel free to add my: Reviewed-by: James Hogan Cheers James