Bug 301103

Summary: Allow direct rendering with fglrx
Product: [Plasma] kwin Reporter: Weng Xuetian <wengxt>
Component: scene-openglAssignee: KWin default assignee <kwin-bugs-null>
Severity: wishlist CC: jammy.zhou, russianneuromancer
Priority: NOR Flags: mgraesslin: Catalyst+
Version: 4.7.3   
Target Milestone: 4.10   
Platform: Chakra   
OS: Linux   
Latest Commit: Version Fixed In: 4.10
Sentry Crash Report:
Attachments: Whitelist Catalyst driver for kwin direct rendering

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:

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.


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)


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

Comment 10 Jammy Zhou 2012-08-23 01:39:05 UTC
Thanks Martin for pushing the patch. ;-)