Re: [PATCH 3/3] drm: do not sleep on vblank while holding a mutex

On Don, 2011-10-27 at 22:19 -0500, Ilija Hadzic wrote:

On Thu, 27 Oct 2011, Daniel Vetter wrote:

...

So I think the right thing to do is

• Kill dev->last_vblank_wait (in a prep patch).

Agreed. Also drm_vblank_info function can go away

Actually, I was rather going to submit a patch to hook it up again — AFAICT it was unhooked without any justification. It could be useful for debugging vblank related hangs. Any issues with it, such as last_vblank_wait not being guaranteed to really be the last one, can always be improved later on.