Detect 64-bit-ness in PowerPC Book III-E
authorYao Qi <yao@codesourcery.com>
Fri, 28 Nov 2014 01:18:26 +0000 (09:18 +0800)
committerYao Qi <yao@codesourcery.com>
Wed, 14 Jan 2015 14:28:27 +0000 (22:28 +0800)
This patch is to teach both GDB and GDBServer to detect 64-bit inferior
correctly.  We find a problem that GDBServer is unable to detect on a
e5500 core processor.  Current GDBServer assumes that MSR is a 64-bit
register, but MSR is a 32-bit register in Book III-E.  This patch is
to fix this problem by checking the right bit in MSR, in order to handle
both Book III-S and Book III-E.  In order to detect Book III-S and
Book III-E, we check the PPC_FEATURE_BOOKE from the host's HWCAP (by
getauxval on glibc >= 2.16.  If getauxval doesn't exist, we implement
the fallback by parsing /proc/self/auxv), because it should an invariant
on the same machine cross different processes.

In order to share code, I add nat/ppc-linux.c for both GDB and
GDBserver side.

gdb:

2015-01-14  Yao Qi  <yao@codesourcery.com>

* Makefile.in (ppc-linux.o): New rule.
* config/powerpc/ppc64-linux.mh (NATDEPFILES): Add ppc-linux.o.
* configure.ac: AC_CHECK_FUNCS(getauxval).
* config.in: Re-generated.
* configure: Re-generated.
* nat/ppc-linux.h [__powerpc64__] (ppc64_64bit_inferior_p):
Declare.
* nat/ppc-linux.c: New file.
* ppc-linux-nat.c (ppc_linux_target_wordsize) [__powerpc64__]:
Call ppc64_64bit_inferior_p.

gdb/gdbserver:

2015-01-14  Yao Qi  <yao@codesourcery.com>

* Makefile.in (SFILES): Add nat/ppc-linux.c.
(ppc-linux.o): New rule.
* configure.srv (powerpc*-*-linux*): Add ppc-linux.o.
* configure.ac: AC_CHECK_FUNCS(getauxval).
* config.in: Re-generated.
* configure: Re-generated.
* linux-ppc-low.c (ppc_arch_setup) [__powerpc64__]: Call
ppc64_64bit_inferior_p

16 files changed:
gdb/ChangeLog
gdb/Makefile.in
gdb/config.in
gdb/config/powerpc/ppc64-linux.mh
gdb/configure
gdb/configure.ac
gdb/gdbserver/ChangeLog
gdb/gdbserver/Makefile.in
gdb/gdbserver/config.in
gdb/gdbserver/configure
gdb/gdbserver/configure.ac
gdb/gdbserver/configure.srv
gdb/gdbserver/linux-ppc-low.c
gdb/nat/ppc-linux.c [new file with mode: 0644]
gdb/nat/ppc-linux.h
gdb/ppc-linux-nat.c

index 3b954f82ce674d5e44fbd80b3870e0c1228b43f2..4557c3ebfde12d2c204b02e0bb5404d62e6a5e9d 100644 (file)
@@ -1,3 +1,16 @@
+2015-01-14  Yao Qi  <yao@codesourcery.com>
+
+       * Makefile.in (ppc-linux.o): New rule.
+       * config/powerpc/ppc64-linux.mh (NATDEPFILES): Add ppc-linux.o.
+       * configure.ac: AC_CHECK_FUNCS(getauxval).
+       * config.in: Re-generated.
+       * configure: Re-generated.
+       * nat/ppc-linux.h [__powerpc64__] (ppc64_64bit_inferior_p):
+       Declare.
+       * nat/ppc-linux.c: New file.
+       * ppc-linux-nat.c (ppc_linux_target_wordsize) [__powerpc64__]:
+       Call ppc64_64bit_inferior_p.
+
 2015-01-14  Yao Qi  <yao@codesourcery.com>
 
        * ppc-linux-nat.c (PT_ORIG_R3, PT_TRAP): Move to
index 968dc50879684d998b2ec23141816d7b3029a0a0..f9701765ef0352c605c00a332be01dfc3d9a0c8d 100644 (file)
@@ -2270,6 +2270,10 @@ mips-linux-watch.o: ${srcdir}/nat/mips-linux-watch.c
        $(COMPILE) $(srcdir)/nat/mips-linux-watch.c
        $(POSTCOMPILE)
 
+ppc-linux.o: ${srcdir}/nat/ppc-linux.c
+       $(COMPILE) $(srcdir)/nat/ppc-linux.c
+       $(POSTCOMPILE)
+
 #
 # gdb/tui/ dependencies
 #
index 9d3f32d6b6733cd7b03afc94c2d0eea372a9d255..806cbace072d971ef84fafba1bb55835b9d64c4e 100644 (file)
 /* Define if <sys/procfs.h> has fpregset_t. */
 #undef HAVE_FPREGSET_T
 
+/* Define to 1 if you have the `getauxval' function. */
+#undef HAVE_GETAUXVAL
+
 /* Define to 1 if you have the `getgid' function. */
 #undef HAVE_GETGID
 
index 4b91408fbe678ef51b8af6c936d3542e61c3f5d2..1b3fbc76c00e331b40aee46d6baed2f0b7139284 100644 (file)
@@ -6,7 +6,7 @@ NAT_FILE= config/nm-linux.h
 NATDEPFILES= inf-ptrace.o fork-child.o \
        ppc-linux-nat.o proc-service.o linux-thread-db.o \
        linux-nat.o linux-osdata.o linux-fork.o linux-procfs.o linux-ptrace.o \
-       linux-waitpid.o
+       linux-waitpid.o ppc-linux.o
 NAT_CDEPS = $(srcdir)/proc-service.list
 
 # The PowerPC has severe limitations on TOC size, and uses them even
index 7ff74badbf7ceaeec6320a3739c25d9e86cb0eb9..fdcf215253c78583d72ad686a797101a2272b1ff 100755 (executable)
@@ -10490,7 +10490,7 @@ $as_echo "#define HAVE_WORKING_FORK 1" >>confdefs.h
 
 fi
 
-for ac_func in getrusage getuid getgid \
+for ac_func in getauxval getrusage getuid getgid \
                pipe poll pread pread64 pwrite resize_term \
                sbrk setpgid setpgrp setsid \
                sigaction sigprocmask sigsetmask socketpair \
index ec776d71e87e458dcdb4bc03bb56bc3725a2a207..cc181745bc8b921088a9429c59da7ecb8648a22b 100644 (file)
@@ -1311,7 +1311,7 @@ AC_C_BIGENDIAN
 
 AC_FUNC_MMAP
 AC_FUNC_VFORK
