Ticket #711 (closed bug: fixed)

Opened 7 years ago

Last modified 4 years ago

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: Difficulty: Easy (less than 1 hour)
Test Case: N/A Blocked By:
Blocking: Related Tickets:

Description

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

Attachments

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

Change History

Changed 7 years ago by lennart.augustsson@…

Patch to deallocate memory for win32.

Changed 7 years ago by simonmar

  • owner set to simonmar
  • component changed from Compiler to Runtime System
  • milestone set to 6.4.2

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

Changed 7 years ago by simonmar

  • priority changed from normal to low
  • summary changed from shutdownHaskell() does not return allocated memory to shutdownHaskell() does not return allocated memory on Unix
  • severity changed from normal to minor
  • milestone 6.4.2 deleted

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

Changed 7 years ago by igloo

  • testcase set to N/A
  • milestone set to 6.6.1

Changed 6 years ago by simonmar

  • milestone changed from 6.6.1 to _|_

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

Changed 5 years ago by simonmar

  • architecture changed from Unknown to Unknown/Multiple

Changed 5 years ago by simonmar

  • os changed from Multiple to Unknown/Multiple

Changed 4 years ago by thoughtpolice

patch to fix bug on unix

Changed 4 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.)

Changed 4 years ago by simonmar

  • owner changed from igloo to simonmar
  • priority changed from low to high
  • milestone changed from _|_ to 6.12.1

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

Changed 4 years ago by simonmar

  • status changed from new to closed
  • resolution set to fixed

Applied, thanks! 

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

Changed 4 years ago by simonmar

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