RFR: 8301992: Embed SymbolTable CHT node [v3]
Calvin Cheung
ccheung at openjdk.org
Thu Feb 16 16:58:28 UTC 2023
On Thu, 16 Feb 2023 01:12:51 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> Oh but if it overflows the refcount to become permanent, the permanence is sticky. We won't free it, so it's not a bug.
>
> From the comment above this code, it seems we do free permanent symbols, but only when the current thread lost when competing to insert a symbol for the boot loader. So the old code is correct, and the new code would have a leak.
I've added back the `Afree()` for permanent symbols. It passed tier1 testing. Running other tiers testing currently.
-------------
PR: https://git.openjdk.org/jdk/pull/12562
More information about the hotspot-dev
mailing list