Integrated: 8371923: Update LockFreeStack for Atomic<T>

Kim Barrett kbarrett at openjdk.org
Wed Nov 19 20:07:57 UTC 2025


On Fri, 14 Nov 2025 18:35:10 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this change to the `LockFreeStack` utility to allow clients to
> use `Atomic<T>` as the type of the "next" member used in the linked-list
> representation of the stack. It also continues to allow clients to use the old
> (pre-`Atomic<T>`) form where the "next" member is volatile. This allows
> clients to be updated incrementally after this change, rather than requiring
> all clients to be updated in conjunction with the update of this class. Once
> all clients have been updated, support for the old form can be removed.
> 
> The associated gtests have been updated to use `Atomic<T>`, with testing of
> the old form is no longer being done. The non-updated uses provide some
> testing, and that's all expected to go away soon. So parameterizing the gtests
> for both forms seems like a bunch of work that will just be deleted soon, with
> very little benefit.
> 
> Testing: mach5 tier1

This pull request has now been integrated.

Changeset: 6f1c5733
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6f1c5733ed4a1d1a1e099681f1f292acf827d9dc
Stats:     53 lines in 2 files changed: 21 ins; 0 del; 32 mod

8371923: Update LockFreeStack for Atomic<T>

Reviewed-by: iwalulya, dholmes

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

PR: https://git.openjdk.org/jdk/pull/28329


More information about the hotspot-dev mailing list