drivers: virt: acrn: Don't use %pK through printk

In the past %pK was preferable to %p as it would not leak raw pointer
values into the kernel log.
Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
the regular %p has been improved to avoid this issue.
Furthermore, restricted pointers ("%pK") were never meant to be used
through printk(). They can still unintentionally leak raw pointers or
acquire sleeping locks in atomic contexts.

Switch to the regular pointer formatting which is safer and
easier to reason about.

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Link: https://lore.kernel.org/r/20250718-restricted-pointers-virt-v1-1-12913fceaf52@linutronix.de
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/virt/acrn/ioreq.c b/drivers/virt/acrn/ioreq.c
index e94358239a4b54013b2d770b3626175c6c312059..55ddfa4840afc4c0bf72a73b9dac1a74ddbddb09 100644 (file)
--- a/drivers/virt/acrn/ioreq.c
+++ b/drivers/virt/acrn/ioreq.c
@@ -626,7 +626,7 @@ int acrn_ioreq_init(struct acrn_vm *vm, u64 buf_vma)
        }
 
        dev_dbg(acrn_dev.this_device,
-               "Init ioreq buffer %pK!\n", vm->ioreq_buf);
+               "Init ioreq buffer %p!\n", vm->ioreq_buf);
        ret = 0;
 free_buf:
        kfree(set_buffer);
@@ -638,7 +638,7 @@ void acrn_ioreq_deinit(struct acrn_vm *vm)
        struct acrn_ioreq_client *client, *next;
 
        dev_dbg(acrn_dev.this_device,
-               "Deinit ioreq buffer %pK!\n", vm->ioreq_buf);
+               "Deinit ioreq buffer %p!\n", vm->ioreq_buf);
        /* Destroy all clients belonging to this VM */
        list_for_each_entry_safe(client, next, &vm->ioreq_clients, list)
                acrn_ioreq_client_destroy(client);

diff --git a/drivers/virt/acrn/mm.c b/drivers/virt/acrn/mm.c
index 4c2f28715b703957eb8ef1aef2a719cd704cb1ef..bfb3031885e8d6c288d0d34a01c2509c808249cb 100644 (file)
--- a/drivers/virt/acrn/mm.c
+++ b/drivers/virt/acrn/mm.c
@@ -68,7 +68,7 @@ int acrn_mm_region_add(struct acrn_vm *vm, u64 user_gpa, u64 service_gpa,
        ret = modify_region(vm, region);
 
        dev_dbg(acrn_dev.this_device,
-               "%s: user-GPA[%pK] service-GPA[%pK] size[0x%llx].\n",
+               "%s: user-GPA[%p] service-GPA[%p] size[0x%llx].\n",
                __func__, (void *)user_gpa, (void *)service_gpa, size);
        kfree(region);
        return ret;
@@ -99,7 +99,7 @@ int acrn_mm_region_del(struct acrn_vm *vm, u64 user_gpa, u64 size)
 
        ret = modify_region(vm, region);
 
-       dev_dbg(acrn_dev.this_device, "%s: user-GPA[%pK] size[0x%llx].\n",
+       dev_dbg(acrn_dev.this_device, "%s: user-GPA[%p] size[0x%llx].\n",
                __func__, (void *)user_gpa, size);
        kfree(region);
        return ret;
@@ -224,7 +224,7 @@ int acrn_vm_ram_map(struct acrn_vm *vm, struct acrn_vm_memmap *memmap)
 
                if (ret) {
                        dev_dbg(acrn_dev.this_device,
-                               "Failed to lookup PFN at VMA:%pK.\n", (void *)memmap->vma_base);
+                               "Failed to lookup PFN at VMA:%p.\n", (void *)memmap->vma_base);
                        return ret;
                }
 
@@ -326,7 +326,7 @@ int acrn_vm_ram_map(struct acrn_vm *vm, struct acrn_vm_memmap *memmap)
        kfree(regions_info);
 
        dev_dbg(acrn_dev.this_device,
-               "%s: VM[%u] service-GVA[%pK] user-GPA[%pK] size[0x%llx]\n",
+               "%s: VM[%u] service-GVA[%p] user-GPA[%p] size[0x%llx]\n",
                __func__, vm->vmid,
                remap_vaddr, (void *)memmap->user_vm_pa, memmap->len);
        return ret;