From: Dave A. <ai...@gm...> - 2010-02-18 08:16:12
|
2010/2/18 Rafał Miłecki <za...@gm...>: > 2010/2/18 Dave Airlie <ai...@gm...>: >> From: Dave Airlie <ai...@re...> >> >> This patch adds a check on avivo chips to see if we are in the VBL >> region for the active crtcs when we trigger the engine change. >> >> I appear to have glitches locally on pm transistion (not sure all >> fixes are in yet) and this at least seems to be correct here, >> maybe others can test on systems with no glitches. > > Because we are totally out of sync with VBLANK. If we correctly sync > with it, it's just a lucky case. Please check my > [PATCH] drm/radeon/kms: really wait for VBLANK in PM code > and reply in thread where I posted it. Yeah I'll put this in when I've ran it here, it looks sane. > It adds some reading & printing steps before every reclock, while we > really want it to happen as soon as possible. Maybe you could execute > this only on some > #ifdef RADEON_PM_DEBUG I don't think it'll be a major overhead, though doing it under a debug would be fine, but it would be nice to make it into a WARN_ON so we could track where this actually happens for ppl with something like kerneloops. Dave. |