Quoting Tvrtko Ursulin (2019-04-02 17:09:18) > > On 25/03/2019 09:03, Chris Wilson wrote: > > @@ -2120,20 +2150,20 @@ eb_select_engine(struct drm_i915_private *dev_priv, > > } else { > > DRM_DEBUG("execbuf with unknown bsd ring: %u\n", > > bsd_idx); > > - return NULL; > > + return -EINVAL; > > } > > > > - engine = dev_priv->engine[_VCS(bsd_idx)]; > > + engine = eb->i915->engine[_VCS(bsd_idx)]; > > } else { > > - engine = dev_priv->engine[user_ring_map[user_ring_id]]; > > + engine = eb->i915->engine[user_ring_map[user_ring_id]]; > > Cache dev_priv/i915 in a local? i915_gem_do_execbuffer 4742 4740 -2 Barely seems worth it this time. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx