Skip to content

Commit

Permalink
8267271: Fix gc/arguments/TestNewRatioFlag.java expectedNewSize calcu…
Browse files Browse the repository at this point in the history
…lation

Backport-of: 59de99d
  • Loading branch information
GoeLin committed Jun 10, 2022
1 parent c6e4a44 commit 6ff4ba2
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions test/hotspot/jtreg/gc/arguments/TestNewRatioFlag.java
Expand Up @@ -124,11 +124,12 @@ public static void main(String args[]) {
public static void verifyDefNewNewRatio(int expectedRatio) {
long initEden = HeapRegionUsageTool.getEdenUsage().getInit();
long initSurv = HeapRegionUsageTool.getSurvivorUsage().getInit();
long initOld = HeapRegionUsageTool.getOldUsage().getInit();
long initHeap = HeapRegionUsageTool.getHeapUsage().getInit();

long newSize = initEden + 2 * initSurv;

long expectedNewSize = HeapRegionUsageTool.alignDown(initOld / expectedRatio,
// See GenArguments::scale_by_NewRatio_aligned for calculation in the JVM.
long expectedNewSize = HeapRegionUsageTool.alignDown(initHeap / (expectedRatio + 1),
wb.getHeapSpaceAlignment());

if (expectedNewSize != newSize) {
Expand All @@ -145,11 +146,12 @@ public static void verifyDefNewNewRatio(int expectedRatio) {
public static void verifyPSNewRatio(int expectedRatio) {
long initEden = HeapRegionUsageTool.getEdenUsage().getInit();
long initSurv = HeapRegionUsageTool.getSurvivorUsage().getInit();
long initOld = HeapRegionUsageTool.getOldUsage().getInit();
long initHeap = HeapRegionUsageTool.getHeapUsage().getInit();

long newSize = initEden + 2 * initSurv;

long alignedDownNewSize = HeapRegionUsageTool.alignDown(initOld / expectedRatio,
// See GenArguments::scale_by_NewRatio_aligned for calculation in the JVM.
long alignedDownNewSize = HeapRegionUsageTool.alignDown(initHeap / (expectedRatio + 1),
wb.getHeapSpaceAlignment());
long expectedNewSize = HeapRegionUsageTool.alignUp(alignedDownNewSize,
wb.psVirtualSpaceAlignment());
Expand Down

1 comment on commit 6ff4ba2

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.