Thiago Jung Bauermann <[email protected]> writes: > Dave Young <[email protected]> writes: > >> On 03/06/18 at 07:22pm, AKASHI Takahiro wrote: >>> As arch_kexec_kernel_image_{probe,load}(), >>> arch_kimage_file_post_load_cleanup() and arch_kexec_kernel_verify_sig() >>> are almost duplicated among architectures, they can be commonalized with >>> an architecture-defined kexec_file_ops array. So let's factor them out. >>> >>> Signed-off-by: AKASHI Takahiro <[email protected]> >>> Cc: Dave Young <[email protected]> >>> Cc: Vivek Goyal <[email protected]> >>> Cc: Baoquan He <[email protected]> >>> Cc: Michael Ellerman <[email protected]> >>> Cc: Thiago Jung Bauermann <[email protected]> >>> --- >>> arch/powerpc/include/asm/kexec.h | 2 +- >>> arch/powerpc/kernel/kexec_elf_64.c | 2 +- >>> arch/powerpc/kernel/machine_kexec_file_64.c | 39 ++----------------- >>> arch/x86/include/asm/kexec-bzimage64.h | 2 +- >>> arch/x86/kernel/kexec-bzimage64.c | 2 +- >>> arch/x86/kernel/machine_kexec_64.c | 45 +--------------------- >>> include/linux/kexec.h | 13 +++---- >>> kernel/kexec_file.c | 60 >>> +++++++++++++++++++++++++++-- >>> 8 files changed, 71 insertions(+), 94 deletions(-) >>> >> >> For this patch it also needs some review from powerpc people. > > FWIW: > > Reviewed-by: Thiago Jung Bauermann <[email protected]> > > Also, tested on a ppc64le KVM guest: > > Tested-by: Thiago Jung Bauermann <[email protected]>
Thanks for testing. It looks OK to me. Acked-by: Michael Ellerman <[email protected]> (powerpc) cheers
