From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29830 invoked by alias); 16 Nov 2002 00:35:01 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 29823 invoked from network); 16 Nov 2002 00:34:59 -0000 Received: from unknown (HELO localhost.redhat.com) (216.138.202.10) by sources.redhat.com with SMTP; 16 Nov 2002 00:34:59 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id 866F63E4B; Fri, 15 Nov 2002 19:34:58 -0500 (EST) Message-ID: <3DD592B2.5020304@redhat.com> Date: Fri, 15 Nov 2002 16:35:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.0) Gecko/20020824 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Duncan Roe Cc: gdb-patches@sources.redhat.com Subject: Re: Patch for bug 567 (sparc-sun-solaris2.8) References: <20021115153648.A15888@pinot> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2002-11/txt/msg00436.txt.bz2 > Hi, > > Bug 567 complains that when you build gdb for sparc-sun-solaris2.8 as a 64-bit > program, then when you use it to debug a 32-bit program things happen like: > > (gdb) p d1 > Cannot access memory at address 0xffbef7a0 > (gdb) x/fg &d1 > 0xffbef7a0: 3.2999999999999998 > > (Actually 567 shows the wrong value being printed to x/fg, but this seems to be > fixed at gcc-3.2 / gdb 5.2.1). > > It turns out that you can't "p" any kind of variable: the problem isn't limited > to double. > > The problem seemed to me that 32-bit negative stack offsets became +ve 64-bit > quantities. This would give a 33-bit address (the wanted address with an extra > "1" on the left). > > Ideally, I think gdb should just mask off the extra bit when debugging 32-bit > code. But I couldn't figure out how to do that, so the attached patch > sign-extends symbol values as they are read in. From the point of view of GDB's core, the patch below may be closer to correct than you think (don't know how well it fits into the definition of the debug info - debug maintainer problem :-). GDB converts all external (debug info, et.al.) addresses into a canonical form. That form won't involve masking but can, on ocasions, involve sign extension. This is so that GDB can correctly debug a 32 bit ABI on a 64 bit target. In such a situtation, while a pointer might be 32 bits, registers and the address space would be the full 64 bits, gdb extending everything out to the size of CORE_ADDR. This is what makes it possible for GDB to debug an o32 ABI on a MIPS 64 platform (eg IRIX 6.5). BTW, can you post the corresponding debug info? Andrew > GCC 3.2 configuration: > > /tmp/gcc-3.2/configure --prefix=/usr/local/gcc-3.2 > > GDB 5.2.1 configuration > > PATH=/usr/local/gcc-3.2/bin:$PATH > export PATH > CC="gcc -m64" > export CC > CFLAGS="-g -O2" > export CFLAGS > ./configure --prefix=/usr/local/solaris2.8_64 > > You need to put "-m64" in CC rather than CFLAGS else "make install" fails when > trying to build "chew". This has the unfortunate side-effect that "make check" > also uses "gcc -m64", i.e. it never exercises "gcc" which would build a 32-bit > program. > > Cheers ... Duncan. > > > > diff -r -u gdb-5.2.1.bu/gdb/dbxread.c gdb-5.2.1/gdb/dbxread.c > --- gdb-5.2.1.bu/gdb/dbxread.c Fri Apr 5 08:33:49 2002 > +++ gdb-5.2.1/gdb/dbxread.c Thu Nov 14 16:23:40 2002 > @@ -2591,6 +2591,27 @@ > fill_symbuf (abfd); > bufp = &symbuf[symbuf_idx++]; > INTERNALIZE_SYMBOL (nlist, bufp, abfd); > + > + /* ----------------------------------------------- */ > + /* Horrible fix for when gdb is built with "-m64" */ > + /* (sparc-sun-solaris2.8): */ > + /* sign-extend the 32-bit result in nlist.n_value. */ > + /* */ > + /* This fixes the testcase in bug 567, */ > + /* in that you can "p d1" successfully. */ > + /* Actually you can "p" *anything* */ > + /* (previously, you couldn't). */ > + /* */ > + /* I expect this will break something else, */ > + /* we'll just have to wait to see what. */ > + /* */ > + /* The proper fix is for gdb to know that it's */ > + /* working on a 32-bit program and */ > + /* truncate addresses to 32 bits before using them */ > + /* ----------------------------------------------- */ > + > + nlist.n_value = (long)(int)nlist.n_value; > + > OBJSTAT (objfile, n_stabs++); > > type = bfd_h_get_8 (abfd, bufp->e_type); > > > > /* > * From GDB bug report 567 > gcc -g -m64 -o double double.c > gcc -g -o double double.c > * > * The report says to breakpoint on the printf line > * & print the value of d1 > */ > extern int printf(const char *, ...); > > int > main() > { > double d1; > > d1 = 3.3; > printf("d1 = %f\n", d1); > > return 0; > }