Re: [PATCH igt] igt/gem_spin_batch: Skip overloading aliased BSD engines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Tvrtko Ursulin (2017-12-21 10:02:06)
> 
> On 20/12/2017 17:56, Chris Wilson wrote:
> > BSD == BSD1 or BSD2. Since we already emit spinners to the explicit BSD
> > rins, skip the aliased ring.
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104352
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > ---
> >   tests/gem_spin_batch.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tests/gem_spin_batch.c b/tests/gem_spin_batch.c
> > index 896311304..cccba75a7 100644
> > --- a/tests/gem_spin_batch.c
> > +++ b/tests/gem_spin_batch.c
> > @@ -77,7 +77,7 @@ static void spin_on_all_engines(int fd, unsigned int timeout_sec)
> >       unsigned engine;
> >   
> >       for_each_engine(fd, engine) {
> > -             if (engine == 0)
> > +             if (engine == 0 || engine == I915_EXEC_BSD)
> 
> You forget the other annoyance of the VCS selection uAPI where explicit 
> flags can only be used on dual-VCS platforms? :) So I think you need to 
> skip on engine & I915_EXEC_BSD_RING1, unless I am missing something.

No way, the uapi can't be that stupid. It is.

Can we do the s/for_each_engine/for_each_ring/ yet?
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux