RFR(S) 8077400: Unnecessary and incorrect "Code Cache Roots" G1 log entry
Thomas Schatzl
thomas.schatzl at oracle.com
Fri Apr 10 12:22:48 UTC 2015
Hi Mikael,
On Fri, 2015-04-10 at 14:14 +0200, Mikael Gerdin wrote:
> Hi all,
>
> When "JDK-8027962 Per-phase timing measurements for strong roots
> processing" was ported to the G1RootProcessor change it still contained
> a left over reference to "Code Cache Roots". Code Cache scanning is
> already done and measured lower in the call chain in G1.
> The current "Code Cache Roots" measurement is actually the sum of the
> UpdateRS, ScanRS and Code Root Scan times.
>
> I also took the liberty to remove the comment about the complement of
> the collection set, since it should be more clear now that the call to
> oops_into_collection_set_do is in a separate function with a fairly
> descriptive name.
>
> I also updated the TestGCLogMessages test to not expect the Code Cache
> Roots log entry.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8077400
> Webrev: http://cr.openjdk.java.net/~mgerdin/8077400/webrev.0
looks good.
Thomas
More information about the hotspot-gc-dev
mailing list