Opened 11 years ago

Closed 8 years ago

Last modified 7 years ago

#711 closed bug (fixed)

shutdownHaskell() does not return allocated memory on Unix

Reported by: lennart.augustsson@… Owned by: simonmar
Priority: high Milestone: 6.12.1
Component: Runtime System Version: 6.5
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case: N/A
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

Calling shutdownHaskell() doesn't actually return the memory allocated. This can be very bad when loading and unloading DLLs many times.

Attachments (2)

rts.patch (35.3 KB) - added by lennart.augustsson@… 11 years ago.
Patch to deallocate memory for win32.
rts.dpatch (197.3 KB) - added by thoughtpolice 8 years ago.
patch to fix bug on unix

Download all attachments as: .zip

Change History (13)

Changed 11 years ago by lennart.augustsson@…

Attachment: rts.patch added

Patch to deallocate memory for win32.

comment:1 Changed 11 years ago by simonmar

Component: CompilerRuntime System
Milestone: 6.4.2
Owner: set to simonmar

comment:2 Changed 11 years ago by simonpj

Lennart says "I'm not implying anything, except that I've plugged the space leak of 256M every time a DLL is loaded & unloaded." See the attached patch.

comment:3 Changed 11 years ago by simonmar

Milestone: 6.4.2
Priority: normallow
severity: normalminor
Summary: shutdownHaskell() does not return allocated memoryshutdownHaskell() does not return allocated memory on Unix

committed; thanks. Problem still not addressed on Unix, so I've left the bug open.

comment:4 Changed 10 years ago by igloo

Milestone: 6.6.1
Test Case: N/A

comment:5 Changed 10 years ago by simonmar

Milestone: 6.6.1_|_

Milestone _|_: this doesn't affect anyone (yet)

comment:6 Changed 8 years ago by simonmar

Architecture: UnknownUnknown/Multiple

comment:7 Changed 8 years ago by simonmar

Operating System: MultipleUnknown/Multiple

Changed 8 years ago by thoughtpolice

Attachment: rts.dpatch added

patch to fix bug on unix

comment:8 Changed 8 years ago by thoughtpolice

Owner: changed from simonmar to igloo

Hopefully this patch should fix this issue.

Fri Mar 20 07:49:57 CDT 2009  Austin Seipp <mad.one@gmail.com>
  * FIX #711 implement osFreeAllMBlocks for unix

I'm assigning this to igloo so he can take a look at it and merge it when necessary into HEAD I suppose (I've run validate and everything turned up OK.)

comment:9 Changed 8 years ago by simonmar

Milestone: _|_6.12.1
Owner: changed from igloo to simonmar
Priority: lowhigh

Just noticed this patch. I'll review and commit.

comment:10 Changed 8 years ago by simonmar

Resolution: fixed
Status: newclosed

Applied, thanks!

Thu Sep 10 01:46:30 PDT 2009  Austin Seipp <mad.one@gmail.com>
  * FIX #711 implement osFreeAllMBlocks for unix

comment:11 Changed 7 years ago by simonmar

difficulty: Easy (1 hr)Easy (less than 1 hour)
Note: See TracTickets for help on using tickets.