Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 487792 - x11-drivers/ati-drivers - Memory leak with kernels >= 3.10
Summary: x11-drivers/ati-drivers - Memory leak with kernels >= 3.10
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2013-10-12 16:43 UTC by Mathieu Belanger
Modified: 2017-03-15 19:33 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to fix initial proc patch memory leak. (linux-3.10-proc2.patch,2.34 KB, patch)
2013-10-12 16:43 UTC, Mathieu Belanger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mathieu Belanger 2013-10-12 16:43:14 UTC
Created attachment 360718 [details, diff]
Patch to fix initial proc patch memory leak.

Hi,

Since fglrx got adapted for the new proc structure in Kernel 3.10, it suffer from a memory leak and cause crash on some system including mine (running out of memory is bad)

The attached patch will patch all ebuild that compile on 3.10+ and it need to be after the proc diff patch on fglrx that don't officially compile on 3.10. AMD just took that initial bugged proc patch from a dude in the Phoronix forum and decided to include in there fglrx without much testing/qa...
Comment 1 emil karlson 2013-10-12 20:41:56 UTC
Can you explain what the patch is supposed to do or link to the thread, thanks?
Comment 2 Mathieu Belanger 2013-10-12 22:48:36 UTC
You have two way to create proc entry.

the fglrx was using the default way to manage it but the patch to initially fix the compilation on 3.10 kernel create the proc entry on another way with is not compatible with the way fglrx use it, creating a leak.

Here the link to the unofficial AMD bugzilla entry : http://ati.cchtml.com/show_bug.cgi?id=866
Comment 3 emil karlson 2013-10-13 14:21:01 UTC
Could you state which versions do you think need this patch and which ones have you tested (and how)? The only big proc related patch we are using right now is written by me and is not the one posted on phoronix forums, so not everything posted there applies to gentoo.
Comment 4 Matt Turner gentoo-dev 2017-03-15 19:33:51 UTC
x11-drivers/ati-drivers has been removed, per bug 582406.