RFR: JDK-8318566: Heap walking functions should not use FilteredFieldStream [v4]
Serguei Spitsyn
sspitsyn at openjdk.org
Sat Feb 3 00:40:03 UTC 2024
On Sat, 3 Feb 2024 00:14:15 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
>> FilteredFieldStream used by heap walking functions to iterate through klass/superclasses/interfaces fields are known to have poor performance (see [JDK-8317692](https://bugs.openjdk.org/browse/JDK-8317692) for details).
>> Heap walking API implementation is the last user of the klasses.
>> The fix reworks iteration through klass/superclasses/interfaces fields and drops FilteredFieldStream-related code.
>> Additionally removed/updated includes of reflectionUtils.hpp.
>>
>> Testing:
>> - tier1..4;
>> - test/hotspot/jtreg/vmTestbase/nsk/jvmti (contains tests for different heap walking functions);
>> - new test from #17580 (now the test runs several times faster).
>
> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
>
> updated comment
Marked as reviewed by sspitsyn (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/17661#pullrequestreview-1860685491
More information about the serviceability-dev
mailing list