答复: [PATCH] Initialize max_pfn_mapped as initial ident mapping size for x86_64

[Zhenzhong Duan <zhenzhong.duan@oracle.com>]

----- zhenzhong.duan@oracle.com 写道：

> It's better to initialize max_pfn_mapped as initial ident
> mapping size but not initial kernel map size for x86_64.
> This is also in accordance with i386 code.
> 
> It lead to page tables allocation to as high as 1G for
> init_memory_mapping,
> this will allow larger crashkernel reservation.
> 
> Tested on HP DL980G7, it can reserve more than 521MiB space for kdump
> kernel now.
> http://bugzilla.oracle.com/bugzilla/show_bug.cgi?id=13405
> 
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> Cc: Yinghai Lu <yinghai@kernel.org>
> Signed-off-by: Zhenzhong Duan <zhenzhong.duan@oracle.com>
> Tested-by: Dawei Pang <dawei.pang@hp.com>
> ---
>  arch/x86/kernel/head64.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/x86/kernel/head64.c b/arch/x86/kernel/head64.c
> index 3a3b779..3beb94f 100644
> --- a/arch/x86/kernel/head64.c
> +++ b/arch/x86/kernel/head64.c
> @@ -77,7 +77,7 @@ void __init x86_64_start_kernel(char *
> real_mode_data)
>  	/* Make NULL pointers segfault */
>  	zap_identity_mappings();
>  
> -	max_pfn_mapped = KERNEL_IMAGE_SIZE >> PAGE_SHIFT;
> +	max_pfn_mapped = PUD_PAGE_SIZE >> PAGE_SHIFT;
>  
>  	for (i = 0; i < NUM_EXCEPTION_VECTORS; i++) {
>  #ifdef CONFIG_EARLY_PRINTK
> -- 
> 1.7.3

Hmm, any comment is welcome
thanks

zduan