[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8275445: RunThese30M.java failed "assert(ZAddress::is_marked(addr)) failed: Should be ma
From:       Markus =?UTF-8?B?R3LDtm5sdW5k?= <mgronlun () openjdk ! java ! net>
Date:       2021-10-19 13:03:57
Message-ID: 6tkPfLd6OEyTujY4e03WWYA6RmCFUynq9Rfp7Sd69Vg=.90d78563-e326-48bd-8203-c9c961e0907f () github ! com
[Download RAW message or body]

On Tue, 19 Oct 2021 09:39:06 GMT, Markus Grönlund <mgronlun@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@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


[prev in list] [next in list] [prev in thread] [next in thread]