[igt-dev] [PATCH v24 04/14] lib: igt_gt: make gem_engine_can_store_dword() check engine class

[Andi Shyti]

Hi Chris,

On Tue, May 14, 2019 at 10:07:57AM +0100, Chris Wilson wrote:
> Quoting Andi Shyti (2019-05-13 18:56:03)
> > -bool gem_can_store_dword(int fd, unsigned int engine)
> > +bool gem_class_can_store_dword(int fd, int class)
> >  {
> >         uint16_t devid = intel_get_drm_devid(fd);
> >         const struct intel_device_info *info = intel_get_device_info(devid);
> > @@ -569,8 +569,8 @@ bool gem_can_store_dword(int fd, unsigned int engine)
> >         if (gen == 3 && (info->is_grantsdale || info->is_alviso))
> >                 return false; /* only supports physical addresses */
> >  
> > -       if (gen == 6 && ((engine & 0x3f) == I915_EXEC_BSD))
> > -               return false; /* kills the machine! */
> > +       if (gen == 6 && class == I915_ENGINE_CLASS_VIDEO)
> > +               return false;
> 
> Probably wise to keep the note about why it is excluded. It's a nasty
> surprise.

I have another patch ready on this one with one of your Suggested-by
that I will send later in a different patchset.

Andi