RFR: 8236981: Remove ShenandoahTraversalUpdateRefsClosure

Aleksey Shipilev shade at redhat.com
Fri Mar 6 11:13:53 UTC 2020


Hi,

The removal looks okay.

On 3/6/20 7:37 AM, Aditya Mandaleeka wrote:
> diff -r 92cf8efd381d src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp
> --- a/src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp Fri Mar 06 10:27:24 2020 +0530
> +++ b/src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp Thu Mar 05 22:30:55 2020 -0800
> @@ -605,8 +605,11 @@
>      // that results the TLAB/GCLAB not usable. Retire them here.
>      _heap->make_parsable(true);
> 
> +    // Do this fixup before the call to parallel_cleaning to ensure that all
> +    // forwarded objects (including those that are no longer in the cset) are
> +    // updated by the time we do weak root processing.
> +    fixup_roots();
>      _heap->parallel_cleaning(false);
> -    fixup_roots();
> 
>      _heap->set_has_forwarded_objects(false);

...but this thing reverts recent fix!
  https://hg.openjdk.java.net/jdk/jdk/rev/b997e5b9479b#l3.1

I wonder what is up with that, Roman?

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list