Re: [PATCH 07/17] drm/xe/oa/uapi: Define and parse OA stream properties

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Thu, 07 Dec 2023 22:43:19 -0800, Ashutosh Dixit wrote:
>
> +/**
> + * struct drm_xe_oa_open_param - Params for opening an OA stream
> + *
> + * Stream params are specified as a chain of @drm_xe_ext_set_property
> + * struct's, with @property values from enum @drm_xe_oa_property_id and
> + * @xe_user_extension base.name set to @DRM_XE_OA_EXTENSION_SET_PROPERTY
> + */
> +struct drm_xe_oa_open_param {
> +#define DRM_XE_OA_EXTENSION_SET_PROPERTY	0
> +	/** @extensions: Pointer to the first extension struct */
> +	__u64 extensions;
> +};

Harish pointed out that this struct is not really needed. We might as well
point the @param field in struct @drm_xe_perf_param to the first
@drm_xe_ext_set_property struct. I'll make this change in the next
revision. Thanks Harish.