Page MenuHomeFreeBSD
Differential D33804

Remove the arm FIQ support
ClosedPublic
Actions

Authored by andrew on Jan 10 2022, 2:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 6, 10:13 PM
Unknown Object (File)
Sun, Sep 6, 5:41 AM
Unknown Object (File)
Sun, Sep 6, 5:11 AM
Unknown Object (File)
Sat, Sep 5, 7:56 AM
Unknown Object (File)
Sat, Sep 5, 4:33 AM
Unknown Object (File)
Sat, Sep 5, 3:08 AM
Unknown Object (File)
Sat, Sep 5, 3:08 AM
Unknown Object (File)
Sat, Sep 5, 2:13 AM
View All Files
Subscribers
imp
mmel

Details

Reviewers
mmel
Group Reviewers
ARM
Commits
rG6abad5b6062d: Remove the arm FIQ support
Summary

It isn't used, and only masks/unmasks FIQs on the local CPU so will be
broken on SMP.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 43803
Build 40691: arc lint + arc unit

Event Timeline

andrew created this revision.Jan 10 2022, 2:17 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 10 2022, 2:17 PM
andrew requested review of this revision.Jan 10 2022, 2:17 PM
Harbormaster completed remote builds in B43802: Diff 101195.Jan 10 2022, 2:17 PM
andrew updated this revision to Diff 101198.Jan 10 2022, 2:43 PM

Remove more fiq bits

Harbormaster completed remote builds in B43803: Diff 101198.Jan 10 2022, 2:43 PM
mmel added a subscriber: mmel.Jan 10 2022, 2:49 PM
mmel added inline comments.
sys/arm/arm/exception.S
381–382

.fiqv should be also removed

382

and also this .space and _C_LABEL(page0_data) on the next line.
In addition, the arm_vector_init() function in machdep.c should be modified accordingly.

andrew updated this revision to Diff 139511.Jun 5 2024, 2:34 PM

Remove more FIQ support

Harbormaster completed remote builds in B58066: Diff 139511.Jun 5 2024, 2:34 PM
mmel accepted this revision.Jun 9 2024, 3:29 PM
This revision is now accepted and ready to land.Jun 9 2024, 3:29 PM
Closed by commit rG6abad5b6062d: Remove the arm FIQ support (authored by andrew). · Explain WhyJun 10 2024, 3:30 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG6abad5b6062d: Remove the arm FIQ support.

Revision Contents
Changeset List

PathSize
sys/
arm/
arm/
21 lines
include/
conf/
2 lines

Diff 101198

View Options

sys/arm/arm/exception.S

Loading...
View Options

sys/arm/arm/fiq.c

Loading...
View Options

sys/arm/arm/fiq_subr.S

Loading...
View Options

sys/arm/include/fiq.h

Loading...
View Options

sys/conf/files.arm

Loading...