Skip to content

Commit

Permalink
Set _updateFreeMemoryMinPeriod option to 300 ms on Power
Browse files Browse the repository at this point in the history
The current default value for this option (50 ms) has been shown
to cause a small footprint regression on Power. This commit changes
the default value of `_updateFreeMemoryMinPeriod` to 300 ms on Power
to eliminate the regression.

Signed-off-by: Marius Pirvu <mpirvu@ca.ibm.com>
  • Loading branch information
mpirvu committed Dec 14, 2022
1 parent f6d01fd commit 3651005
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion runtime/compiler/control/J9Options.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -266,8 +266,11 @@ int32_t J9::Options::_numCodeCachesToCreateAtStartup = 0; // 0 means no change f
int32_t J9::Options::_dataCacheQuantumSize = 64;
int32_t J9::Options::_dataCacheMinQuanta = 2;

#if defined(TR_TARGET_POWER)
int32_t J9::Options::_updateFreeMemoryMinPeriod = 300; // 300 ms
#else
int32_t J9::Options::_updateFreeMemoryMinPeriod = 50; // 50 ms

#endif /* defined(TR_TARGET_POWER) */
size_t J9::Options::_scratchSpaceLimitKBWhenLowVirtualMemory = 64*1024; // 64MB; currently, only used on 32 bit Windows

int32_t J9::Options::_scratchSpaceFactorWhenJSR292Workload = JSR292_SCRATCH_SPACE_FACTOR;
Expand Down

0 comments on commit 3651005

Please sign in to comment.