[PATCH] drm/xe/uapi: Correct sync type definition in comments
Lin, Shuicheng
shuicheng.lin at intel.com
Sun Jun 8 23:06:38 UTC 2025
Hi Lucas,
Thanks very much for the detail check. I really appreciate it. And will follow it for my future patches.
Shuicheng
On Sat, June 7, 2025 1:10 PM De Marchi, Lucas wrote:
> On Sat, Jun 07, 2025 at 02:34:42AM +0000, Shuicheng Lin wrote:
> >DRM_XE_SYNC_FLAG_SYNCOBJ is renamed to DRM_XE_SYNC_TYPE_SYNCOBJ.
> >DRM_XE_SYNC_FLAG_TIMELINE_SYNCOBJ is renamed to
> DRM_XE_SYNC_TYPE_TIMELINE_SYNCOBJ.
> >DRM_XE_SYNC_FLAG_DMA_BUF is removed.
>
> This can be seen by the diff. Here it'd be better to explain why, not
> what:
>
> >
> >Fixes: dc787f8931af ("drm/xe/uapi: Split xe_sync types from flags")
>
> this is not really correct. The rename happened in that commit, but the commits
> using the old names came later. So I think the commit message should be
> something like this:
>
> Commit dc787f8931af ("drm/xe/uapi: Split xe_sync types from flags") renamed
> some DRM_XE_SYNC_* defines but later commits kept using the old names.
>
>
> Fixes: 9329f0667215 ("drm/xe/uapi: Use LR abbrev for long-running vms")
> Fixes: 4b437893a826 ("drm/xe/uapi: More uAPI documentation additions and
> cosmetic updates")
>
> with that, Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>
>
>
> thanks
> Lucas De Marchi
>
> >Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> >Cc: Francois Dugast <francois.dugast at intel.com>
> >Cc: Zongyao Bai <zongyao.bai at intel.com>
> >Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>
> >---
> > include/uapi/drm/xe_drm.h | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> >diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> >index 6a702ba7817c..5f1524f466a7 100644
> >--- a/include/uapi/drm/xe_drm.h
> >+++ b/include/uapi/drm/xe_drm.h
> >@@ -925,9 +925,9 @@ struct drm_xe_gem_mmap_offset {
> > * - %DRM_XE_VM_CREATE_FLAG_LR_MODE - An LR, or Long Running VM
> accepts
> > * exec submissions to its exec_queues that don't have an upper time
> > * limit on the job execution time. But exec submissions to these
> >- * don't allow any of the flags DRM_XE_SYNC_FLAG_SYNCOBJ,
> >- * DRM_XE_SYNC_FLAG_TIMELINE_SYNCOBJ,
> DRM_XE_SYNC_FLAG_DMA_BUF,
> >- * used as out-syncobjs, that is, together with DRM_XE_SYNC_FLAG_SIGNAL.
> >+ * don't allow any of the sync types DRM_XE_SYNC_TYPE_SYNCOBJ,
> >+ * DRM_XE_SYNC_TYPE_TIMELINE_SYNCOBJ, used as out-syncobjs, that is,
> >+ * together with sync flag DRM_XE_SYNC_FLAG_SIGNAL.
> > * LR VMs can be created in recoverable page-fault mode using
> > * DRM_XE_VM_CREATE_FLAG_FAULT_MODE, if the device supports it.
> > * If that flag is omitted, the UMD can not rely on the slightly
> >@@ -1394,7 +1394,7 @@ struct drm_xe_sync {
> >
> > /**
> > * @timeline_value: Input for the timeline sync object. Needs to be
> >- * different than 0 when used
> with %DRM_XE_SYNC_FLAG_TIMELINE_SYNCOBJ.
> >+ * different than 0 when used
> with %DRM_XE_SYNC_TYPE_TIMELINE_SYNCOBJ.
> > */
> > __u64 timeline_value;
> >
> >--
> >2.25.1
> >
More information about the Intel-xe
mailing list