RFR: 8306843: JVMTI tag map extremely slow after JDK-8292741 [v4]

Serguei Spitsyn sspitsyn at openjdk.org
Tue May 9 16:52:40 UTC 2023


On Tue, 9 May 2023 14:02:26 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> The ResourceHashtable conversion for JDK-8292741 didn't add the resizing code.  The old hashtable code was tuned for resizing in anticipation of large hashtables for JVMTI tags.  This patch ports over the old hashtable resizing code.  It also adds a ResourceHashtable::put_fast() function that prepends to the bucket list, which is also reclaims the performance of the old hashtable for this test with 10M tags.  The ResourceHashtable put function is really a put_if_absent. This can be cleaned up in a future change.  Also, the remove function needed a lambda to destroy the WeakHandle, since resizing requires copying entries.
>> 
>> Tested with JVMTI and JDI tests locally, and tier1-4 tests.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   One line and comment making obj null in copy constructor.

Thank you for the update.
It looks okay to me.
Thanks,
Serguei

-------------

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13818#pullrequestreview-1419067300


More information about the hotspot-dev mailing list