public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Lakshmi Ramasubramanian <nramas@linux•microsoft.com>
To: zohar@linux•ibm.com, bauerman@linux•ibm.com, robh@kernel•org,
	takahiro.akashi@linaro•org, gregkh@linuxfoundation•org,
	will@kernel•org, joe@perches•com, catalin.marinas@arm•com,
	mpe@ellerman•id.au
Cc: mark.rutland@arm•com, bhsharma@redhat•com, tao.li@vivo•com,
	paulus@samba•org, vincenzo.frascino@arm•com,
	frowand.list@gmail•com, sashal@kernel•org, masahiroy@kernel•org,
	jmorris@namei•org, linux-arm-kernel@lists•infradead.org,
	serge@hallyn•com, devicetree@vger•kernel.org,
	pasha.tatashin@soleen•com, prsriva@linux•microsoft.com,
	hsinyi@chromium•org, allison@lohutok•net,
	christophe.leroy@c-s•fr, mbrugger@suse•com,
	balajib@linux•microsoft.com, dmitry.kasatkin@gmail•com,
	linux-kernel@vger•kernel.org, james.morse@arm•com,
	linux-integrity@vger•kernel.org, linuxppc-dev@lists•ozlabs.org
Subject: [PATCH v16 01/12] powerpc: Rename kexec elfcorehdr_addr to elf_headers_mem
Date: Thu,  4 Feb 2021 08:41:24 -0800	[thread overview]
Message-ID: <20210204164135.29856-2-nramas@linux.microsoft.com> (raw)
In-Reply-To: <20210204164135.29856-1-nramas@linux.microsoft.com>

From: Rob Herring <robh@kernel•org>

The architecture specific field, elfcorehdr_addr in struct kimage_arch,
that holds the address of the buffer in memory for ELF core header for
powerpc has a different name than the one used for arm64.  This makes
it hard to have a common code for setting up the device tree for
kexec system call.

Rename elfcorehdr_addr to elf_headers_mem to align with arm64 name so
common code can use it.

Signed-off-by: Rob Herring <robh@kernel•org>
Reviewed-by: Thiago Jung Bauermann <bauerman@linux•ibm.com>
Reviewed-by: Lakshmi Ramasubramanian <nramas@linux•microsoft.com>
---
 arch/powerpc/include/asm/kexec.h  | 2 +-
 arch/powerpc/kexec/file_load.c    | 4 ++--
 arch/powerpc/kexec/file_load_64.c | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/powerpc/include/asm/kexec.h b/arch/powerpc/include/asm/kexec.h
