[jdk17u-dev] Integrated: 8298061: vmTestbase/nsk/sysdict/vm/stress/btree/btree012/btree012.java failed with "fatal error: refcount has gone to zero"

[Dmitry Chuyko]

On Wed, 5 Mar 2025 19:05:34 GMT, Dmitry Chuyko <dchuyko at openjdk.org> wrote:

> Please review the backport of the fix for JDK-8298061 to jdk17u. The bug leads to rare crashes. For instance, they cause the Spring Boot build to fail under certain circumstances. In particular on AMD EPYC 7R13 and Ubuntu 24.04.1.
> 
> Original patch doesn't apply cleanly. First, technically clear_supername() was only added by JDK-8292286 in JDK 20 and Symbol::maybe_increment/decrement_refcount were added by JDK-8291457 also in JDK 20, all before the original patch. 
> Second, PlaceholderTable::find_and_remove() actually needs what clear_supername() call does in 20 to make the fix complete (and to match find_and_add()). That second part was later reworked in JDK 21 (JDK-8302108) and after.
> 
> PlaceholderEntry::set_supername() was modified to perform same work as before under the same condition as in the original patch.
> 
> The early version of the original change by @coleenp had an implementation of the second part which is suitable for 17u so it was made a part of this backport. See https://github.com/openjdk/jdk/pull/11726
> 
> Without that second part part we did observe assert failures.
> 
> Testing: jtreg tier 1-3 on linux-amd64, dedicated stress test of the Spring Boot build.

This pull request has now been integrated.

Changeset: 41b35416
Author:    Dmitry Chuyko <dchuyko at openjdk.org>
Committer: Paul Hohensee <phh at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/41b35416615c6078e6c31ef76120a86fe56fe84d
Stats:     12 lines in 2 files changed: 9 ins; 0 del; 3 mod

8298061: vmTestbase/nsk/sysdict/vm/stress/btree/btree012/btree012.java failed with "fatal error: refcount has gone to zero"

Reviewed-by: coleenp
Backport-of: 03d992728e27bd3dcd00d1af8a7b7179281e626f

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

PR: https://git.openjdk.org/jdk17u-dev/pull/3328