Bug 46425

Summary: assert_fail in sna_pixmap_move_area_to_gpu()
Product: xorg Reporter: Joe Nahmias <joe>
Component: Driver/intelAssignee: Chris Wilson <chris>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: git   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Backtrace while running with --enable-debug showing assert_fail()
none
Another backtrace none

Description Joe Nahmias 2012-02-21 19:56:56 UTC
Created attachment 57431 [details]
Backtrace while running with --enable-debug showing assert_fail()

Well, it looks like we closed #46346 a bit prematurely.  I got another assert_fail in sna_pixmap_move_area_to_gpu().  Backtrace is attached.

Feel free to merge this with 46346 if that's how you'd like me to handle it in the future.

Thanks,
--Joe
Comment 1 Joe Nahmias 2012-02-21 20:26:59 UTC
Created attachment 57432 [details]
Another backtrace

Here's another backtrace showing a similar (to me) assert_fail.
Comment 2 Chris Wilson 2012-02-22 01:11:42 UTC
It is a subtly different assertion failure. On reflection, it looks like I relaxed an invariant that the assertion is checking...
Comment 3 Chris Wilson 2012-02-22 01:18:08 UTC
commit b02f866d67d60538368619849f2acda4c1706476
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Feb 22 09:15:25 2012 +0000

    sna: Ensure we restore the shadow pixels before uploading CPU data
    
    Reported-by: Joe Nahmias <joe@nahmias.net>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=46425
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

Next! ;)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.