Bug 39538

Summary: [g45 sna] tearing with partial screen updates
Product: xorg Reporter: Achim Frase <achim.frase>
Component: Driver/intelAssignee: Chris Wilson <chris>
Status: RESOLVED MOVED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: medium CC: kokoko3k, tomi
Version: git   
Hardware: Other   
OS: All   
i915 platform: i915 features:
Description Flags
systeminformation none

Description Achim Frase 2011-07-26 01:15:53 UTC
I have noticed since I use SNA that partial screen updates have sometimes tearing.

For example when moving windows, or in videos that are displayed in a window.

I think Chris Wilson was talking about that with Dave Airlie but I don't remember what was the conclusion.

With UXA there was no tearing visible.
Comment 1 Achim Frase 2011-07-26 01:17:46 UTC
Created attachment 49553 [details]
Comment 2 Chris Wilson 2011-08-26 14:27:26 UTC
It's a bone of contention in that the specification does not state whether those updates should or should not be synchronous with screen updates and the dri2 code calls the same function from a code path that is supposed to be asynchronous. The change is trivial and if the xserver patches ever land then there is no confusion either...
Comment 3 Tomas Janousek 2011-08-26 14:51:22 UTC
Which xserver patches?
Comment 4 Eugeni Dodonov 2011-09-02 05:27:13 UTC
@Tomas - I believe Chris was talking about patches in http://cgit.freedesktop.org/~ickle/xserver/log/. Could you try them out please?
Comment 5 Chris Wilson 2011-09-02 05:39:50 UTC
Once you have the async vblank hook, you can then make DRI2CopyRegion synchronous (changing the sna_dri_copy_region function to pass true instead of false).
Comment 6 Eugeni Dodonov 2011-09-02 07:35:56 UTC
(In other words - apply the patch at http://cgit.freedesktop.org/~eugeni/xf86-video-intel/commit/?id=4e78dfc6b8c26312922e63002c423fb6637de472 after Chris' patches).
Comment 7 Eugeni Dodonov 2011-09-02 12:47:50 UTC
No changes here with changing sna_dri_copy_region function to true though, tearing still occurs when moving windows around.
Comment 8 Chris Wilson 2011-09-02 13:02:17 UTC
Are you absolutely sure at this point? I'm tempted to say turn on debugging again and verify for yourself that the scanline waits are inserted into the batch just before the dri copy. The only platform where it won't work is SNB, as we just have no clue yet how to make it work.
Comment 9 Eugeni Dodonov 2012-04-11 17:18:27 UTC
Just wondering of those xserver patches have finally landed somewhere or they are still MIA.
Comment 10 Chris Wilson 2012-04-12 00:53:09 UTC
I've got a couple more reviews this time around...
Comment 11 Antonio Orefice 2012-07-18 07:34:33 UTC
For what is worth, i've serious tearing in kde if opengl compositing is enabled when i use sna.
No matter if i force vsync in kwin and/or via .drirc.

chipset is 945gm
Comment 12 libo 2013-05-28 02:51:16 UTC
The bug can't be reproduced on my side. Our test environment as following:
Mesa:     (9.1)f00ae9c773dc944f0d900ef5c50c3b417ad95e84
Xserver:  (server-1.13-branch)xorg-server-1.13.4
Kernel:(drm-intel-fixes) 3598706b52cb45ba0a9e8aa99ce5ac59140f2b8b
Os:     Fedora17
Comment 13 Chris Wilson 2013-05-28 08:35:38 UTC
This bug stems from the broken design of DRI2. Nothing can be done without revving the protocol.
Comment 14 Chris Wilson 2014-02-01 11:18:28 UTC
It should be noted that at the current time DRI3/Present does not fix this problem.
Comment 15 Martin Peres 2019-11-27 13:28:26 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xorg/driver/xf86-video-intel/issues/9.

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.