Yu, Fenghua wrote:
Since in patch 2/8 we already implement the generic part, this will add the rest part for ia64. Signed-off-by: WANG Cong <amwang@xxxxxxxxxx> Cc: Fenghua Yu <fenghua.yu@xxxxxxxxx> Cc: Tony Luck <tony.luck@xxxxxxxxx> --- Index: linux-2.6/arch/ia64/include/asm/kexec.h =================================================================== --- linux-2.6.orig/arch/ia64/include/asm/kexec.h +++ linux-2.6/arch/ia64/include/asm/kexec.h @@ -1,6 +1,7 @@ #ifndef _ASM_IA64_KEXEC_H #define _ASM_IA64_KEXEC_H +#include <asm-generic/kexec.h> /* Maximum physical address we can use pages from */ #define KEXEC_SOURCE_MEMORY_LIMIT (-1UL) @@ -19,6 +20,49 @@ flush_icache_range(page_addr, page_addr + PAGE_SIZE); \ } while(0) +#ifdef CONFIG_KEXEC_AUTO_RESERVE +#define MBYTES(n) ((n)*1024*1024ULL) +#define GBYTES(n) ((n)*1024*1024*1024ULL) +/* + Memory size Reserved memory + =========== =============== + [4G, 12G) 256M + [12G, 128G) 512M + [128G, 256G) 768M + [256G, 378G) 1024M + [378G, 512G) 1536M + [512G, 768G) 2048M + [768G, ) 3072M + */ +static inline +unsigned long long arch_default_crash_size(unsigned long long total_size) +{This function shouldn't be inlined (Andrew commented this before). You might remove inline and add __init for this function.
Yeah, seems I missed Andrew's comment... :-/ I will update it.
+ unsigned long long ret; + + if (total_size >= GBYTES(4) && total_size < GBYTES(12)) + ret = MBYTES(256); + else if (total_size >= GBYTES(12) && total_size < GBYTES(128)) + ret = MBYTES(512); + else if (total_size >= GBYTES(128) && total_size < GBYTES(256)) + ret = MBYTES(768); + else if (total_size >= GBYTES(256) && total_size < GBYTES(378)) + ret = MBYTES(1024); + else if (total_size >= GBYTES(318) && total_size < GBYTES(512)) + ret = MBYTES(1536); + else if (total_size >= GBYTES(512) && total_size < GBYTES(768)) + ret = MBYTES(2048); + else + ret = MBYTES(3072); + if (!ia64_platform_is("sn2") && !ia64_platform_is("uv")) + if (ret > GBYTES(4)) + ret = ret / GBYTES(4) * MBYTES(256);This code doesn't make sense to me. What you really want is following code, right? + if (total_size > GBYTES(4)) + ret = total_size / GBYTES(4) * MBYTEST(256)
Nope. Check check_crashkernel_memory(). Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |