Commit 71338aa7d050c ("net: convert %p usage to %pK") which is from 2011 and changed the %p annotation for pointer to %pK. Back then the default behaviour for %p was to print the pointer. The %pK modifier was introduced to able to control the behaviour of specific pointer values without changing the behaviour of %p for everyone. It was dedicated to avoid leaking pointers via /proc. Things changed over time. The default behaviour for %p is now to print a hash pointer which does not leak the address but allows to correlate if two pointers are equal. I intend to remove %pK from the library. The policy on %p is to not introduce new ones. This is somehow in between since it already exists. The driver uses %p to print the address of skb, %p for the address of a transfer ring and %pK for the address of the status ring. For the skb it is probably enough to print if there is one mapped. The address of the ring probably not matter given the physical is address is shown and it is virtually mapped. Remove the %p and %pK line from wil_print_.*() so it does not end in debugfs' output (debugfs is not ABI so removing does not break anything). Replace the skb pointer with yes/ no. Signed-off-by: Sebastian Andrzej Siewior --- drivers/net/wireless/ath/wil6210/debugfs.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c b/drivers/net/wireless/ath/wil6210/debugfs.c index e06e45323272c..093ec04e4a052 100644 --- a/drivers/net/wireless/ath/wil6210/debugfs.c +++ b/drivers/net/wireless/ath/wil6210/debugfs.c @@ -80,7 +80,6 @@ static void wil_print_ring(struct seq_file *s, struct wil6210_priv *wil, seq_printf(s, "RING %s = {\n", name); seq_printf(s, " pa = %pad\n", &ring->pa); - seq_printf(s, " va = 0x%p\n", ring->va); seq_printf(s, " size = %d\n", ring->size); if (wil->use_enhanced_dma_hw && ring->is_rx) seq_printf(s, " swtail = %u\n", *ring->edma_rx_swtail.va); @@ -200,7 +199,6 @@ static void wil_print_sring(struct seq_file *s, struct wil6210_priv *wil, seq_printf(s, "Status Ring %s [ %d ] = {\n", sring->is_rx ? "RX" : "TX", sring_idx); seq_printf(s, " pa = %pad\n", &sring->pa); - seq_printf(s, " va = 0x%pK\n", sring->va); seq_printf(s, " size = %d\n", sring->size); seq_printf(s, " elem_size = %zu\n", sring->elem_size); seq_printf(s, " swhead = %d\n", sring->swhead); @@ -1140,7 +1138,7 @@ static int txdesc_show(struct seq_file *s, void *data) u[0], u[1], u[2], u[3]); seq_printf(s, " DMA = 0x%08x 0x%08x 0x%08x 0x%08x\n", u[4], u[5], u[6], u[7]); - seq_printf(s, " SKB = 0x%p\n", skb); + seq_printf(s, " SKB = %s\n", skb ? "yes" : "no"); if (skb) { skb_get(skb); -- 2.55.0