Re: another attempt to kill off linux/config.h

[Sam Ravnborg]

On Wed, Oct 04, 2006 at 02:01:22PM -0400, Dave Jones wrote:
> On Wed, Oct 04, 2006 at 01:45:32PM +0200, Arjan van de Ven wrote:
> 
>  > > Removing it for real will be a pain for external modules.
>  > > They could of course detect that it is missing and then
>  > > drop it.
>  > > I would suggest to keep the #warning in 2.6.19 and only
>  > > remove it for real for 2.6.20.
>  > 
>  > they'll have to change anyway; delaying it one release doesn't actually
>  > change that. And you can bet on most modules ignoring the warning anyway
>  > and wait until the thing really is gone... making the value that this
>  > extra delay has basically zero. While the cost is that more false users
>  > will sneak into the kernel ;(
> 
> My thoughts exactly.  Since when did we give a damn about keeping
> external modules compiling anyway?

In kbuild quite a lot of effort has been done to make life easier
for those dealing with external modules. There are several
reasons to deal with external modules:
- Alpha stage module that is not yet ready for kernel inclusion
- Special stuff that does not belong in the kernel for som reason
- Experimental stuff of any kind

This covers maybe one fifth of all external modules but that is enough
to make proper support anyway.

That said I continue to be mystified over how much effort the external
module people put into avoiding using kbuild.
I have even seen external modules that mixed up proper kbuild
supported modules and old 2.4 style modules in the same Makefile.

And the effort trying to deal with vermagic stuff in external modules,
inspecting CFLAGS setting etc. is just countless :-(

But that does not change my point about giving these people a proper
warning in at least one kernel release.

	Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/