Page MenuHomeFreeBSD
Differential D28132

Handle functions that use a nop in the arm64 fbt
ClosedPublic
Actions

Authored by andrew on Jan 13 2021, 11:22 AM.
Tags
None
Referenced Files
F166934256: D28132.id82202.diff
Mon, Aug 17, 8:55 PM
Unknown Object (File)
Fri, Aug 14, 10:05 PM
Unknown Object (File)
Mon, Aug 10, 5:16 PM
Unknown Object (File)
Mon, Aug 10, 12:08 PM
Unknown Object (File)
Mon, Aug 10, 11:33 AM
Unknown Object (File)
Tue, Aug 4, 3:18 AM
Unknown Object (File)
Tue, Aug 4, 2:44 AM
Unknown Object (File)
Mon, Aug 3, 4:04 AM
View All Files
Subscribers
emaste
imp

Details

Reviewers
rwatson
gnn
manu
Group Reviewers
arm64
Commits
rGefe35aecb801: Handle functions that use a nop in the arm64 fbt
rG28d945204ea1: Handle functions that use a nop in the arm64 fbt
Summary

To trace leaf asm functions we can insert a single nop instruction as
the first instruction in a function and trigger off this.

Sponsored by: Innovate UK

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36165
Build 33054: arc lint + arc unit

Event Timeline

andrew created this revision.Jan 13 2021, 11:22 AM
Herald added a reviewer: gnn. · View Herald TranscriptJan 13 2021, 11:22 AM
Herald added a reviewer: manu. · View Herald Transcript
Herald added subscribers: emaste, imp. · View Herald Transcript
Harbormaster completed remote builds in B36165: Diff 82202.Jan 13 2021, 11:22 AM
andrew requested review of this revision.Jan 13 2021, 11:22 AM
andrew added a comment.Jan 13 2021, 11:30 AM

Tested with dtrace -n ":::". This will trigger on memcpy as called by dtrace_getnanouptime, however it seems to be inlined by the in tree clang.

gnn accepted this revision.Jan 13 2021, 6:00 PM
This revision is now accepted and ready to land.Jan 13 2021, 6:00 PM
Closed by commit rG28d945204ea1: Handle functions that use a nop in the arm64 fbt (authored by andrew). · Explain WhyMar 3 2021, 2:27 PM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG28d945204ea1: Handle functions that use a nop in the arm64 fbt.
andrew added a commit: rGefe35aecb801: Handle functions that use a nop in the arm64 fbt.Feb 22 2022, 4:40 PM

Revision Contents
Changeset List

PathSize
sys/
arm64/
include/
8 lines
cddl/
contrib/
opensolaris/
uts/
common/
sys/
2 lines
dev/
dtrace/
aarch64/
5 lines
fbt/
aarch64/
51 lines

Diff 82202

View Options

sys/arm64/include/asm.h

Loading...
View Options

sys/cddl/contrib/opensolaris/uts/common/sys/dtrace.h

Loading...
View Options

sys/cddl/dev/dtrace/aarch64/dtrace_subr.c

Loading...
View Options

sys/cddl/dev/fbt/aarch64/fbt_isa.c

Loading...