[PATCH 3/3] drm/xe: Print vm parameter in xe_vma trace

[Matthew Brost]

On Tue, Dec 17, 2024 at 06:14:20PM -0500, Oak Zeng wrote:
> Print the vm that the vma belongs to in the vma trace.
> 

Again maybe say why this useful. e.g., To correlate VMA operations to
the VM.

> Signed-off-by: Oak Zeng <oak.zeng at intel.com>

With better commit:
Reviewed-by: Matthew Brost <matthew.brost at intel.com>

> ---
>  drivers/gpu/drm/xe/xe_trace_bo.h | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_trace_bo.h b/drivers/gpu/drm/xe/xe_trace_bo.h
> index a46f7cfd92c6f..ce4d0d7e81017 100644
> --- a/drivers/gpu/drm/xe/xe_trace_bo.h
> +++ b/drivers/gpu/drm/xe/xe_trace_bo.h
> @@ -92,6 +92,7 @@ DECLARE_EVENT_CLASS(xe_vma,
>  		    TP_STRUCT__entry(
>  			     __string(dev, __dev_name_vma(vma))
>  			     __field(struct xe_vma *, vma)
> +			     __field(struct xe_vm *, vm)
>  			     __field(u32, asid)
>  			     __field(u64, start)
>  			     __field(u64, end)
> @@ -101,14 +102,16 @@ DECLARE_EVENT_CLASS(xe_vma,
>  		    TP_fast_assign(
>  			   __assign_str(dev);
>  			   __entry->vma = vma;
> +			   __entry->vm = xe_vma_vm(vma);
>  			   __entry->asid = xe_vma_vm(vma)->usm.asid;
>  			   __entry->start = xe_vma_start(vma);
>  			   __entry->end = xe_vma_end(vma) - 1;
>  			   __entry->ptr = xe_vma_userptr(vma);
>  			   ),
>  
> -		    TP_printk("dev=%s, vma=%p, asid=0x%05x, start=0x%012llx, end=0x%012llx, userptr=0x%012llx,",
> -			      __get_str(dev), __entry->vma, __entry->asid, __entry->start,
> +		    TP_printk("dev=%s, vma=%p, vm=%p, asid=0x%05x, start=0x%012llx, end=0x%012llx, userptr=0x%012llx",
> +			      __get_str(dev), __entry->vma, __entry->vm,
> +			      __entry->asid, __entry->start,
>  			      __entry->end, __entry->ptr)
>  )
>  
> -- 
> 2.26.3
>