[PATCH] printllval: fix 64bit unpacking on mips/o32 and xtensa
Mike Frysinger
vapier at gentoo.org
Sun May 5 22:30:21 UTC 2013
While ARM's EABI does aligning of 64bit values when making syscalls, its
OABI does not. Note that this doesn't work atm in case someone feels like
fixing it :). We don't keep tracking of the target ABI anywhere, so this
will require tweaking a few core places.
The mips/o32 ABI and xtensa arch also do 64bit aligning, so add it to the
printllval list for this.
Normally for sys_fadvise64_64 we'd handle the same list of arches, but
mips/o32 ABI is funky and doesn't shift -- it has 7 args. So just add
xtensa to it.
* file.c (sys_fadvise64_64): Add XTENSA to the shifted list.
* util.c (printllval): Add LINUX_MIPSO32 and XTENSA to the shifted list.
---
file.c | 2 +-
util.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/file.c b/file.c
index e5fc259..07f1ad9 100644
--- a/file.c
+++ b/file.c
@@ -2677,7 +2677,7 @@ sys_fadvise64_64(struct tcb *tcp)
printfd(tcp, tcp->u_arg[0]);
argn = printllval(tcp, ", %lld, ", 1);
argn = printllval(tcp, "%lld, ", argn);
-#if defined ARM || defined AARCH64 || defined POWERPC
+#if defined ARM || defined AARCH64 || defined POWERPC || defined XTENSA
printxval(advise, tcp->u_arg[1], "POSIX_FADV_???");
#else
printxval(advise, tcp->u_arg[argn], "POSIX_FADV_???");
diff --git a/util.c b/util.c
index 970faa4..f44c7b6 100644
--- a/util.c
+++ b/util.c
@@ -207,8 +207,9 @@ printllval(struct tcb *tcp, const char *format, int arg_no)
arg_no += 2;
}
#else
-# if defined(ARM) || defined(POWERPC)
+# if defined(ARM) || defined(POWERPC) || defined(LINUX_MIPSO32) || defined(XTENSA)
/* Align arg_no to the next even number. */
+ /* Note: This is broken for ARM/OABI. */
arg_no = (arg_no + 1) & 0xe;
# endif
tprintf(format, LONG_LONG(tcp->u_arg[arg_no], tcp->u_arg[arg_no + 1]));
--
1.8.2.1
More information about the Strace-devel
mailing list