On Fri, Jan 08, 2016 at 03:51:20PM +0200, Mika Kuoppala wrote:
> If we go into suspend with unclaimed access detected,
> it would be nice to catch that access on a next suspend path.
> So instead of just notifying about it, arm the unclaimed
> mmio checks on suspend side.
> 
> We want to keep the asymmetry on resume, as if it was
> on resume path, it was not driver that is responsible so
> no point in arming mmio debugs.
> 
> Cc: Chris Wilson <[email protected]>
> Signed-off-by: Mika Kuoppala <[email protected]>

Reviewed-by: Chris Wilson <[email protected]>

I actually even like the extra DRM_ERROR here. Driver unload is another
candidate for

  if (intel_uncore_unclaimed_mmio(dev_priv))
        DRM_ERROR("Unclaimed access detected prior to unload\n");

Might as well tickle the CI if nothing else.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to