[igt-dev] [PATCH i-g-t 1/2] lib/xe/xe_spin: Integrate igt_spin_new with Xe.

[Zbigniew Kempczyński]

On Thu, Jun 01, 2023 at 07:59:49AM +0000, Kumar, Janga Rahul wrote:

<cut>

> > >  typedef struct igt_spin {
> > > @@ -83,6 +85,14 @@ typedef struct igt_spin {  #define SPIN_CLFLUSH (1
> > > << 0)
> > >
> > >  	struct igt_spin_factory opts;
> > > +
> > > +	struct xe_spin *xe_spin;
> > > +	size_t bo_size;
> > > +	uint64_t address;
> > > +	unsigned int engine;
> > > +	uint32_t vm;
> vm defined inside spin_factory as well, can we avoid this if it is redundant

I think it can be useful to assign external vm or create a new one in it.
Then on free path you're destroying only vm you own.

--
Zbigniew
> 
> Thanks,
> Rahul
> 
> > > +	uint32_t syncobj;