Integrated: 8270870: Simplify G1ServiceThread

Kim Barrett kbarrett at openjdk.java.net
Fri Jul 23 12:18:35 UTC 2021


On Thu, 22 Jul 2021 10:15:08 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this simplification of the internal control structure of
> G1ServiceThread, particularly around the entry and exit of monitor contexts.
> 
> In addition, the run_task logging now includes the delay between the
> scheduled time to run and the actual, as that seems like useful information
> to have sometimes.
> 
> I also changed the monitor rank from nonleaf to leaf.  There was no reason
> to use nonleaf, and leaf avoids questions about why nonleaf was used.
> 
> Testing:
> mach5 tier1-3.  This includes the gtest for G1ServiceThread.
> 
> Ran a test program with gc+task logging enabled and checked that the logging
> output looked okay.

This pull request has now been integrated.

Changeset: 0cc4bb72
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/0cc4bb729e3746537e0983a8f2665044ad2689b1
Stats:     109 lines in 3 files changed: 20 ins; 36 del; 53 mod

8270870: Simplify G1ServiceThread

Reviewed-by: tschatzl, iwalulya

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

PR: https://git.openjdk.java.net/jdk/pull/4868


More information about the hotspot-gc-dev mailing list