RFR: 8243325: Cleanup TaskQueueSuper<>::peek

Kim Barrett kim.barrett at oracle.com
Tue May 5 09:17:30 UTC 2020


> On May 5, 2020, at 5:00 AM, stefan.johansson at oracle.com wrote:
> 
> Hi Kim,
> 
> On 2020-05-05 07:39, Kim Barrett wrote:
>> Please review this change to some assertions in taskTerminator that
>> are verifying all the queues are empty.  Previously this was
>> accomplished by using the peek operation on the queues, verifying all
>> returned false.  That's the only use of the peek operation.  This
>> change replaces that usage with new assert_empty helper functions, and
>> eliminates the now unused peek operations.
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8243325
>> Webrev:
>> https://cr.openjdk.java.net/~kbarrett/8243325/open.00/
> Looks good,
> Stefan
>> Testing:
>> mach5 tier1-5, in combination with fix for JDK-8244307.
>> Locally (linux-x64) hotspot:tier1 without JDK-8244307.

Thanks.




More information about the hotspot-gc-dev mailing list