Skip to content

Commit

Permalink
[Misc] change default gc in aarch64 from CMS to PS
Browse files Browse the repository at this point in the history
Summary: The same as title

Test Plan: CICD

Reviewed-by: D-D-H, yuleil

Issue: #548
  • Loading branch information
sandlerwang committed Jul 11, 2023
1 parent 1640100 commit 55698c6
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
4 changes: 4 additions & 0 deletions hotspot/src/share/vm/runtime/arguments.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1605,7 +1605,11 @@ void Arguments::select_gc_ergonomically() {

void Arguments::select_gc() {
if (!gc_selected()) {
#ifdef AARCH64
select_gc_ergonomically();
#else
FLAG_SET_ERGO(bool, UseConcMarkSweepGC, true);
#endif
}
}

Expand Down
6 changes: 6 additions & 0 deletions hotspot/test/gc/TestDefaultGC.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,13 @@
# @run shell TestDefaultGC.sh
#


arch=$(uname -m)
if [[ ${arch} == 'aarch64' ]]; then
${TESTJAVA}/bin/java -XX:+PrintFlagsFinal -version | grep UseParallelGC |grep true
else
${TESTJAVA}/bin/java -XX:+PrintFlagsFinal -version | grep UseConcMarkSweepGC |grep true
fi

if [ "0" == $? ];
then
Expand Down

0 comments on commit 55698c6

Please sign in to comment.