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

	Unknown Object (File)
	Mon, Jan 27, 4:35 PM

	Unknown Object (File)
	Nov 24 2024, 4:23 PM

	Unknown Object (File)
	Nov 21 2024, 6:47 AM

	Unknown Object (File)
	Oct 12 2024, 2:18 PM

	Unknown Object (File)
	Oct 12 2024, 2:04 PM

	Unknown Object (File)
	Oct 1 2024, 6:57 AM

	Unknown Object (File)
	Oct 1 2024, 2:12 AM

	Unknown Object (File)
	Sep 27 2024, 5:33 PM

View All 38 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 Not Applicable

Unit

Tests Not Applicable

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 128819

sys/arm64/arm64/trap.c

Loading...