RFR(s): 8079579: Add SuspendibleThreadSetLeaver and make SuspendibleThreadSet::joint()/leave() private

Per Liden per.liden at oracle.com
Mon May 11 11:33:15 UTC 2015

Thanks David.


On 2015-05-11 12:36, David Lindholm wrote:
> Hi Per,
> Changes looks good! Nice cleanup.
> /David
> On 2015-05-07 13:13, Per Liden wrote:
>> Hi,
>> This patch adds a SuspendibleThreadSetLeaver (to accompany the
>> SuspendibleThreadSetJoiner), and get rid off all direct calls to
>> SuspendibleThreadSet::joint()/leave(). To cater for all use cases the
>> existing SuspendibleThreadSetJoiner also needed a change, with an
>> added argument to the constructor to allow the joiner/leaver to be
>> disabled.
>> The patch also does a number of renames of the sts variable to
>> sts_join, to make it distinct from the new sts_leave.
>> Webrev: http://cr.openjdk.java.net/~pliden/8079579/webrev.0/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8079579
>> Testing: jprt, gc-test-suite
>> PS. I've had this patch sitting in a local repo for quite some time.
>> Bengt and I have discussed other enhancements in this area, but I my
>> time has been taken up by other things. Instead of letting this patch
>> rot I suggest we do this enhancement as a first step.
>> cheers,
>> /Per

More information about the hotspot-gc-dev mailing list