]> git.ipfire.org Git - thirdparty/elfutils.git/commitdiff
Add frame pointer unwinding as fallback on x86_64
authorUlf Hermann <ulf.hermann@qt.io>
Fri, 10 Feb 2017 14:19:40 +0000 (15:19 +0100)
committerMark Wielaard <mark@klomp.org>
Tue, 2 May 2017 13:25:36 +0000 (15:25 +0200)
If we don't find any debug information for a given frame, we usually
cannot unwind any further. However, the binary in question might have
been compiled with frame pointers, in which case we can look up the
well known frame pointer locations in the stack snapshot and use them
to bridge the frames without debug information.

The "unwind" hook is the right place for this as it is so far only
used on s390 and called only after trying to unwind with debug
information.

Signed-off-by: Ulf Hermann <ulf.hermann@qt.io>
backends/ChangeLog
backends/Makefile.am
backends/x86_64_init.c
backends/x86_64_unwind.c [new file with mode: 0644]
tests/ChangeLog
tests/Makefile.am
tests/backtrace.x86_64.fp.core.bz2 [new file with mode: 0644]
tests/backtrace.x86_64.fp.exec.bz2 [new file with mode: 0644]
tests/run-backtrace-fp-core-x86_64.sh [new file with mode: 0755]

index c6e0e08bf598ca3d03af59b93d6fd46463d236df..04f579daacadfbeae036ff5f66cffddde3442894 100644 (file)
@@ -1,3 +1,9 @@
+2017-02-09  Ulf Hermann  <ulf.hermann@qt.io>
+
+       * x86_64_unwind.c: New file
+       * Makefile.am (x86_64_SRCS): Add x86_64_unwind.c
+       * x86_64_init.c (x86_64_init): Hook x86_64_unwind
+
 2017-04-20  Ulf Hermann <ulf.hermann@qt.io>
 
        * aarch64_initreg.c: Compile register initialization only on linux.
index b553ec34c37781f1e7ce8afe6adea739b595b661..60917b925641d40b2d138bdfd742cb1fe3cbb1ae 100644 (file)
@@ -59,7 +59,7 @@ am_libebl_sh_pic_a_OBJECTS = $(sh_SRCS:.c=.os)
 
 x86_64_SRCS = x86_64_init.c x86_64_symbol.c x86_64_corenote.c x86_64_cfi.c \
              x86_64_retval.c x86_64_regs.c i386_auxv.c x86_64_syscall.c \
-             x86_64_initreg.c x32_corenote.c
+             x86_64_initreg.c x86_64_unwind.c x32_corenote.c
 cpu_x86_64 = ../libcpu/libcpu_x86_64.a
 libebl_x86_64_pic_a_SOURCES = $(x86_64_SRCS)
 am_libebl_x86_64_pic_a_OBJECTS = $(x86_64_SRCS:.c=.os)
index cfd0158ca4ebe7d733e2ee6f49a5cef8f3a4056e..adfa4791aa6d4c976dae89993e089b35dba331a7 100644 (file)
@@ -68,6 +68,7 @@ x86_64_init (Elf *elf __attribute__ ((unused)),
   /* gcc/config/ #define DWARF_FRAME_REGISTERS.  */
   eh->frame_nregs = 17;
   HOOK (eh, set_initial_registers_tid);
+  HOOK (eh, unwind);
 
   return MODVERSION;
 }
