Paths

Table of Contentst

Differential D54558

arm64: Enable MOPS in userspace
Needs ReviewPublic
Actions

Authored by sarah.walker2_arm.com on Tue, Jan 6, 3:41 PM.

Tags

None

Referenced Files

	F141916469: D54558.id169311.diff
	Mon, Jan 12, 11:49 AM

	F141913039: D54558.id169311.diff
	Mon, Jan 12, 10:41 AM

	F141912222: D54558.id169311.diff
	Mon, Jan 12, 10:24 AM

	F141911950: D54558.id169311.diff
	Mon, Jan 12, 10:19 AM

	F141864420: D54558.id169311.diff
	Sun, Jan 11, 3:41 PM

	Unknown Object (File)
	Sat, Jan 10, 4:14 PM

	Unknown Object (File)
	Fri, Jan 9, 11:04 AM

	Unknown Object (File)
	Fri, Jan 9, 6:48 AM

View All 20 Files

Subscribers

Details

Reviewers

andrew
manu
jrtc27

Group Reviewers

Summary

Detect presence of FEAT_MOPS, and enable instruction set and set HWCAP2 flag
if present.

Add handler for MOE exceptions.

Sponsored by: Arm Ltd

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69710
Build 66593: arc lint + arc unit

Event Timeline

sarah.walker2_arm.com created this revision.Tue, Jan 6, 3:41 PM

Herald added a reviewer: andrew. · View Herald TranscriptTue, Jan 6, 3:41 PM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

Herald added subscribers: emaste, imp. · View Herald Transcript

sarah.walker2_arm.com requested review of this revision.Tue, Jan 6, 3:41 PM

Harbormaster completed remote builds in B69661: Diff 169188.Tue, Jan 6, 3:41 PM

sarah.walker2_arm.com added a child revision: D54559: rtld-elf: Pass HWCAP flags to ifunc resolver functions.Tue, Jan 6, 3:41 PM

andrew added a reviewer: arm64.Tue, Jan 6, 4:05 PM

andrew added inline comments.

sys/arm64/arm64/machdep.c
240	Do we need an `isb` here and below? I assume it's probably safe for userspace only MOPS. In the kernel I expect we would need it as there may not be a context synchronising event before a MOPS instruction is used.
sys/arm64/include/armreg.h
603–606	In the Arm ARM M.a.a there is a single `FormatOption` field holding the old `OptionA` and `WrongOption` fields (bits 17:16).

Address review comments. Update SHA IFUNC usage.

Harbormaster completed remote builds in B69710: Diff 169310.Thu, Jan 8, 10:49 AM

Address review comments.

Harbormaster completed remote builds in B69711: Diff 169311.Thu, Jan 8, 10:51 AM

andrew accepted this revision.Thu, Jan 8, 3:46 PM

This revision is now accepted and ready to land.Thu, Jan 8, 3:46 PM

jrtc27 requested changes to this revision.Thu, Jan 8, 3:59 PM

jrtc27 added a subscriber: jrtc27.

jrtc27 added inline comments.

sys/arm64/arm64/identcpu.c
1098–1099	Indented with spaces not tabs

This revision now requires changes to proceed.Thu, Jan 8, 3:59 PM

jrtc27 added inline comments.Thu, Jan 8, 4:02 PM

sys/arm64/include/armreg.h
603	This define through OPTION_B2 uses spaces not tabs between the name and value
604	The 5 lines starting here exceed 80 chars, though there are already various violations of that in this file...
680	Uses spaces not tab before comment

Address review comments.

Harbormaster completed remote builds in B69758: Diff 169397.Fri, Jan 9, 4:03 PM

andrew accepted this revision.Fri, Jan 9, 5:35 PM

Revision Contents
Changeset List

Path

Size

sys/

arm64/

arm64/

8 lines

35 lines

64 lines

include/

26 lines

Diff 169310

sys/arm64/arm64/identcpu.c

Loading...

sys/arm64/arm64/machdep.c

Loading...

sys/arm64/arm64/trap.c

Loading...

sys/arm64/include/armreg.h

Loading...