lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 09 Oct 2018 21:51:42 +0530 From: Sibi Sankar <sibis@...eaurora.org> To: Bjorn Andersson <bjorn.andersson@...aro.org> Cc: linux-remoteproc@...r.kernel.org, linux-kernel@...r.kernel.org, ohad@...ery.com, kyan@...eaurora.org, sricharan@...eaurora.org, akdwived@...eaurora.org, linux-arm-msm@...r.kernel.org, tsoni@...eaurora.org Subject: Re: [PATCH v3 5/6] remoteproc: qcom: q6v5-pil: Register segments/dumpfn for coredump Hi Bjorn, Thanks for the review ! On 2018-10-08 12:18, Bjorn Andersson wrote: > On Fri 27 Jul 08:20 PDT 2018, Sibi Sankar wrote: > >> Register the MDT segments and custom dumpfn with the remoteproc core >> dump functionality. >> >> Signed-off-by: Sibi Sankar <sibis@...eaurora.org> >> --- >> drivers/remoteproc/qcom_q6v5_pil.c | 40 >> ++++++++++++++++++++++++++++++ >> 1 file changed, 40 insertions(+) >> >> diff --git a/drivers/remoteproc/qcom_q6v5_pil.c >> b/drivers/remoteproc/qcom_q6v5_pil.c >> index ac3342f9ea5a..22bb049c3e7f 100644 >> --- a/drivers/remoteproc/qcom_q6v5_pil.c >> +++ b/drivers/remoteproc/qcom_q6v5_pil.c >> @@ -1058,10 +1058,50 @@ static void *q6v5_da_to_va(struct rproc >> *rproc, u64 da, int len) >> return qproc->mpss_region + offset; >> } >> >> +static int qcom_q6v5_register_dump_segments(struct rproc *rproc, >> + const struct firmware *fw_unused) > > How about naming it mba_fw instead of unused? Just as unused, but > easier > to understand why it isn't used. > sure >> +{ >> + const struct firmware *fw; >> + const struct elf32_phdr *phdrs; >> + const struct elf32_phdr *phdr; >> + const struct elf32_hdr *ehdr; >> + struct q6v5 *qproc = (struct q6v5 *)rproc->priv; > > No need for an explicit typecast from void *. > will remove it > The rest looks good! > > Regards, > Bjorn -- -- Sibi Sankar -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.
Powered by blists - more mailing lists