diff --git a/backends/x86_64_unwind.c b/backends/x86_64_unwind.c
new file mode 100644 (file)
index 0000000..ade64c0
--- /dev/null
@@ -0,0 +1,86 @@
+/* Get previous frame state for an existing frame state.
+   Copyright (C) 2016 The Qt Company Ltd.
+   This file is part of elfutils.
+
+   This file is free software; you can redistribute it and/or modify
+   it under the terms of either
+
+     * the GNU Lesser General Public License as published by the Free
+       Software Foundation; either version 3 of the License, or (at
+       your option) any later version
+
+   or
+
+     * the GNU General Public License as published by the Free
+       Software Foundation; either version 2 of the License, or (at
+       your option) any later version
+
+   or both in parallel, as here.
+
+   elfutils is distributed in the hope that it will be useful, but
+   WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   General Public License for more details.
+
+   You should have received copies of the GNU General Public License and
+   the GNU Lesser General Public License along with this program.  If
+   not, see <http://www.gnu.org/licenses/>.  */
+
+#ifdef HAVE_CONFIG_H
+# include <config.h>
+#endif
+
+#include <stdlib.h>
+#include <assert.h>
+
+#define BACKEND x86_64_
+#include "libebl_CPU.h"
+
+/* There was no CFI. Maybe we happen to have a frame pointer and can unwind from that?  */
+
+bool
+x86_64_unwind (Ebl *ebl __attribute__ ((unused)),
+               Dwarf_Addr pc __attribute__ ((unused)),
+               ebl_tid_registers_t *setfunc, ebl_tid_registers_get_t *getfunc,
+               ebl_pid_memory_read_t *readfunc, void *arg,
+               bool *signal_framep __attribute__ ((unused)))
+{
+  // Register 6 is supposed to be rbp, thus the conventional frame pointer
+  const int fpReg = 6;
+  const int spReg = 7;
+
+  Dwarf_Word fp;
+  if (!getfunc(fpReg, 1, &fp, arg) || fp == 0)
+    return false;
+
+  // Try to read old sp, so that we can avoid infinite loops below
+  Dwarf_Word sp;
+  if (!getfunc(spReg, 1, &sp, arg))
+    sp = 0;
+
+  Dwarf_Word prev_fp;
+  if (!readfunc(fp, &prev_fp, arg))
+    prev_fp = 0;
+
+  Dwarf_Word ret;
+  if (!readfunc(fp + 8, &ret, arg))
+    return false;
+
+  if (!setfunc(fpReg, 1, &prev_fp, arg))
+    return false;
+
+  fp += 16; // Pop fp and return address and write result to sp
+  if (!setfunc(spReg, 1, &fp, arg))
+    return false;
+
+  if (!setfunc(-1, 1, &ret, arg))
+    return false;
+
+  // If the sp didn't move up we don't actually have a new stack
+  // frame but rather some random data that doesn't include frame
+  // pointers. Break the unwinding then.
+  if (sp >= fp)
+    return false;
+
+  return true;
+}
index 73485458a3b1481632d694c5b9449c98759ee1de..db71f8e3259ae7e212532cb289a9e5e75a48c55c 100644 (file)
@@ -1,3 +1,10 @@
+2017-02-09  Ulf Hermann  <ulf.hermann@qt.io>
+
+       * Makefile.am: Add test for unwinding with frame pointers on x86_64
+       * backtrace.x86_64.fp.core.bz2: New file
+       * backtrace.x86_64.fp.exec.bz2: New file
+       * run-backtrace-fp-core-x86_64.sh: New file
+
 2017-04-25  Mark Wielaard  <mark@klomp.org>
 
        * backtrace-subr.sh (check_backtracegen): New function.
index f6d8b0d885a8b500cb5742d8b956979d81e88ebf..b0db19f0dbb654c1a96fe1a5c16adf0a02c728eb 100644 (file)
@@ -115,6 +115,7 @@ TESTS = run-arextract.sh run-arsymtest.sh newfile test-nlist \
        run-backtrace-native.sh run-backtrace-data.sh run-backtrace-dwarf.sh \
        run-backtrace-native-biarch.sh run-backtrace-native-core.sh \
        run-backtrace-native-core-biarch.sh run-backtrace-core-x86_64.sh \
+       run-backtrace-fp-core-x86_64.sh \
        run-backtrace-core-x32.sh \
        run-backtrace-core-i386.sh run-backtrace-core-ppc.sh \
        run-backtrace-core-s390x.sh run-backtrace-core-s390.sh \
@@ -293,9 +294,11 @@ EXTRA_DIST = run-arextract.sh run-arsymtest.sh \
             run-backtrace-native.sh run-backtrace-native-biarch.sh \
             run-backtrace-native-core.sh run-backtrace-native-core-biarch.sh \
             run-backtrace-core-x86_64.sh run-backtrace-core-i386.sh \
+            run-backtrace-fp-core-x86_64.sh \
             run-backtrace-core-x32.sh \
             backtrace-subr.sh backtrace.i386.core.bz2 backtrace.i386.exec.bz2 \
             backtrace.x86_64.core.bz2 backtrace.x86_64.exec.bz2 \
+            backtrace.x86_64.fp.core.bz2 backtrace.x86_64.fp.exec.bz2 \
             backtrace.ppc.core.bz2 backtrace.ppc.exec.bz2 \
             run-backtrace-core-ppc.sh testfile66.bz2 testfile66.core.bz2 \
             backtrace.s390x.core.bz2 backtrace.s390x.exec.bz2 \
diff --git a/tests/backtrace.x86_64.fp.core.bz2 b/tests/backtrace.x86_64.fp.core.bz2
new file mode 100644 (file)
index 0000000..e773ca2
Binary files /dev/null and b/tests/backtrace.x86_64.fp.core.bz2 differ
diff --git a/tests/backtrace.x86_64.fp.exec.bz2 b/tests/backtrace.x86_64.fp.exec.bz2
new file mode 100644 (file)
index 0000000..0695845
Binary files /dev/null and b/tests/backtrace.x86_64.fp.exec.bz2 differ
diff --git a/tests/run-backtrace-fp-core-x86_64.sh b/tests/run-backtrace-fp-core-x86_64.sh
new file mode 100755 (executable)
index 0000000..348eb18
--- /dev/null
@@ -0,0 +1,29 @@
+#! /bin/bash
+# Copyright (C) 2017 The Qt Company
+# This file is part of elfutils.
+#
+# This file is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# elfutils is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+. $srcdir/backtrace-subr.sh
+
+# The binary is generated by compiling with eh_frame CFI, but with frame
+# pointers.
+#
+# gcc -static -O2 -fno-omit-frame-pointer -fno-asynchronous-unwind-tables \
+#     -D_GNU_SOURCE -pthread -o tests/backtrace.x86_64.fp.exec -I. -Ilib \
+#     tests/backtrace-child.c
+#
+# The core is generated by calling the binary with --gencore
+
+check_core x86_64.fp