Re: [PATCH v2] mm/Kconfig: add MMU dependency for MIGRATION.

[Chen Gang <gang.chen@asianux.com>]

On 08/05/2013 05:20 PM, Chen Gang wrote:
> 
> 
> 

Sorry for my careless sending: reserving so many waste empty lines.

> MIGRATION need depend on MMU, or allmodconfig for sh architecture which
> without MMU will be fail for compiling.
> 
> The related error: 
> 
>     CC      mm/migrate.o
>   mm/migrate.c: In function 'remove_migration_pte':
>   mm/migrate.c:134:3: error: implicit declaration of function 'pmd_trans_huge' [-Werror=implicit-function-declaration]
>      if (pmd_trans_huge(*pmd))
>      ^
>   mm/migrate.c:149:2: error: implicit declaration of function 'is_swap_pte' [-Werror=implicit-function-declaration]
>     if (!is_swap_pte(pte))
>     ^
>   ...
> 
> Also need let CMA depend on MMU, or when NOMMU, if select CMA, it will
> select MIGRATION by force.
> 
> 
> Signed-off-by: Chen Gang <gang.chen@asianux.com>
> Reviewed-by: Michal Hocko <mhocko@suse.cz>
> ---
>  mm/Kconfig |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/mm/Kconfig b/mm/Kconfig
> index 256bfd0..e847f19 100644
> --- a/mm/Kconfig
> +++ b/mm/Kconfig
> @@ -245,7 +245,7 @@ config COMPACTION
>  config MIGRATION
>  	bool "Page migration"
>  	def_bool y
> -	depends on NUMA || ARCH_ENABLE_MEMORY_HOTREMOVE || COMPACTION || CMA
> +	depends on (NUMA || ARCH_ENABLE_MEMORY_HOTREMOVE || COMPACTION || CMA) && MMU
>  	help
>  	  Allows the migration of the physical location of pages of processes
>  	  while the virtual addresses are not changed. This is useful in
> @@ -522,7 +522,7 @@ config MEM_SOFT_DIRTY
>  
>  config CMA
>  	bool "Contiguous Memory Allocator"
> -	depends on HAVE_MEMBLOCK
> +	depends on HAVE_MEMBLOCK && MMU
>  	select MIGRATION
>  	select MEMORY_ISOLATION
>  	help
> 


-- 
Chen Gang

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>