remoteproc: replace "%p" with "%pK"
The format specifier "%p" can leak kernel addresses. Use "%pK" instead. This patch proposes changes for remoteproc core only. Acked-by: Suman Anna <s-anna@ti.com> Signed-off-by: Loic Pallardy <loic.pallardy@st.com> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
This commit is contained in:
parent
12c2b509c0
commit
276ec99342
3 changed files with 6 additions and 6 deletions
|
@ -241,7 +241,7 @@ int rproc_alloc_vring(struct rproc_vdev *rvdev, int i)
|
|||
if (notifyid > rproc->max_notifyid)
|
||||
rproc->max_notifyid = notifyid;
|
||||
|
||||
dev_dbg(dev, "vring%d: va %p dma %pad size 0x%x idr %d\n",
|
||||
dev_dbg(dev, "vring%d: va %pK dma %pad size 0x%x idr %d\n",
|
||||
i, va, &dma, size, notifyid);
|
||||
|
||||
rvring->va = va;
|
||||
|
@ -499,7 +499,7 @@ static int rproc_handle_trace(struct rproc *rproc, struct fw_rsc_trace *rsc,
|
|||
|
||||
rproc->num_traces++;
|
||||
|
||||
dev_dbg(dev, "%s added: va %p, da 0x%x, len 0x%x\n",
|
||||
dev_dbg(dev, "%s added: va %pK, da 0x%x, len 0x%x\n",
|
||||
name, ptr, rsc->da, rsc->len);
|
||||
|
||||
return 0;
|
||||
|
@ -637,7 +637,7 @@ static int rproc_handle_carveout(struct rproc *rproc,
|
|||
goto free_carv;
|
||||
}
|
||||
|
||||
dev_dbg(dev, "carveout va %p, dma %pad, len 0x%x\n",
|
||||
dev_dbg(dev, "carveout va %pK, dma %pad, len 0x%x\n",
|
||||
va, &dma, rsc->len);
|
||||
|
||||
/*
|
||||
|
|
|
@ -231,7 +231,7 @@ static int rproc_rsc_table_show(struct seq_file *seq, void *p)
|
|||
}
|
||||
break;
|
||||
default:
|
||||
seq_printf(seq, "Unknown resource type found: %d [hdr: %p]\n",
|
||||
seq_printf(seq, "Unknown resource type found: %d [hdr: %pK]\n",
|
||||
hdr->type, hdr);
|
||||
break;
|
||||
}
|
||||
|
@ -260,7 +260,7 @@ static int rproc_carveouts_show(struct seq_file *seq, void *p)
|
|||
|
||||
list_for_each_entry(carveout, &rproc->carveouts, node) {
|
||||
seq_puts(seq, "Carveout memory entry:\n");
|
||||
seq_printf(seq, "\tVirtual address: %p\n", carveout->va);
|
||||
seq_printf(seq, "\tVirtual address: %pK\n", carveout->va);
|
||||
seq_printf(seq, "\tDMA address: %pad\n", &carveout->dma);
|
||||
seq_printf(seq, "\tDevice address: 0x%x\n", carveout->da);
|
||||
seq_printf(seq, "\tLength: 0x%x Bytes\n\n", carveout->len);
|
||||
|
|
|
@ -96,7 +96,7 @@ static struct virtqueue *rp_find_vq(struct virtio_device *vdev,
|
|||
size = vring_size(len, rvring->align);
|
||||
memset(addr, 0, size);
|
||||
|
||||
dev_dbg(dev, "vring%d: va %p qsz %d notifyid %d\n",
|
||||
dev_dbg(dev, "vring%d: va %pK qsz %d notifyid %d\n",
|
||||
id, addr, len, rvring->notifyid);
|
||||
|
||||
/*
|
||||
|
|
Loading…
Reference in a new issue