8230425: Shenandoah forces +UseNUMAInterleaving even after explicitly disabled

shipilev · shipilev · commit c25e2fd8ec4d · 2019-09-02T12:10:21.000+02:00
Reviewed-by: rkennke
diff --git a/src/hotspot/share/gc/shenandoah/shenandoahArguments.cpp b/src/hotspot/share/gc/shenandoah/shenandoahArguments.cpp
@@ -60,11 +60,9 @@ void ShenandoahArguments::initialize() {
   }
 
   // Enable NUMA by default. While Shenandoah is not NUMA-aware, enabling NUMA makes
-  // storage allocation code NUMA-aware, and NUMA interleaving makes the storage
-  // allocated in consistent manner (interleaving) to minimize run-to-run variance.
+  // storage allocation code NUMA-aware.
   if (FLAG_IS_DEFAULT(UseNUMA)) {
     FLAG_SET_DEFAULT(UseNUMA, true);
-    FLAG_SET_DEFAULT(UseNUMAInterleaving, true);
   }
 
   // Set up default number of concurrent threads. We want to have cycles complete fast

Original file line number	Diff line number	Diff line change
`@@ -60,11 +60,9 @@ void ShenandoahArguments::initialize() {`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`// Enable NUMA by default. While Shenandoah is not NUMA-aware, enabling NUMA makes`
`63`		`- // storage allocation code NUMA-aware, and NUMA interleaving makes the storage`
`64`		`- // allocated in consistent manner (interleaving) to minimize run-to-run variance.`
	`63`	`+ // storage allocation code NUMA-aware.`
`65`	`64`	`if (FLAG_IS_DEFAULT(UseNUMA)) {`
`66`	`65`	`FLAG_SET_DEFAULT(UseNUMA, true);`
`67`		`- FLAG_SET_DEFAULT(UseNUMAInterleaving, true);`
`68`	`66`	`}`
`69`	`67`
`70`	`68`	`// Set up default number of concurrent threads. We want to have cycles complete fast`