Add the SRCID of the faulting hardware unit to the return of the
xe_vm_get_property_ioctl fault report.

Signed-off-by: Jonathan Cavitt <[email protected]>
Cc: Matthew Brost <[email protected]>
Cc: Thomas Hellström <[email protected]>
Cc: Rodrigo Vivi <[email protected]>
Cc: David Airlie <[email protected]>
Cc: Simona Vetter <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Cc: Maxime Ripard <[email protected]>
Cc: Thomas Zimmermann <[email protected]>
---
 drivers/gpu/drm/xe/xe_vm.c       | 11 +++++++++--
 drivers/gpu/drm/xe/xe_vm_types.h |  2 ++
 include/uapi/drm/xe_drm.h        |  4 ++--
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c
index 080c2fff0e95..18d5cdafd76b 100644
--- a/drivers/gpu/drm/xe/xe_vm.c
+++ b/drivers/gpu/drm/xe/xe_vm.c
@@ -636,6 +636,7 @@ void xe_vm_add_fault_entry_pf(struct xe_vm *vm, struct 
xe_pagefault *pf)
                                  pf->consumer.fault_type_level);
        e->fault_level = FIELD_GET(XE_PAGEFAULT_LEVEL_MASK,
                                   pf->consumer.fault_type_level);
+       e->srcid = pf->consumer.srcid;
 
        list_add_tail(&e->list, &vm->faults.list);
        vm->faults.len++;
@@ -4108,6 +4109,11 @@ static u8 xe_to_user_fault_level(u8 fault_level)
        return fault_level;
 }
 
+static u8 xe_to_user_srcid(u8 srcid)
+{
+       return srcid;
+}
+
 static int fill_faults(struct xe_vm *vm,
                       struct drm_xe_vm_get_property *args)
 {
@@ -4135,6 +4141,8 @@ static int fill_faults(struct xe_vm *vm,
                fault_entry.fault_type = 
xe_to_user_fault_type(entry->fault_type);
                fault_entry.fault_level = 
xe_to_user_fault_level(entry->fault_level);
 
+               fault_entry.srcid = xe_to_user_srcid(entry->srcid);
+
                memcpy(&fault_list[i], &fault_entry, entry_size);
 
                i++;
@@ -4190,8 +4198,7 @@ int xe_vm_get_property_ioctl(struct drm_device *drm, void 
*data,
        int ret = 0;
 
        if (XE_IOCTL_DBG(xe, (args->reserved[0] || args->reserved[1] ||
-                             args->reserved[2] || args->extensions ||
-                             args->pad)))
+                             args->reserved[2] || args->extensions)))
                return -EINVAL;
 
        vm = xe_vm_lookup(xef, args->vm_id);
diff --git a/drivers/gpu/drm/xe/xe_vm_types.h b/drivers/gpu/drm/xe/xe_vm_types.h
index 635ed29b9a69..e8aea75341cc 100644
--- a/drivers/gpu/drm/xe/xe_vm_types.h
+++ b/drivers/gpu/drm/xe/xe_vm_types.h
@@ -196,6 +196,7 @@ struct xe_device;
  * @access_type: type of address access that resulted in fault
  * @fault_type: type of fault reported
  * @fault_level: fault level of the fault
+ * @srcid: ID of the faulting hardware unit
  */
 struct xe_vm_fault_entry {
        struct list_head list;
@@ -204,6 +205,7 @@ struct xe_vm_fault_entry {
        u8 access_type;
        u8 fault_type;
        u8 fault_level;
+       u8 srcid;
 };
 
 struct xe_vm {
diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
index 48e9f1fdb78d..2b45b691a032 100644
--- a/include/uapi/drm/xe_drm.h
+++ b/include/uapi/drm/xe_drm.h
@@ -1294,8 +1294,8 @@ struct xe_vm_fault {
 #define FAULT_LEVEL_PML4       3
 #define FAULT_LEVEL_PML5       4
        __u8 fault_level;
-       /** @pad: MBZ */
-       __u8 pad;
+       /** @srcid: ID of the faulting hardware unit */
+       __u8 srcid;
        /** @reserved: MBZ */
        __u64 reserved[4];
 };
-- 
2.53.0

Reply via email to