-AC_CHECK_FUNCS([getrusage getuid getgid \
+AC_CHECK_FUNCS([getauxval getrusage getuid getgid \
                pipe poll pread pread64 pwrite resize_term \
                sbrk setpgid setpgrp setsid \
                sigaction sigprocmask sigsetmask socketpair \
index 2b844a09dddcfe71f3283cfaa918d68c6f5badbe..f762dafd412d8af0d909baa45a7e994b8d4bf2b3 100644 (file)
@@ -1,3 +1,14 @@
+2015-01-14  Yao Qi  <yao@codesourcery.com>
+
+       * Makefile.in (SFILES): Add nat/ppc-linux.c.
+       (ppc-linux.o): New rule.
+       * configure.srv (powerpc*-*-linux*): Add ppc-linux.o.
+       * configure.ac: AC_CHECK_FUNCS(getauxval).
+       * config.in: Re-generated.
+       * configure: Re-generated.
+       * linux-ppc-low.c (ppc_arch_setup) [__powerpc64__]: Call
+       ppc64_64bit_inferior_p
+
 2015-01-14  Yao Qi  <yao@codesourcery.com>
 
        * linux-ppc-low.c: Include "nat/ppc-linux.h".
index 1ed2ec83bdf5e9d853c8d3382bed7b042a85adaa..0e442fcd973fb2f73bdd2d18516a43520f065a3b 100644 (file)
@@ -155,7 +155,7 @@ SFILES=     $(srcdir)/gdbreplay.c $(srcdir)/inferiors.c $(srcdir)/dll.c \
        $(srcdir)/linux-m32r-low.c \
        $(srcdir)/linux-m68k-low.c $(srcdir)/linux-mips-low.c \
        $(srcdir)/linux-nios2-low.c \
-       $(srcdir)/linux-ppc-low.c \
+       $(srcdir)/linux-ppc-low.c $(srcdir)/nat/ppc-linux.c \
        $(srcdir)/linux-s390-low.c \
        $(srcdir)/linux-sh-low.c $(srcdir)/linux-sparc-low.c \
        $(srcdir)/linux-x86-low.c \
@@ -581,6 +581,9 @@ linux-waitpid.o: ../nat/linux-waitpid.c
 mips-linux-watch.o: ../nat/mips-linux-watch.c
        $(COMPILE) $<
        $(POSTCOMPILE)
+ppc-linux.o: ../nat/ppc-linux.c
+       $(COMPILE) $<
+       $(POSTCOMPILE)
 
 aarch64.c : $(srcdir)/../regformats/aarch64.dat $(regdat_sh)
        $(SHELL) $(regdat_sh) $(srcdir)/../regformats/aarch64.dat aarch64.c
index fd80adc9542038c6aaf3a0edb42273ca43d479a1..8f68ed25668774cc7dd505af22b310f71afeb9c0 100644 (file)
@@ -60,6 +60,9 @@
 /* Define to 1 if you have the `fdwalk' function. */
 #undef HAVE_FDWALK
 
+/* Define to 1 if you have the `getauxval' function. */
+#undef HAVE_GETAUXVAL
+
 /* Define to 1 if you have the `getrlimit' function. */
 #undef HAVE_GETRLIMIT
 
index 45efc51e4b066d6698d6d355b4215354b9ad00c3..55bd2c5b83c241eb67f54d88c190f79cdd6295ac 100755 (executable)
@@ -4745,7 +4745,7 @@ fi
 
 done
 
-for ac_func in pread pwrite pread64
+for ac_func in getauxval pread pwrite pread64
 do :
   as_ac_var=`$as_echo "ac_cv_func_$ac_func" | $as_tr_sh`
 ac_fn_c_check_func "$LINENO" "$ac_func" "$as_ac_var"
index 02082ccb8ddc2199cd988ff857c294adf22e2a15..39e3a06764d3ab2ea4ed4d5962f5bdc0bfd82aaa 100644 (file)
@@ -90,7 +90,7 @@ AC_CHECK_HEADERS(sgtty.h termio.h termios.h sys/reg.h string.h dnl
                 fcntl.h signal.h sys/file.h dnl
                 sys/ioctl.h netinet/in.h sys/socket.h netdb.h dnl
                 netinet/tcp.h arpa/inet.h)
-AC_CHECK_FUNCS(pread pwrite pread64)
+AC_CHECK_FUNCS(getauxval pread pwrite pread64)
 
 GDB_AC_COMMON
 
index 679fc9f065b7d8313f16ea7651f14c1116a617ff..f52aee25e3dcf34bd9e09cf294cbf510b2f6e03e 100644 (file)
@@ -217,7 +217,7 @@ case "${target}" in
                        srv_regobj="${srv_regobj} powerpc-isa205-64l.o"
                        srv_regobj="${srv_regobj} powerpc-isa205-altivec64l.o"
                        srv_regobj="${srv_regobj} powerpc-isa205-vsx64l.o"
-                       srv_tgtobj="$srv_linux_obj linux-ppc-low.o"
+                       srv_tgtobj="$srv_linux_obj linux-ppc-low.o ppc-linux.o"
                        srv_xmlfiles="rs6000/powerpc-32l.xml"
                        srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-altivec32l.xml"
                        srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-cell32l.xml"
index 6e880c19376314e548a5ef98b784abae5cb7871d..188fac05ac37c22bd61ebddca612c20a0828a4c1 100644 (file)
@@ -398,13 +398,11 @@ ppc_arch_setup (void)
   current_process ()->tdesc = tdesc;
   ppc_hwcap = 0;
 
-  /* Only if the high bit of the MSR is set, we actually have
-     a 64-bit inferior.  */
   regcache = new_register_cache (tdesc);
   fetch_inferior_registers (regcache, find_regno (tdesc, "msr"));
   collect_register_by_name (regcache, "msr", &msr);
   free_register_cache (regcache);
-  if (msr < 0)
+  if (ppc64_64bit_inferior_p (msr))
     {
       ppc_get_hwcap (&ppc_hwcap);
       if (ppc_hwcap & PPC_FEATURE_CELL)
diff --git a/gdb/nat/ppc-linux.c b/gdb/nat/ppc-linux.c
new file mode 100644 (file)
index 0000000..76b3574
--- /dev/null
@@ -0,0 +1,75 @@
+/*Copyright (C) 2015 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   This program 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.
+
+   This program 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/>.  */
+
+#include "common-defs.h"
+#include "ppc-linux.h"
+#include <elf.h>
+
+#ifdef HAVE_GETAUXVAL
+#include <sys/auxv.h>
+#endif
+
+#ifdef __powerpc64__
+
+/* Get the HWCAP from the process of GDB or GDBserver.  If success,
+   save it in *VALP.   */
+
+static void
+ppc64_host_hwcap (unsigned long *valp)
+{
+#ifdef HAVE_GETAUXVAL
+  *valp = getauxval (AT_HWCAP);
+#else
+  unsigned long data[2];
+  FILE *f = fopen ("/proc/self/auxv", "r");
+
+  if (f == NULL)
+    return;
+
+  while (fread (data, sizeof (data), 1, f) > 0)
+    {
+      if (data[0] == AT_HWCAP)
+       {
+         *valp = data[1];
+         break;
+       }
+    }
+
+  fclose (f);
+#endif /* HAVE_GETAUXVAL */
+}
+
+int
+ppc64_64bit_inferior_p (long msr)
+{
+  unsigned long ppc_host_hwcap = 0;
+
+  /* Get host's HWCAP to check whether the machine is Book E.  */
+  ppc64_host_hwcap (&ppc_host_hwcap);
+
+  /* We actually have a 64-bit inferior only if the certain bit of the
+     MSR is set.  The PowerISA Book III-S MSR is different from the
+     PowerISA Book III-E MSR.  The Book III-S MSR is 64 bits wide, and
+     its MSR[SF] is the bit 0 of a 64-bit value.  Book III-E MSR is 32
+     bits wide, and its MSR[CM] is the bit 0 of a 32-bit value.   */
+  if (ppc_host_hwcap & PPC_FEATURE_BOOKE)
+    return msr & 0x80000000;
+  else
+    return msr < 0;
+}
+
+#endif
index 30d936fdb9b623c2f3733ec9439fca9f9dea072f..0ff222321cef26f85bfd445dcc1397d5547f441f 100644 (file)
 #define PTRACE_SETEVRREGS 21
 #endif
 
+#ifdef __powerpc64__
+/* Return whether the inferior is 64bit or not by checking certain bit
+   in MSR.  */
+int ppc64_64bit_inferior_p (long msr);
+#endif
+
 #endif
index c5a286bb97a07b57dfd0093bf4cf6def501f13b8..88ca49e3d449fff8b28f6ee310e7232af0a7b6a4 100644 (file)
@@ -2367,7 +2367,7 @@ ppc_linux_target_wordsize (void)
 
   errno = 0;
   msr = (long) ptrace (PTRACE_PEEKUSER, tid, PT_MSR * 8, 0);
-  if (errno == 0 && msr < 0)
+  if (errno == 0 && ppc64_64bit_inferior_p (msr))
     wordsize = 8;
 #endif