[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4B84D20B.9080600@osadl.org>
Date: Wed, 24 Feb 2010 08:15:23 +0100
From: Carsten Emde <Carsten.Emde@...dl.org>
To: Dan Carpenter <error27@...il.com>,
Thomas Gleixner <tglx@...utronix.de>,
LKML <linux-kernel@...r.kernel.org>,
rt-users <linux-rt-users@...r.kernel.org>,
Ingo Molnar <mingo@...e.hu>,
Steven Rostedt <rostedt@...dmis.org>,
Peter Zijlstra <peterz@...radead.org>,
Clark Williams <williams@...hat.com>,
Frank Rowand <frank.rowand@...sony.com>,
Robin Gareus <robin@...eus.org>,
Gregory Haskins <ghaskins@...ell.com>,
Philippe Reynes <philippe.reynes@...smpp.fr>,
Fernando Lopez-Lezcano <nando@...ma.Stanford.EDU>,
Will Schmidt <will_schmidt@...t.ibm.com>,
Darren Hart <dvhltc@...ibm.com>, Jan Blunck <jblunck@...e.de>,
Sven-Thorsten Dietrich <sdietrich@...ell.com>,
Jon Masters <jcm@...hat.com>,
Mark Knecht <markknecht@...il.com>,
John Kacur <jkacur@...hat.com>,
Nick Piggin <nickpiggin@...oo.com.au>
Subject: Re: [ANNOUNCE] 2.6.33-rc8-rt1
On 02/22/2010 02:29 PM, Dan Carpenter wrote:
> kernel/trace/latency_hist.c
> 373 static ssize_t
> 374 latency_hist_show_maxlatproc(struct file *filp, char __user *ubuf,
> 375 size_t cnt, loff_t *ppos)
> 376 {
> 377 char buf[1024];
>
> This is a large buffer to put on the stack.
Thanks!
Remove stack allocation of buffer space, use dyn memory instead.
Use a better assumption to estimate the required buffer space.
Signed-off-by: Carsten Emde <C.Emde@...dl.org>
View attachment "histogram-maxlatproc-use-dyn-mem.patch" of type "text/x-patch" (1224 bytes)
Powered by blists - more mailing lists