Quoting Chris Wilson (2018-07-14 18:36:03) > From: Michał Winiarski <[email protected]> > > We're seeing "RPM wakelock ref not held during HW access" warning > otherwise. Since IRQs are synced for runtime suspend we can just disable > the wakeref asserts. > > Reported-by: Marta Löfstedt <[email protected]> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105710 > Signed-off-by: Michał Winiarski <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Michal Wajdeczko <[email protected]> > --- > drivers/gpu/drm/i915/intel_guc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_guc.c > b/drivers/gpu/drm/i915/intel_guc.c > index e12bd259df17..560c7406ae40 100644 > --- a/drivers/gpu/drm/i915/intel_guc.c > +++ b/drivers/gpu/drm/i915/intel_guc.c > @@ -466,11 +466,13 @@ void intel_guc_to_host_event_handler_mmio(struct > intel_guc *guc) > * could happen that GuC sets the bit for 2nd interrupt but Host > * clears out the bit on handling the 1st interrupt. > */
GEM_BUG_ON(!in_interrupt()); /* may be overkill */ > + disable_rpm_wakeref_asserts(dev_priv); > spin_lock(&guc->irq_lock); > val = I915_READ(SOFT_SCRATCH(15)); > msg = val & guc->msg_enabled_mask; > I915_WRITE(SOFT_SCRATCH(15), val & ~msg); > spin_unlock(&guc->irq_lock); > + enable_rpm_wakeref_asserts(dev_priv); > > intel_guc_to_host_process_recv_msg(guc, msg); Reviewed-by: Chris Wilson <[email protected]> -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
