On Fri, Feb 28, 2014 at 08:44:45PM -0300, Rodrigo Vivi wrote: > Baytrail cannot easily detect screen updates and force PSR exit. > So we inactivate it on {busy_ioctl, set_domain, sw_finish and mark_busy} > and update to enable it back it later with a delayed workqueue.
Why are we not checking if the object being accessed is indeed being used for PSR? In set-domain, you only care about writes. sw-finish and busy are too late for psr_exit, the damage has already been done and presumably the content may already be corrupted? Can you please explain that it is safe to do an psr_exit after the damage is already in the scanount based on the panel refresh cycle. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx