|Summary:||[g45 sna] tearing with partial screen updates|
|Product:||xorg||Reporter:||Achim Frase <achim.frase>|
|Component:||Driver/intel||Assignee:||Chris Wilson <chris>|
|Status:||RESOLVED MOVED||QA Contact:||Intel GFX Bugs mailing list <intel-gfx-bugs>|
|i915 platform:||i915 features:|
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 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 Xf86_video_intel:(master)2.21.7-5-g87295b1ef85505689ce326137c2794230fb3f35f Cairo:(master)631bf299256e11a17511977f357e0353fb5615f7 Libva_intel_driver:(master)31caada2967b94705d78ab7f6d07965ad7f13d42 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.