RFR: 8248346: Move OopStorage mutex setup out from OopStorageSet

Stefan Karlsson stefan.karlsson at oracle.com
Fri Jun 26 09:48:10 UTC 2020


Thanks, Kim.

StefanK

On 2020-06-26 11:43, Kim Barrett wrote:
>> On Jun 25, 2020, at 3:40 PM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>
>> Hi all,
>>
>> Please review this small patch to move the OopStorage mutex creation out from oopStorageSet.cpp and put it inside the OopStorage constructor.
>>
>> https://cr.openjdk.java.net/~stefank/8248346/webrev.01
>> https://bugs.openjdk.java.net/browse/JDK-8248346
>>
>> So far I've only tested this locally with gtest. The product code should be the same, but the test used slightly different values when initializing the mutexes. That doesn't seem to affect the tests.
>>
>> Thanks,
>> StefanK
> Looks good.
>



More information about the hotspot-gc-dev mailing list