SF.net SVN: jikesrvm:[15793] rvmroot/trunk/MMTk/src/org/mmtk/utility/heap/ FreeListPageResource.java

[steveb-oss]

Revision: 15793
          http://jikesrvm.svn.sourceforge.net/jikesrvm/?rev=15793&view=rev
Author:   steveb-oss
Date:     2009-11-12 11:47:04 +0000 (Thu, 12 Nov 2009)

Log Message:
-----------
Fix boundary condition bug which probably lead to a 4MB memory leak in each Immix space.  Sigh.

Modified Paths:
--------------
    rvmroot/trunk/MMTk/src/org/mmtk/utility/heap/FreeListPageResource.java

Modified: rvmroot/trunk/MMTk/src/org/mmtk/utility/heap/FreeListPageResource.java
===================================================================
--- rvmroot/trunk/MMTk/src/org/mmtk/utility/heap/FreeListPageResource.java 2009-11-05 06:01:07 UTC (rev 15792)
+++ rvmroot/trunk/MMTk/src/org/mmtk/utility/heap/FreeListPageResource.java 2009-11-12 11:47:04 UTC (rev 15793)
@@ -159,7 +159,7 @@
     } else {
       pagesCurrentlyOnFreeList -= pages;
       if (pageOffset > highWaterMark) {
-        if ((pageOffset ^ highWaterMark) > EmbeddedMetaData.PAGES_IN_REGION) {
+        if (highWaterMark == 0 || (pageOffset ^ highWaterMark) > EmbeddedMetaData.PAGES_IN_REGION) {
           int regions = 1 + ((pageOffset - highWaterMark) >> EmbeddedMetaData.LOG_PAGES_IN_REGION);
           int metapages = regions * metaDataPagesPerRegion;
           reserved += metapages;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Jikesrvm-commits mailing list
Jikesrvm-commits@...
https://lists.sourceforge.net/lists/listinfo/jikesrvm-commits