Re: [PATCH] move SWAP option in menu

[Gabriel Paubert]

On Thu, Mar 06, 2003 at 11:43:32AM -0700, Tom Rini wrote:
> 
> How's this look?  I picked MMU=x implies SWAP=x for defaults, just
> because that's how they were before...

I'd be very surprised if it were possible to have swap on a MMU-less 
machine (no virtual memory, page faults, etc.). Except for this nitpick, 
the patch looks fine, but my knowledge of MM is close to zero (and 
also of the new config language, but I'll have to learn it soon).

> ===== init/Kconfig 1.10 vs edited =====
> --- 1.10/init/Kconfig	Mon Feb  3 13:19:37 2003
> +++ edited/init/Kconfig	Thu Mar  6 11:41:51 2003
> @@ -37,6 +37,16 @@
>  
>  menu "General setup"
>  
> +config SWAP
> +	bool "Support for paging of anonymous memory"
> +	default y if MMU
> +	default n if !MMU

Should rather be (from just reading kconfig-language.txt):

config SWAP
	depends on MMU
	bool "Support for paging of anonymous memory"
	default y

Comments?

	Gabriel.