Bug 301103

Summary:	Allow direct rendering with fglrx
Product:	[Plasma] kwin	Reporter:	Weng Xuetian <wengxt>
Component:	scene-opengl	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
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:	http://commits.kde.org/kde-workspace/1efa6a9d41237339298e45f0c791ffe1269077f4	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:
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. ;-)