Page MenuHomeFreeBSD
Differential D59280

dtrace/arm: Fix and de-pessimize dtrace_copy
ClosedPublic
Actions

Authored by freebsd_dev.thsi.be on Sun, Aug 30, 10:51 PM.
Tags
None
Referenced Files
F172960590: D59280.diff
Tue, Sep 22, 10:36 AM
Unknown Object (File)
Sun, Sep 20, 2:48 AM
Unknown Object (File)
Thu, Sep 17, 2:35 PM
Unknown Object (File)
Tue, Sep 15, 9:10 AM
Unknown Object (File)
Mon, Sep 14, 7:22 AM
Unknown Object (File)
Sun, Sep 13, 8:57 AM
Unknown Object (File)
Sat, Sep 12, 9:48 AM
Unknown Object (File)
Sat, Sep 12, 8:31 AM
View All 53 Files
Subscribers
imp

Details

Reviewers
andrew
fuz
gnn
markj
Commits
rGc874d312e78f: dtrace/arm: Fix and de-pessimize dtrace_copy
Summary

Use an unprivileged load to access user memory from dtrace_copy, which
is running in SVC mode.

Abort the loop if the load is trapped, as it is useless, hence
wasteful, to keep faulting on successive addresses. I believe that
this de-pessimization should also be done on aarch64 and riscv.

PR: 298064
MFC after: 1 month

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

freebsd_dev.thsi.be created this revision.Sun, Aug 30, 10:51 PM
Herald added a subscriber: imp. · View Herald TranscriptSun, Aug 30, 10:51 PM
freebsd_dev.thsi.be requested review of this revision.Sun, Aug 30, 10:51 PM
Harbormaster completed remote builds in B76339: Diff 185460.Sun, Aug 30, 10:51 PM
freebsd_dev.thsi.be added a parent revision: D59279: sys/arm: Fix DTrace trap hook.Sun, Aug 30, 10:51 PM
freebsd_dev.thsi.be added a child revision: D59281: dtrace/arm: Really trap translation faults.
freebsd_dev.thsi.be updated this revision to Diff 185502.Mon, Aug 31, 5:46 PM
Harbormaster completed remote builds in B76354: Diff 185502.Mon, Aug 31, 5:46 PM
markj accepted this revision.Wed, Sep 2, 1:15 PM
This revision is now accepted and ready to land.Wed, Sep 2, 1:15 PM
Closed by commit rGc874d312e78f: dtrace/arm: Fix and de-pessimize dtrace_copy (authored by freebsd_dev.thsi.be, committed by markj). · Explain WhyThu, Sep 3, 3:05 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rGc874d312e78f: dtrace/arm: Fix and de-pessimize dtrace_copy.
freebsd_dev.thsi.be mentioned this in D59341: dtrace/arm64: de-pessimize dtrace_copy.Thu, Sep 3, 3:13 PM

Revision Contents
Changeset List

PathSize
sys/
cddl/
dev/
dtrace/
arm/
11 lines

Diff 185706

View Options

sys/cddl/dev/dtrace/arm/dtrace_asm.S

Loading...