RFR (M): 6672778: G1 should trim task queues more aggressively during evacuation pauses

Stefan Johansson stefan.johansson at oracle.com
Thu Apr 19 09:27:42 UTC 2018



On 2018-04-19 10:09, Thomas Schatzl wrote:
> Hi all,
> 
>    I unfortunately found another issue with timing:
> 
> when calculating the "Other" time, we add the SATBFiltering phase to
> the known worker time - however that time is already included in the
> ext root scan time, so it is double-counted, and you occasionally get
> slightly negative "Other" times.
> 
> This problem had been introduced in the refactoring too :(
> 
> This change fixes that.
> 
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/6672778/webrev.3_to_4/ (diff)
> http://cr.openjdk.java.net/~tschatzl/6672778/webrev.4/ (full)
Good that you caught that! Still good.

Thanks,
Stefan

> 
> Thanks,
>    Thomas
> 
> On Mon, 2018-04-16 at 14:58 +0200, Stefan Johansson wrote:
>> Hi Thomas,
>>
>> On 2018-04-16 13:08, Thomas Schatzl wrote:
>>> Hi all,
>>>
>>> On Fri, 2018-04-13 at 14:25 +0200, Stefan Johansson wrote:
>>>>
>>>> On 2018-04-13 10:35, Thomas Schatzl wrote:
>>>>> Hi Stefan,
>>>>>
>>>>>      thanks for your review... :)
>>>>>
>>>>> On Thu, 2018-04-12 at 17:15 +0200, Stefan Johansson wrote:
>>>>>> Hi Thomas,
>>>>>>
>>>>>> On 2018-04-11 13:46, Thomas Schatzl wrote:
>>>>>>> Hi all,
>>>>>>>
>>>>>>>       I updated and (hopefully) improved the change a bit
>>>>>>> after
>>>>>>> some
>>>>>>> more thinking.
>>>>>
>>>>> [...]
>>>
>>> Also fixed a problem with the "-" operator of Tickspan introduced
>>> in
>>> all this refactoring. This caused negative times being reported
>>> sometimes in the logs.
>>>
>>> http://cr.openjdk.java.net/~tschatzl/6672778/webrev.2_to_3/ (diff)
>>> http://cr.openjdk.java.net/~tschatzl/6672778/webrev.3/ (full)
>>>
>>> The change looks really nice now imho, thanks Stefan!
>>
>> I agree, ship it :)
>>
>> Thanks,
>> Stefan
>>>
>>> Thanks,
>>>     Thomas
>>>
> 



More information about the hotspot-gc-dev mailing list