Received: by 10.213.65.68 with SMTP id h4csp707296imn; Tue, 13 Mar 2018 19:15:49 -0700 (PDT) X-Google-Smtp-Source: AG47ELuWTCNAfYnCP+RPTyqj0v8tZ+IUTVzk+zK3Tzy12hy3TND/C/qwqaAu7NtJX2JzN32OH9Fn X-Received: by 10.167.129.129 with SMTP id g1mr2637769pfi.224.1520993749333; Tue, 13 Mar 2018 19:15:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1520993749; cv=none; d=google.com; s=arc-20160816; b=lFk0Ny9koqe/cWxXlNcA2jn2DrVM0dqlzIuzDtzePF7jwpDqF5G1fbwfxbj+cku5zV MGQd44H3Rk2cqnip5BYXQnm4vy4zOeWMMWCYdK7neKTnf8YQPCGrpVOgOgeUe75qmMer tubd8Eyf5qJRvb62DhoNIwAMX45yjLQMOvC0o9ccCw7BMyhJ867QgAqHd9ZOcpJ1NSzI Vgb59aG/nGviMqzXRbnMvRUNFwOpQXvF5hTiU+P7Z3+CwxYpCYFPrKSK2hVoruDvDO1B +5N7r8zCHHkog5SG83zYYInjy6BCTPqogswfmysC+wXg+Cea11i88pw5yU2RY7ogl03D k1Bg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:arc-authentication-results; bh=QVyrnNeBZLA6exh2LkQq9/sunkrz3e16cXnVLfrZtDg=; b=GN9gGLSag13OSjGi3ZAgnMnpzM/eHotzQ57lQucG4qnvzlXF0OC2alLTc8J3o5RRl5 CWIYKiJsvR0YDrdZ4Wu1OcOzzLiSB1lmIRVLEeP/7jCiNx1yq9gqbsp7bYdbWKYZZGxv WuxKl/3bmvevAZx6jwOGJVB/ApOFk2IWDbXlKcwzw7qCn6vYmr5x4kirJRjjOO7RHmFi bEYEQkqbxG8RmTBod5ViY3PbbhSxyrMWIJICd+wJn0XzAUXH3axQhkhKilhkiA+MWMCi TAvtlwt2YWTMz0XHCppkRx5elWgjO/M5e+NHD9YirbM9pkyYnZrg8DtH4Zwb6NbW0+q7 nUSw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b1-v6si1140516plm.172.2018.03.13.19.15.34; Tue, 13 Mar 2018 19:15:49 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933191AbeCNCOn (ORCPT + 99 others); Tue, 13 Mar 2018 22:14:43 -0400 Received: from ozlabs.org ([103.22.144.67]:48797 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932809AbeCNCOm (ORCPT ); Tue, 13 Mar 2018 22:14:42 -0400 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPSA id 401FfT06wnz9sTc; Wed, 14 Mar 2018 13:14:41 +1100 (AEDT) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=ellerman.id.au From: Michael Ellerman To: Thiago Jung Bauermann , Dave Young Cc: AKASHI Takahiro , vgoyal@redhat.com, bhe@redhat.com, prudo@linux.vnet.ibm.com, kexec@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org Subject: Re: [PATCH v2 2/7] kexec_file, x86, powerpc: factor out kexec_file_ops functions In-Reply-To: <87zi3b8o94.fsf@morokweng.localdomain> References: <20180306102303.9063-1-takahiro.akashi@linaro.org> <20180306102303.9063-3-takahiro.akashi@linaro.org> <20180308011303.GB2600@dhcp-128-65.nay.redhat.com> <87zi3b8o94.fsf@morokweng.localdomain> Date: Wed, 14 Mar 2018 13:14:40 +1100 Message-ID: <87o9jrflnz.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Thiago Jung Bauermann writes: > Dave Young 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 >>> Cc: Dave Young >>> Cc: Vivek Goyal >>> Cc: Baoquan He >>> Cc: Michael Ellerman >>> Cc: Thiago Jung Bauermann >>> --- >>> 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 > > Also, tested on a ppc64le KVM guest: > > Tested-by: Thiago Jung Bauermann Thanks for testing. It looks OK to me. Acked-by: Michael Ellerman (powerpc) cheers