#10190 NORM 1.0-sof: suspend during Record activity results in wackier behaviour than before (was: suspend during Record activity results in wacky colours)

[Zarro Boogs per Child]

#10190: suspend during Record activity results in wackier behaviour than before
--------------------------------+-------------------------------------------
           Reporter:  dsd       |       Owner:  dsd                
               Type:  defect    |      Status:  new                
           Priority:  normal    |   Milestone:  1.0-software-update
          Component:  kernel    |     Version:  not specified      
         Resolution:            |    Keywords:                     
        Next_action:  diagnose  |    Verified:  0                  
Deployment_affected:            |   Blockedby:                     
           Blocking:            |  
--------------------------------+-------------------------------------------
Changes (by dsd):

 * cc: corbet (added)


Comment:

 OK, I got sidetracked with looking at the video driver.
 There are 2 bugs at hand:

 The psychadelic colours after resume is a bug in lxfb or something. Xv
 video goes funky if you suspend/resume while it happens. Filed as #10200

 The regression to this bug, in that on F11 images the colours do not
 return to normal even after moving the mouse is due to a bug in cafe_ccic
 (the colours go from psychadelic to pinky/bluey). Simple test case to
 reproduce, avoiding the Xv bug by using regular X11 rendering:

 {{{
 stop prefdm
 xinit /usr/bin/gst-launch v4l2src ! ffmpegcolorspace ! ximagesink
 }}}

 With the camera feed on-screen, close the lid to suspend the laptop. Open
 it again, and observe pinky/bluey colours.

 The cause of this regression is that this commit was never sent upstream
 or carried forward:
 http://dev.laptop.org/git/olpc-2.6/commit/?h=testing&id=6db90c293d78cfa7bbbcc6fc035e856a491eaec3
 "cafe_ccic: fix up locking during suspend (hopefully..)"

 I suspect the core part of the commit is the fact that we don't
 temporarily power down the camera during resume if someone is using it.

 I'm going to forward port the commit to the 2.6.31 kernel. Jon, is this
 suitable for upstream, or can you suggest an alternate version?

-- 
Ticket URL: <http://dev.laptop.org/ticket/10190#comment:2>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system