Skip to content
  • Dave Young's avatar
    x86/kexec: Fix a kexec_file_load() failure · 993a1103
    Dave Young authored
    Commit
    
      b6664ba4 ("s390, kexec_file: drop arch_kexec_mem_walk()")
    
    changed the behavior of kexec_locate_mem_hole(): it will try to allocate
    free memory only when kbuf.mem is initialized to zero.
    
    However, x86's kexec_file_load() implementation reuses a struct
    kexec_buf allocated on the stack and its kbuf.mem member gets set by
    each kexec_add_buffer() invocation.
    
    The second kexec_add_buffer() will reuse the same kbuf but not
    reinitialize kbuf.mem.
    
    Therefore, explictily reset kbuf.mem each time in order for
    kexec_locate_mem_hole() to locate a free memory region each time.
    
     [ bp: massage commit message. ]
    
    Fixes: b6664ba4
    
     ("s390, kexec_file: drop arch_kexec_mem_walk()")
    Signed-off-by: default avatarDave Young <dyoung@redhat.com>
    Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
    Acked-by: default avatarBaoquan He <bhe@redhat.com>
    Cc: "Eric W. Biederman" <ebiederm@xmission.com>
    Cc: "H. Peter Anvin" <hpa@zytor.com>
    Cc: AKASHI Takahiro <takahiro.akashi@linaro.org>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Cc: Ingo Molnar <mingo@redhat.com>
    Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
    Cc: Philipp Rudo <prudo@linux.vnet.ibm.com>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: Vivek Goyal <vgoyal@redhat.com>
    Cc: Yannik Sembritzki <yannik@sembritzki.me>
    Cc: Yi Wang <wang.yi59@zte.com.cn>
    Cc: kexec@lists.infradead.org
    Cc: x86-ml <x86@kernel.org>
    Link: https://lkml.kernel.org/r/20181228011247.GA9999@dhcp-128-65.nay.redhat.com
    993a1103