index 55d6ede30c19..dbf09d2f36d0 100644
--- a/arch/powerpc/include/asm/kexec.h
+++ b/arch/powerpc/include/asm/kexec.h
@@ -108,7 +108,7 @@ struct kimage_arch {
 	unsigned long backup_start;
 	void *backup_buf;
 
-	unsigned long elfcorehdr_addr;
+	unsigned long elf_headers_mem;
 	unsigned long elf_headers_sz;
 	void *elf_headers;
 
diff --git a/arch/powerpc/kexec/file_load.c b/arch/powerpc/kexec/file_load.c
index 9a232bc36c8f..e452b11df631 100644
--- a/arch/powerpc/kexec/file_load.c
+++ b/arch/powerpc/kexec/file_load.c
@@ -45,7 +45,7 @@ char *setup_kdump_cmdline(struct kimage *image, char *cmdline,
 		return NULL;
 
 	elfcorehdr_strlen = sprintf(cmdline_ptr, "elfcorehdr=0x%lx ",
-				    image->arch.elfcorehdr_addr);
+				    image->arch.elf_headers_mem);
 
 	if (elfcorehdr_strlen + cmdline_len > COMMAND_LINE_SIZE) {
 		pr_err("Appending elfcorehdr=<addr> exceeds cmdline size\n");
@@ -263,7 +263,7 @@ int setup_new_fdt(const struct kimage *image, void *fdt,
 		 * Avoid elfcorehdr from being stomped on in kdump kernel by
 		 * setting up memory reserve map.
 		 */
-		ret = fdt_add_mem_rsv(fdt, image->arch.elfcorehdr_addr,
+		ret = fdt_add_mem_rsv(fdt, image->arch.elf_headers_mem,
 				      image->arch.elf_headers_sz);
 		if (ret) {
 			pr_err("Error reserving elfcorehdr memory: %s\n",
diff --git a/arch/powerpc/kexec/file_load_64.c b/arch/powerpc/kexec/file_load_64.c
index c69bcf9b547a..a05c19b3cc60 100644
--- a/arch/powerpc/kexec/file_load_64.c
+++ b/arch/powerpc/kexec/file_load_64.c
@@ -815,7 +815,7 @@ static int load_elfcorehdr_segment(struct kimage *image, struct kexec_buf *kbuf)
 		goto out;
 	}
 
-	image->arch.elfcorehdr_addr = kbuf->mem;
+	image->arch.elf_headers_mem = kbuf->mem;
 	image->arch.elf_headers_sz = headers_sz;
 	image->arch.elf_headers = headers;
 out:
@@ -851,7 +851,7 @@ int load_crashdump_segments_ppc64(struct kimage *image,
 		return ret;
 	}
 	pr_debug("Loaded elf core header at 0x%lx, bufsz=0x%lx memsz=0x%lx\n",
-		 image->arch.elfcorehdr_addr, kbuf->bufsz, kbuf->memsz);
+		 image->arch.elf_headers_mem, kbuf->bufsz, kbuf->memsz);
 
 	return 0;
 }
-- 
2.30.0


  reply	other threads:[~2021-02-04 16:46 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-04 16:41 [PATCH v16 00/12] Carry forward IMA measurement log on kexec on ARM64 Lakshmi Ramasubramanian
2021-02-04 16:41 ` Lakshmi Ramasubramanian [this message]
2021-02-04 16:41 ` [PATCH v16 02/12] of: Add a common kexec FDT setup function Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 03/12] arm64: Use common of_kexec_setup_new_fdt() Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 04/12] powerpc: " Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 05/12] powerpc: Move ima buffer fields to struct kimage Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 06/12] powerpc: Move arch independent ima kexec functions to drivers/of/kexec.c Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 07/12] kexec: Use fdt_appendprop_addrrange() to add ima buffer to FDT Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 08/12] powerpc: Delete unused function delete_fdt_mem_rsv() Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 09/12] of: Define functions to allocate and free FDT Lakshmi Ramasubramanian
2021-02-04 16:41 ` [PATCH v16 10/12] arm64: Use OF alloc and free functions for FDT Lakshmi Ramasubramanian
2021-02-04 18:00   ` Will Deacon
2021-02-04 16:41 ` [PATCH v16 11/12] powerpc: Use OF alloc and free " Lakshmi Ramasubramanian
2021-02-04 19:26   ` Rob Herring
2021-02-04 23:23     ` Lakshmi Ramasubramanian
2021-02-04 23:36       ` Rob Herring
2021-02-04 23:42         ` Lakshmi Ramasubramanian
2021-02-08  4:12     ` Michael Ellerman
2021-02-04 16:41 ` [PATCH v16 12/12] arm64: Enable passing IMA log to next kernel on kexec Lakshmi Ramasubramanian

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210204164135.29856-2-nramas@linux.microsoft.com \
    --to=nramas@linux$(echo .)microsoft.com \
    --cc=allison@lohutok$(echo .)net \
    --cc=balajib@linux$(echo .)microsoft.com \
    --cc=bauerman@linux$(echo .)ibm.com \
    --cc=bhsharma@redhat$(echo .)com \
    --cc=catalin.marinas@arm$(echo .)com \
    --cc=christophe.leroy@c-s$(echo .)fr \
    --cc=devicetree@vger$(echo .)kernel.org \
    --cc=dmitry.kasatkin@gmail$(echo .)com \
    --cc=frowand.list@gmail$(echo .)com \
    --cc=gregkh@linuxfoundation$(echo .)org \
    --cc=hsinyi@chromium$(echo .)org \
    --cc=james.morse@arm$(echo .)com \
    --cc=jmorris@namei$(echo .)org \
    --cc=joe@perches$(echo .)com \
    --cc=linux-arm-kernel@lists$(echo .)infradead.org \
    --cc=linux-integrity@vger$(echo .)kernel.org \
    --cc=linux-kernel@vger$(echo .)kernel.org \
    --cc=linuxppc-dev@lists$(echo .)ozlabs.org \
    --cc=mark.rutland@arm$(echo .)com \
    --cc=masahiroy@kernel$(echo .)org \
    --cc=mbrugger@suse$(echo .)com \
    --cc=mpe@ellerman$(echo .)id.au \
    --cc=pasha.tatashin@soleen$(echo .)com \
    --cc=paulus@samba$(echo .)org \
    --cc=prsriva@linux$(echo .)microsoft.com \
    --cc=robh@kernel$(echo .)org \
    --cc=sashal@kernel$(echo .)org \
    --cc=serge@hallyn$(echo .)com \
    --cc=takahiro.akashi@linaro$(echo .)org \
    --cc=tao.li@vivo$(echo .)com \
    --cc=vincenzo.frascino@arm$(echo .)com \
    --cc=will@kernel$(echo .)org \
    --cc=zohar@linux$(echo .)ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox