Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH-ppc 3/5] Add VSX support for GDBServer
@ 2008-07-25 20:16 Luis Machado
  2008-08-08 15:18 ` Luis Machado
  2008-08-14 19:54 ` Ulrich Weigand
  0 siblings, 2 replies; 6+ messages in thread
From: Luis Machado @ 2008-07-25 20:16 UTC (permalink / raw)
  To: gdb-patches

This patch adds the required bits to support VSX on GDBServer.

--- 
2008-07-25  Luis Machado  <luisgpm@br.ibm.com>

	* Makefile.in: Add required vsx dependencies.

	* linux-ppc-low: Define PPC_FEATURE_HAS_VSX.
	Declare init_registers_powerpc_vsx32l.
	Declare init_registers_powerpc_vsx64l.
	Define PTRACE_GETVSXREGS and PTRACE_SETVSXREGS.
	(ppc_arch_setup): Check for VSX in hwcap.
	(ppc_fill_vsxregset): New function.
	(ppc_store_vsxregset): New function.
	Add new VSX entry in regset_info target_regsets.

	* configure.srv: Add new VSX dependencies.

Index: gdb/gdbserver/Makefile.in
===================================================================
--- gdb.orig/gdbserver/Makefile.in	2008-07-23 09:27:56.000000000 -0700
+++ gdb/gdbserver/Makefile.in	2008-07-23 09:31:44.000000000 -0700
@@ -223,8 +223,8 @@
 	rm -f reg-cris.c reg-crisv32.c reg-x86-64-linux.c reg-xtensa.c
 	rm -f arm-with-iwmmxt.c mips-linux.c mips64-linux.c
 	rm -f powerpc-32l.c powerpc-64l.c powerpc-e500l.c
-	rm -f powerpc-altivec32l.c powerpc-altivec64l.c
-	rm -f xml-builtin.c stamp-xml
+	rm -f powerpc-altivec32l.c powerpc-vsx32l.c powerpc-altivec64l.c
+	rm -f powerpc-vsx64l.c xml-builtin.c stamp-xml
 
 maintainer-clean realclean distclean: clean
 	rm -f nm.h tm.h xm.h config.status config.h stamp-h config.log
@@ -368,6 +368,9 @@
 powerpc-altivec32l.o : powerpc-altivec32l.c $(regdef_h)
 powerpc-altivec32l.c : $(srcdir)/../regformats/rs6000/powerpc-altivec32l.dat $(regdat_sh)
 	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-altivec32l.dat powerpc-altivec32l.c
+powerpc-vsx32l.o : powerpc-vsx32l.c $(regdef_h)
+powerpc-vsx32l.c : $(srcdir)/../regformats/rs6000/powerpc-vsx32l.dat $(regdat_sh)
+	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-vsx32l.dat powerpc-vsx32l.c
 powerpc-e500l.o : powerpc-e500l.c $(regdef_h)
 powerpc-e500l.c : $(srcdir)/../regformats/rs6000/powerpc-e500l.dat $(regdat_sh)
 	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-e500l.dat powerpc-e500l.c
@@ -377,6 +380,9 @@
 powerpc-altivec64l.o : powerpc-altivec64l.c $(regdef_h)
 powerpc-altivec64l.c : $(srcdir)/../regformats/rs6000/powerpc-altivec64l.dat $(regdat_sh)
 	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-altivec64l.dat powerpc-altivec64l.c
+powerpc-vsx64l.o : powerpc-vsx64l.c $(regdef_h)
+powerpc-vsx64l.c : $(srcdir)/../regformats/rs6000/powerpc-vsx64l.dat $(regdat_sh)
+	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-vsx64l.dat powerpc-vsx64l.c
 reg-s390.o : reg-s390.c $(regdef_h)
 reg-s390.c : $(srcdir)/../regformats/reg-s390.dat $(regdat_sh)
 	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/reg-s390.dat reg-s390.c
Index: gdb/gdbserver/linux-ppc-low.c
===================================================================
--- gdb.orig/gdbserver/linux-ppc-low.c	2008-07-23 09:27:56.000000000 -0700
+++ gdb/gdbserver/linux-ppc-low.c	2008-07-23 09:31:44.000000000 -0700
@@ -25,6 +25,7 @@
 #include <asm/ptrace.h>
 
 /* These are in <asm/cputable.h> in current kernels.  */
+#define PPC_FEATURE_HAS_VSX		0x00000080
 #define PPC_FEATURE_HAS_ALTIVEC         0x10000000
 #define PPC_FEATURE_HAS_SPE             0x00800000
 
@@ -35,12 +36,16 @@
 void init_registers_powerpc_32l (void);
 /* Defined in auto-generated file powerpc-altivec32l.c.  */
 void init_registers_powerpc_altivec32l (void);
+/* Defined in auto-generated file powerpc-vsx32l.c.  */
+void init_registers_powerpc_vsx32l (void);
 /* Defined in auto-generated file powerpc-e500l.c.  */
 void init_registers_powerpc_e500l (void);
 /* Defined in auto-generated file powerpc-64l.c.  */
 void init_registers_powerpc_64l (void);
 /* Defined in auto-generated file powerpc-altivec64l.c.  */
 void init_registers_powerpc_altivec64l (void);
+/* Defined in auto-generated file powerpc-vsx64l.c.  */
+void init_registers_powerpc_vsx64l (void);
 
 #define ppc_num_regs 73
 
@@ -254,9 +259,10 @@
   if (msr < 0)
     {
       ppc_get_hwcap (&ppc_hwcap);
-      if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
+      if (ppc_hwcap & PPC_FEATURE_HAS_VSX)
+	init_registers_powerpc_vsx64l ();
+      else if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
 	init_registers_powerpc_altivec64l ();
-
       return;
     }
 #endif
@@ -265,9 +271,12 @@
   init_registers_powerpc_32l ();
 
   ppc_get_hwcap (&ppc_hwcap);
-  if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
+  if (ppc_hwcap & PPC_FEATURE_HAS_VSX)
+    init_registers_powerpc_vsx32l ();
+  else if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
     init_registers_powerpc_altivec32l ();
 
+
   /* On 32-bit machines, check for SPE registers.
      Set the low target's regmap field as appropriately.  */
 #ifndef __powerpc64__
@@ -316,6 +325,41 @@
     ppc_collect_ptrace_register (i, (char *) buf + ppc_regmap[i]);
 }
 
+#ifndef PTRACE_GETVSXREGS
+#define PTRACE_GETVSXREGS 27
+#define PTRACE_SETVSXREGS 28
+#endif
+
+#define SIZEOF_VSXREGS 32*8
+
+static void
+ppc_fill_vsxregset (void *buf)
+{
+  int i, base;
+  char *regset = buf;
+
+  if (!(ppc_hwcap & PPC_FEATURE_HAS_VSX))
+    return;
+
+  base = find_regno ("vs0h");
+  for (i = 0; i < 32; i++)
+    collect_register (base + i, &regset[i * 8]);
+}
+
+static void
+ppc_store_vsxregset (const void *buf)
+{
+  int i, base;
+  const char *regset = buf;
+
+  if (!(ppc_hwcap & PPC_FEATURE_HAS_VSX))
+    return;
+
+  base = find_regno ("vs0h");
+  for (i = 0; i < 32; i++)
+    supply_register (base + i, &regset[i * 8]);
+}
+
 #ifndef PTRACE_GETVRREGS
 #define PTRACE_GETVRREGS 18
 #define PTRACE_SETVRREGS 19
@@ -408,6 +452,8 @@
      fetch them every time, but still fall back to PTRACE_PEEKUSER for the
      general registers.  Some kernels support these, but not the newer
      PPC_PTRACE_GETREGS.  */
+  { PTRACE_GETVSXREGS, PTRACE_SETVSXREGS, SIZEOF_VSXREGS, EXTENDED_REGS,
+  ppc_fill_vsxregset, ppc_store_vsxregset },
   { PTRACE_GETVRREGS, PTRACE_SETVRREGS, SIZEOF_VRREGS, EXTENDED_REGS,
     ppc_fill_vrregset, ppc_store_vrregset },
   { PTRACE_GETEVRREGS, PTRACE_SETEVRREGS, 32 * 4 + 8 + 4, EXTENDED_REGS,
Index: gdb/gdbserver/configure.srv
===================================================================
--- gdb.orig/gdbserver/configure.srv	2008-07-23 09:27:56.000000000 -0700
+++ gdb/gdbserver/configure.srv	2008-07-23 09:31:44.000000000 -0700
@@ -108,13 +108,17 @@
 			;;
   powerpc*-*-linux*)	srv_regobj="powerpc-32l.o"
 			srv_regobj="${srv_regobj} powerpc-altivec32l.o"
