Bug 83425

Summary:	ndiswrapper 1.0 leak patch
Product:	Gentoo Linux	Reporter:	Karol Krizka <kkrizka>
Component:	New packages	Assignee:	AMD64 Project <amd64>
Status:	RESOLVED FIXED
Severity:	minor
Priority:	High
Version:	unspecified
Hardware:	AMD64
OS:	Linux
URL:	http://article.gmane.org/gmane.linux.drivers.ndiswrapper.general/4132
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Parak's Patch

Description Karol Krizka 2005-02-26 18:17:42 UTC

Hello,
There is a memory leak in ndiswrapper 1.0 for amd64. See the attached link for more info. From what I heard, it has been fixed in the next version but that dosn't work under amd64, just makes the kernel crash. Someone has created a patch for 1.0 and this problem and it works great on FC3 and Debian amd64. I haven't tried this on gentoo but I am installing it right now and will tell you about it's success if you want me to. IS there any more information you would like?

Reproducible: Always
Steps to Reproduce:
1.modprobe ndiswrapper
2.wait for a while
3.system starts slowing down

Actual Results:  
The system slows down and slabtop shows size-64 to be around 99% of the memory.

Expected Results:  
System should run normal speed and slabtop shows size-64 to be 552K.

Comment 1 Karol Krizka 2005-02-26 18:19:17 UTC

Created attachment 52212 [details, diff]
Parak's Patch

Patch for ndiswrapper 1.0

Comment 2 Georg Schild 2005-02-28 23:54:48 UTC

what giri said on the ml the patch from parag is not the best solution and the problem got fixed after 1.0. There is _no_ memory leak present since 1.1-rc1 so an ebuild fro 1.1-rc3, which will maybe become 1.1, would be better then this patch

Comment 3 Karol Krizka 2005-03-01 15:47:31 UTC

But no version after 1.0 works on AMD64. It causes a kernel panic. Someone has captured it with netconsole here:
http://sourceforge.net/mailarchive/message.php?msg_id=11012875

Comment 4 Carsten Niehaus 2005-03-05 07:00:24 UTC

From reading the devel-mailinglist I do think that 1.1-rc3 fixed the 64-bit problems. 1.1-final is already out so we should add an ebuild for the final-version and might even drop 1.0 completly.

Comment 5 Robin Munn 2005-03-05 12:53:33 UTC

I've created an enhancement request (#84219) for an ndiswrapper-1.1 ebuild.

Comment 6 Karol Krizka 2005-03-11 07:55:07 UTC

Yes, the leak has been fixed in 1.1 and it works again on amd64.

Comment 7 Karol Krizka 2005-03-11 22:58:18 UTC

Workd in new version of ndiswrapper