Page MenuHomeFreeBSD
Differential D21888

riscv: use the common sub-word {,f}cmpset implementation
ClosedPublic
Actions

Authored by kevans on Oct 3 2019, 7:25 PM.
Tags
None
Referenced Files
F82793515: D21888.id.diff
Thu, May 2, 3:47 PM
F82793156: D21888.id62950.diff
Thu, May 2, 3:43 PM
F82793151: D21888.id62880.diff
Thu, May 2, 3:43 PM
F82789226: D21888.diff
Thu, May 2, 3:06 PM
Unknown Object (File)
Thu, Apr 25, 3:31 PM
Unknown Object (File)
Dec 20 2023, 1:37 AM
Unknown Object (File)
Nov 14 2023, 2:33 PM
Unknown Object (File)
Nov 8 2023, 8:20 PM
View All 20 Files
Subscribers
imp

Details

Reviewers
mhorne
markj
Commits
rS353131: riscv: use the common sub-word {,f}cmpset implementation
Summary

(For now? I do not know if riscv includes necessary instructions to do sub-word atomics)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kevans created this revision.Oct 3 2019, 7:25 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 3 2019, 7:25 PM
Harbormaster completed remote builds in B26851: Diff 62880.Oct 3 2019, 7:25 PM
mhorne accepted this revision.Oct 4 2019, 2:35 AM

Looks good to me. There are no sub-word atomic instructions for RISC-V.

This revision is now accepted and ready to land.Oct 4 2019, 2:35 AM
Closed by commit rS353131: riscv: use the common sub-word {,f}cmpset implementation (authored by kevans). · Explain WhyOct 6 2019, 1:35 AM
This revision was automatically updated to reflect the committed changes.
kevans added a commit: rS353131: riscv: use the common sub-word {,f}cmpset implementation.

Revision Contents
Changeset List

PathSize
head/
sys/
riscv/
include/
154 lines

Diff 62950

View Options

head/sys/riscv/include/atomic.h

Loading...