Re: [PATCH] checkpatch: Test for kmalloc/memset(0) pairs
From: Nicolas Palix
Date: Sun Mar 20 2011 - 07:32:55 EST
On Sun, Mar 20, 2011 at 12:06 PM, Pekka Enberg <penberg@xxxxxxxxxx> wrote:
> On Sun, Mar 20, 2011 at 12:54 PM, Ingo Molnar <mingo@xxxxxxx> wrote:
>>> Indeed that one seems to be quite out of date. ÂYou can get the most
>>> recent version here: https://launchpad.net/~npalix/+archive/coccinelle
>> With tools/coccinelle/ you would never run into such problems of distributing
>> the latest stable version to your fellow kernel developers: it would always be
>> available in tools/coccinelle/.
>> Integration, synergy, availability, distribution and half a dozen other
>> buzzwords come to mind as to why it's a good idea to have kernel-focused
>> tools hosted in the kernel repo :-)
Our usage is mainly kernel-focused but not the tool. It is C-program focused
and we have used it on other programs like Wine, OpenSSL, VLC. Others
use it on other projects like Davecot or close-source projets.
So, IMHO Coccinelle should no be part of Linux.
Integrating kernel-focused SmPL scripts is on the other hand a great idea
to check the kernel and to ease kernel developer life.
It is what have been done so far. It is certainly possible to improve that,
at least by adding more and more scripts.
>> IMO it's an option to consider.
> That's my thinking too. Yes, 80 KLOC of OCaml in the kernel tree
> sounds crazy but I think the practical advantages might be enough to
> justify it. Btw, would git-submodule be something to consider here?
At every RC, we push the Coccinelle code on github. Using git-submodule
seems the way to go thus. Moreover, it will ease the maintenance of
scripts as we may assume users have one of the latest versions.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/