Bug 70675 - steam error dialogs are garbled
Summary: steam error dialogs are garbled
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/Gallium/r600 (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-20 14:47 UTC by almos
Modified: 2014-01-16 22:40 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
patch (526 bytes, patch)
2013-10-20 19:46 UTC, Vadim Girlin
Details | Splinter Review

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 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.