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)
next prev parent 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