Separate compact reason for rasdump

The COMPACT_FORCED_GC reason was reused what indicated that an explicit command line option was specified confusing GCMV parser to incorrectly conclude there was an option while there was only a RAS dump. Now we use a dedicated reason with a different string issued by Verbose GC. Signed-off-by: Aleksandar Micic <Aleksandar_Micic@ca.ibm.com>
eclipse-openj9 · Nov 1, 2023 · 82a191a · 82a191a
1 parent f8fecbe
commit 82a191a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/gc/base/gcutils.cpp b/gc/base/gcutils.cpp
@@ -87,6 +87,8 @@ getCompactionReasonAsString(CompactReason reason)
 			return "page granularity fragmentation";	
 		case COMPACT_MICRO_FRAG:
 			return "micro fragmentation";	
+		case COMPACT_RASDUMP:
+			return "rasdump";
 		default:
 			return "unknown";
 	}

diff --git a/gc/base/standard/ParallelGlobalGC.cpp b/gc/base/standard/ParallelGlobalGC.cpp
@@ -589,7 +589,7 @@ MM_ParallelGlobalGC::shouldCompactThisCycle(MM_EnvironmentBase *env, MM_Allocate
 	 * a compact we always honour it in order to produce optimal heap dumps
 	 */
 	if (J9MMCONSTANT_EXPLICIT_GC_RASDUMP_COMPACT == gcCode.getCode()) {
-		compactReason = COMPACT_FORCED_GC;
+		compactReason = COMPACT_RASDUMP;
 		goto compactionReqd;
 	}	
 

diff --git a/include_core/omrgcconsts.h b/include_core/omrgcconsts.h
@@ -294,7 +294,8 @@ typedef enum {
 	COMPACT_CONTRACT = 11,
 	COMPACT_AGGRESSIVE= 12,
 	COMPACT_PAGE = 13,
-	COMPACT_MICRO_FRAG = 14
+	COMPACT_MICRO_FRAG = 14,
+	COMPACT_RASDUMP = 15
 } CompactReason;
 
 typedef enum {