Paths

Table of Contentst

Differential D42018

arm64: Raise a SIGBUS on a user external abort
ClosedPublic
Actions

Authored by andrew on Sep 29 2023, 1:59 PM.

Tags

None

Referenced Files

	F114911813: D42018.id128003.diff
	Fri, Apr 18, 11:06 AM

	F114890527: D42018.id128000.diff
	Fri, Apr 18, 4:51 AM

	Unknown Object (File)
	Wed, Apr 16, 9:39 PM

	Unknown Object (File)
	Sat, Apr 5, 11:20 AM

	Unknown Object (File)
	Mar 4 2025, 7:22 AM

	Unknown Object (File)
	Feb 15 2025, 7:55 PM

	Unknown Object (File)
	Feb 7 2025, 10:59 PM

	Unknown Object (File)
	Feb 6 2025, 6:57 AM

View All 46 Files

Subscribers

Details

Reviewers

markj
kib
manu

Group Reviewers

Commits

rG890008575a43: arm64: Raise a SIGBUS on a user external abort
rGf323ecb4e075: arm64: Raise a SIGBUS on a user external abort
rGa69450102ff1: arm64: Raise a SIGBUS on a user external abort

Summary

When userspace triggers an external abort allow it to handle the abort
by raising a SIGBUS.

Sponsored by: Arm Ltd

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 53766
Build 50657: arc lint + arc unit

Event Timeline

andrew created this revision.Sep 29 2023, 1:59 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 29 2023, 1:59 PM

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

andrew requested review of this revision.Sep 29 2023, 1:59 PM

Harbormaster completed remote builds in B53764: Diff 128000.Sep 29 2023, 1:59 PM

markj added inline comments.Sep 29 2023, 2:19 PM

sys/arm64/arm64/trap.c
230	Does this mean that userspace can't reliably distinguish between a SIGBUS due to a vm_fault() failure and a SIGBUS due to an external abort?
239	No need to test `!lower` now.
246	Same here.

Remove unneeded lower checks

Harbormaster completed remote builds in B53766: Diff 128003.Sep 29 2023, 3:40 PM

kib added inline comments.Sep 29 2023, 5:03 PM

sys/arm64/arm64/trap.c
230	The app gets the fault address from FAR, which should be enough for it to understand the specific mapping that caused the trap. In other words, I think that there is no need for additional arch-specific BUS_XXX code for this situation.

Ping?

kib accepted this revision.Oct 11 2023, 5:38 PM

This revision is now accepted and ready to land.Oct 11 2023, 5:38 PM

markj accepted this revision.Oct 11 2023, 7:56 PM

Closed by commit rGa69450102ff1: arm64: Raise a SIGBUS on a user external abort (authored by andrew). · Explain WhyOct 16 2023, 10:48 AM

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rGa69450102ff1: arm64: Raise a SIGBUS on a user external abort.

andrew added a commit: rGf323ecb4e075: arm64: Raise a SIGBUS on a user external abort.Feb 19 2024, 4:46 PM

andrew added a commit: rG890008575a43: arm64: Raise a SIGBUS on a user external abort.

Revision Contents
Changeset List

Path

Size

sys/

arm64/

arm64/

10 lines

Diff 128003

sys/arm64/arm64/trap.c

Loading...