Bug 301103 - Allow direct rendering with fglrx
Summary: Allow direct rendering with fglrx
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: scene-opengl (show other bugs)
Version: 4.7.3
Platform: Chakra Linux
: NOR wishlist
Target Milestone: 4.10
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-03 11:55 UTC by Weng Xuetian
Modified: 2012-08-23 01:39 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.10
Sentry Crash Report:
mgraesslin: Catalyst+


Attachments
Whitelist Catalyst driver for kwin direct rendering (1.85 KB, patch)
2012-08-15 10:49 UTC, Jammy Zhou
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Weng Xuetian 2012-06-03 11:55:30 UTC
AMD release a new driver catalyst 12.6/fglrx 8.98, which seems to be the first version can work with kwin direct rendering without problem.

So maybe driver newer than this can be removed from black list.

Phoronix news:
http://www.phoronix.com/scan.php?page=news_item&px=MTExMTU

So far no problem here.

Reproducible: Always
Comment 1 Martin Flöser 2012-06-03 12:02:05 UTC
Some comments:

* It's not a blacklist, but a whitelist
* I have info that not everything is working yet (e.g. Slideback seems broken)

When we enable the driver in the whitelist there is no easy way for users to go back. Because of that I'm reluctant to change it. Everything *has* to work perfectly with direct rendering.
Comment 2 Jammy Zhou 2012-08-08 07:06:47 UTC
Hi Martin,

The slideback issue has been confirmed to be an issue of kwin, which happens for both NV and AMD (radeon and fglrx). And bug has been reported below.

https://bugs.kde.org/show_bug.cgi?id=304479

So currently there is no direct rendering specific issue for fglrx now. Maybe we can consider the whitelisting at some proper point of KDE 4.10.
Comment 3 Martin Flöser 2012-08-14 15:58:25 UTC
I'm fine with setting the latest Catalyst to direct rendering in current master, but we need someone to write that code. To my knowledge no KWin developer is using the Catalyst driver .
Comment 4 Jammy Zhou 2012-08-15 10:48:59 UTC
The reference patch is attached.
Comment 5 Jammy Zhou 2012-08-15 10:49:56 UTC
Created attachment 73178 [details]
Whitelist Catalyst driver for kwin direct rendering
Comment 6 Thomas Lübking 2012-08-15 11:23:08 UTC
Please use git.reviewboard.kde.org to get the patch reviewed (imo quite necessary)

However:

+bool getCatalystversion(char *version)
+{
+    const char *RV = "ReleaseVersion";
+    char *RVbuf = NULL;
+    char buf[MAX_LINE_LEN];
+    FILE *fp = fopen("/etc/ati/amdpcsdb.default","rb");
...

-> is there really no version to get that info out of glGetString(GL_VERSION) or similar?
Comment 7 Jammy Zhou 2012-08-16 01:55:02 UTC
Sure, I will send review request to git.reviewboard.kde.org later.

-> is there really no version to get that info out of glGetString(GL_VERSION) or similar?
The release version cannot be got from GL_VERSION currently, and only /etc/ati/amdpcsdb.default can be leveraged on.
Comment 8 Jammy Zhou 2012-08-16 03:11:31 UTC
The review request has been submitted.
https://git.reviewboard.kde.org/r/106050/
Comment 9 Martin Flöser 2012-08-22 18:33:55 UTC
Git commit 1efa6a9d41237339298e45f0c791ffe1269077f4 by Martin Gräßlin, on behalf of Jammy Zhou.
Committed on 22/08/2012 at 08:05.
Pushed by graesslin into branch 'master'.

Whitelist AMD Catalyst for kwin direct rendering
FIXED-IN: 4.10
REVIEW: 106050

M  +34   -0    kwin/opengltest/opengltest.cpp

http://commits.kde.org/kde-workspace/1efa6a9d41237339298e45f0c791ffe1269077f4
Comment 10 Jammy Zhou 2012-08-23 01:39:05 UTC
Thanks Martin for pushing the patch. ;-)