Re: 2.6.17-rc5-mm1

[Martin J. Bligh]

> This is nonsense.
> If you do this do it at least correctly, e.g. something like:
>
> config DEBUG_RUNTIME_CHECKS
> 	bool "Enable runtime debug checks"
>
> config DEBUG_RUNTIME_CHECKS_ALL
> 	bool "Enable all runtime debug checks"
> 	depends on DEBUG_RUNTIME_CHECKS
>
> config DEBUG_KERNEL
> 	bool "Kernel debugging"
>
> ...
>
> config DEBUG_FOO
> 	bool "foo" if DEBUG_KERNEL
> 	default DEBUG_RUNTIME_CHECKS

That doesn't seem to cover what we talked about clearly at all ?
I suppose the _ALL stuff is meant to cover stuff with overhead,
but frankly, what Ingo did seemed much clearer to me.

M.
-
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/