From: Timo A. <tja...@cc...> - 2009-01-30 22:57:59
|
On Fri, 30 Jan 2009, Jesse Barnes wrote: > On Friday, January 30, 2009 11:16 am Timo Aaltonen wrote: >> On Fri, 30 Jan 2009, Jesse Barnes wrote: >>> I'm not sure about Timo's problem though; this patch helps with a problem >>> he was seeing too, but he sees it after running for awhile, not at VT >>> switch. It's possible that his problem could be due to wraparound, if the >>> jumps we see at interrupt off->on time are big enough, a few DPMS events >>> could cause a wraparound related hang. Timo, do you have a reliable way >>> of reproducing your problem? It sounds separate from 18041 so we may >>> want to fix it differently. >> >> I understood that 18041 was already fixed by the recent'ish commits to >> drm, and I haven't been able to reproduce that since they were added to >> the jaunty kernel (and vblank was again enabled in mesa). >> >> I can reproduce the momentary freezes after changing the VT a couple of >> times. That's what happens on a suspend/resume cycle as well, so it's >> probably what triggered it in the first place. And the patch you posted >> did fix this issue for me. > > Ah ok, then it probably is the same issue. Sounds like you're just hitting > the timeout in libdrm, which would cause apps to freeze for about a second > after a VT switch. I think the sequence number the apps are waiting for is > actually less than the current one, but our current code prevents that from > being detected. Does this patch also fix things for you? Yes it did. t |