Re: [PATCH v16 3/5] drm/xe/uapi: Define drm_xe_vm_get_property

2025-04-02 Thread Jianxun Zhang
On 4/2/25 18:19, Jianxun Zhang wrote: On 4/1/25 15:26, Jonathan Cavitt wrote: 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

Re: [PATCH v16 3/5] drm/xe/uapi: Define drm_xe_vm_get_property

2025-04-02 Thread Jianxun Zhang
On 4/1/25 15:26, Jonathan Cavitt wrote: 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) v4: - Remove engine

Re: [PATCH v14 6/6] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-27 Thread Jianxun Zhang
Jonathan, I triggered several page faults by hacking UMD and can see multiple faults from 4 engines are generated and reported from KMD with this series (patchwork rev15). Although the hack cannot cover every case, I belive the new ioctl interface is working now. I will let you know any findi

Re: [PATCH v13 6/6] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-25 Thread Jianxun Zhang
On 3/25/25 15:27, Jonathan Cavitt wrote: Add support for userspace to request a list of observed faults from a specified VM. v2: - Only allow querying of failed pagefaults (Matt Brost) v3: - Remove unnecessary size parameter from helper function, as it is a property of the arguments. (jca

Re: [PATCH v12 5/5] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-25 Thread Jianxun Zhang
On 3/24/25 16:09, Jonathan Cavitt wrote: Add support for userspace to request a list of observed faults from a specified VM. v2: - Only allow querying of failed pagefaults (Matt Brost) v3: - Remove unnecessary size parameter from helper function, as it is a property of the arguments. (jca

Re: [PATCH v8 6/6] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-19 Thread Jianxun Zhang
On 3/13/25 11:34, Jonathan Cavitt wrote: Add support for userspace to request a list of observed faults from a specified VM. v2: - Only allow querying of failed pagefaults (Matt Brost) v3: - Remove unnecessary size parameter from helper function, as it is a property of the arguments. (jca

Re: [PATCH v9 3/6] drm/xe/uapi: Define drm_xe_vm_get_property

2025-03-19 Thread Jianxun Zhang
On 3/19/25 14:13, Cavitt, Jonathan wrote: -Original Message- From: Zhang, Jianxun Sent: Wednesday, March 19, 2025 2:02 PM To: Cavitt, Jonathan ; intel...@lists.freedesktop.org Cc: Gupta, saurabhg ; Zuo, Alex ; joonas.lahti...@linux.intel.com; Brost, Matthew ; Lin, Shuicheng ; dri-de

Re: [PATCH v9 3/6] drm/xe/uapi: Define drm_xe_vm_get_property

2025-03-19 Thread Jianxun Zhang
On 3/18/25 10:11, Jonathan Cavitt wrote: 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 ++

Re: [PATCH v8 6/6] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-18 Thread Jianxun Zhang
On 3/18/25 11:12, Cavitt, Jonathan wrote: -Original Message- From: Zhang, Jianxun Sent: Tuesday, March 18, 2025 10:48 AM To: Cavitt, Jonathan ; intel...@lists.freedesktop.org Cc: Gupta, saurabhg ; Zuo, Alex ; joonas.lahti...@linux.intel.com; Brost, Matthew ; Lin, Shuicheng ; dri-dev

Re: [PATCH v8 6/6] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-03-18 Thread Jianxun Zhang
On 3/13/25 11:34, Jonathan Cavitt wrote: Add support for userspace to request a list of observed faults from a specified VM. v2: - Only allow querying of failed pagefaults (Matt Brost) v3: - Remove unnecessary size parameter from helper function, as it is a property of the arguments. (jca

Re: [PATCH v8 3/6] drm/xe/uapi: Define drm_xe_vm_get_property

2025-03-13 Thread Jianxun Zhang
On 3/13/25 11:34, Jonathan Cavitt wrote: Add initial declarations for the drm_xe_vm_get_property ioctl. Signed-off-by: Jonathan Cavitt --- include/uapi/drm/xe_drm.h | 69 +++ 1 file changed, 69 insertions(+) diff --git a/include/uapi/drm/xe_drm.h b/inc