arm: Implement atomic_testandset_acq_long as a simple wrapper
ClosedPublic
Actions

Authored by jhb on Nov 15 2024, 8:40 PM.

Details

Reviewers

kib
jrtc27
andrew

Commits

rG994b7127f670: arm: Implement atomic_testandset_acq_long as a simple wrapper
rG987c5a194418: arm: Implement atomic_testandset_acq_long as a simple wrapper

Summary

Use a memory barrier after calling the existing atomic_testandset_long
rather than using the fcmpset-based fallback version from
<sys/_atomic_subword.h>.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Nov 15 2024, 8:40 PM

Herald added a reviewer: andrew. · View Herald TranscriptNov 15 2024, 8:40 PM

Herald added a subscriber: imp. · View Herald Transcript

jhb requested review of this revision.Nov 15 2024, 8:40 PM

jhb added a parent revision: D47627: riscv: Add implementations of atomic_testand(set|clear)_(32|64|long).

Harbormaster completed remote builds in B60620: Diff 146565.Nov 15 2024, 8:40 PM

jhb added a child revision: D47629: atomic(9): Remove fcmpset-based fallback for atomic_testand(clear|set).Nov 15 2024, 8:40 PM

kib accepted this revision.Nov 16 2024, 6:58 PM

This revision is now accepted and ready to land.Nov 16 2024, 6:58 PM

Closed by commit rG987c5a194418: arm: Implement atomic_testandset_acq_long as a simple wrapper (authored by jhb). · Explain WhyNov 19 2024, 3:28 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG987c5a194418: arm: Implement atomic_testandset_acq_long as a simple wrapper.

jhb added a commit: rG994b7127f670: arm: Implement atomic_testandset_acq_long as a simple wrapper.Dec 1 2024, 4:58 AM

Revision Contents
Changeset List

Path

Size

sys/

arm/

include/

atomic.h

12 lines

Diff 146693

View Options

arm: Implement atomic_testandset_acq_long as a simple wrapperClosedPublicActions