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