linux-toradex.git/drivers/gpu/drm/xe/xe_trace.h, branch v6.10-rc2

drm/xe/xe_tracer: Align fence output format in ftrace log

2024-03-13T19:36:53+00:00

The fence print in xe_gt_tlb_invalidation_fence and xe_hw_fence
is with "%p", change fence print in xe_sched_job to "%p" also.

Cc: Lucas De Marchi 
Signed-off-by: Shuicheng Lin 
Reviewed-by: Lucas De Marchi 
Link: https://patchwork.freedesktop.org/patch/msgid/20240313025052.1410833-1-shuicheng.lin@intel.com
Signed-off-by: Lucas De Marchi

drm/xe: Invalidate userptr VMA on page pin fault

2024-03-13T18:40:03+00:00

Rather than return an error to the user or ban the VM when userptr VMA
page pin fails with -EFAULT, invalidate VMA mappings. This supports the
UMD use case of freeing userptr while still having bindings.

Now that non-faulting VMs can invalidate VMAs, drop the usm prefix for
the tile_invalidated member.

v2:
 - Fix build error (CI)
v3:
 - Don't invalidate VMA if in fault mode, rather kill VM (Thomas)
 - Update commit message with tile_invalidated name chagne (Thomas)
 - Wait VM bookkeep slots with VM resv lock (Thomas)
v4:
 - Move list_del_init(&userptr.repin_link) after error check (Thomas)
 - Assert not in fault mode (Matthew)

Fixes: dd08ebf6c352 ("drm/xe: Introduce a new DRM driver for Intel GPUs")
Signed-off-by: Matthew Brost 
Reviewed-by: Thomas Hellström 
Link: https://patchwork.freedesktop.org/patch/msgid/20240312183907.933835-1-matthew.brost@intel.com

drm/xe/xe_trace: Add move_lacks_source detail to xe_bo_move trace

2024-02-29T10:51:58+00:00

Add move_lacks_source detail to xe_bo_move trace to make it readable
that is to check if it is migrate clear or migrate copy.

Cc: Thomas Hellström 
Signed-off-by: Priyanka Dandamudi 
Reviewed-by: Thomas Hellström 
Fixes: a0df2cc858c3 ("drm/xe/xe_bo_move: Enhance xe_bo_move trace")
Signed-off-by: Thomas Hellström 
Link: https://patchwork.freedesktop.org/patch/msgid/20240221101950.1019312-1-priyanka.dandamudi@intel.com

drm/xe: Use pointers in trace events

2024-02-22T22:22:39+00:00

Commit a0df2cc858c3 ("drm/xe/xe_bo_move: Enhance xe_bo_move trace")
inadvertently reverted commit 8d038f49c1f3 ("drm/xe: Fix cast on trace
variable"), breaking the build on 32bits.

As noted by Ville, there's no point in converting the pointers to u64
and add casts everywhere. In fact, it's better to just use %p and let
the address be hashed. Convert all the cases in xe_trace.h to use
pointers.

Cc: Ville Syrjälä 
Cc: Matt Roper 
Cc: Priyanka Dandamudi 
Cc: Oak Zeng 
Cc: Thomas Hellström 
Signed-off-by: Lucas De Marchi 
Reviewed-by: Thomas Hellström 
Link: https://patchwork.freedesktop.org/patch/msgid/20240222144125.2862546-1-lucas.demarchi@intel.com

drm/xe/xe_bo_move: Enhance xe_bo_move trace

2024-02-20T07:35:14+00:00

Enhanced xe_bo_move trace to be more readable.
It will help to show the migration details.
Src and dst details.

v2: Modify trace_xe_bo_move(), it takes the integer mem_type
rather than a string.
Make mem_type_to_name() extern, it will be used by trace.(Thomas)

v3: Move mem_type_to_name() to xe_bo.[ch] (Thomas, Matt)

v4: Add device details to reduce ambiquity related to vram0/vram1. (Oak)

v5: Rename mem_type_to_name to xe_mem_type_to_name. (Thomas)

v6: Optimised code to use xe_bo_device(__entry->bo). (Thomas)

Cc: Thomas Hellström 
Cc: Oak Zeng 
Cc: Kempczynski Zbigniew 
Cc: Matthew Brost 
Cc: Brian Welty 
Signed-off-by: Priyanka Dandamudi 
Reviewed-by: Oak Zeng 
Reviewed-by: Thomas Hellström 
Signed-off-by: Thomas Hellström 
Link: https://patchwork.freedesktop.org/patch/msgid/20240220044748.948496-1-priyanka.dandamudi@intel.com

drm/xe: Fix cast on trace variable

2024-01-19T21:09:04+00:00

Cast the pointer to unsigned long and let it be implicitly extended to
u64. This fixes the build on 32bits arch.

Cc: Matthew Brost 
Cc: Niranjana Vishwanathapura 
Cc: Rodrigo Vivi 
Reviewed-by: Matt Roper 
Signed-off-by: Lucas De Marchi 
Link: https://patchwork.freedesktop.org/patch/msgid/20240119001612.2991381-5-lucas.demarchi@intel.com

drm/xe: Update SPDX deprecated license identifier

2023-12-21T16:43:34+00:00

The "GPL-2.0" SPDX license identifier is deprecated. Update the
code to use "GPL-2.0-only" instead. Choose this identifier over
"GPL-2.0-or-later" since it's the most restrictive of the two and it's
not fully clear that "GPL-2.0" also allows "GPL-2.0-or-later".

Cc: Francois Dugast 
Cc: Rodrigo Vivi 
Signed-off-by: Thomas Hellström 
Reviewed-by: Francois Dugast 
Link: https://patchwork.freedesktop.org/patch/msgid/20231107082440.7568-1-thomas.hellstrom@linux.intel.com
Signed-off-by: Rodrigo Vivi

drm/xe/trace: Optimize trace definition

2023-12-21T16:43:21+00:00

Make use of EVENT_CLASS to group similar trace events

Signed-off-by: Balasubramani Vivekanandan 
Reviewed-by: Haridhar Kalvala 
Link: https://lore.kernel.org/intel-xe/20231019093140.1901665-3-balasubramani.vivekanandan@intel.com/
Signed-off-by: Rodrigo Vivi

drm/xe: Add event tracing for CTB

2023-12-21T16:43:19+00:00

Event tracing enabled for CTB submissions.

Additional minor refactor - Removed a unnecessary ct_to_xe() call.

v2: Remove a unwanted comment (Hari)
    Add missing change to commit message

Signed-off-by: Balasubramani Vivekanandan 
Reviewed-by: Haridhar Kalvala 
Link: https://lore.kernel.org/intel-xe/20231019093140.1901665-2-balasubramani.vivekanandan@intel.com/
Signed-off-by: Rodrigo Vivi

drm/xe: Rename engine to exec_queue

2023-12-21T16:39:20+00:00

Engine was inappropriately used to refer to execution queues and it
also created some confusion with hardware engines. Where it applies
the exec_queue variable name is changed to q and comments are also
updated.

Link: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/162
Signed-off-by: Francois Dugast 
Reviewed-by: Rodrigo Vivi 
Signed-off-by: Rodrigo Vivi