Bug 43607

Summary: [SNB] Unity don't have shadows with SNA enabled
Product: xorg Reporter: Victor Machado <machado.prx>
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
With SNA enabled
none
With SNA disabled
none
Xorg log none

Description Victor Machado 2011-12-07 17:00:06 UTC 
There are no shadows in the windows when ""Sandy Bridge New Acceleration"" is enabled, with UXA acceleration the problem does not appear.

Hardware:
Processor: Intel Core i7-2630QM @ 2.00GHz (8 Cores), Chipset: Intel 2nd Generation Core Family, Memory: 4096MB, Graphics: Intel 2nd Generation Core Family IGP
Software:
OS: Ubuntu 11.10, Kernel: 3.2.0-3-generic (x86_64), Desktop: Unity 4.24.0, Display Server: X Server 1.11.2.901 (1.11.3 RC 1), Display Driver: intel 2.17.0, OpenGL: 2.1 Mesa 7.12-devel
Comment 1 Chris Wilson 2011-12-07 17:55:11 UTC 
Please include a couple of snapshots and you Xorg.log so that I have some clue as to what you mean.
Comment 2 Victor Machado 2011-12-07 18:39:11 UTC 
Created attachment 54202 [details]
With SNA enabled

Screenshot with SNA enabled
Comment 3 Victor Machado 2011-12-07 18:40:25 UTC 
Created attachment 54203 [details]
With SNA disabled

Screenshot with SNA disabled
Comment 4 Victor Machado 2011-12-07 18:43:12 UTC 
Created attachment 54204 [details]
Xorg log

Xorg log
Comment 5 Chris Wilson 2011-12-08 04:38:06 UTC 
That was a slightly more subtle bug than the one I was thinking of:

commit 874c722c86b06ac0aa10e6943d3faf76fcb751be
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Thu Dec 8 12:07:28 2011 +0000

    sna: Beware flushing partial buffers before they are written
    
    A partial buffer is considered finished upon the next batch submission,
    so one needs to be careful that it is completely written to before such
    an event is triggered. move-to-cpu is such a trigger as demonstrated by
    the picture fixup routine for handling convolution filters.
    
    Reported-by: Victor Machado <machado.prx@gmail.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43607
    Signed-off-by: Chris Wilson <ickle@crestline.(none)>
Comment 6 Victor Machado 2011-12-08 04:50:53 UTC 
Very thanks Chris, the problem was resolved.

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.