drm/xe: Add properties line to VM snapshot capture

Add properties line to VM snapshot capture which includes additional
information about VMA being dumped. This is helpful for debug purposes
but also to build a robust GPU hang replay tool.

The current format is:

[<vma address>]: <permissions>|<type>

Permissions has two options, either "read_only" or "read_write".

Type has two options, either "userptr" or "bo".

Cc: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Link: https://patch.msgid.link/20251126185952.546277-2-matthew.brost@intel.com

diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c
index 8ab726289583c1ea85d9b9e2f359145965a5a2df..880877b10d9187f0d9818d4faaa37a94305231e9 100644 (file)
--- a/drivers/gpu/drm/xe/xe_vm.c
+++ b/drivers/gpu/drm/xe/xe_vm.c
@@ -4047,6 +4047,9 @@ struct xe_vm_snapshot {
        struct {
                u64 ofs, bo_ofs;
                unsigned long len;
+#define XE_VM_SNAP_FLAG_USERPTR                BIT(0)
+#define XE_VM_SNAP_FLAG_READ_ONLY      BIT(1)
+               unsigned long flags;
                struct xe_bo *bo;
                void *data;
                struct mm_struct *mm;
@@ -4087,6 +4090,8 @@ struct xe_vm_snapshot *xe_vm_snapshot_capture(struct xe_vm *vm)
 
                snap->snap[i].ofs = xe_vma_start(vma);
                snap->snap[i].len = xe_vma_size(vma);
+               snap->snap[i].flags = xe_vma_read_only(vma) ?
+                       XE_VM_SNAP_FLAG_READ_ONLY : 0;
                if (bo) {
                        snap->snap[i].bo = xe_bo_get(bo);
                        snap->snap[i].bo_ofs = xe_vma_bo_offset(vma);
@@ -4100,6 +4105,7 @@ struct xe_vm_snapshot *xe_vm_snapshot_capture(struct xe_vm *vm)
                                snap->snap[i].data = ERR_PTR(-EFAULT);
 
                        snap->snap[i].bo_ofs = xe_vma_userptr(vma);
+                       snap->snap[i].flags |= XE_VM_SNAP_FLAG_USERPTR;
                } else {
                        snap->snap[i].data = ERR_PTR(-ENOENT);
                }
@@ -4169,6 +4175,12 @@ void xe_vm_snapshot_print(struct xe_vm_snapshot *snap, struct drm_printer *p)
        for (i = 0; i < snap->num_snaps; i++) {
                drm_printf(p, "[%llx].length: 0x%lx\n", snap->snap[i].ofs, snap->snap[i].len);
 
+               drm_printf(p, "[%llx].properties: %s|%s\n", snap->snap[i].ofs,
+                          snap->snap[i].flags & XE_VM_SNAP_FLAG_READ_ONLY ?
+                          "read_only" : "read_write",
+                          snap->snap[i].flags & XE_VM_SNAP_FLAG_USERPTR ?
+                          "userptr" : "bo");
+
                if (IS_ERR(snap->snap[i].data)) {
                        drm_printf(p, "[%llx].error: %li\n", snap->snap[i].ofs,
                                   PTR_ERR(snap->snap[i].data));