On Wed, May 07, 2025 at 12:32:49PM -0700, Cavitt, Jonathan wrote: > -----Original Message----- > From: Vivi, Rodrigo <[email protected]> > Sent: Wednesday, May 7, 2025 12:24 PM > To: Cavitt, Jonathan <[email protected]> > Cc: [email protected]; Gupta, saurabhg > <[email protected]>; Zuo, Alex <[email protected]>; > [email protected]; Brost, Matthew <[email protected]>; > Zhang, Jianxun <[email protected]>; Lin, Shuicheng > <[email protected]>; [email protected]; Wajdeczko, Michal > <[email protected]>; Mrozek, Michal <[email protected]>; > Jadav, Raag <[email protected]>; Harrison, John C > <[email protected]>; Briano, Ivan <[email protected]>; Auld, > Matthew <[email protected]>; [email protected] > Subject: Re: [PATCH v24 3/5] drm/xe/uapi: Define drm_xe_vm_get_property > > > > On Wed, May 07, 2025 at 03:57:24PM +0000, 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 class and instance (Ivan) > > > > > > v5: > > > - Add 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 <[email protected]> > > > Reviewed-by: Shuicheng Lin <[email protected]> > > > Acked-by: Matthew Brost <[email protected]> > > > Cc: Zhang Jianxun <[email protected]> > > > Cc: Ivan Briano <[email protected]> > > > Cc: Matthew Brost <[email protected]> > > > > We are already in the v25 of this series and we still don't have the > > userspace > > PR/MR to reference here and no ack from the user space? > > The userspace PR/MR is in the cover letter: > """ > uAPI: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/32987 > """ > > > > > Sometimes faster series iterations are counterintuitively slowing things > > down. > > Who should I be messaging to get userspace acks from? > -Jonathan Cavitt >
Acked-by: Ivan Briano <[email protected]> Do note that the MR you point at was closed for reasons, though we can certainly just reopen it and update it to the UAPI changes since then. But the actual work to get rid of the scratch page that is a pre-requisite to enable this extension is not done yet and will still take a bit. We do need the page fault reports for it though.
