Received: by 10.223.185.116 with SMTP id b49csp421723wrg; Fri, 23 Feb 2018 00:35:13 -0800 (PST) X-Google-Smtp-Source: AH8x225vXSogrlCiFIQZYQ8DSMpwmDMssPeFhQx46sJpuDNe0KLTF/cKMuD0z8xBqR/9c/O2Givb X-Received: by 2002:a17:902:8605:: with SMTP id f5-v6mr996045plo.120.1519374912922; Fri, 23 Feb 2018 00:35:12 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519374912; cv=none; d=google.com; s=arc-20160816; b=cOmufdfHUchcLuLR5jbZmtkBWgV/xfMpfFbP3AEMpszvVs0pFxRXY+qo6xhXP2bs0r AiZ9qylfEzQy1HlXemIEun1szCLtWpswqH+XSZBKUPkVqOz+CnS93m9FGqT41gDZ301w MJCg7Lw9BZXI+lDY/VxfJK5rBZphhycpkqSSpO5Bi1sb0ZI5sPvLxLrqS8x1QTq2mcKX FgFU54788xRyShIYsGkNhpuxV0UFlgakQp5dKGXYZP1ctxCKb6zgVLYCMEgFCGOQyG9o 8+G1qu6c6JBJTaSiSKXFwsx7AHdqbMIDzTPWQapUlhBYYPNo9YHhwvBiLTZ4mtxDCi7m PfCQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=8hRR0nW9I246HyhnDuwf+YHZPtVScqhrvMFEkNFDetQ=; b=piLE3RVjeS5CAYKmjhD06LIJZGPumhQXfXDRHM1F1kDpeK5OS7no/Ki4/u5yt7YAWP vB58+qm3tkw27NbgRh8aB8H2g83eIyxo2sKIHWg1tw74nGECk6R+2vtIEetcnrXV2v5H 6+OXuevTHNweVYGhxI9IvDhexD+nDnM8s0y+sDHLGuRHPLzjkrJ8tRYPdeYXmqkmkvJ0 WI0YDK16jIae6qd4nWuxSYkW/trd0UShtEWG6hQ8yhB1bF+JWYv34yJCRxUWFbMpxY64 wJFPVTFufo5ELgiJsX131oJzZRmcYmFn2rERZnddmPBEgCppKKKkye0QlJ5TkADREtGH WZeQ== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r63si1464530pfj.95.2018.02.23.00.34.58; Fri, 23 Feb 2018 00:35:12 -0800 (PST) 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751681AbeBWIeT (ORCPT + 99 others); Fri, 23 Feb 2018 03:34:19 -0500 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:37334 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751273AbeBWIeP (ORCPT ); Fri, 23 Feb 2018 03:34:15 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id D345580401C1; Fri, 23 Feb 2018 08:34:14 +0000 (UTC) Received: from dhcp-128-65.nay.redhat.com (ovpn-12-55.pek2.redhat.com [10.72.12.55]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 97385B300D; Fri, 23 Feb 2018 08:34:09 +0000 (UTC) Date: Fri, 23 Feb 2018 16:34:05 +0800 From: Dave Young To: Philipp Rudo Cc: kexec@lists.infradead.org, linux-s390@vger.kernel.org, Michael Ellerman , x86@kernel.org, Heiko Carstens , linux-kernel@vger.kernel.org, Martin Schwidefsky , Eric Biederman , Thiago Jung Bauermann , Andrew Morton , Vivek Goyal , AKASHI Takahiro Subject: Re: [PATCH 00/17] Add kexec_file_load support to s390 Message-ID: <20180223083405.GA29498@dhcp-128-65.nay.redhat.com> References: <20180212100754.55121-1-prudo@linux.vnet.ibm.com> <20180214073543.GA12328@dhcp-128-65.nay.redhat.com> <20180214105449.1325bfbe@ThinkPad> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180214105449.1325bfbe@ThinkPad> User-Agent: Mutt/1.9.1 (2017-09-22) X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Fri, 23 Feb 2018 08:34:14 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Fri, 23 Feb 2018 08:34:14 +0000 (UTC) for IP:'10.11.54.5' DOMAIN:'int-mx05.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'dyoung@redhat.com' RCPT:'' Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/14/18 at 10:54am, Philipp Rudo wrote: > Hi Dave, > > I just browsed AKASHI's patches (v7). The way I see it the common code changes > are on different areas of kexec_file and shouldn't get in conflict. Only in the > arch code (s390 and arm64) some function calls might need an update. But I have > to take a closer look on the patches first. I'm tring both in local branch with both series (V8 from AKASHI and this series), but there are indeed some conflicts, may because AKASHI factored out some arch code from ppc/x86 to common code. > > @AKASHI: Welcome to the list. While browsing I couldn't find any > arch_apply_relocation function for arm64. These require the biggest changes in > my patch set. Did I just miss it or does arm64 work differently? > > @Dave: Happy holiday Thank you! > > Thanks > Philipp > > On Wed, 14 Feb 2018 15:35:43 +0800 > Dave Young wrote: > > > Hi Philipp, > > > > I added AKASHI in cc, he posted arm64 kexec_file series previously. > > I would like to read both series especially the general part, but > > maybe at the end of this month because of a holiday.. > > > > From the patch log the cleanup looks nice, but still need read the > > details. > > > > On 02/12/18 at 11:07am, Philipp Rudo wrote: > > > Hi everybody > > > > > > resending the series as there was no reaction, yet. Furthermore i was told > > > that Andrew and the x86 list should also be CCed, so welcome. > > > > > > No changes made to the patches since first time i sent them. The patches > > > apply to the current master (v4.16-rc1). > > > > > > Thanks > > > Philipp > > > > > > --- > > > > > > this series adds the kexec_file_load system call to s390. Before the system > > > call is added there are some preparations/clean ups to common > > > kexec_file_load. In detail this series contains: > > > > > > Patch #1&2: Minor cleanups/fixes. > > > > > > Patch #3-9: Clean up the purgatory load/relocation code. Especially remove > > > the mis-use of the purgatory_info->sechdrs->sh_offset field, currently > > > holding a pointer into either kexec_purgatory (ro) or purgatory_buf (rw) > > > depending on the section. With these patches the section address will be > > > calculated verbosely and sh_offset will contain the offset of the section > > > in the stripped purgatory binary (purgatory_buf). > > > > > > Patch #10: Allows architectures to set the purgaory load address. This > > > patch is important for s390 as the kernel and purgatory have to be loaded > > > to fixed addresses. In current code this is impossible as the purgatory > > > load is opaque to the architecture. > > > > > > Patch #11: Moves x86 purgatories sha implementation to common lib/ > > > directory. > > > > > > Patches #12-17 finally adds the kexec_file_load system call to s390. > > > > > > Please note that I had to touch arch code for x86 and power a little. In > > > theory this should not change the behavior but I don't have a way to test > > > it. Cross-compiling with defconfig(*) works fine for both. > > > > > > Thanks > > > Philipp > > > > > > (*) On x86 with the orc unwinder turned off. objtool SEGFAULTs on s390... > > > > > > Philipp Rudo (17): > > > kexec_file: Silence compile warnings > > > kexec_file: Remove checks in kexec_purgatory_load > > > kexec_file: Make purgatory_info->ehdr const > > > kexec_file: Search symbols in read-only kexec_purgatory > > > kexec_file: Use read-only sections in arch_kexec_apply_relocations* > > > kexec_file: Split up __kexec_load_puragory > > > kexec_file: Simplify kexec_purgatory_setup_sechdrs 1 > > > kexec_file: Simplify kexec_purgatory_setup_sechdrs 2 > > > kexec_file: Remove mis-use of sh_offset field > > > kexec_file: Allow archs to set purgatory load address > > > kexec_file: Move purgatories sha256 to common code > > > s390/kexec_file: Prepare setup.h for kexec_file_load > > > s390/kexec_file: Add purgatory > > > s390/kexec_file: Add kexec_file_load system call > > > s390/kexec_file: Add image loader > > > s390/kexec_file: Add crash support to image loader > > > s390/kexec_file: Add ELF loader > > > > > > arch/powerpc/kernel/kexec_elf_64.c | 9 +- > > > arch/s390/Kbuild | 1 + > > > arch/s390/Kconfig | 4 + > > > arch/s390/include/asm/kexec.h | 23 ++ > > > arch/s390/include/asm/purgatory.h | 17 ++ > > > arch/s390/include/asm/setup.h | 40 ++- > > > arch/s390/kernel/Makefile | 1 + > > > arch/s390/kernel/asm-offsets.c | 5 + > > > arch/s390/kernel/compat_wrapper.c | 1 + > > > arch/s390/kernel/kexec_elf.c | 149 ++++++++++ > > > arch/s390/kernel/kexec_image.c | 78 +++++ > > > arch/s390/kernel/machine_kexec_file.c | 291 +++++++++++++++++++ > > > arch/s390/kernel/syscalls/syscall.tbl | 1 + > > > arch/s390/purgatory/Makefile | 37 +++ > > > arch/s390/purgatory/head.S | 279 ++++++++++++++++++ > > > arch/s390/purgatory/purgatory.c | 42 +++ > > > arch/x86/kernel/kexec-bzimage64.c | 8 +- > > > arch/x86/kernel/machine_kexec_64.c | 66 ++--- > > > arch/x86/purgatory/Makefile | 3 + > > > arch/x86/purgatory/purgatory.c | 2 +- > > > include/linux/kexec.h | 38 +-- > > > {arch/x86/purgatory => include/linux}/sha256.h | 10 +- > > > kernel/kexec_file.c | 375 ++++++++++++------------- > > > {arch/x86/purgatory => lib}/sha256.c | 4 +- > > > 24 files changed, 1200 insertions(+), 284 deletions(-) > > > create mode 100644 arch/s390/include/asm/purgatory.h > > > create mode 100644 arch/s390/kernel/kexec_elf.c > > > create mode 100644 arch/s390/kernel/kexec_image.c > > > create mode 100644 arch/s390/kernel/machine_kexec_file.c > > > create mode 100644 arch/s390/purgatory/Makefile > > > create mode 100644 arch/s390/purgatory/head.S > > > create mode 100644 arch/s390/purgatory/purgatory.c > > > rename {arch/x86/purgatory => include/linux}/sha256.h (63%) > > > rename {arch/x86/purgatory => lib}/sha256.c (99%) > > > > > > -- > > > 2.13.5 > > > > > > > > > _______________________________________________ > > > kexec mailing list > > > kexec@lists.infradead.org > > > http://lists.infradead.org/mailman/listinfo/kexec > > > > Thanks > > Dave > > >