Quoting Tvrtko Ursulin (2018-09-20 15:01:38) > > On 19/09/2018 20:55, Chris Wilson wrote: > > As we emit requests or touch HW directly for some of the live tests, the > > requirement is that we hold the rpm wakeref before doing so. We want a > > mix of granularity since we will want to test runtime suspend, so try to > > mark up only the critical sections where we need rpm for the live test. > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > --- > Should we have a rpm assert in i915_request_alloc? The intel_runtime_pm_get_noresume() we use there complains if we don't have an intel rpm wakeref. > Or at least in the > backend context pin if that's the only place where we can need it. Yup, it spits one out when we touch HW without a wakeref. Just it seems that wakerefs abound and we rarely catch ourselves out ;) -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
