[PATCH v9 2/8] drm/xe/uapi: Introduce API for EU stall sampling
Dixit, Ashutosh
ashutosh.dixit at intel.com
Mon Feb 10 23:53:26 UTC 2025
On Mon, 10 Feb 2025 15:07:51 -0800, Dixit, Ashutosh wrote:
>
> > +
> > + ret = xe_eu_stall_user_extensions(xe, data, &props);
> > + if (ret)
> > + return ret;
> > +
> > + if (!props.gt) {
> > + drm_dbg(&xe->drm, "GT ID not provided for EU stall sampling\n");
> > + return -EINVAL;
> > + }
>
> This check is not needed. props.gt cannot be NULL the way
> set_prop_eu_stall_gt_id is implemented.
>
> If you need it, use xe_assert(props->gt) in set_prop_eu_stall_gt_id.
Sorry, this is needed in case gt_id is not provided by userland. Unless
default gt_id can be 0, in which case this is not needed?
More information about the Intel-xe
mailing list