On 08/03/2019 17:11, Chris Wilson wrote:
Quoting Tvrtko Ursulin (2019-03-08 16:31:51)
Looks okay. But one more thing is needed:

https://cgit.freedesktop.org/~tursulin/drm-intel/commit/?h=media&id=38266bfe99469de9e13774a13fa641c377988c67

drm/i915: Allow SSEU configuration to be set on virtual engine

        /* Only render engine supports RPCS configuration. */
-       if (engine->class != RENDER_CLASS)
+       if (engine->class != RENDER_CLASS &&
+           !(engine->flags & I915_ENGINE_IS_VIRTUAL &&
+             ctx->engines[1]->class == RENDER_CLASS))
                return -ENODEV;

A virtual engine composed of RCS engines would have
engine->class == RENDER_CLASS.

So it's just the engine->id BUG_ON that needs lifting?

If so then yes. Must be a recent change since I was sure the class was set to other before.

Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to