Commit 78b2770c authored by Sean Anderson's avatar Sean Anderson Committed by Christoph Hellwig

dma-mapping: fix tracing dma_alloc/free with vmalloc'd memory

Not all virtual addresses have physical addresses, such as if they were
vmalloc'd. Just trace the virtual address instead of trying to trace a
physical address. This aligns with the API, and is good enough to
associate dma_alloc with dma_free.

Fixes: 038eb433 ("dma-mapping: add tracing for dma-mapping API calls")
Reported-by: syzbot+b4bfacdec173efaa8567@syzkaller.appspotmail.com
Closes: https://lore.kernel.org/all/670ebde5.050a0220.d9b66.0154.GAE@google.com/Signed-off-by: default avatarSean Anderson <sean.anderson@linux.dev>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent c964ced7
...@@ -121,7 +121,7 @@ TRACE_EVENT(dma_alloc, ...@@ -121,7 +121,7 @@ TRACE_EVENT(dma_alloc,
TP_STRUCT__entry( TP_STRUCT__entry(
__string(device, dev_name(dev)) __string(device, dev_name(dev))
__field(u64, phys_addr) __field(void *, virt_addr)
__field(u64, dma_addr) __field(u64, dma_addr)
__field(size_t, size) __field(size_t, size)
__field(gfp_t, flags) __field(gfp_t, flags)
...@@ -130,18 +130,18 @@ TRACE_EVENT(dma_alloc, ...@@ -130,18 +130,18 @@ TRACE_EVENT(dma_alloc,
TP_fast_assign( TP_fast_assign(
__assign_str(device); __assign_str(device);
__entry->phys_addr = virt_to_phys(virt_addr); __entry->virt_addr = virt_addr;
__entry->dma_addr = dma_addr; __entry->dma_addr = dma_addr;
__entry->size = size; __entry->size = size;
__entry->flags = flags; __entry->flags = flags;
__entry->attrs = attrs; __entry->attrs = attrs;
), ),
TP_printk("%s dma_addr=%llx size=%zu phys_addr=%llx flags=%s attrs=%s", TP_printk("%s dma_addr=%llx size=%zu virt_addr=%p flags=%s attrs=%s",
__get_str(device), __get_str(device),
__entry->dma_addr, __entry->dma_addr,
__entry->size, __entry->size,
__entry->phys_addr, __entry->virt_addr,
show_gfp_flags(__entry->flags), show_gfp_flags(__entry->flags),
decode_dma_attrs(__entry->attrs)) decode_dma_attrs(__entry->attrs))
); );
...@@ -153,7 +153,7 @@ TRACE_EVENT(dma_free, ...@@ -153,7 +153,7 @@ TRACE_EVENT(dma_free,
TP_STRUCT__entry( TP_STRUCT__entry(
__string(device, dev_name(dev)) __string(device, dev_name(dev))
__field(u64, phys_addr) __field(void *, virt_addr)
__field(u64, dma_addr) __field(u64, dma_addr)
__field(size_t, size) __field(size_t, size)
__field(unsigned long, attrs) __field(unsigned long, attrs)
...@@ -161,17 +161,17 @@ TRACE_EVENT(dma_free, ...@@ -161,17 +161,17 @@ TRACE_EVENT(dma_free,
TP_fast_assign( TP_fast_assign(
__assign_str(device); __assign_str(device);
__entry->phys_addr = virt_to_phys(virt_addr); __entry->virt_addr = virt_addr;
__entry->dma_addr = dma_addr; __entry->dma_addr = dma_addr;
__entry->size = size; __entry->size = size;
__entry->attrs = attrs; __entry->attrs = attrs;
), ),
TP_printk("%s dma_addr=%llx size=%zu phys_addr=%llx attrs=%s", TP_printk("%s dma_addr=%llx size=%zu virt_addr=%p attrs=%s",
__get_str(device), __get_str(device),
__entry->dma_addr, __entry->dma_addr,
__entry->size, __entry->size,
__entry->phys_addr, __entry->virt_addr,
decode_dma_attrs(__entry->attrs)) decode_dma_attrs(__entry->attrs))
); );
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment