RFR: 8268139: CDS ArchiveBuilder may reference unloaded classes [v2]

Ioi Lam iklam at openjdk.java.net
Fri Jun 4 06:43:16 UTC 2021


On Thu, 3 Jun 2021 05:08:33 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Looks good. In systemDictionaryShared.cpp
> 
> `432 assert(DumpTimeTable_lock->owned_by_self(), "sanity");`
> 
> Can the above be changed to `assert_lock_strong(DumpTimeTable_lock);`?

Hi Calvin, thanks for the review. I've changed it as you suggested. This makes the code more consistent.

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

PR: https://git.openjdk.java.net/jdk/pull/4322


More information about the hotspot-runtime-dev mailing list