Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Pierre Muller" <pierre.muller@ics-cnrs.unistra.fr>
To: "'Jan Kratochvil'" <jan.kratochvil@redhat.com>,
	       "'Ulrich Weigand'" <uweigand@de.ibm.com>
Cc: "'Andreas Arnez'" <arnez@linux.vnet.ibm.com>,
	<gdb-patches@sourceware.org>
Subject: RFA: Fix build regression on 32-bit hosts  [Re: [RFA][PATCH v5 2/2] S/390: Add TDB regset]
Date: Sun, 15 Sep 2013 21:33:00 -0000	[thread overview]
Message-ID: <002701ceb25b$2e880fc0$8b982f40$@muller@ics-cnrs.unistra.fr> (raw)
In-Reply-To: <20130915155211.GA20499@host2.jankratochvil.net>

The patch below fixes the two build failure reports
due to last s390-tdep.c commit.

This is done by replacing system elf.h header inclusion
by use of GDB include/elf/common.h.
This fixes my bug report.
For Jan's problem of third parameter type,
I simply change hwcap local variable type to 
CORE_ADDR as expected for target_auxv_search function.

As and extra bonus, when looking for AT_HWCAP,
I noticed that arm-linux-tdep.c had it's own definition
instead of using the one from elf/common.h.
(This could be split from that patch and submitted separately if
requested...)


Pierre Muller


It also 
2013-09-15  Pierre Muller  <muller@sourceware.org>

        * arm-linux-tdep.c: Add "elf/common.h" header.
        Remove AT_HWCAP macro definintion as it is provided in
        added include file.
        * s390-tdep.c: Remove system header <elf.h>
        Add "elf/common.h" header for AT_HWCAP definition.
        (s390_core_read_description): Use correct CORE_ADDR
        for hwcap local variable used as third parameter
        of function target_auxv_search.

Index: src/gdb/arm-linux-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/arm-linux-tdep.c,v
retrieving revision 1.96
diff -u -p -r1.96 arm-linux-tdep.c
--- src/gdb/arm-linux-tdep.c    22 Aug 2013 20:32:54 -0000      1.96
+++ src/gdb/arm-linux-tdep.c    15 Sep 2013 21:23:40 -0000
@@ -49,12 +49,9 @@
 #include "parser-defs.h"
 #include "user-regs.h"
 #include <ctype.h>
-
+#include "elf/common.h"
 #include "gdb_string.h"

-/* This is defined in <elf.h> on ARM GNU/Linux systems.  */
-#define AT_HWCAP        16
-
 extern int arm_apcs_32;

 /* Under ARM GNU/Linux the traditional way of performing a breakpoint
Index: src/gdb/s390-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/s390-tdep.c,v
retrieving revision 1.215
diff -u -p -r1.215 s390-tdep.c
--- src/gdb/s390-tdep.c 13 Sep 2013 14:17:29 -0000      1.215
+++ src/gdb/s390-tdep.c 15 Sep 2013 21:23:43 -0000
@@ -52,7 +52,7 @@
 #include "user-regs.h"
 #include "cli/cli-utils.h"
 #include <ctype.h>
-#include <elf.h>
+#include "elf/common.h"

 #include "features/s390-linux32.c"
 #include "features/s390-linux32v1.c"
@@ -807,7 +807,7 @@ s390_core_read_description (struct gdbar
   asection *v1 = bfd_get_section_by_name (abfd, ".reg-s390-last-break");
   asection *v2 = bfd_get_section_by_name (abfd, ".reg-s390-system-call");
   asection *section = bfd_get_section_by_name (abfd, ".reg");
-  unsigned long hwcap = 0;
+  CORE_ADDR hwcap = 0;

   target_auxv_search (target, AT_HWCAP, &hwcap);
   if (!section)


  reply	other threads:[~2013-09-15 21:33 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-11 15:15 [RFA][PATCH v5 0/2] S/390: Add TDB regset support Andreas Arnez
2013-09-11 15:29 ` [RFA][PATCH v5 1/2] S/390 regmap rework Andreas Arnez
2013-09-13 12:40   ` Ulrich Weigand
2013-09-11 15:30 ` [RFA][PATCH v5 2/2] S/390: Add TDB regset Andreas Arnez
2013-09-11 16:28   ` Eli Zaretskii
2013-09-11 16:55     ` Andreas Arnez
2013-09-11 16:59       ` Eli Zaretskii
2013-09-11 17:39         ` Andreas Arnez
2013-09-11 19:21           ` Eli Zaretskii
2013-09-13 12:42   ` Ulrich Weigand
2013-09-15 15:52     ` Build regression on 32-bit hosts [Re: [RFA][PATCH v5 2/2] S/390: Add TDB regset] Jan Kratochvil
2013-09-15 21:33       ` Pierre Muller [this message]
2013-09-16  9:52         ` RFA: Fix build " Ulrich Weigand
2013-09-16 11:35           ` Pierre Muller
2013-09-16 12:23             ` Joel Brobecker
2013-09-16 13:53               ` Eli Zaretskii
     [not found]           ` <38466.0263116986$1379331313@news.gmane.org>
2013-09-17 18:39             ` Tom Tromey
2013-09-16 10:43         ` Andreas Arnez
2013-09-12 15:36 ` [RFA][PATCH v5 0/2] S/390: Add TDB regset support Andreas Arnez
2013-09-13 14:19 ` Ulrich Weigand
2013-09-15 14:42   ` Pierre Muller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='002701ceb25b$2e880fc0$8b982f40$@muller@ics-cnrs.unistra.fr' \
    --to=pierre.muller@ics-cnrs.unistra.fr \
    --cc=arnez@linux.vnet.ibm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=uweigand@de.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox