Hi Lucas, > > > > The following interfaces: > > > > > > > > i915_wedged > > > > i915_forcewake_user > > > > > > > > are dependent on gt values. Put them inside gt/ and drop the > > > > "i915_" prefix name. This would be the new structure: > > > > > > > > dri/0/gt > > > > | > > > > +-- forcewake_user > > > > | > > > > \-- reset > > > > > > > > For backwards compatibility with existing igt (and the slight > > > > semantic difference between operating on the i915 abi entry > > > > points and the deep gt info): > > > > > > > > dri/0 > > > > | > > > > +-- i915_wedged > > > > | > > > > \-- i915_forcewake_user > > > > > > > > remain at the top level. > > > > > > > > Signed-off-by: Andi Shyti <andi.sh...@linux.intel.com> > > > > Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > > > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > > > > Reviewed-by: Lucas De Marchi <lucas.demar...@intel.com> > > > > > > do you want me to push this? > > > > yes, please. > > done, thanks.
Thanks! > Now, about igt: eventually we need to update it to use the gt > debugfs file. Is this something you have already or is it something > we are waiting on multi-gt to land? There is some work done in igt but it's all around multitile. I think it's better to wait for the multitile to land and then update igt. In any case, at the current state, igt shouldn't be affected. Thanks again, Andi
