Page MenuHomeFreeBSD
Differential D45171

arm64: make SPE regs use ALT_NAME macro
ClosedPublic
Actions

Authored by zachary.leaf_arm.com on May 13 2024, 9:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 2, 11:09 PM
Unknown Object (File)
Wed, Sep 2, 11:07 PM
Unknown Object (File)
Wed, Sep 2, 11:06 PM
Unknown Object (File)
Wed, Sep 2, 6:12 PM
Unknown Object (File)
Wed, Sep 2, 11:12 AM
Unknown Object (File)
Mon, Aug 31, 4:50 PM
Unknown Object (File)
Sat, Aug 29, 3:25 PM
Unknown Object (File)
Sat, Aug 29, 3:23 PM
View All Files
Subscribers
emaste
imp

Details

Reviewers
andrew
manu
Group Reviewers
arm64
Commits
rGd8a9e188f279: arm64: make SPE regs use ALT_NAME macro
rG750fbcc0603f: arm64: make SPE regs use ALT_NAME macro
rGf7bdaa103eb8: arm64: make SPE regs use ALT_NAME macro
Summary

When the register is not defined in Armv8.0 i.e. added in a later
extension, like SPE added in v8.2, the alternative name format of:

S<op0>_<op1>_C<crn>_C<crm>_<op2>

should be used; otherwise, calls to {READ,WRITE}_SPECIALREG() will
fail.

Use the MRS_REG_ALT_NAME() macro for SPE changing hex to decimal as
required by the macro.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

zachary.leaf_arm.com created this revision.May 13 2024, 9:53 AM
Herald added a reviewer: manu. · View Herald TranscriptMay 13 2024, 9:53 AM
Herald added subscribers: emaste, imp. · View Herald Transcript
zachary.leaf_arm.com requested review of this revision.May 13 2024, 9:53 AM
Harbormaster completed remote builds in B57673: Diff 138461.May 13 2024, 9:53 AM
zachary.leaf_arm.com added a child revision: D45172: arm64: add PMBSR_MSS_{BSC,FSC} status code field.May 13 2024, 9:53 AM
andrew added inline comments.May 13 2024, 11:35 AM
sys/arm64/include/armreg.h
1638

As with D45173 keep the existing MRS_REG macro and add a _REG one with MRS_REG_ALT_NAME

zachary.leaf_arm.com added inline comments.May 13 2024, 12:55 PM
sys/arm64/include/armreg.h
1638

Would the non-_REG one ever actually be usable or useful in any way? Trying to understand the logic of keeping the existing ones. MRS_REG produces a 24 bit value e.g. 0x189ae0, the _ALT_NAME one S3_0_C9_C10_7.

andrew added inline comments.May 13 2024, 1:30 PM
sys/arm64/include/armreg.h
1638

They are used in identcpu.c when we need to provide a common value across all CPUs. In this case they are unlikely to be used, but I'd prefer to keep the macro names consistent.

zachary.leaf_arm.com updated this revision to Diff 138480.May 13 2024, 4:54 PM

As per review comments

Harbormaster completed remote builds in B57684: Diff 138480.May 13 2024, 4:54 PM
andrew accepted this revision.May 14 2024, 10:00 AM
This revision is now accepted and ready to land.May 14 2024, 10:00 AM
Closed by commit rGf7bdaa103eb8: arm64: make SPE regs use ALT_NAME macro (authored by zachary.leaf_arm.com, committed by andrew). · Explain WhyMay 17 2024, 3:12 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rGf7bdaa103eb8: arm64: make SPE regs use ALT_NAME macro.
andrew added a commit: rG750fbcc0603f: arm64: make SPE regs use ALT_NAME macro.Jul 15 2024, 12:39 PM
andrew added a commit: rGd8a9e188f279: arm64: make SPE regs use ALT_NAME macro.

Revision Contents
Changeset List

PathSize
sys/
arm64/
include/
132 lines

Diff 138651

View Options

sys/arm64/include/armreg.h

Loading...