From: Geert Uytterhoeven Subject: Re: [PATCH][trivial] crypto: tcrypt - reduce stack size Date: Wed, 25 Feb 2009 15:31:10 +0100 (CET) Message-ID: References: <49A54C23.8090209@suse.de> <20090225142000.GA4136@gondor.apana.org.au> <49A55499.6090700@suse.de> <20090225142744.GA4331@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=ISO-8859-15 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Frank Seidel , linux kernel , linux-crypto@vger.kernel.org, Frank Seidel , akpm@linux-foundation.org, "David S. Miller" , nhorman@tuxdriver.com, lho@amcc.com, kaber@trash.net, darrenrjenkins@gmail.com To: Herbert Xu Return-path: Received: from vervifontaine.sonytel.be ([80.88.33.193]:64986 "EHLO vervifontaine.sonycom.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754694AbZBYObP (ORCPT ); Wed, 25 Feb 2009 09:31:15 -0500 In-Reply-To: <20090225142744.GA4331@gondor.apana.org.au> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Wed, 25 Feb 2009, Herbert Xu wrote: > On Wed, Feb 25, 2009 at 03:24:25PM +0100, Frank Seidel wrote: > > Herbert Xu wrote: > > >> - char output[1024]; > > >=20 > > > How about just making it static? > >=20 > > Well, it was static before. But there > > are efforts to reduce current stack memory > > footprints. >=20 > Well if it's static it won't be using any stack memory. Probably people should be reminded tcrypt can only be a module, not bui= lt-in, so it won't waste precious unswappable kernel memory if it's static. With kind regards, Geert Uytterhoeven Software Architect Sony Techsoft Centre Europe The Corporate Village =B7 Da Vincilaan 7-D1 =B7 B-1935 Zaventem =B7 Bel= gium Phone: +32 (0)2 700 8453 =46ax: +32 (0)2 700 8622 E-mail: Geert.Uytterhoeven@sonycom.com Internet: http://www.sony-europe.com/ A division of Sony Europe (Belgium) N.V. VAT BE 0413.825.160 =B7 RPR Brussels =46ortis =B7 BIC GEBABEBB =B7 IBAN BE41293037680010 -- To unsubscribe from this list: send the line "unsubscribe linux-crypto"= in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html