Integrated: 8298376: ZGC: thaws stackChunk with stale oops
Stefan Karlsson
stefank at openjdk.org
Wed Dec 14 14:37:53 UTC 2022
On Thu, 8 Dec 2022 11:05:10 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
> During the recent ZGC + Loom rewrites JDK-8296875 a bug was introduced, which causes Loom to thaw stackChunks with stale oops.
>
> Before that change we made sure that only the GC performed load barriers on the cont oop in stackChunk, so that we could use the oop state as an indication of the age of the stackChunk. That information was then used in ZCollectedHeap::requires_barriers.
>
> In Generational ZGC we changed the implementation of requires_barriers, so that the rest of the JVM didn't need to have special-handling of the cont oop. However, when upstreaming the changes in JDK-8296875 we purposefully didn't update requires_barriers, but we incorrectly brought over the removal of the special-handling of the cont oop.
>
> To limit the fix for this bug, I propose that we restore the special-handling of the cont oop.
>
> I've tested this with a reproducer running Fuzz.java + ZGC.
This pull request has now been integrated.
Changeset: ed8a2120
Author: Stefan Karlsson <stefank at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/ed8a2120ca1e9756c6ab5eeebfe24c15d549f04e
Stats: 19 lines in 4 files changed: 15 ins; 0 del; 4 mod
8298376: ZGC: thaws stackChunk with stale oops
Reviewed-by: eosterlund, pchilanomate
-------------
PR: https://git.openjdk.org/jdk/pull/11585
More information about the hotspot-dev
mailing list