Add a counter to xe_drm_client that tracks the number of times the
engine has been reset since the drm client was created.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/xe/xe_drm_client.c | 2 ++
drivers/gpu/drm/xe/xe_drm_client.h | 2 ++
drivers/gpu/drm/xe/xe_guc_submit.c | 4 +++-
3
variants will be useful for debugging later.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 27 ---
drivers/gpu/drm/xe/xe_gt_pagefault.h | 51
2 files changed, 51 insertions(+), 27 deletions(-)
diff --git a/drivers/gpu/drm/xe
not caused by said
pagefault after the reset.
v2: Remove exec queue from blame list on destroy and recreate (Joonas)
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/xe/xe_drm_client.c | 128
drivers/gpu/drm/xe/xe_drm_client.h | 36 +++
drivers/gpu/drm/xe
caused by said
pagefault after the reset.
Also add a tracker that counts the number of times the drm client has
experienced an engine reset.
Signed-off-by: Jonathan Cavitt
Jonathan Cavitt (4):
drm/xe/xe_exec_queue: Add ID param to exec queue struct
drm/xe/xe_gt_pagefault: Migrate pagefault
Add the exec queue id to the exec queue struct. This is useful for
performing a reverse lookup into the xef->exec_queue xarray.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/xe/xe_exec_queue.c | 1 +
drivers/gpu/drm/xe/xe_exec_queue_types.h | 2 ++
2 files changed, 3 inserti
)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 21 ++
drivers/gpu/drm/xe/xe_vm.c | 59
drivers/gpu/drm/xe/xe_vm.h
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
VMs as well (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag Jadav
Jonathan Cavitt (5):
drm/xe/xe_gt_pagefault: Disallow writes to read-only VMAs
drm
)
v5:
- Store fault and access type instead of address type (Jianxun)
v6:
- Store pagefaults in non-fault-mode VMs as well (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
---
drivers/gpu/drm/xe
)
- Assert valid size (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
---
drivers/gpu/drm/xe/xe_device.c | 3 ++
drivers/gpu/drm/xe/xe_vm.c | 97 ++
drivers/gpu/drm/xe/xe_vm.h | 2 +
3
Add initial declarations for the drm_xe_vm_get_property ioctl.
v2:
- Expand kernel docs for drm_xe_vm_get_property (Jianxun)
v3:
- Remove address type external definitions (Jianxun)
- Add fault type to xe_drm_fault struct (Jianxun)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
---
include
)
- Assert valid size (jcavitt)
v11:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
---
drivers/gpu/drm/xe/xe_device.c | 3 ++
drivers/gpu
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
Add initial declarations for the drm_xe_vm_get_property ioctl.
v2:
- Expand kernel docs for drm_xe_vm_get_property (Jianxun)
v3:
- Remove address type external definitions (Jianxun)
- Add fault type to xe_drm_fault struct (Jianxun)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
---
include
, xe_gt_pagefault_types.h (Michal W)
- Add kernel docs (Michal W)
v4:
- Fix includes usage (Michal W)
- Reference Bspec (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 42 +++
drivers/gpu/drm/xe
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
Add initial declarations for the drm_xe_vm_get_property ioctl.
v2:
- Expand kernel docs for drm_xe_vm_get_property (Jianxun)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
---
include/uapi/drm/xe_drm.h | 80 +++
1 file changed, 80 insertions(+)
diff
Add a new field to the xe_pagefault struct, address_type, that tracks
the type of fault the pagefault incurred.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 3 +++
drivers/gpu/drm/xe/xe_gt_pagefault_types.h | 2 ++
2 files changed, 5 insertions(+)
diff --git
)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 133 +
drivers/gpu/drm/xe/xe_vm.h | 2 +
3 files changed, 138 insertions(+)
diff
logic (jcavitt)
v9:
- Fix includes usage (Michal W)
- More kernel doc fixes (Michal W, Jianxun)
- Better error handling (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal
)
v5:
- Store fault and access type instead of address type (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 21 ++
drivers/gpu/drm/xe/xe_vm.c
extern with access type (Jianxun)
- Add fault type to xe_drm_fault (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Jonathan Cavitt (5):
drm/xe/xe_gt_pagefault
engine class and instance (Ivan)
v3:
- Add declares for fault type, access type, and fault level (Matt Brost,
Ivan)
- Add reference to KMD change (Kamil)
References:
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Suggested-by: Zhang Jianxun
Suggested
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
)
v5:
- Store fault and access type instead of address type (Jianxun)
v6:
- Store pagefaults in non-fault-mode VMs as well (Jianxun)
v7:
- Fix kernel docs and comments (Michal W)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
, Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 109 +
drivers/gpu/drm/xe/xe_vm.h | 2 +
3
declares for fault type, access type, and fault level (Matt Brost,
Ivan)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
Cc: Ivan Briano
Cc: Matthew Brost
---
include/uapi/drm/xe_drm.h | 86 +++
1 file changed, 86 insertions(+)
diff --git a/include/uapi/drm
VMs as well (Jianxun)
v12:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
- Fix kernel docs and comments (Michal W)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc
,
Ivan)
Signed-off-by: Jonathan Cavitt
Cc: Ivan Briano
---
include/drm-uapi/xe_drm.h | 85 +++
1 file changed, 85 insertions(+)
diff --git a/include/drm-uapi/xe_drm.h b/include/drm-uapi/xe_drm.h
index 154f947ef0..8380508d18 100644
--- a/include/drm-uapi
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
)
- Assert valid size (jcavitt)
v11:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
---
drivers/gpu/drm/xe/xe_device.c | 3 ++
drivers/gpu
)
v5:
- Store fault and access type instead of address type (Jianxun)
v6:
- Store pagefaults in non-fault-mode VMs as well (Jianxun)
v7:
- Fix kernel docs and comments (Michal W)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal
VMs as well (Jianxun)
v12:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
- Fix kernel docs and comments (Michal W)
v13:
- Move xe and user engine class mapping arrays to header (John H)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Add initial declarations for the drm_xe_vm_get_property ioctl.
v2:
- Expand kernel docs for drm_xe_vm_get_property (Jianxun)
v3:
- Remove address type external definitions (Jianxun)
- Add fault type to xe_drm_fault struct (Jianxun)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
---
include
.
Signed-off-by: Jonathan Cavitt
Suggested-by: John Harrison
---
drivers/gpu/drm/xe/xe_hw_engine.c | 24
drivers/gpu/drm/xe/xe_hw_engine_types.h | 5 +
drivers/gpu/drm/xe/xe_query.c | 18 +-
3 files changed, 22 insertions(+), 25
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
Add initial declarations for the drm_xe_vm_get_property ioctl.
v2:
- Expand kernel docs for drm_xe_vm_get_property (Jianxun)
v3:
- Remove address type external definitions (Jianxun)
- Add fault type to xe_drm_fault struct (Jianxun)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
---
include
of int (Raag)
- Remove unnecessary includes (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag Jadav
Cc: John Harrison
Jonathan Cavitt (6):
drm/xe
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
)
- Assert valid size (jcavitt)
v11:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
v12:
- Use size_t instead of int (Raag)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
---
drivers/gpu
)
v5:
- Store fault and access type instead of address type (Jianxun)
v6:
- Store pagefaults in non-fault-mode VMs as well (Jianxun)
v7:
- Fix kernel docs and comments (Michal W)
v8:
- Fix double-locking issue (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng
.
v2:
- Remove unnecessary includes (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: John Harrison
---
drivers/gpu/drm/xe/xe_hw_engine.c | 24
drivers/gpu/drm/xe/xe_hw_engine_types.h | 3 +++
drivers/gpu/drm/xe/xe_query.c | 18
)
v9:
- Future-proof size logic for zero-size properties (jcavitt)
- Add access and fault types (Jianxun)
- Remove address type (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
Align with 20250326165751.72881-1-jonathan.cav...@intel.com
Add initial declarations for the xe_vm_get_property_ioctl call, including
necessary structures and IOCTL macros.
Signed-off-by: Jonathan Cavitt
---
include/drm-uapi/xe_drm.h | 78 +++
1 file changed
.
v2:
- Remove unnecessary includes (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: John Harrison
---
drivers/gpu/drm/xe/xe_hw_engine.c | 24
drivers/gpu/drm/xe/xe_hw_engine_types.h | 3 +++
drivers/gpu/drm/xe/xe_query.c | 18
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
)
- Assert valid size (jcavitt)
v11:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
v12:
- Use size_t instead of int (Raag)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
---
drivers/gpu
of int (Raag)
- Remove unnecessary includes (jcavitt)
v15:
- Do not report faults from reserved engines (Jianxun)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag
: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 26
drivers/gpu/drm/xe/xe_vm.c | 89
drivers/gpu/drm/xe/xe_vm.h | 9
.
Signed-off-by: Jonathan Cavitt
---
lib/xe/xe_ioctl.c | 5 +
lib/xe/xe_ioctl.h | 1 +
2 files changed, 6 insertions(+)
diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c
index 6d83889188..a82714cdfd 100644
--- a/lib/xe/xe_ioctl.c
+++ b/lib/xe/xe_ioctl.c
@@ -232,6 +232,11 @@ void
Add a test to xe_vm that determines if pagefaults are correctly tracked
and reported by the DRM_IOCTL_XE_VM_GET_PROPERTY.
Signed-off-by: Jonathan Cavitt
Suggested-by: Jianxun Zhang
Suggested-by: Stuart Summers
---
tests/intel/xe_vm.c | 162 +++-
1 file
Add tests that exercise the new drm_xe_vm_get_property uapi,
specifically for ioctl input validation and for determining that
pagefaults are correctly tracked and returned by the reporter.
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Suggested-by
Add tests to xe_vm that exercise the new DRM_IOCTL_XE_VM_GET_PROPERTY
ioctl. Specifically, add input validation tests that exercise the
return values for improperly formatted ioctl structures.
Signed-off-by: Jonathan Cavitt
---
tests/intel/xe_vm.c | 82
Add a test to xe_vm that determines if pagefaults are correctly tracked
and reported by the DRM_IOCTL_XE_VM_GET_PROPERTY.
Signed-off-by: Jonathan Cavitt
Suggested-by: Jianxun Zhang
Suggested-by: Stuart Summers
---
tests/intel/xe_vm.c | 159 +++-
1 file
Align with 20250401222657.78545-1-jonathan.cav...@intel.com
Add initial declarations for the xe_vm_get_property_ioctl call, including
necessary structures and IOCTL macros.
v2:
- Remove engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Cc: Ivan Briano
---
include/drm-uapi
Add tests that exercise the new drm_xe_vm_get_property uapi,
specifically for ioctl input validation and for determining that
pagefaults are correctly tracked and returned by the reporter.
v2:
- Remove engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
.
Signed-off-by: Jonathan Cavitt
---
lib/xe/xe_ioctl.c | 5 +
lib/xe/xe_ioctl.h | 1 +
2 files changed, 6 insertions(+)
diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c
index 6d83889188..a82714cdfd 100644
--- a/lib/xe/xe_ioctl.c
+++ b/lib/xe/xe_ioctl.c
@@ -232,6 +232,11 @@ void
engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 26 +
drivers/gpu/drm/xe/xe_vm.c | 87
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
---
drivers/
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
of int (Raag)
- Remove unnecessary includes (jcavitt)
v15:
- Do not report faults from reserved engines (Jianxun)
v16:
- Remove engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc
: Jonathan Cavitt
Cc: Zhang Jianxun
Cc: Ivan Briano
---
include/uapi/drm/xe_drm.h | 75 +++
1 file changed, 75 insertions(+)
diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
index 616916985e3f..45f7560879cb 100644
--- a/include/uapi/drm
)
- Assert valid size (jcavitt)
v11:
- Remove unnecessary else condition
- Correct backwards helper function size logic (jcavitt)
v12:
- Use size_t instead of int (Raag)
v13:
- Remove engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc
engine class and instance (Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 26 +
drivers/gpu/drm/xe/xe_vm.c | 87
letter
uAPI: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32987
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag Jadav
Cc: John Harrison
Cc: Ivan Briano
, Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 109 +
drivers/gpu/drm/xe/xe_vm.h | 2 +
3
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
declares for fault type, access type, and fault level (Matt Brost,
Ivan)
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
Cc: Ivan Briano
Cc: Matthew Brost
---
include/uapi/drm/xe_drm.h | 86 +++
1 file changed, 86 insertions(+)
diff --git a/include/uapi/drm
.
Signed-off-by: Jonathan Cavitt
---
lib/xe/xe_ioctl.c | 5 +
lib/xe/xe_ioctl.h | 1 +
2 files changed, 6 insertions(+)
diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c
index fb8c4aef13..dc329c4625 100644
--- a/lib/xe/xe_ioctl.c
+++ b/lib/xe/xe_ioctl.c
@@ -232,6 +232,11 @@ void
,
Ivan)
Signed-off-by: Jonathan Cavitt
Cc: Ivan Briano
---
include/drm-uapi/xe_drm.h | 85 +++
1 file changed, 85 insertions(+)
diff --git a/include/drm-uapi/xe_drm.h b/include/drm-uapi/xe_drm.h
index 154f947ef0..8380508d18 100644
--- a/include/drm-uapi
Add tests to xe_vm that exercise the new DRM_IOCTL_XE_VM_GET_PROPERTY
ioctl. Specifically, add input validation tests that exercise the
return values for improperly formatted ioctl structures.
v2:
- Make vm creation consistent between patches (jcavitt)
Signed-off-by: Jonathan Cavitt
---
tests
Add a test to xe_vm that determines if pagefaults are correctly tracked
and reported by the DRM_IOCTL_XE_VM_GET_PROPERTY.
v2:
- s/pageproperty/pagefaults
Signed-off-by: Jonathan Cavitt
Suggested-by: Jianxun Zhang
Suggested-by: Stuart Summers
---
tests/intel/xe_vm.c | 156
level (Matt Brost,
Ivan)
v4:
- Make vm creation consistent between patches (jcavitt)
- s/pageproperty/pagefaults
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Suggested-by: Zhang Jianxun
Suggested-by: Stuart Summers
CC: Shuicheng Lin
Cc: Ivan Briano
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Reviewed-by: Shuicheng Lin
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 49 +
drivers/gpu/drm/xe
: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag Jadav
Cc: John Harrison
Cc: Ivan Briano
Cc: Matthew Auld
Jonathan Cavitt (5):
drm/xe/xe_gt_pagefault: Disallow writes to read-only
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shu
, Ivan)
v15:
- Remove unnecessary size assertion (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shuicheng Lin
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 107
, Ivan)
v15:
- Remove unnecessary size assertion (jcavitt)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 107
letter
v19:
- Perform kzalloc outside of lock (Auld)
v20:
- Fix inconsistent use of whitespace in defines
v21:
- Remove unnecessary size assertion (jcavitt)
uAPI: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32987
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested
declares for fault type, access type, and fault level (Matt Brost,
Ivan)
v6:
- Fix inconsistent use of whitespace in defines
Signed-off-by: Jonathan Cavitt
Cc: Zhang Jianxun
Cc: Ivan Briano
Cc: Matthew Brost
---
include/uapi/drm/xe_drm.h | 86 +++
1 file
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Reviewed-by: Shuicheng Lin
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 49 +
drivers/gpu/drm/xe
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shu
engine class and instance (Ivan)
v11:
- Perform kzalloc outside of lock (Auld)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
Cc: Ivan Briano
Cc: Matthew Auld
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 26
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Reviewed-by: Shuicheng Lin
Acked-by: Matthew Brost
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 49
The page fault handler should reject write/atomic access to read only
VMAs. Add code to handle this in handle_pagefault after the VMA lookup.
Fixes: 3d420e9fa848 ("drm/xe: Rework GPU page fault handling")
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shu
engine class and instance (Ivan)
v11:
- Perform kzalloc outside of lock (Auld)
v12:
- Fix xe_vm_fault_entry kernel docs (Shuicheng)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shuicheng Lin
Acked-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal
declares for fault type, access type, and fault level (Matt Brost,
Ivan)
v6:
- Fix inconsistent use of whitespace in defines
Signed-off-by: Jonathan Cavitt
Reviewed-by: Shuicheng Lin
Acked-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Ivan Briano
Cc: Matthew Brost
---
include/uapi/drm/xe_drm.h | 86
/-/merge_requests/32987
Signed-off-by: Jonathan Cavitt
Suggested-by: Joonas Lahtinen
Suggested-by: Matthew Brost
Reviewed-by: Shuicheng Lin
Acked-by: Matthew Brost
Cc: Zhang Jianxun
Cc: Shuicheng Lin
Cc: Michal Wajdeczko
Cc: Michal Mrozek
Cc: Raag Jadav
Cc: John Harrison
Cc: Ivan Briano
Cc: Matthew
, Ivan)
v15:
- Remove unnecessary size assertion (jcavitt)
v16:
- Nit fixes (Matt Brost)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Reviewed-by: Shuicheng Lin
Acked-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe
engine class and instance (Ivan)
v11:
- Perform kzalloc outside of lock (Auld)
v12:
- Fix xe_vm_fault_entry kernel docs (Shuicheng)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
Cc: Ivan Briano
Cc
declares for fault type, access type, and fault level (Matt Brost,
Ivan)
v6:
- Fix inconsistent use of whitespace in defines
Signed-off-by: Jonathan Cavitt
Reviewed-by: Shuicheng Lin
Cc: Zhang Jianxun
Cc: Ivan Briano
Cc: Matthew Brost
---
include/uapi/drm/xe_drm.h | 86
engine class and instance (Ivan)
v11:
- Perform kzalloc outside of lock (Auld)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Shuicheng Lin
Cc: Jianxun Zhang
Cc: Michal Wajdeczko
Cc: Michal Mzorek
Cc: Ivan Briano
Cc: Matthew Auld
---
drivers/gpu/drm/xe/xe_gt_pagefault.c | 26
:
- Convert enums to defines in regs folder (Michal W)
- Move xe_guc_pagefault_desc to regs folder (Michal W)
Bspec: 77412
Signed-off-by: Jonathan Cavitt
Cc: Michal Wajdeczko
---
drivers/gpu/drm/xe/regs/xe_pagefault_desc.h | 50 +
drivers/gpu/drm/xe/xe_gt_pagefault.c| 43
, Ivan)
Signed-off-by: Jonathan Cavitt
Suggested-by: Matthew Brost
Cc: Jainxun Zhang
Cc: Shuicheng Lin
Cc: Raag Jadav
Cc: Ivan Briano
---
drivers/gpu/drm/xe/xe_device.c | 3 +
drivers/gpu/drm/xe/xe_vm.c | 109 +
drivers/gpu/drm/xe/xe_vm.h | 2 +
3
1 - 100 of 229 matches
Mail list logo
