Page MenuHomeFreeBSD
Differential D59180

arm64: Fix the capitalization of CurrentEL
ClosedPublic
Actions

Authored by andrew on Aug 25 2026, 2:37 PM.
Tags
None
Referenced Files
F174026904: D59180.diff
Wed, Sep 30, 1:55 AM
F173968061: D59180.id186266.diff
Tue, Sep 29, 5:04 PM
F173967845: D59180.id185034.diff
Tue, Sep 29, 5:03 PM
F173955569: D59180.diff
Tue, Sep 29, 3:25 PM
Unknown Object (File)
Sun, Sep 27, 1:43 PM
Unknown Object (File)
Thu, Sep 24, 6:47 AM
Unknown Object (File)
Thu, Sep 24, 4:42 AM
Unknown Object (File)
Sat, Sep 19, 2:44 PM
View All 49 Files
Subscribers
emaste
imp
kajetan.puchalski_arm.com

Details

Reviewers
manu
kajetan.puchalski_arm.com
Group Reviewers
arm64
Commits
rGdeb43dafdcd2: arm64: Fix the capitalization of CurrentEL
Summary

Use the same capitalization as the Arm documentation.

Sponsored by: Arm Ltd

Diff Detail

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

Event Timeline

andrew created this revision.Aug 25 2026, 2:37 PM
Herald added a reviewer: manu. · View Herald TranscriptAug 25 2026, 2:37 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
andrew requested review of this revision.Aug 25 2026, 2:37 PM
Harbormaster completed remote builds in B76153: Diff 185034.Aug 25 2026, 2:37 PM
andrew added a parent revision: D59179: arm64: Add ICC_SGI1R_IRM_VAL and use it.Aug 25 2026, 2:37 PM
andrew added a child revision: D59181: arm64: Move the CNTV_CTL_EL0 fields to armreg.h.
kajetan.puchalski_arm.com accepted this revision.Tue, Sep 8, 2:58 PM
kajetan.puchalski_arm.com added a subscriber: kajetan.puchalski_arm.com.

There is one lowercase 'currentel' in the machdep file, though in a block of all-lowercase registers so maybe we don't want to capitalise that one. LGTM otherwise.

sys/arm64/arm64/machdep.c
1000:	PRINT_REG(currentel);
This revision is now accepted and ready to land.Tue, Sep 8, 2:58 PM
andrew added a comment.Wed, Sep 9, 2:20 PM

That code probably needs to be updated, e.g. it's missing some ID registers

Closed by commit rGdeb43dafdcd2: arm64: Fix the capitalization of CurrentEL (authored by andrew). · Explain WhyWed, Sep 9, 2:23 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rGdeb43dafdcd2: arm64: Fix the capitalization of CurrentEL.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
4 lines
6 lines
include/
12 lines

Diff 185034

View Options

sys/arm64/arm64/locore.S

Loading...
View Options

sys/arm64/arm64/machdep.c

Loading...
View Options

sys/arm64/include/armreg.h

Loading...