Bug 70675

Summary:	steam error dialogs are garbled
Product:	Mesa	Reporter:	almos <aaalmosss>
Component:	Drivers/Gallium/r600	Assignee:	Default DRI bug account <dri-devel>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium	CC:	alexandre.f.demers
Version:	git
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	patch

Description almos 2013-10-20 14:47:25 UTC

Steps to reproduce:
1. Go to the store tab, on the top there is a category dropdown, select free-to-play
2. Select any free-to-play game from the list that doesn't support Linux
3. Click the green "play game" button
4. An error dialog tells you that "this game is not available for your current platform"

With r600g the dialog is garbled, with llvmpipe it's OK.

Report on steam issue tracker:
https://github.com/ValveSoftware/steam-for-linux/issues/2732

Comment 1 Vadim Girlin 2013-10-20 19:46:48 UTC

Created attachment 87888 [details] [review]
patch

IIRC this is related to tiling and not limited to steam windows, I saw this in different apps with r600g when window height is in range from 120 to 128. Just reducing the height of glxgears window to that range was enough to trigger the bug (or setting the height in command line, e.g. "glxgears -geometry 100x121").

Attached patch for xf86-video-ati fixed it for me, but I'm not sure if it's a correct solution.

Comment 2 Alexandre Demers 2013-10-20 22:59:53 UTC

I confirm the bug, I've seen it at the same places you listed and in the same range. I can't tell if the proposed patch is the real fix or just a workaround though.

Comment 3 almos 2013-10-21 10:05:47 UTC

I can confirm what Vadim said. Although, those geometries are fine. The height has to be in range 121-127, and width must be 128 or greater to trigger the bug.

I'll try the patch later. It looks like it is a fix for an already existing workaround.

Comment 4 Alex Deucher 2013-10-23 23:58:26 UTC

I think the patch is fine.  I think 64 is too small for 2D tiling on some asics depending on the number of memory channels they have.  It shouldn't hurt anything.

Comment 5 Alexandre Demers 2013-11-01 02:25:25 UTC

I just tested the patch on my HD 6950 and it fixes the visual bug. Can the patch be pushed in the ddx tree?

Comment 6 Alex Deucher 2013-11-01 16:45:59 UTC

Pushed:
http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=3b38701a72fa1cad1e4610a2f4330b3da4cc6391

Comment 7 almos 2014-01-16 22:38:28 UTC

I'm wondering if a new version of xf86-video-ati is scheduled to be released in the foreseeable future. Apart from this fix, there seem to be some useful commits since 7.2.0.

Comment 8 Alex Deucher 2014-01-16 22:40:23 UTC

(In reply to comment #7)
> I'm wondering if a new version of xf86-video-ati is scheduled to be released
> in the foreseeable future. Apart from this fix, there seem to be some useful
> commits since 7.2.0.

Hopefully next week.  Just waiting on a glamor release since we currently depend on glamor from git.

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.