+			srv_regobj="${srv_regobj} powerpc-vsx32l.o"
 			srv_regobj="${srv_regobj} powerpc-e500l.o"
 			srv_regobj="${srv_regobj} powerpc-64l.o"
 			srv_regobj="${srv_regobj} powerpc-altivec64l.o"
+			srv_regobj="${srv_regobj} powerpc-vsx64l.o"
 			srv_tgtobj="linux-low.o linux-ppc-low.o"
 			srv_xmlfiles="rs6000/powerpc-32l.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-altivec32l.xml"
+			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-vsx32l.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power-altivec.xml"
+			srv_xmlfiles="${srv_xmlfiles} rs6000/power-vsx.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power-core.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power-linux.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power-fpu.xml"
@@ -122,6 +126,7 @@
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power-spe.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-64l.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-altivec64l.xml"
+			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-vsx64l.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power64-core.xml"
 			srv_xmlfiles="${srv_xmlfiles} rs6000/power64-linux.xml"
 			srv_linux_usrregs=yes



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH-ppc 3/5] Add VSX support for GDBServer
  2008-07-25 20:16 [PATCH-ppc 3/5] Add VSX support for GDBServer Luis Machado
@ 2008-08-08 15:18 ` Luis Machado
  2008-08-14 19:54 ` Ulrich Weigand
  1 sibling, 0 replies; 6+ messages in thread
From: Luis Machado @ 2008-08-08 15:18 UTC (permalink / raw)
  To: gdb-patches

Ping?

