From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 105339] Deadlock inside glClientWaitSync Date: Fri, 09 Mar 2018 21:48:24 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0751202935==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [IPv6:2610:10:20:722:a800:ff:fe98:4b55]) by gabe.freedesktop.org (Postfix) with ESMTP id C67A76EC2B for ; Fri, 9 Mar 2018 21:48:23 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0751202935== Content-Type: multipart/alternative; boundary="15206321030.3D3CdCe.5631" Content-Transfer-Encoding: 7bit --15206321030.3D3CdCe.5631 Date: Fri, 9 Mar 2018 21:48:23 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D105339 --- Comment #5 from Matias N. Goldberg --- I traced the regression to commit: commit bc65dcab3bc48673ff6180afb036561a4b8b1119 Author: Nicolai H=C3=A4hnle Date: Fri Nov 10 10:58:10 2017 +0100 radeonsi: avoid syncing the driver thread in si_fence_finish It is really only required when we need to flush for deferred fences. Reviewed-by: Marek Ol=C5=A1=C3=A1k Although I slightly suspect the former code was just making the race condit= ion much harder to trigger, considering I've played other Dolphin games in the = past (before this regression) and they ocasionally hanged in a similar way after= 2-4 hours of continuous play or so (extremely rare to trigger) and it wouldn't always happen (But that may have been a different bug). --=20 You are receiving this mail because: You are the assignee for the bug.= --15206321030.3D3CdCe.5631 Date: Fri, 9 Mar 2018 21:48:23 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated

Commen= t # 5 on bug 10533= 9 from Matias N. Goldberg
I traced the regression to commit:

commit bc65dcab3bc48673ff6180afb036561a4b8b1119
Author: Nicolai H=C3=A4hnle <nicolai.haehnle@amd.com>
Date:   Fri Nov 10 10:58:10 2017 +0100

    radeonsi: avoid syncing the driver thread in si_fence_finish

    It is really only required when we need to flush for deferred fences.

    Reviewed-by: Marek Ol=C5=A1=C3=A1k <marek.olsak@amd.com>


Although I slightly suspect the former code was just making the race condit=
ion
much harder to trigger, considering I've played other Dolphin games in the =
past
(before this regression) and they ocasionally hanged in a similar way after=
 2-4
hours of continuous play or so (extremely rare to trigger) and it wouldn't
always happen (But that may have been a different bug).


You are receiving this mail because:
  • You are the assignee for the bug.
= --15206321030.3D3CdCe.5631-- --===============0751202935== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0751202935==--