Re: [PATCH v5 7/9] drm/xe/uapi: Add DRM_XE_EXEC_QUEUE_SET_HANG_REPLAY_STATE

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Nov 26, 2025 at 08:44:51PM +0000, Souza, Jose wrote:
> On Wed, 2025-11-26 at 10:59 -0800, Matthew Brost wrote:
> > Add DRM_XE_EXEC_QUEUE_SET_HANG_REPLAY_STATE which accepts a user
> > pointer
> > to populate the exec queue state so that a GPU hang can be replayed
> > via
> > a Mesa tool.
> > 
> > v2: Update the value for HANG_REPLAY_STATE flag
> > 
> 
> Acked-by: José Roberto de Souza <jose.souza@intel.com>

Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> 
> > Cc: José Roberto de Souza <jose.souza@intel.com>
> > Signed-off-by: Matthew Brost <matthew.brost@intel.com>
> > Signed-off-by: Carlos Santa
> > <carlos.santa@intel-corp-partner.google.com>
> > Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> > ---
> >  include/uapi/drm/xe_drm.h | 9 +++++++--
> >  1 file changed, 7 insertions(+), 2 deletions(-)
> > 
> > diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> > index 47853659a705..37881b1eb6ba 100644
> > --- a/include/uapi/drm/xe_drm.h
> > +++ b/include/uapi/drm/xe_drm.h
> > @@ -210,8 +210,12 @@ struct drm_xe_ext_set_property {
> >  	/** @pad: MBZ */
> >  	__u32 pad;
> >  
> > -	/** @value: property value */
> > -	__u64 value;
> > +	union {
> > +		/** @value: property value */
> > +		__u64 value;
> > +		/** @ptr: pointer to user value */
> > +		__u64 ptr;
> > +	};
> >  
> >  	/** @reserved: Reserved */
> >  	__u64 reserved[2];
> > @@ -1292,6 +1296,7 @@ struct drm_xe_exec_queue_create {
> >  #define   DRM_XE_EXEC_QUEUE_SET_PROPERTY_PRIORITY		0
> >  #define   DRM_XE_EXEC_QUEUE_SET_PROPERTY_TIMESLICE		1
> >  #define   DRM_XE_EXEC_QUEUE_SET_PROPERTY_PXP_TYPE		2
> > +#define   DRM_XE_EXEC_QUEUE_SET_HANG_REPLAY_STATE		3
> >  	/** @extensions: Pointer to the first extension struct, if
> > any */
> >  	__u64 extensions;
> >