RFR: Add 'generational' as value choice for JVM option 'ShenandoahGCMode'.

Roman Kennke rkennke at openjdk.java.net
Tue Oct 27 10:13:14 UTC 2020


On Wed, 21 Oct 2020 12:53:41 GMT, Bernd Mathiske <bmathiske at openjdk.org> wrote:

> All upcoming GenShen code will be guarded by this JVM option. If generational mode is not set, Shenandoah will function in its original single-generational form, without changes or degradations.

You should merge-in the master branch from openjdk/shenandoah, this should hopefully fix the jcheck complaints.

src/hotspot/share/gc/shenandoah/mode/shenandoahGenerationalMode.cpp line 2:

> 1: /*
> 2:  * Copyright (c) 2019, 2020, Red Hat, Inc. All rights reserved.

I believe "Copyright (c) 2020 ..." would be correct there.

src/hotspot/share/gc/shenandoah/mode/shenandoahGenerationalMode.cpp line 32:

> 30: #include "gc/shenandoah/heuristics/shenandoahStaticHeuristics.hpp"
> 31: #include "gc/shenandoah/mode/shenandoahGenerationalMode.hpp"
> 32: #include "logging/log.hpp"

Those includes for logging/* look unused. Probably leftovers?

src/hotspot/share/gc/shenandoah/mode/shenandoahGenerationalMode.hpp line 2:

> 1: /*
> 2:  * Copyright (c) 2019, Red Hat, Inc. All rights reserved.

Same as above. 2020 is the year ;-)

src/hotspot/share/gc/shenandoah/mode/shenandoahMode.hpp line 28:

> 26: #define SHARE_GC_SHENANDOAH_MODE_SHENANDOAHMODE_HPP
> 27: 
> 28: #include "runtime/java.hpp"

This include looks unused.

-------------

Changes requested by rkennke (Reviewer).

PR: https://git.openjdk.java.net/shenandoah/pull/4


More information about the shenandoah-dev mailing list