On Fri, 2008-07-25 at 17:15 -0300, Luis Machado wrote:
> This patch adds the required bits to support VSX on GDBServer.
> 
> --- 
> 2008-07-25  Luis Machado  <luisgpm@br.ibm.com>
> 
> 	* Makefile.in: Add required vsx dependencies.
> 
> 	* linux-ppc-low: Define PPC_FEATURE_HAS_VSX.
> 	Declare init_registers_powerpc_vsx32l.
> 	Declare init_registers_powerpc_vsx64l.
> 	Define PTRACE_GETVSXREGS and PTRACE_SETVSXREGS.
> 	(ppc_arch_setup): Check for VSX in hwcap.
> 	(ppc_fill_vsxregset): New function.
> 	(ppc_store_vsxregset): New function.
> 	Add new VSX entry in regset_info target_regsets.
> 
> 	* configure.srv: Add new VSX dependencies.
> 
> Index: gdb/gdbserver/Makefile.in
> ===================================================================
> --- gdb.orig/gdbserver/Makefile.in	2008-07-23 09:27:56.000000000 -0700
> +++ gdb/gdbserver/Makefile.in	2008-07-23 09:31:44.000000000 -0700
> @@ -223,8 +223,8 @@
>  	rm -f reg-cris.c reg-crisv32.c reg-x86-64-linux.c reg-xtensa.c
>  	rm -f arm-with-iwmmxt.c mips-linux.c mips64-linux.c
>  	rm -f powerpc-32l.c powerpc-64l.c powerpc-e500l.c
> -	rm -f powerpc-altivec32l.c powerpc-altivec64l.c
> -	rm -f xml-builtin.c stamp-xml
> +	rm -f powerpc-altivec32l.c powerpc-vsx32l.c powerpc-altivec64l.c
> +	rm -f powerpc-vsx64l.c xml-builtin.c stamp-xml
> 
>  maintainer-clean realclean distclean: clean
>  	rm -f nm.h tm.h xm.h config.status config.h stamp-h config.log
> @@ -368,6 +368,9 @@
>  powerpc-altivec32l.o : powerpc-altivec32l.c $(regdef_h)
>  powerpc-altivec32l.c : $(srcdir)/../regformats/rs6000/powerpc-altivec32l.dat $(regdat_sh)
>  	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-altivec32l.dat powerpc-altivec32l.c
> +powerpc-vsx32l.o : powerpc-vsx32l.c $(regdef_h)
> +powerpc-vsx32l.c : $(srcdir)/../regformats/rs6000/powerpc-vsx32l.dat $(regdat_sh)
> +	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-vsx32l.dat powerpc-vsx32l.c
>  powerpc-e500l.o : powerpc-e500l.c $(regdef_h)
>  powerpc-e500l.c : $(srcdir)/../regformats/rs6000/powerpc-e500l.dat $(regdat_sh)
>  	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-e500l.dat powerpc-e500l.c
> @@ -377,6 +380,9 @@
>  powerpc-altivec64l.o : powerpc-altivec64l.c $(regdef_h)
>  powerpc-altivec64l.c : $(srcdir)/../regformats/rs6000/powerpc-altivec64l.dat $(regdat_sh)
>  	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-altivec64l.dat powerpc-altivec64l.c
> +powerpc-vsx64l.o : powerpc-vsx64l.c $(regdef_h)
> +powerpc-vsx64l.c : $(srcdir)/../regformats/rs6000/powerpc-vsx64l.dat $(regdat_sh)
> +	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/rs6000/powerpc-vsx64l.dat powerpc-vsx64l.c
>  reg-s390.o : reg-s390.c $(regdef_h)
>  reg-s390.c : $(srcdir)/../regformats/reg-s390.dat $(regdat_sh)
>  	$(SHELL) $(regdat_sh) $(srcdir)/../regformats/reg-s390.dat reg-s390.c
> Index: gdb/gdbserver/linux-ppc-low.c
> ===================================================================
> --- gdb.orig/gdbserver/linux-ppc-low.c	2008-07-23 09:27:56.000000000 -0700
> +++ gdb/gdbserver/linux-ppc-low.c	2008-07-23 09:31:44.000000000 -0700
> @@ -25,6 +25,7 @@
>  #include <asm/ptrace.h>
> 
>  /* These are in <asm/cputable.h> in current kernels.  */
> +#define PPC_FEATURE_HAS_VSX		0x00000080
>  #define PPC_FEATURE_HAS_ALTIVEC         0x10000000
>  #define PPC_FEATURE_HAS_SPE             0x00800000
> 
> @@ -35,12 +36,16 @@
>  void init_registers_powerpc_32l (void);
>  /* Defined in auto-generated file powerpc-altivec32l.c.  */
>  void init_registers_powerpc_altivec32l (void);
> +/* Defined in auto-generated file powerpc-vsx32l.c.  */
> +void init_registers_powerpc_vsx32l (void);
>  /* Defined in auto-generated file powerpc-e500l.c.  */
>  void init_registers_powerpc_e500l (void);
>  /* Defined in auto-generated file powerpc-64l.c.  */
>  void init_registers_powerpc_64l (void);
>  /* Defined in auto-generated file powerpc-altivec64l.c.  */
>  void init_registers_powerpc_altivec64l (void);
> +/* Defined in auto-generated file powerpc-vsx64l.c.  */
> +void init_registers_powerpc_vsx64l (void);
> 
>  #define ppc_num_regs 73
> 
> @@ -254,9 +259,10 @@
>    if (msr < 0)
>      {
>        ppc_get_hwcap (&ppc_hwcap);
> -      if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
> +      if (ppc_hwcap & PPC_FEATURE_HAS_VSX)
> +	init_registers_powerpc_vsx64l ();
> +      else if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
>  	init_registers_powerpc_altivec64l ();
> -
>        return;
>      }
>  #endif
> @@ -265,9 +271,12 @@
>    init_registers_powerpc_32l ();
> 
>    ppc_get_hwcap (&ppc_hwcap);
> -  if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
> +  if (ppc_hwcap & PPC_FEATURE_HAS_VSX)
> +    init_registers_powerpc_vsx32l ();
> +  else if (ppc_hwcap & PPC_FEATURE_HAS_ALTIVEC)
>      init_registers_powerpc_altivec32l ();
> 
> +
>    /* On 32-bit machines, check for SPE registers.
>       Set the low target's regmap field as appropriately.  */
>  #ifndef __powerpc64__
> @@ -316,6 +325,41 @@
>      ppc_collect_ptrace_register (i, (char *) buf + ppc_regmap[i]);
>  }
> 
> +#ifndef PTRACE_GETVSXREGS
> +#define PTRACE_GETVSXREGS 27
> +#define PTRACE_SETVSXREGS 28
> +#endif
> +
> +#define SIZEOF_VSXREGS 32*8
> +
> +static void
> +ppc_fill_vsxregset (void *buf)
> +{
> +  int i, base;
> +  char *regset = buf;
> +
> +  if (!(ppc_hwcap & PPC_FEATURE_HAS_VSX))
> +    return;
> +
> +  base = find_regno ("vs0h");
> +  for (i = 0; i < 32; i++)
> +    collect_register (base + i, &regset[i * 8]);
> +}
> +
> +static void
> +ppc_store_vsxregset (const void *buf)
> +{
> +  int i, base;
> +  const char *regset = buf;
> +
> +  if (!(ppc_hwcap & PPC_FEATURE_HAS_VSX))
> +    return;
> +
> +  base = find_regno ("vs0h");
> +  for (i = 0; i < 32; i++)
> +    supply_register (base + i, &regset[i * 8]);
> +}
> +
>  #ifndef PTRACE_GETVRREGS
>  #define PTRACE_GETVRREGS 18
>  #define PTRACE_SETVRREGS 19
> @@ -408,6 +452,8 @@
>       fetch them every time, but still fall back to PTRACE_PEEKUSER for the
>       general registers.  Some kernels support these, but not the newer
>       PPC_PTRACE_GETREGS.  */
> +  { PTRACE_GETVSXREGS, PTRACE_SETVSXREGS, SIZEOF_VSXREGS, EXTENDED_REGS,
> +  ppc_fill_vsxregset, ppc_store_vsxregset },
>    { PTRACE_GETVRREGS, PTRACE_SETVRREGS, SIZEOF_VRREGS, EXTENDED_REGS,
>      ppc_fill_vrregset, ppc_store_vrregset },
>    { PTRACE_GETEVRREGS, PTRACE_SETEVRREGS, 32 * 4 + 8 + 4, EXTENDED_REGS,
> Index: gdb/gdbserver/configure.srv
> ===================================================================
> --- gdb.orig/gdbserver/configure.srv	2008-07-23 09:27:56.000000000 -0700
> +++ gdb/gdbserver/configure.srv	2008-07-23 09:31:44.000000000 -0700
> @@ -108,13 +108,17 @@
>  			;;
>    powerpc*-*-linux*)	srv_regobj="powerpc-32l.o"
>  			srv_regobj="${srv_regobj} powerpc-altivec32l.o"
> +			srv_regobj="${srv_regobj} powerpc-vsx32l.o"
>  			srv_regobj="${srv_regobj} powerpc-e500l.o"
>  			srv_regobj="${srv_regobj} powerpc-64l.o"
>  			srv_regobj="${srv_regobj} powerpc-altivec64l.o"
> +			srv_regobj="${srv_regobj} powerpc-vsx64l.o"
>  			srv_tgtobj="linux-low.o linux-ppc-low.o"
>  			srv_xmlfiles="rs6000/powerpc-32l.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-altivec32l.xml"
> +			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-vsx32l.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power-altivec.xml"
> +			srv_xmlfiles="${srv_xmlfiles} rs6000/power-vsx.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power-core.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power-linux.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power-fpu.xml"
> @@ -122,6 +126,7 @@
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power-spe.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-64l.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-altivec64l.xml"
> +			srv_xmlfiles="${srv_xmlfiles} rs6000/powerpc-vsx64l.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power64-core.xml"
>  			srv_xmlfiles="${srv_xmlfiles} rs6000/power64-linux.xml"
>  			srv_linux_usrregs=yes
> 
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH-ppc 3/5] Add VSX support for GDBServer
  2008-07-25 20:16 [PATCH-ppc 3/5] Add VSX support for GDBServer Luis Machado
  2008-08-08 15:18 ` Luis Machado
@ 2008-08-14 19:54 ` Ulrich Weigand
  2008-08-15  5:13   ` Luis Machado
  1 sibling, 1 reply; 6+ messages in thread
From: Ulrich Weigand @ 2008-08-14 19:54 UTC (permalink / raw)
  To: luisgpm; +Cc: gdb-patches

Luis Machado wrote:

> 2008-07-25  Luis Machado  <luisgpm@br.ibm.com>
> 
> 	* Makefile.in: Add required vsx dependencies.
> 
> 	* linux-ppc-low: Define PPC_FEATURE_HAS_VSX.
> 	Declare init_registers_powerpc_vsx32l.
> 	Declare init_registers_powerpc_vsx64l.
> 	Define PTRACE_GETVSXREGS and PTRACE_SETVSXREGS.
> 	(ppc_arch_setup): Check for VSX in hwcap.
> 	(ppc_fill_vsxregset): New function.
> 	(ppc_store_vsxregset): New function.
> 	Add new VSX entry in regset_info target_regsets.
> 
> 	* configure.srv: Add new VSX dependencies.

This looks good to me, but Dan needs to approve gdbserver patches ...

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH-ppc 3/5] Add VSX support for GDBServer
  2008-08-14 19:54 ` Ulrich Weigand
