[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1214121824.5255512.1472050804153.JavaMail.zimbra@redhat.com>
Date: Wed, 24 Aug 2016 11:00:04 -0400 (EDT)
From: Paolo Bonzini <pbonzini@...hat.com>
To: SF Markus Elfring <elfring@...rs.sourceforge.net>
Cc: Julia Lawall <julia.lawall@...6.fr>, walter harms <wharms@....de>,
kvm@...r.kernel.org, linux-s390@...r.kernel.org,
Christian Bornträger <borntraeger@...ibm.com>,
Cornelia Huck <cornelia.huck@...ibm.com>,
David Hildenbrand <dahi@...ux.vnet.ibm.com>,
Heiko Carstens <heiko.carstens@...ibm.com>,
Martin Schwidefsky <schwidefsky@...ibm.com>,
Radim Krčmář <rkrcmar@...hat.com>,
LKML <linux-kernel@...r.kernel.org>,
kernel-janitors@...r.kernel.org
Subject: Re: Replacing specific kmalloc() calls by kmalloc_array()?
----- Original Message -----
> From: "SF Markus Elfring" <elfring@...rs.sourceforge.net>
> To: "Paolo Bonzini" <pbonzini@...hat.com>
> Cc: "Julia Lawall" <julia.lawall@...6.fr>, "walter harms" <wharms@....de>, kvm@...r.kernel.org,
> linux-s390@...r.kernel.org, "Christian Bornträger" <borntraeger@...ibm.com>, "Cornelia Huck"
> <cornelia.huck@...ibm.com>, "David Hildenbrand" <dahi@...ux.vnet.ibm.com>, "Heiko Carstens"
> <heiko.carstens@...ibm.com>, "Martin Schwidefsky" <schwidefsky@...ibm.com>, "Radim Krčmář" <rkrcmar@...hat.com>,
> "LKML" <linux-kernel@...r.kernel.org>, kernel-janitors@...r.kernel.org
> Sent: Wednesday, August 24, 2016 2:10:13 PM
> Subject: Re: Replacing specific kmalloc() calls by kmalloc_array()?
>
> > Or kmalloc_array, since zeroing is not necessary. Might be an idea for
> > a new Coccinelle script, like
> >
> > - kmalloc (N * sizeof T, GFP)
> > + kmalloc_array(N, sizeof T, GFP)
>
> I have picked your idea up. The corresponding script for the semantic
> patch language became longer than your general suggestion
> (if additional source code control flow aspects are integrated).
>
> Would it make sense to check any more function combinations
> in a similar way?
I don't know :) but I'm interested in seeing the semantic patch!
Paolo
Powered by blists - more mailing lists