Fixes regression in minAddressHint() refactoring, which lazily initia…

…lizes pageSize. So, we must not use roundUpToPage() before calling minAddressHint(). This is okay, because sqAllocateMemorySegmentOfSizeAboveAllocatedSizeInto(...) does the rounding anyway.
OpenSmalltalk · Jul 12, 2021 · 4ce584b · 4ce584b
1 parent ee88a81
commit 4ce584b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/platforms/win32/vm/sqWin32SpurAlloc.c b/platforms/win32/vm/sqWin32SpurAlloc.c
@@ -83,7 +83,7 @@ sqAllocateMemory(usqInt minHeapSize, usqInt desiredHeapSize)
 #endif
 
 	alloc = sqAllocateMemorySegmentOfSizeAboveAllocatedSizeInto
-				(roundUpToPage(desiredHeapSize), minAddressHint(), &allocBytes);
+				(desiredHeapSize, minAddressHint(), &allocBytes);
 	if (!alloc) {
 		sqMessageBox(MB_OK | MB_ICONSTOP, TEXT("VM Error:"),
 					 TEXT("sqAllocateMemory: initial alloc failed!\n"));
@@ -225,7 +225,7 @@ allocateJITMemory(usqInt *desiredSize)
 	sqInt allocBytes;
 
 	char *alloc = sqAllocateMemorySegmentOfSizeAboveAllocatedSizeInto
-				(roundUpToPage(*desiredSize), minAddressHint(), &allocBytes);
+				(*desiredSize, minAddressHint(), &allocBytes);
 
 	if (!alloc) {
 		perror("Could not allocate JIT memory");