@ 2008-08-15  5:13   ` Luis Machado
  2008-08-16 20:40     ` Daniel Jacobowitz
  0 siblings, 1 reply; 6+ messages in thread
From: Luis Machado @ 2008-08-15  5:13 UTC (permalink / raw)
  To: Ulrich Weigand; +Cc: gdb-patches, Daniel Jacobowitz

On Thu, 2008-08-14 at 21:53 +0200, Ulrich Weigand wrote:
> Luis Machado wrote:
> 
> > 2008-07-25  Luis Machado  <luisgpm@br.ibm.com>
> > 
> > 	* Makefile.in: Add required vsx dependencies.
> > 
> > 	* linux-ppc-low: Define PPC_FEATURE_HAS_VSX.
> > 	Declare init_registers_powerpc_vsx32l.
> > 	Declare init_registers_powerpc_vsx64l.
> > 	Define PTRACE_GETVSXREGS and PTRACE_SETVSXREGS.
> > 	(ppc_arch_setup): Check for VSX in hwcap.
> > 	(ppc_fill_vsxregset): New function.
> > 	(ppc_store_vsxregset): New function.
> > 	Add new VSX entry in regset_info target_regsets.
> > 
> > 	* configure.srv: Add new VSX dependencies.
> 
> This looks good to me, but Dan needs to approve gdbserver patches ...
> 
> Bye,
> Ulrich

