RFR(S) 8077400: Unnecessary and incorrect "Code Cache Roots" G1 log entry

Bengt Rutisson bengt.rutisson at oracle.com
Fri Apr 10 12:38:06 UTC 2015

Hi Mikael,

On 2015-04-10 14:14, 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. Thanks for finding and fixing this!


> Testing: JPRT
> /Mikael

More information about the hotspot-gc-dev mailing list