RFR: 8275445: RunThese30M.java failed "assert(ZAddress::is_marked(addr)) failed: Should be marked" [v2]

[Coleen Phillimore]

On Tue, 19 Oct 2021 10:05:15 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

>> Greetings,
>> 
>> This fixes the issue seen in testing when accessing an oop as part of unloading (introduced with [JDK-8266936](https://bugs.openjdk.java.net/browse/JDK-8266936)).
>> 
>> Instead, oop accesses will be done outside of unloading and the result, i.e the codesource attribute, will be cached and reused in the FinalizerEntry.
>> 
>> Testing: tier1-3, jdk_jfr
>> 
>> Thanks
>> Markus
>> 
>> PS one effect of this is that classes that unload before they have allocated anything will not have a codesource attribute. This can be fixed by letting classes register with the table as part of class loading, instead of during allocation. I will follow-up with a separate change for that.
>
> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
> 
>   header

Looks good Markus.

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

Marked as reviewed by coleenp (Reviewer).

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