Thanks for reviewing.

Daniel, are you OK with this one?

Regards,
Luis


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH-ppc 3/5] Add VSX support for GDBServer
  2008-08-15  5:13   ` Luis Machado
@ 2008-08-16 20:40     ` Daniel Jacobowitz
  2008-08-18 16:02       ` Luis Machado
  0 siblings, 1 reply; 6+ messages in thread
From: Daniel Jacobowitz @ 2008-08-16 20:40 UTC (permalink / raw)
  To: Luis Machado; +Cc: Ulrich Weigand, gdb-patches

On Fri, Aug 15, 2008 at 02:12:09AM -0300, Luis Machado wrote:
> Daniel, are you OK with this one?

Yes, this one's OK too.

-- 
Daniel Jacobowitz
CodeSourcery


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH-ppc 3/5] Add VSX support for GDBServer
  2008-08-16 20:40     ` Daniel Jacobowitz
@ 2008-08-18 16:02       ` Luis Machado
  0 siblings, 0 replies; 6+ messages in thread
From: Luis Machado @ 2008-08-18 16:02 UTC (permalink / raw)
  To: Daniel Jacobowitz; +Cc: Ulrich Weigand, gdb-patches

Thanks.

Checked in.

Luis
On Sat, 2008-08-16 at 16:40 -0400, Daniel Jacobowitz wrote:
> On Fri, Aug 15, 2008 at 02:12:09AM -0300, Luis Machado wrote:
> > Daniel, are you OK with this one?
> 
> Yes, this one's OK too.
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2008-08-18 16:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-25 20:16 [PATCH-ppc 3/5] Add VSX support for GDBServer Luis Machado
2008-08-08 15:18 ` Luis Machado
2008-08-14 19:54 ` Ulrich Weigand
2008-08-15  5:13   ` Luis Machado
2008-08-16 20:40     ` Daniel Jacobowitz
2008-08-18 16:02       ` Luis Machado

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox