[PATCH i-g-t 1/3] lib/xe/xe_spin: Update xe_spin with exec_sync

[Zbigniew Kempczyński]

On Wed, Mar 20, 2024 at 09:24:28PM +0530, sai.gowtham.ch at intel.com wrote:
> From: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
> 
> When a spinner is submitted with LR mode, exec_sync can be used
> as GPU address when sync is passed in via exec IOCTL.
> 
> Cc:Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Signed-off-by: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
> ---
>  lib/xe/xe_spin.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/xe/xe_spin.h b/lib/xe/xe_spin.h
> index 5c8c45143..87b7c1716 100644
> --- a/lib/xe/xe_spin.h
> +++ b/lib/xe/xe_spin.h
> @@ -38,6 +38,7 @@ struct xe_spin {
>  	uint32_t start;
>  	uint32_t end;
>  	uint32_t ticks_delta;
> +	uint64_t exec_sync;

Agree, you need userfence variable gpu may write into on the completion.
It would be good to add userfence path to xe_spin (via some opts) but
it is not required at the moment. So:

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

>  };
>  
>  igt_spin_t *xe_spin_create(int fd, const struct igt_spin_factory *opt);
> -- 
> 2.39.1
>