RFR 8214821: Remove ConcurrentHashTable::get_copy

Kim Barrett kim.barrett at oracle.com
Tue Jan 29 04:04:15 UTC 2019


> On Jan 23, 2019, at 1:49 PM, Gerard Ziemski <gerard.ziemski at oracle.com> wrote:
> 
> Hi all,
> 
> Please review this simple fix that removes “ConcurrentHashTable::get_copy” API, whose only client was a single gtest. Doing so allows us to remove CONFIG::notfound()
> 
> That gtest replaces the usage of “get_copy” API with a local “cht_get_copy” method, which itself is implemented using "ConcurrentHashTable::get” API
> 
> I also took this opportunity to clean up a few method names in the test a bit.
> 
> References:
> 
> https://bugs.openjdk.java.net/browse/JDK-8214821
> http://cr.openjdk.java.net/~gziemski/8214821_rev1
> 
> Passes gtest locally on Mac and passes Mach5 hs-tier1,2,3,4,5,6
> 
> 
> Cheers

Looks good.



More information about the hotspot-runtime-dev mailing list