From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesper Juhl Subject: Re: [PATCH] cgroup: Avoid a memset by using vzalloc Date: Thu, 4 Nov 2010 22:43:56 +0100 (CET) Message-ID: References: <20101031173336.GA28141@balbir.in.ibm.com> <1288799284.15729.27.camel@Joe-Laptop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Christoph Lameter Cc: jovi zhang , containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, Joe Perches , Paul Menage , Balbir Singh List-Id: containers.vger.kernel.org On Wed, 3 Nov 2010, Christoph Lameter wrote: > On Wed, 3 Nov 2010, Joe Perches wrote: > > > On Wed, 2010-11-03 at 23:20 +0800, jovi zhang wrote: > > > On Wed, Nov 3, 2010 at 10:38 PM, Christoph Lameter wrote: > > > > On Mon, 1 Nov 2010, Jesper Juhl wrote: > > > > > > > >> On Sun, 31 Oct 2010, Balbir Singh wrote: > > > > > > > >> > > There are so many placed need vzalloc. > > > >> > > Thanks, Jesper. > > > > > > > > > > > > Could we avoid this painful exercise with a "semantic patch"? > > > > There's an existing cocci kmalloc/memset script. > > I have it in > /usr/share/doc/coccinelle/examples/janitorings/kzalloc-orig.cocci.gz > > (Ubuntu coccinelle package) > > > Perhaps this is good enough? > > > > cp scripts/coccinelle/api/alloc/kzalloc-simple.cocci scripts/coccinelle/api/alloc/vzalloc-simple.cocci > > sed -i -e 's/kmalloc/vmalloc/g' -e 's/kzalloc/vzalloc/g' scripts/coccinelle/api/alloc/vzalloc-simple.cocci > > Not sure if that is the same script but certainly a good start. Try it and > see if it catches all the locations that you know of? > I'm aware of coccinelle, but I've never used it and it looks like it'll take more than just a few hours to learn, so I'm sticking with bash+egrep+manual inspection for now until I get a bit more time on my hands to learn coccinelle/spatch. I assume that not using spatch is not going to be an obstacle to patches such as this one getting merged...? -- Jesper Juhl http://www.chaosbits.net/ Plain text mails only, please http://www.expita.com/nomime.html Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html