RFR: 8292375: Convert ProtectionDomainCacheTable to ResourceHashtable [v7]
David Holmes
dholmes at openjdk.org
Thu Sep 1 04:29:07 UTC 2022
On Wed, 31 Aug 2022 12:39:08 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> Please review this simple conversion for the ProtectionDomainCacheTable from Old Hashtable to ResourceHashtable. There are specific tests for this table in test/hotspot/jtreg/runtime/Dictionary and serviceability/dcmd/vm/DictionaryStatsTest.java.
>> Also tested with tier1-7.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix comments, add assert.
src/hotspot/share/classfile/protectionDomainCache.cpp line 186:
> 184:
> 185: // The object_no_keepalive() call peeks at the phantomly reachable oop without
> 186: // keeping it alive. This is used for traversing DictionaryEntry pd_set.
My confusion here was that the comment belonged to the `ProtectionDomainCacheEntry::object_no_keepalive` method, but you have deleted that and are now applying the comment to the `ProtectionDomainEntry::object_no_keepalive()` method.
The end result is okay.
-------------
PR: https://git.openjdk.org/jdk/pull/10043
More information about the hotspot-dev
mailing list