[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#210359: kernel CONFIG_TR interferes with busybox CONFIG_TR



>>>>> On Thu, 18 Sep 2003 08:09:33 +0900, GOTO Masanori <gotom@debian.or.jp> said:

  Goto> Yes, it's not libc6 own problem, but kernel source problem.
  Goto> In some kernels, this problem should also be occured on alpha.
  Goto> One way to fix is to enclose #ifdef __KERNEL__.

  Goto> David, could you look this namespace pollution issue? at:

  Goto> 	http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=210359

  Goto> This experimental patch is for 2003-09-17 bk latest.  Only
  Goto> ia64 part.  Is this ok?  Is this CONFIG_IA64_HP_SIM part still
  Goto> needed for debugging purpose?

  Goto> --- include/asm-ia64/param.h 2002-10-07 13:20:02.000000000
  Goto> +0900 +++ include/asm-ia64/param.h.new 2003-09-17
  Goto> 19:13:34.000000000 +0900 @@ -8,6 +8,8 @@ * David
  Goto> Mosberger-Tang <davidm@hpl.hp.com> */

  Goto> +#ifdef __KERNEL__ + #include <linux/config.h>

  Goto>  #ifdef CONFIG_IA64_HP_SIM @@ -20,6 +22,8 @@ # define HZ 1024
  Goto> #endif

  Goto> +#endif + #define EXEC_PAGESIZE 65536

  Goto>  #ifndef NGROUPS

  Goto> Regards, -- gotom

Looks like a nasty problem.  If we can't include <linux/config.h>,
there is no guarantee that the header contents will be correct.  We
can obviously move the definition of HZ into the (existing)
__KERNEL__-only section, but then apps which rely on the HZ macro
won't compile anymore.  Perhaps that's OK (they probably should use
sysconf() anyhow), though.

	--david



Reply to: