On 26.06.2014 19:39, Christian K?nig wrote: > Am 26.06.2014 11:29, schrieb Michel D?nzer: >> From: Michel D?nzer <michel.daenzer at amd.com> >> >> Prevents radeon_crtc_handle_flip() from running before >> radeon_flip_work_func(), resulting in a kernel panic due to the BUG_ON() >> in drm_vblank_put(). >> >> Tested-by: Dieter N?tzel <Dieter at nuetzel-hh.de> >> Signed-off-by: Michel D?nzer <michel.daenzer at amd.com> > > Does patch #2 alone fixes the problem as well?
It should avoid the panic as well. > I would rather want to avoid turning the pflip interrupt on and off. What's the problem with that? It's not like we're saving any register writes by not doing it. The diagnostic messages Dieter was getting with only patch #2 show that the pflip interrupt often triggers unnecessarily, potentially wasting power by waking up the CPU from a power saving state pointlessly. -- Earthling Michel D?nzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
