JDK-8190358 : test/hotspot/jtreg/gc/logging/TestPrintReferences.java does not include balance queue time
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Affected Version: 10
  • Priority: P3
  • Status: Closed
  • Resolution: Duplicate
  • Submitted: 2017-10-30
  • Updated: 2018-05-15
  • Resolved: 2018-05-15
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
tbd_majorResolved
Related Reports
Duplicate :  
Relates :  
Relates :  
Description
When working on JDK-8190353 I realized that the verification code doesn't include time possibly spent balancing queues. This is likely because this has not been done in the past since the Full GC hasn't done parallel reference processing. That will not be true once JEP-307 is completed, here is an example when using parallel threads:
[0,379s][debug][gc,phases,ref  ] GC(13) Reference Processing: 41,8ms
[0,379s][debug][gc,phases,ref  ] GC(13)   SoftReference: 41,7ms
[0,379s][debug][gc,phases,ref  ] GC(13)     Balance queues: 0,6ms
[0,379s][debug][gc,phases,ref  ] GC(13)     Phase1: 14,3ms
[0,379s][debug][gc,phases,ref  ] GC(13)     Phase2: 13,4ms
[0,379s][debug][gc,phases,ref  ] GC(13)     Phase3: 13,4ms
[0,379s][debug][gc,phases,ref  ] GC(13)     Discovered: 457511
[0,379s][debug][gc,phases,ref  ] GC(13)     Cleared: 0

Adding the balance queue time should be fairly straight forward, using the same mechanism as for the other phases.
Comments
Balance queue time logging verification has been implemented since at least JDK-8186465.
15-05-2018