lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 14 Mar 2007 01:05:17 +0300 From: Alexey Dobriyan <adobriyan@...il.com> To: "Robert P. J. Day" <rpjday@...dspring.com> Cc: linux-kernel@...r.kernel.org, kernel-janitors@...ts.osdl.org Subject: Re: module.h and moduleparam.h: more header file pedantry On Mon, Mar 12, 2007 at 12:59:20PM -0400, Robert P. J. Day wrote: > to my surprise, i learned only today that module.h includes > moduleparam.h, which flies in the face of all of the documentation > i've ever read which was adamant that i *had* to include moduleparam.h > if i was using parameters. i'm guessing this comes as a surprise to > the 400+ header files which include both unnecessarily. > > so ... in a perfect world, should a module source file that doesn't > use parameters *at all* need to include moduleparam.h? Probably not. > as it stands > now, yes, it does, given some ugly inter-dependencies between the two > files. > > so, at the very least, programmers can stop including moduleparam.h, > unless there's a cleaner way to do all that. Regardless, of what you'll do: cross-compile test! After aforementioned removal and adding "struct kernel_param;" + akmk arm-assabet -k CHK include/linux/version.h make[2]: `include/asm-arm/mach-types.h' is up to date. Using /home/linux/linux-irq-flags-t as source for kernel GEN /home/linux/build/arm-assabet/Makefile CHK include/linux/utsrelease.h CHK include/linux/compile.h CC arch/arm/nwfpe/fpmodule.o arch/arm/nwfpe/fpmodule.c:179: error: syntax error before string constant arch/arm/nwfpe/fpmodule.c:179: warning: type defaults to `int' in declaration of `__MODULE_INFO' arch/arm/nwfpe/fpmodule.c:179: warning: function declaration isn't a prototype arch/arm/nwfpe/fpmodule.c:179: warning: data definition has no type or storage class - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists