RFR(M) 8043575: Dynamically parallelize reference processing work

Kim Barrett kim.barrett at oracle.com
Fri Jun 15 16:42:28 UTC 2018


> On Jun 14, 2018, at 7:34 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi all,
> 
>  after some talk about making parallel ref processing default for G1
> the suggestion came up to extract that part into a separate CR. I will
> post that one shortly.
> 
> However, this means that there is a trivial change in this webrev to be
> looked at:
> 
> http://cr.openjdk.java.net/~tschatzl/8043575/webrev.3_to_4 (diff)
> http://cr.openjdk.java.net/~tschatzl/8043575/webrev.4 (full)
> 
> This is the full change, for reference:
> 
> --- old/src/hotspot/share/gc/g1/g1Arguments.cpp	2018-06-14
> 13:30:08.166027425 +0200
> +++ new/src/hotspot/share/gc/g1/g1Arguments.cpp	2018-06-14
> 13:30:07.818016510 +0200
> @@ -122,10 +122,6 @@
>     FLAG_SET_DEFAULT(GCPauseIntervalMillis, MaxGCPauseMillis + 1);
>   }
> 
> -  if (FLAG_IS_DEFAULT(ParallelRefProcEnabled) && ParallelGCThreads >
> 1) {
> -    FLAG_SET_DEFAULT(ParallelRefProcEnabled, true);
> -  }
> -
>   log_trace(gc)("MarkStackSize: %uk  MarkStackSizeMax: %uk", (unsigned
> int) (MarkStackSize / K), (uint) (MarkStackSizeMax / K));
> 
>   // By default do not let the target stack size to be more than 1/4
> of the entries
> 
> Thanks,
>  Thomas

Looks good.




More information about the hotspot-gc-dev mailing list