Re: PATCH] Kconfig.debug family

From: Sam Ravnborg
Date: Thu Apr 15 2004 - 16:32:59 EST


On Thu, Apr 15, 2004 at 01:52:29PM -0700, Randy.Dunlap wrote:
> On Wed, 14 Apr 2004 16:25:39 -0500 Matt Mackall wrote:
>
> | Sticking this in arch/*/Kconfig seems silly (as does much of the
> | duplication in said files). Can we stick this and other debug bits
> | under the kallsyms option in init/Kconfig instead? Or alternately move
> | debugging bits into their own file that gets included as appropriate.
>
>
> This patch:
> - creates lib/Kconfig.debug for generic kernel debug options
> - creates arch/*/Kconfig.debug for arch-specific debug options
> - moves KALLSYMS to the generic kernel debug options list
>
>
> This is a first cut for review/comments. I will double-check
> the generic options list to see how it needs to be corrected...

Looks good to me - and I really like concentrating this in
a more logic common place.

If you could move even more to the generic part it would be fine.

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/