RE: [PATCH v3 1/5] remoteproc: fix the wrong default value of is_iomem
From: Peng Fan (OSS) <hidden>
Date: 2021-08-05 10:00:37
Also in:
linux-remoteproc
Subject: [PATCH v3 1/5] remoteproc: fix the wrong default value of is_iomem
Currently the is_iomem is a random value in the stack which may be default
to true even on those platforms that not use iomem to store firmware.
Cc: Bjorn Andersson <redacted>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Fixes: 40df0a91b2a5 ("remoteproc: add is_iomem to da_to_va")
Reviewed-by: Peng Fan <peng.fan@nxp.com>
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>Tested-by: Peng Fan <peng.fan@nxp.com> with https://patchwork.kernel.org/project/linux-remoteproc/patch/20210629014752.5659-1-peng.fan@oss.nxp.com/
quoted hunk ↗ jump to hunk
--- v2->v3: * no changes v1->v2: * update rproc_copy_segment as well --- drivers/remoteproc/remoteproc_coredump.c | 2 +- drivers/remoteproc/remoteproc_elf_loader.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)diff --git a/drivers/remoteproc/remoteproc_coredump.cb/drivers/remoteproc/remoteproc_coredump.c index aee657cc08c6..c892f433a323 100644--- a/drivers/remoteproc/remoteproc_coredump.c +++ b/drivers/remoteproc/remoteproc_coredump.c@@ -152,8 +152,8 @@ static void rproc_copy_segment(struct rproc *rproc,void *dest, struct rproc_dump_segment *segment, size_t offset, size_t size) { + bool is_iomem = false; void *ptr; - bool is_iomem; if (segment->dump) { segment->dump(rproc, segment, dest, offset, size); diff --git a/drivers/remoteproc/remoteproc_elf_loader.c b/drivers/remoteproc/remoteproc_elf_loader.c index e8078efb3dec..3cd1798f17a3 100644--- a/drivers/remoteproc/remoteproc_elf_loader.c +++ b/drivers/remoteproc/remoteproc_elf_loader.c@@ -178,8 +178,8 @@ int rproc_elf_load_segments(struct rproc *rproc,const struct firmware *fw) u64 filesz = elf_phdr_get_p_filesz(class, phdr); u64 offset = elf_phdr_get_p_offset(class, phdr); u32 type = elf_phdr_get_p_type(class, phdr); + bool is_iomem = false; void *ptr; - bool is_iomem; if (type != PT_LOAD) continue; -- 2.25.1
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel