RFR(T): 8230332: G1DirtyCardQueueSet _notify_when_complete is always true
Stefan Johansson
stefan.johansson at oracle.com
Thu Aug 29 07:10:00 UTC 2019
Hi Kim,
On 2019-08-29 03:05, Kim Barrett wrote:
> Please review this trivial cleanup of G1DirtyCardQueueSet. With the
> separation of G1RedirtyCardsQueueSet from G1DirtyCardQueueSet, the
> latter is now a singleton class. As a result, the
> _notify_when_complete member (used to control whether adding completed
> buffers should notify the completed buffer monitor) is always true.
>
> This change removes that member and changes the conditional
> notifications to be unconditional. Also cleaned up some locker usage
> for _cbl_mon when notification is needed, and changed to consistently
> use notify_all() (there's no good reason to use notify(), and
> definitely no good reason to use a mix of the two here).
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8230332
>
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8230332/open.00/
Looks good, nice cleanup.
Stefan
>
> Testing:
> Local (linux-x64) hotspot:tier1
>
More information about the hotspot-gc-dev
mailing list