Re: [PATCH] init/main.c

[Andrzej Krzysztofowicz]

Hi,
   What do you aim at doing this changes ?

> The following patch against 2.4.6-ac5 does
[...]
> (b) wraps certain root_dev_names[] into #ifdefs.
>     I hope I found the correct CONFIG_BLK_* variables
[...]
>  #endif
> +#ifdef CONFIG_BLK_DEV_DAC960
>  	{ "rd/c0d0p",0x3000 },
>  	{ "rd/c0d1p",0x3008 },
>  	{ "rd/c0d2p",0x3010 },
> @@ -296,6 +298,8 @@
>  	{ "rd/c0d13p",0x3068 },
>  	{ "rd/c0d14p",0x3070 },
>  	{ "rd/c0d15p",0x3078 },
> +#endif
> +#ifdef CONFIG_BLK_CPQ_DA
... etc.

AFAIR, the above (and similar) #ifdefs were intentionally removed as they
break using root=... kernel parameters for apropriate drivers loaded as
module from initrd. In these cases there are 3 solutions:

1. Use numeric values (not convenient)
2. #if defined(CONFIG_FOO) || defined(CONFIG_FOO_MODULE)
   This is ugly (main kernel depends on too many module settings).
   Also, Linux (Alan too ?) hate too many unnecessary
3. Remove all #ifdefs here. (prefered as this is init code...)

Alan, please DON'T apply this patch.

Andrzej