RFR: 8252141: Rename G1YoungRemSetSamplingThread to better reflect its purpose
thomas.schatzl at oracle.com
Mon Aug 31 13:20:13 UTC 2020
On 26.08.20 09:18, stefan.johansson at oracle.com wrote:
> Hi Kim,
> Thanks for reviewing.
> On 2020-08-25 04:44, Kim Barrett wrote:
>>> On Aug 24, 2020, at 5:36 PM, stefan.johansson at oracle.com wrote:
>>> Please review this rename of the G1 Young RemSet Sampling thread.
> Nice catch, updated in:
> Full: http://cr.openjdk.java.net/~sjohanss/8252141/01/
> Inc: http://cr.openjdk.java.net/~sjohanss/8252141/00-01/
> I also filed a issue to take care of the name at some point:
out->print_cr(" Concurrent sampling threads times (s)");
out->print_cr(" %5.2f", service_thread_vtime());
should probably be "Service thread time (s)".
- if you are bored, some copyright dates need updates
- pre-existing: maybe the includes in G1ServiceThread.hpp could be
fixed; at least runtime/mutex.hpp should be included for Monitor.
More information about the hotspot-gc-dev