Page MenuHomeFreeBSD
Differential D51763

bhyve/arm64: Stop assuming the CPU index is MPIDR
ClosedPublic
Actions

Authored by andrew on Aug 6 2025, 4:20 PM.
Tags
None
Referenced Files
F145241516: D51763.id159964.diff
Tue, Feb 17, 12:08 PM
Unknown Object (File)
Mon, Feb 16, 8:11 AM
Unknown Object (File)
Fri, Feb 13, 3:24 AM
Unknown Object (File)
Wed, Jan 28, 2:32 AM
Unknown Object (File)
Jan 16 2026, 3:11 PM
Unknown Object (File)
Jan 16 2026, 5:25 AM
Unknown Object (File)
Jan 8 2026, 7:10 AM
Unknown Object (File)
Jan 7 2026, 7:42 PM
View All 46 Files
Subscribers
bcran
emaste
imp
rgrimes

Details

Reviewers
markj
Group Reviewers
arm64
bhyve
Commits
rG6227d0f81a5e: bhyve/arm64: Stop assuming the CPU index is MPIDR
Summary

We need the MPIDR value in a few places in userspace. Rather than
calculate it ask the kernel to give it to us. This allows us to change
how it is calculated without having to change userspace.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 66060
Build 62943: arc lint + arc unit

Event Timeline

andrew created this revision.Aug 6 2025, 4:20 PM
Herald added subscribers: bcran, rgrimes, emaste, imp. · View Herald TranscriptAug 6 2025, 4:20 PM
andrew requested review of this revision.Aug 6 2025, 4:20 PM
Harbormaster completed remote builds in B66021: Diff 159877.Aug 6 2025, 4:20 PM
andrew added a parent revision: D51762: arm64/vmm: Support reading MPIDR_EL1.Aug 6 2025, 4:20 PM
andrew added a child revision: D51764: arm64/vmm: Support PMU v3p9.
markj added inline comments.Aug 6 2025, 6:09 PM
usr.sbin/bhyve/aarch64/bhyverun_machdep.c
380

We should check for an error, even if we just assert error == 0.

usr.sbin/bhyve/aarch64/vmexit.c
174–175

This TODO comment can be dropped, no?

andrew updated this revision to Diff 159963.Aug 7 2025, 12:56 PM

Update based on feedback from @markj
Add a missing header file

Harbormaster completed remote builds in B66059: Diff 159963.Aug 7 2025, 12:56 PM
andrew updated this revision to Diff 159964.Aug 7 2025, 2:28 PM

Rebase past 5d56a04d1b99c0187c7799be0d1e93586312bd55

Harbormaster completed remote builds in B66060: Diff 159964.Aug 7 2025, 2:28 PM
markj accepted this revision as: markj.Aug 7 2025, 2:47 PM
This revision is now accepted and ready to land.Aug 7 2025, 2:47 PM
Closed by commit rG6227d0f81a5e: bhyve/arm64: Stop assuming the CPU index is MPIDR (authored by andrew). · Explain WhyAug 7 2025, 4:42 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG6227d0f81a5e: bhyve/arm64: Stop assuming the CPU index is MPIDR.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
aarch64/
1 line
21 lines
3 lines
23 lines

Diff 159964

View Options

usr.sbin/bhyve/aarch64/bhyve_machdep.h

Loading...
View Options

usr.sbin/bhyve/aarch64/bhyverun_machdep.c

Loading...
View Options

usr.sbin/bhyve/aarch64/fdt.c

Loading...
View Options

usr.sbin/bhyve/aarch64/vmexit.c

Loading...