summaryrefslogtreecommitdiffstats
path: root/sys/cddl
diff options
context:
space:
mode:
authormarkj <markj@FreeBSD.org>2015-09-11 03:31:22 +0000
committermarkj <markj@FreeBSD.org>2015-09-11 03:31:22 +0000
commitb30bc0e211f2c169b84e83127d7d57de24fb688d (patch)
treebd69e05b9aff86437d0df31f23df93b3f0537bc4 /sys/cddl
parentdfb0cc5c03efdfd24efc1469b804742fd2d9bc5d (diff)
downloadFreeBSD-src-b30bc0e211f2c169b84e83127d7d57de24fb688d.zip
FreeBSD-src-b30bc0e211f2c169b84e83127d7d57de24fb688d.tar.gz
Remove the arg0 field from struct amd64_frame. Its existence was a bug,
since on amd64 the first argument to a function is generally not on the stack. Revert an old DTrace bug fix to some code that assumed that sizeof(struct amd64_frame) == 16. Reviewed by: jhb, kib Sponsored by: EMC / Isilon Storage Division Differential Revision: https://reviews.freebsd.org/D3255
Diffstat (limited to 'sys/cddl')
-rw-r--r--sys/cddl/dev/dtrace/amd64/dtrace_isa.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/cddl/dev/dtrace/amd64/dtrace_isa.c b/sys/cddl/dev/dtrace/amd64/dtrace_isa.c
index 07a1b0a..e05932d 100644
--- a/sys/cddl/dev/dtrace/amd64/dtrace_isa.c
+++ b/sys/cddl/dev/dtrace/amd64/dtrace_isa.c
@@ -440,7 +440,7 @@ dtrace_getarg(int arg, int aframes)
}
arg -= (inreg + 1);
- stack = (uintptr_t *)fp + 2;
+ stack = (uintptr_t *)&fp[1];
load:
DTRACE_CPUFLAG_SET(CPU_DTRACE_NOFAULT);
OpenPOWER on IntegriCloud