On 15/09/22 2:12 pm, Sourabh Jain wrote:
Restricting kexec tool to allocate hole for kexec segments below 768MB may not be relavent now since first memory block size can be 1024MB and more. Removing rma_top restriction will give more space to find holes for kexec segments and existing in-place checks make sure that kexec segment allocation doesn't cross the first memory block because every kexec segment has to be within first memory block for kdump kernel to boot properly. Signed-off-by: Sourabh Jain <sourabhjain@xxxxxxxxxxxxx>
Yeah. The restriction seems arbitrary. Even more so since commit 47478ea66d43 ("kexec-tools: ppc64: fix how RMA top is deduced") with which kexec_load was sure to place all segments within the first memory block. So, the change looks good to me. Acked-by: Hari Bathini <hbathini@xxxxxxxxxxxxx>
--- kexec/arch/ppc64/kexec-ppc64.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/kexec/arch/ppc64/kexec-ppc64.c b/kexec/arch/ppc64/kexec-ppc64.c index 5b17740..611809f 100644 --- a/kexec/arch/ppc64/kexec-ppc64.c +++ b/kexec/arch/ppc64/kexec-ppc64.c @@ -717,8 +717,6 @@ static int get_devtree_details(unsigned long kexec_flags) if (base < rma_base) { rma_base = base; rma_top = base + be64_to_cpu(((uint64_t *)buf)[1]); - if (rma_top > 0x30000000UL) - rma_top = 0x30000000UL; } fclose(file);
_______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec