Page MenuHomeFreeBSD
Differential D38494

powerpc: remove nexus_get_cpus()
ClosedPublic
Actions

Authored by mhorne on Feb 10 2023, 5:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 5:57 PM
Unknown Object (File)
Sun, Jan 12, 2:18 PM
Unknown Object (File)
Nov 23 2024, 9:18 PM
Unknown Object (File)
Nov 20 2024, 11:06 AM
Unknown Object (File)
Nov 18 2024, 1:32 PM
Unknown Object (File)
Nov 18 2024, 1:10 PM
Unknown Object (File)
Oct 5 2024, 12:32 AM
Unknown Object (File)
Oct 3 2024, 8:10 PM
View All 45 Files
Subscribers
None

Details

Reviewers
jhb
imp
jhibbits
Commits
rG3ca605d19b2e: powerpc: remove nexus_get_cpus()
rG660609698622: powerpc: remove nexus_get_cpus()
Summary

It is identical in functionality to root_get_cpus().

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 49740
Build 46630: arc lint + arc unit

Event Timeline

mhorne requested review of this revision.Feb 10 2023, 5:36 PM
mhorne created this revision.
mhorne added a parent revision: D38493: ofwbus: only allow unit number zero.
Harbormaster completed remote builds in B49670: Diff 116951.Feb 10 2023, 5:36 PM
jhb added inline comments.Feb 10 2023, 6:10 PM
sys/powerpc/powerpc/nexus.c
222

This one does return all_cpus so is a duplicate.

sys/x86/x86/nexus.c
709 ↗(On Diff #116951)

root_get_cpus returns all_cpus here, not intr_cpus. The difference is that intr_cpus doesn't include SMT threads.

mhorne added inline comments.Feb 10 2023, 6:14 PM
sys/x86/x86/nexus.c
709 ↗(On Diff #116951)

Oh indeed, I read that a bit too fast.

mhorne updated this revision to Diff 117106.Feb 13 2023, 5:39 PM

Drop the x86 changes.

Harbormaster completed remote builds in B49740: Diff 117106.Feb 13 2023, 5:39 PM
mhorne retitled this revision from nexus: remove nexus_get_cpus() to powerpc: remove nexus_get_cpus().Feb 13 2023, 5:39 PM
mhorne edited the summary of this revision. (Show Details)
jhibbits accepted this revision.Feb 15 2023, 2:24 PM
This revision is now accepted and ready to land.Feb 15 2023, 2:24 PM
Closed by commit rG660609698622: powerpc: remove nexus_get_cpus() (authored by mhorne). · Explain WhyFeb 16 2023, 9:30 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rG660609698622: powerpc: remove nexus_get_cpus().
mhorne added a commit: rG3ca605d19b2e: powerpc: remove nexus_get_cpus().Mar 7 2023, 4:55 PM

Revision Contents
Changeset List

PathSize
sys/
powerpc/
powerpc/
21 lines

Diff 117106

View Options

sys/powerpc/powerpc/nexus.c

Loading...