Bug 106372

Summary: Infinite client hang in xcb_wait_for_special_event() - xserver 1.20rcX
Product: xorg Reporter: Tobias Klausmann <tobias.johannes.klausmann>
Component: Driver/modesettingAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED DUPLICATE QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium CC: answer2002, maggu2810
Version: git   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
gdb backtrace of hanging plasmashell thread
none
Disable new code none

Description Tobias Klausmann 2018-05-03 12:02:53 UTC
Created attachment 139302 [details]
gdb backtrace of hanging plasmashell thread

Starting with bisected commit 44d5f2eb8a2f92571698adec39ac569b71da5a1b - "xf86-video-modesetting: Support new vblank kernel API [v2]" Clients seem to miss events, here plasmashell [1]. This is easily reproduced by opening and closing the menu until plasmashell hangs. It seems to me that the server/driver does not properly report all vblank events.

Working around by calling xcb_poll_for_special_event() instead of xcb_wait_for_special_event() in the client call path "resolves" the issue in a dirty way on the client side.

Just to collect a bit more info:
HW: VGA compatible controller: Intel Corporation Device 591b (rev 04)
tested kernel: 4.16.x & 4.17rc

If you need more info, please let me know!
Comment 1 Michel Dänzer 2018-05-03 15:02:38 UTC
Created attachment 139311 [details] [review]
Disable new code

Does this patch on top of current xserver Git master still prevent the problem?
Comment 2 Tobias Klausmann 2018-05-03 15:48:09 UTC
(In reply to Michel Dänzer from comment #1)
> Created attachment 139311 [details] [review] [review]
> Disable new code
> 
> Does this patch on top of current xserver Git master still prevent the
> problem?

Yes the old code/interfaces does still work fine on top of latest master.
Comment 3 Tobias Klausmann 2018-05-03 22:05:22 UTC
This might actually be a duplicate of bug #106351 which i haven't checked :/. Yet we already made a bit of progress on this bugreport with bisecting and testing old & new code, leaving this one open for now seems reasonable...
Comment 4 Michel Dänzer 2018-05-16 06:57:45 UTC

*** This bug has been marked as a duplicate of bug 106351 ***

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.