RFR: 8317240: Promptly free OopMapEntry after fail to insert the entry to OopMapCache

Thu Oct 5 13:05:07 UTC 2023

On Thu, 28 Sep 2023 12:27:52 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

> In OopMapCache::lookup(), after fail to insert newly created OopMapEntry to the cache, the new entry can be free immediately, because it is not published, not accessible by other thread.
> 
> The patch also refactored code a bit to reduce duplicate.
> 
> Test:
>    hotspot_gc on MacOSX (fastdebug and release)

LGTM

Fred

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

Marked as reviewed by fparain (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15963#pullrequestreview-1659808672