Am 27.06.2014 03:03, schrieb Michel D?nzer: > On 27.06.2014 09:53, Dieter N?tzel wrote: >> Am 26.06.2014 12:39, schrieb Christian K?nig: >>> 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? >> >> With #2 alone I get this during boot up (before plymouth): > > [...] > >> [ 15.259867] [drm:radeon_crtc_handle_flip] *ERROR* >> radeon_crtc->flip_status = 0 != RADEON_FLIP_SUBMITTED(2) > > That's the original patch I sent to you along with two others for > testing. The patch I submitted in this series has these messages > downgraded to debugging messages, as they just show the patch > preventing > bad stuff from happening as designed. > > The question is, can you reproduce the panic or the 'impossible msc' > lines in the Xorg log with only patch #2?
Oh, shit sorry. I'll redo in a minute before I went to bed! -Dieter >> But with Michel's #1+2 and 3 I got this in Xorg.0.log: >> (See Xorg.0.log.old.xz) >> >> (EE) [mi] EQ overflowing. Additional events will be discarded until >> existing events are processed. > > That may not be directly related to the page flipping issues.