From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 101651] [radeonsi][hawaii] Borderlands 2 rendering issues with recent mesa/llvm Date: Thu, 29 Jun 2017 17:58:09 +0000 Message-ID: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1269815145==" 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 CA1556E6CA for ; Thu, 29 Jun 2017 17:58:09 +0000 (UTC) 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 --===============1269815145== Content-Type: multipart/alternative; boundary="14987590890.09BAEA3.2195"; charset="UTF-8" --14987590890.09BAEA3.2195 Date: Thu, 29 Jun 2017 17:58:09 +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=3D101651 Bug ID: 101651 Summary: [radeonsi][hawaii] Borderlands 2 rendering issues with recent mesa/llvm Product: Mesa Version: git Hardware: Other OS: All Status: NEW Severity: normal Priority: medium Component: Drivers/Gallium/radeonsi Assignee: dri-devel@lists.freedesktop.org Reporter: ptpzz@yandex.ru QA Contact: dri-devel@lists.freedesktop.org Created attachment 132354 --> https://bugs.freedesktop.org/attachment.cgi?id=3D132354&action=3Dedit screenshot (wrong rendering) AFAICS it's related to the VCCZ bug. There is a workaround for SI/CIK in the AMDGPU LLVM backend (SIInsertWaitcnts.cpp) but it doesn't handle this case. The attached patch fixes it for me by relaxing the workaround condition, but probably it can be improved if there is a detailed info on that bug. OTOH I guess it won't have a noticeable effect on performance anyway, so maybe it's safer to use the patch as is. The issue is reproducible with the reduced OpenCL test in the attachment: $ make // run on cpu - reference result $ ./testcpu 117 // OpenCL/gpu - wrong result $ LD_PRELOAD=3D/opt/llvm64_master/lib/libLLVM-5.0svn.so ./testgpu 99 // OpenCL/gpu + patch - correct result $ LD_PRELOAD=3D/opt/llvm64_patched/lib/libLLVM-5.0svn.so ./testgpu 117 --=20 You are receiving this mail because: You are the assignee for the bug.= --14987590890.09BAEA3.2195 Date: Thu, 29 Jun 2017 17:58:09 +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
Bug ID 101651
Summary [radeonsi][hawaii] Borderlands 2 rendering issues with recent= mesa/llvm
Product Mesa
Version git
Hardware Other
OS All
Status NEW
Severity normal
Priority medium
Component Drivers/Gallium/radeonsi
Assignee dri-devel@lists.freedesktop.org
Reporter ptpzz@yandex.ru
QA Contact dri-devel@lists.freedesktop.org

Created attachment =
132354 [details]
screenshot (wrong rendering)

AFAICS it's related to the VCCZ bug. There is a workaround for SI/CIK in the
AMDGPU LLVM backend (SIInsertWaitcnts.cpp) but it doesn't handle this case.

The attached patch fixes it for me by relaxing the workaround condition, but
probably it can be improved if there is a detailed info on that bug. OTOH I
guess it won't have a noticeable effect on performance anyway, so maybe it's
safer to use the patch as is.

The issue is reproducible with the reduced OpenCL test in the attachment:

  $ make

  // run on cpu - reference result

  $ ./testcpu
  117

  // OpenCL/gpu - wrong result

  $ LD_PRELOAD=3D/opt/llvm64_master/lib/libLLVM-5.0svn.so ./testgpu
  99

  // OpenCL/gpu + patch - correct result

  $ LD_PRELOAD=3D/opt/llvm64_patched/lib/libLLVM-5.0svn.so ./testgpu
  117


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