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>

diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
index 38a5e98d7dd73cdd0486828cd255f04c60dd61ba..aa6206706fe335008060a9e3da7a8a97d2f91e50 100644 (file)
--- a/drivers/remoteproc/remoteproc_core.c
+++ b/drivers/remoteproc/remoteproc_core.c
@@ -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);
 
        /*

diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c
index a20488336aa091b6a0ec69941b0e4bbeb1b84184..a5c29f2764a34f4d12b3080cbcec30d4fc908506 100644 (file)
--- a/drivers/remoteproc/remoteproc_debugfs.c
+++ b/drivers/remoteproc/remoteproc_debugfs.c
@@ -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);

diff --git a/drivers/remoteproc/remoteproc_virtio.c b/drivers/remoteproc/remoteproc_virtio.c
index b0633fd4c04141e9a0ff27f6a80e9dd0dfc8bad6..bbecd44df7e8de3b28ee10b7066c42913aec540a 100644 (file)
--- a/drivers/remoteproc/remoteproc_virtio.c
+++ b/drivers/remoteproc/remoteproc_virtio.c
@@ -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);
 
        /*