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

	Unknown Object (File)
	Feb 18 2024, 7:22 PM

	Unknown Object (File)
	Feb 8 2024, 11:13 AM

	Unknown Object (File)
	Feb 7 2024, 5:47 AM

	Unknown Object (File)
	Dec 20 2023, 8:10 AM

	Unknown Object (File)
	Dec 17 2023, 5:04 PM

	Unknown Object (File)
	Dec 17 2023, 2:08 AM

	Unknown Object (File)
	Dec 4 2023, 5:29 AM

	Unknown Object (File)
	Nov 7 2023, 5:56 AM

View All 29 Files

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...