Paths

Table of Contentst

Differential D15191

Export a breakpoint() function to userland.
ClosedPublic
Actions

Authored by jhb on Apr 25 2018, 2:43 AM.

Tags

None

Referenced Files

	F175726295: D15191.id.diff
	Mon, Oct 12, 8:07 PM

	F175724801: D15191.id44979.diff
	Mon, Oct 12, 7:48 PM

	F175724789: D15191.id44667.diff
	Mon, Oct 12, 7:48 PM

	F175724781: D15191.id42636.diff
	Mon, Oct 12, 7:48 PM

	F175724771: D15191.id41829.diff
	Mon, Oct 12, 7:48 PM

	Unknown Object (File)
	Tue, Sep 29, 12:40 PM

	Unknown Object (File)
	Mon, Sep 21, 2:27 PM

	Unknown Object (File)
	Sep 11 2026, 7:19 PM

Subscribers

Details

Reviewers

br
manu
andrew

Group Reviewers

ARM
arm64

Commits

rS336053: Export a breakpoint() function to userland for arm and arm64.
rS333679: Export a breakpoint() function to userland for riscv.

Summary

As a result, expand coverage of tests using breakpoint() to more
architectures.

Test Plan

I have not tested any of these, so testing by others and/or review would be welcome. I would probably commit these separately, but they are glummed together in the branch this is part of so it's a bit easier to solicit review/testing with a combined patch.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Apr 25 2018, 2:43 AM

Herald added subscribers: emaste, andrew. · View Herald TranscriptApr 25 2018, 2:43 AM

Harbormaster completed remote builds in B16292: Diff 41829.Apr 25 2018, 2:43 AM

andrew added inline comments.Apr 25 2018, 9:50 AM

sys/arm/include/cpufunc.h
503 ↗	(On Diff #41829)	What instruction is this? You might also be able to use the `.inst` directive here.

jhb added inline comments.Apr 25 2018, 4:34 PM

sys/arm/include/cpufunc.h
503 ↗	(On Diff #41829)	This matches GDB5_BREAKPOINT from sys/arm/include/trap.h and it is what GDB uses as the default breakpoint instructions for non-thumb arm: https://github.com/bsdjhb/gdb/blob/master/gdb/arm-tdep.c#L7761 The in-kernel breakpoint() uses .word with the value of KERNEL_BREAKPOINT and I just used that as my template.

andrew added inline comments.Apr 25 2018, 5:09 PM

sys/arm/include/cpufunc.h
503 ↗	(On Diff #41829)	It seems to be `udf 0xfdee`. `udf` is an instruction that raises the undefined instruction exception.

emaste added inline comments.May 1 2018, 1:55 PM

sys/arm/include/cpufunc.h
503 ↗	(On Diff #41829)	Perhaps a comment with that reference?

br accepted this revision as: br.May 1 2018, 3:29 PM

This revision is now accepted and ready to land.May 1 2018, 3:29 PM

Closed by commit rS333679: Export a breakpoint() function to userland for riscv. (authored by jhb). · Explain WhyMay 16 2018, 4:57 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptMay 16 2018, 4:57 PM

jhb reopened this revision.May 16 2018, 6:43 PM

Updated to use 'udf' on 32-bit arm and tested on 32-bit arm.
Still need to verify arm64.

Herald added a reviewer: manu. · View Herald TranscriptJun 29 2018, 9:23 PM

Harbormaster completed remote builds in B17736: Diff 44667.Jun 29 2018, 9:23 PM

Have verified via ptrace_test on arm64 as well now, so both arm and arm64 have been runtime tested.

andrew accepted this revision.Jul 6 2018, 11:40 AM

This revision is now accepted and ready to land.Jul 6 2018, 11:40 AM

Closed by commit rS336053: Export a breakpoint() function to userland for arm and arm64. (authored by jhb). · Explain WhyJul 6 2018, 11:49 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

arm/

include/

15 lines

arm64/

include/

12 lines

tests/

sys/

kern/

11 lines

Diff 44979

head/sys/arm/include/cpufunc.h

Loading...

head/sys/arm64/include/cpufunc.h

Loading...

head/tests/sys/kern/ptrace_test.c

Loading...