RFR (XS) 8242213: Shenandoah: remove unused ShenandoahHeuristics::_bytes_in_cset
Roman Kennke
rkennke at redhat.com
Mon Apr 6 18:28:21 UTC 2020
Ok, let it go! :-)
Roman
> RFE:
> https://bugs.openjdk.java.net/browse/JDK-8242213
>
> That field does not seem to be used for anything, yet we update it on critical path during cset
> selection. We should ditch the field and all "uses" of it. Cursory examination seems to indicate it
> was there since the initial integration.
>
> Fix:
>
> diff -r f30c66ecc928 src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp
> --- a/src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp Mon Apr 06
> 14:46:40 2020 +0200
> +++ b/src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp Mon Apr 06
> 15:04:29 2020 +0200
> @@ -78,3 +78,2 @@
> size_t cur_garbage = 0;
> - _bytes_in_cset = 0;
>
> @@ -92,3 +91,2 @@
> cset->add_region(r);
> - _bytes_in_cset += r->used();
> cur_cset = new_cset;
> diff -r f30c66ecc928 src/hotspot/share/gc/shenandoah/shenandoahHeuristics.cpp
> --- a/src/hotspot/share/gc/shenandoah/shenandoahHeuristics.cpp Mon Apr 06 14:46:40 2020 +0200
> +++ b/src/hotspot/share/gc/shenandoah/shenandoahHeuristics.cpp Mon Apr 06 15:04:29 2020 +0200
> @@ -48,3 +48,2 @@
> _successful_cycles_in_a_row(0),
> - _bytes_in_cset(0),
> _cycle_start(os::elapsedTime()),
> @@ -260,3 +259,3 @@
> void ShenandoahHeuristics::record_allocation_failure_gc() {
> - _bytes_in_cset = 0;
> + // Do nothing.
> }
> @@ -264,4 +263,2 @@
> void ShenandoahHeuristics::record_requested_gc() {
> - _bytes_in_cset = 0;
> -
> // Assume users call System.gc() when external state changes significantly,
> diff -r f30c66ecc928 src/hotspot/share/gc/shenandoah/shenandoahHeuristics.hpp
> --- a/src/hotspot/share/gc/shenandoah/shenandoahHeuristics.hpp Mon Apr 06 14:46:40 2020 +0200
> +++ b/src/hotspot/share/gc/shenandoah/shenandoahHeuristics.hpp Mon Apr 06 15:04:29 2020 +0200
> @@ -84,4 +84,2 @@
>
> - size_t _bytes_in_cset;
> -
> double _cycle_start;
>
> Testing: hotspot_gc_shenandoah
>
More information about the hotspot-gc-dev
mailing list