Re: [KJ] init_module / cleanup_module cleanup

[Domen Puncer <domen@coderock.org>]

[-- Attachment #1: Type: text/plain, Size: 1420 bytes --]

On 28/05/05 12:50 +0200, Reiner Herrmann wrote:
> Hi!
> 
> This is my first kernel patch, so I did a rather simple cleanup.
> Some network-drivers still used the old
> init_module() / cleanup_module() way of de-/registering
> at the kernel.

Hi Reiner.

Nice, you got tabs and linewraps right the first time.
Still, some issues and nitpicks:

Missing "Signed-off-by:";
Patch should probably be split per dir (diffsplit -d) and if some patch
touches >1 file of a driver, per driver.
Also, diffstat -p1 when patching multiple files is great.
(http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt is a nice
 read).

> 
> diff -uprN vanilla-2.6.12-rc5-kj/drivers/net/wireless/wavelan.c linux-2.6.12-rc5-kj/drivers/net/wireless/wavelan.c
> --- vanilla-2.6.12-rc5-kj/drivers/net/wireless/wavelan.c	2005-03-02 08:38:26.000000000 +0100
> +++ linux-2.6.12-rc5-kj/drivers/net/wireless/wavelan.c	2005-05-28 01:10:59.000000000 +0200

This one (and probably some others) won't compile on !MODULE, because
it includes wavelan.p.h with variables only defined on MODULE.

An ugly hack i used to quickly compile test it:
	egrep "^--- " ~/netdrivers_module_cleanup.diff | cut -f2 -d' '| cut -f1 | cut -f2- -d'/' | sed 's/.c$/.o/g' | xargs -n1 make 2>&1

Btw. anyone knows how to make ie.
"make CFLAGS=-DMODULE drivers/net/3c59x.o" work?

Also, when playing with __init and __exit, "make namespacecheck" can
catch some errors.


	Domen

[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors