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