From owner-freebsd-current Fri Apr 12 6:54:47 2002 Delivered-To: freebsd-current@freebsd.org Received: from whale.sunbay.crimea.ua (whale.sunbay.crimea.ua [212.110.138.65]) by hub.freebsd.org (Postfix) with ESMTP id 3F0AF37B416; Fri, 12 Apr 2002 06:54:25 -0700 (PDT) Received: (from ru@localhost) by whale.sunbay.crimea.ua (8.11.6/8.11.2) id g3CDsLQ44256; Fri, 12 Apr 2002 16:54:21 +0300 (EEST) (envelope-from ru) Date: Fri, 12 Apr 2002 16:54:21 +0300 From: Ruslan Ermilov To: Dag-Erling Smorgrav Cc: current@FreeBSD.ORG, Mike Barcroft , "David O'Brien" Subject: Re: alpha tinderbox failure Message-ID: <20020412135421.GC27751@sunbay.com> References: <200204111654.g3BGsmc9069057@beast.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9Ek0hoCL9XbhcSqy" Content-Disposition: inline In-Reply-To: <200204111654.g3BGsmc9069057@beast.freebsd.org> User-Agent: Mutt/1.3.27i Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG --9Ek0hoCL9XbhcSqy Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable I see the same breakage here. The breakage has nothing to do with -Werror. Recent commit by Mike Barcroft to sys/*/endian.h is the culprit. But the actual problem is with gdb. After a lot of experimenting I've found that contrib/gdb.291/gdb/defs.h includes "nm.h" (gnu/usr.bin/binutils/gdb/${MACHINE_ARCH}/nm.h) which are too different between i386 and alpha. In the i386 case, "nm.h" includes which exposes __BSD_VISIBLE (see how these affect the *_ENDIAN macros in sys/*/endian.h). But not in the alpha case. Applying this patch exposes the same bug on i386: %%% Index: nm.h =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /home/ncvs/src/gnu/usr.bin/binutils/gdb/i386/nm.h,v retrieving revision 1.10 diff -u -p -r1.10 nm.h --- nm.h 17 Aug 2000 16:27:26 -0000 1.10 +++ nm.h 12 Apr 2002 13:36:40 -0000 @@ -133,7 +133,9 @@ extern int kernel_writablecore; if (!strcmp(STR, "kgdb")) \ kernel_debugging =3D 1; =20 +#if 0 #include +#endif #include =20 #ifdef PT_GETDBREGS %%% The solution is to fix defs.h: %%% Index: defs.h =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /home/ncvs/src/contrib/gdb.291/gdb/defs.h,v retrieving revision 1.3 diff -u -p -r1.3 defs.h --- defs.h 11 Apr 2001 16:15:19 -0000 1.3 +++ defs.h 12 Apr 2002 13:53:37 -0000 @@ -841,6 +841,8 @@ extern void free (); =20 #ifdef HAVE_ENDIAN_H #include +#else +#include #endif =20 #if !defined (BIG_ENDIAN) %%% On Thu, Apr 11, 2002 at 09:54:48AM -0700, Dag-Erling Smorgrav wrote: > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c: In function `maintenance_print_= symbols': > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c:540: warning: assignment makes p= ointer from integer without a cast > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c: In function `maintenance_print_= psymbols': > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c:777: warning: assignment makes p= ointer from integer without a cast > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c: In function `maintenance_print_= msymbols': > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/symmisc.c:925: warning: assignment makes p= ointer from integer without a cast > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/symtab.c:40: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from wait.h:93, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/target.c:32: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/thread.c:35: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/time.h:40, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/signal.h:45, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/signals.h:27, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/top.c:28: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/top.c: In function `init_main': > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/top.c:3467: warning: assignment from incom= patible pointer type > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/unistd.h:41, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/tracepoint.c:43: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/unistd.h:41, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/utils.c:26: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/fcntl.h:52, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/ser-unix.c:22: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/../../../../contrib/gdb.291/gdb/ser-tcp.c:22: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdb/alpha/freebsd-nat.c:23: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the location = of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the location = of the previous definition > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /alpha/kvm-fbsd.c: In function `kcore_open': > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= /alpha/kvm-fbsd.c:218: warning: assignment makes pointer from integer witho= ut a cast > =3D=3D=3D> gnu/usr.bin/binutils/gdbserver > In file included from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/types.h:49, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/fcntl.h:52, > from /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/de= s/tinderbox/src/alpha/usr/include/sys/file.h:41, > from /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gn= u/usr.bin/binutils/gdbserver/../../../../contrib/gdb.291/gdb/gdbserver/remo= te-utils.c:25: > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:64: warning: `LITTLE_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= server/../../../../contrib/gdb.291/gdb/defs.h:851: warning: this is the loc= ation of the previous definition > /tmp/des/obj/alpha/.amd_mnt/freefall/host/d/home/des/tinderbox/src/alpha/= usr/include/machine/endian.h:65: warning: `BIG_ENDIAN' redefined > /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binutils/gdb= server/../../../../contrib/gdb.291/gdb/defs.h:847: warning: this is the loc= ation of the previous definition > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binu= tils/gdbserver. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin/binu= tils. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu/usr.bin. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src/gnu. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src. > *** Error code 1 >=20 > Stop in /.amd_mnt/freefall/host/d/home/des/tinderbox/src. --=20 Ruslan Ermilov Sysadmin and DBA, ru@sunbay.com Sunbay Software AG, ru@FreeBSD.org FreeBSD committer, +380.652.512.251 Simferopol, Ukraine http://www.FreeBSD.org The Power To Serve http://www.oracle.com Enabling The Information Age --9Ek0hoCL9XbhcSqy Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (FreeBSD) Comment: For info see http://www.gnupg.org iD8DBQE8tucNUkv4P6juNwoRAh+1AJ9PSd9XlrU/PKoYmZJwvhFqh5uW4gCfThpP k4n2QcpqydPac34J+Mc+rbc= =tBTX -----END PGP SIGNATURE----- --9Ek0hoCL9XbhcSqy-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message