RFR (S): 8228857: Refactor PlatformMonitor into PlatformMutex and PlatformMonitor

Per Liden per.liden at oracle.com
Tue Aug 13 09:34:08 UTC 2019


Hi David,

On 8/13/19 5:26 AM, David Holmes wrote:
> Hi Per,
> 
> Are you okay with the latest version of this change?

Yep, I'm ok with that. It seems to be the best we can do for now.

cheers,
Per

> 
> Thanks,
> David
> 
> On 9/08/2019 4:25 pm, David Holmes wrote:
>> Okay here is v2 with Kim's non-copyable adjustments and an alternate 
>> approach to the "impl" class so that the PlatformMutex doesn't have to 
>> pay for the unused pthread_cond_t. Trade-off in this scheme is that 
>> PlatformMonitor now has two "impl" pointers so consumes more space.
>>
>> Also made comment changes suggested by Dan.
>>
>> http://cr.openjdk.java.net/~dholmes/8228857/webrev.v2/
>>
>> Thanks,
>> David
>> -----


More information about the hotspot-dev mailing list