Am 02.07.2014 02:29, schrieb Michel D?nzer: > On 02.07.2014 01:46, Dieter N?tzel wrote: >> Am 01.07.2014 10:14, schrieb Michel D?nzer: >>> From: Michel D?nzer <michel.daenzer at amd.com> >>> >>> But move the programming back to the vertical blank interrupt >>> handler. >>> And signal the flip as being completed immediately after programming >>> it >>> to the hardware. >>> >>> This way we don't have to guess whether or not the hardware will >>> execute >>> the flip in a given vertical blank period, avoiding a whole lot of >>> trouble. >>> >>> Also, not using the page flip interrupt anymore avoids problems due >>> to >>> completing page flips earlier than expected by userspace. >>> >>> Signed-off-by: Michel D?nzer <michel.daenzer at amd.com> >> >> Michel, >> >> against which tree is this first one? >> Don't apply clean on 3.16-rc2. >> >> Can't find this in 'my' source (radeon_display.c). >> >>>> radeon_crtc->flip_status = RADEON_FLIP_SUBMITTED; <<< >> spin_unlock_irqrestore(&crtc->dev->event_lock, flags); >> up_read(&rdev->exclusive_lock); > > The patch applies on top of my previous page flipping fix.
Had some trouble with the first one but... After digikam -> OpenGL Viewer (r600g/RV730 AGP) [-] [ 503.736] (II) RADEON(0): Modeline "1920x1080"x60.0 172.80 1920 2040 2248 2576 1080 1081 1084 1118 -hsync +vsync (67.1 kHz e) [ 999.627] (WW) RADEON(0): flip queue failed: Device or resource busy [ 999.627] (WW) RADEON(0): Page flip failed: Device or resource busy Another day ;-)
