Integrated: 8275445: RunThese30M.java failed "assert(ZAddress::is_marked(addr)) failed: Should be marked"

[Markus Grönlund]

On Tue, 19 Oct 2021 09:39:06 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.

This pull request has now been integrated.

Changeset: 98ab4b03
Author:    Markus Grönlund <mgronlun at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/98ab4b03b311dcd8374cbbb21a898298798750d6
Stats:     109 lines in 4 files changed: 64 ins; 44 del; 1 mod

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

Reviewed-by: egahlin, coleenp

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

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