Re: [igt-dev] [PATCH i-g-t 15/21] gem_wsim: Engine bond command

Tvrtko Ursulin
2019-05-13T13:28:15Z

On 10/05/2019 14:36, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2019-05-08 13:10:52)
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>
>> Engine bonds are an i915 uAPI applicable to load balanced contexts with
>> engine map. They allow expression rules of engine selection between two
>> contexts when submissions are also tied with submit fences.
>>
>> Please refer to the README for a more detailed description.
>
> I would prefer not to have a hexadecimal mask in the file format? That's
> harder than usual to read later on.
>
> bond({master_class:master_instance}, {engine_class:engine_instance}),...
> ?

I agree hexadecimal mask is bad. I'll try to see quickly if piggying
back to some existing list of engines processing could be done cheaply
(quickly). And without adding more horror to the file parsing loop.

Regards,

Tvrtko