diff options
| author | Dag-Erling Smrgrav <des@linpro.no> | 2006-01-02 15:57:06 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-02 08:38:38 -0800 |
| commit | abe842eb98c45e2b77c5868ef106616ca828a3e4 (patch) | |
| tree | f0d89e7d946a7ed9b57bb29e93bae4ce25d2cbc2 /include/asm-x86_64 | |
| parent | f12f4d90308a22396ac87f6c3a7b2620589614c3 (diff) | |
| download | lwn-abe842eb98c45e2b77c5868ef106616ca828a3e4.tar.gz lwn-abe842eb98c45e2b77c5868ef106616ca828a3e4.zip | |
[PATCH] Avoid namespace pollution in <asm/param.h>
In commit 3D59121003721a8fad11ee72e646fd9d3076b5679c, the x86 and x86-64
<asm/param.h> was changed to include <linux/config.h> for the
configurable timer frequency.
However, asm/param.h is sometimes used in userland (it is included
indirectly from <sys/param.h>), so your commit pollutes the userland
namespace with tons of CONFIG_FOO macros. This greatly confuses
software packages (such as BusyBox) which use CONFIG_FOO macros
themselves to control the inclusion of optional features.
After a short exchange, Christoph approved this patch
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/asm-x86_64')
| -rw-r--r-- | include/asm-x86_64/param.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/include/asm-x86_64/param.h b/include/asm-x86_64/param.h index 40b11937180d..5956b23b57c2 100644 --- a/include/asm-x86_64/param.h +++ b/include/asm-x86_64/param.h @@ -1,9 +1,8 @@ -#include <linux/config.h> - #ifndef _ASMx86_64_PARAM_H #define _ASMx86_64_PARAM_H #ifdef __KERNEL__ +# include <linux/config.h> # define HZ CONFIG_HZ /* Internal kernel timer frequency */ # define USER_HZ 100 /* .. some user interfaces are in "ticks */ #define CLOCKS_PER_SEC (USER_HZ) /* like times() */ |