RFR(S): 8173335: Improve logging for j.l.ref.reference processing
sangheon.kim at oracle.com
Wed Aug 9 14:41:49 UTC 2017
On 08/09/2017 02:41 AM, Thomas Schatzl wrote:
> On Thu, 2017-08-03 at 23:46 -0700, sangheon wrote:
>> Hi all,
>> I had some discussions with Thomas and updated webrev with below
>> 0. Agreed to use webrev.3b.
>> 1. Moved ReferenceProcessorPhaseTimes from G1CollectedHeap to
>> - Changed to stack allocate ReferenceProcessPhaseTimes in
>> - ReferenceProcessorPhaseTimes is created from constructor of
>> 2. Simplified the condition at
>> 3. Removed ReferenceProcessorPhaseTimes::processing_is_mt() method.
>> 4. Removed bool processing_is_mt from constructor of
>> ReferenceProcessorPhaseTimes class.
>> 5. Re-based code to latest source code.
>> 6. Updated to use new log classes.
>> 7. Some clean-ups in ReferenceProcessorPhaseTimes.
>> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4 (full)
>> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4_to_3b (inc)
>> Testing: JPRT
> Please fix copyright date in g1DefaultPolicy.cpp and
> otherwise looks good. I do not need a re-review for changing above.
Okay, I will fix it before the push.
More information about the hotspot-gc-dev