GDB 7.5.91 available for testing

GDB 7.5.91 available for testing

[Joel Brobecker]

Hello,

I have just finished creating the gdb-7.5.91 pre-release.
It is available for download at the following location:

    ftp://sourceware.org/pub/gdb/snapshots/branch/gdb-7.5.91.tar.bz2

A gzip'ed version is also available: gdb-7.5.91.tar.gz.

Please give it a test if you can and report any problems you might find.

On behalf of all the GDB contributors, thank you!

-- 
Joel

sim/erc32/Makefile.in patch - was Re: GDB 7.5.91 available for testing

[Joel Sherrill]

[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]

Hi

The sim/erc32 will not build outside the source tree. It is
missing a -I. to find config.h in the build tree.

mkdir b--gdb
cd b-gdb
../gdb-7.5.91/configure --target=sparc-rtems4.11 \
    --prefix=/home/joel/test-gdb/install/ \
    --enable-sim --enable-sim-hardware  \
     --enable-timebase --enable-sim-trace >c.log 2>&1 && \
     make >b.log 2>&1 && \
    make install >i.log 2>&1

Should be enough to reproduce it.

Can you commit this to the branch and head please?

2013-03-13  Joel Sherrill <joel.sherrill@oarcorp.com>

         * Makefile.in: Include build directory in search path to find
         config.h

Thanks.

On 3/12/2013 6:17 PM, Joel Brobecker wrote:
> Hello,
>
> I have just finished creating the gdb-7.5.91 pre-release.
> It is available for download at the following location:
>
>      ftp://sourceware.org/pub/gdb/snapshots/branch/gdb-7.5.91.tar.bz2
>
> A gzip'ed version is also available: gdb-7.5.91.tar.gz.
>
> Please give it a test if you can and report any problems you might find.
>
> On behalf of all the GDB contributors, thank you!
>


-- 
Joel Sherrill, Ph.D.             Director of Research & Development
joel.sherrill@OARcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
Support Available                (256) 722-9985


[-- Attachment #2: sim_erc32.diff --]
[-- Type: text/plain, Size: 713 bytes --]

2013-03-13  Joel Sherrill <joel.sherrill@oarcorp.com>

	* Makefile.in: Include build directory in search path to find
	config.h

Index: sim/erc32/Makefile.in
===================================================================
RCS file: /cvs/src/src/sim/erc32/Makefile.in,v
retrieving revision 1.12
diff -u -r1.12 Makefile.in
--- sim/erc32/Makefile.in	1 Jan 2013 06:41:35 -0000	1.12
+++ sim/erc32/Makefile.in	13 Mar 2013 14:18:43 -0000
@@ -45,7 +45,7 @@
 # There is AC_C_BIGENDIAN but it doesn't handle float endianness.
 # [Are int/float endians every different on a sparc?]
 end: $(srcdir)/end.c
-	$(CC_FOR_BUILD) $(srcdir)/end.c -o end
+	$(CC_FOR_BUILD) -I. $(srcdir)/end.c -o end
 end.h: end
 	./end > end.h
 

Re: sim/erc32/Makefile.in patch - was Re: GDB 7.5.91 available for testing

[Joel Brobecker]

> The sim/erc32 will not build outside the source tree. It is
> missing a -I. to find config.h in the build tree.
> 
> mkdir b--gdb
> cd b-gdb
> ../gdb-7.5.91/configure --target=sparc-rtems4.11 \
>    --prefix=/home/joel/test-gdb/install/ \
>    --enable-sim --enable-sim-hardware  \
>     --enable-timebase --enable-sim-trace >c.log 2>&1 && \
>     make >b.log 2>&1 && \
>    make install >i.log 2>&1
> 
> Should be enough to reproduce it.

Thanks. Reproduced using configure --target=sparc-elf --enable-sim.

> 2013-03-13  Joel Sherrill <joel.sherrill@oarcorp.com>
> 
>         * Makefile.in: Include build directory in search path to find
>         config.h

After a quick review of the patch, it looks reasonable to me.
But Mike F is the Global Maintainer, so I'll defer to him.

> 2013-03-13  Joel Sherrill <joel.sherrill@oarcorp.com>
> 
> 	* Makefile.in: Include build directory in search path to find
> 	config.h
> 
> Index: sim/erc32/Makefile.in
> ===================================================================
> RCS file: /cvs/src/src/sim/erc32/Makefile.in,v
> retrieving revision 1.12
> diff -u -r1.12 Makefile.in
> --- sim/erc32/Makefile.in	1 Jan 2013 06:41:35 -0000	1.12
> +++ sim/erc32/Makefile.in	13 Mar 2013 14:18:43 -0000
> @@ -45,7 +45,7 @@
>  # There is AC_C_BIGENDIAN but it doesn't handle float endianness.
>  # [Are int/float endians every different on a sparc?]
>  end: $(srcdir)/end.c
> -	$(CC_FOR_BUILD) $(srcdir)/end.c -o end
> +	$(CC_FOR_BUILD) -I. $(srcdir)/end.c -o end
>  end.h: end
>  	./end > end.h
>  


-- 
Joel

Re: sim/erc32/Makefile.in patch - was Re: GDB 7.5.91 available for testing

[Mike Frysinger]

[-- Attachment #1: Type: Text/Plain, Size: 924 bytes --]

On Wednesday 13 March 2013 14:17:59 Joel Brobecker wrote:
> > The sim/erc32 will not build outside the source tree. It is
> > missing a -I. to find config.h in the build tree.
> > 
> > mkdir b--gdb
> > cd b-gdb
> > ../gdb-7.5.91/configure --target=sparc-rtems4.11 \
> >    --prefix=/home/joel/test-gdb/install/ \
> >    --enable-sim --enable-sim-hardware  \
> >     --enable-timebase --enable-sim-trace >c.log 2>&1 && \
> >     make >b.log 2>&1 && \
> >    make install >i.log 2>&1
> > 
> > Should be enough to reproduce it.
> 
> Thanks. Reproduced using configure --target=sparc-elf --enable-sim.
> 
> > 2013-03-13  Joel Sherrill <joel.sherrill@oarcorp.com>
> > 
> >         * Makefile.in: Include build directory in search path to find
> >         config.h
> 
> After a quick review of the patch, it looks reasonable to me.
> But Mike F is the Global Maintainer, so I'll defer to him.

OK
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[gdb-7.5.91] mips sim fails to build on mingw32

[Ralf Corsepius]

Hi,

The mips sim fails to build on mingw:
...
x86_64-w64-mingw32-gcc -DHAVE_CONFIG_H     -DPROFILE=1 -DWITH_PROFILE=-1 
   -DWITH_ALIGNMENT=NONSTRICT_ALIGNMENT -DWITH_TARGET_WORD_BITSIZE=32 
-DWITH_TARGET_WORD_MSB=31 -DWITH_DEFAULT_TARGET_BYTE_ORDER=BIG_ENDIAN 
-DWITH_FLOATING_POINT=HARD_FLOATING_POINT 
-DWITH_TARGET_FLOATING_POINT_BITSIZE=32 -DWITH_HW=0 
-DWITH_HOST_BYTE_ORDER=0 -DDEFAULT_INLINE=0   -DWITH_RESERVED_BITS=1 
-DWITH_SMP=0  -Wall -Wdeclaration-after-statement -Wpointer-arith 
-Wformat-nonliteral -Wno-pointer-sign -Wno-unused -Wunused-value 
-Wunused-function -Wno-switch -Wno-char-subscripts -Wmissing-prototypes 
-Wno-format    -DMIPS_MACH_DEFAULT=bfd_mach_mips8000   -I. 
-I../../../gdb-7.5.91/sim/mips -I../common 
-I../../../gdb-7.5.91/sim/mips/../common -I../../include 
-I../../../gdb-7.5.91/sim/mips/../..//include -I../../bfd 
-I../../../gdb-7.5.91/sim/mips/../..//bfd -I../../opcodes 
-I../../../gdb-7.5.91/sim/mips/../..//opcodes  -O2 -g -pipe -Wall 
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions --param=ssp-buffer-size=4 
-mms-bitfields -D__USE_MINGW_ACCESS -c -o interp.o -MT interp.o -MMD -MP 
-MF .deps/interp.Tpo ../../../gdb-7.5.91/sim/mips/interp.c
../../../gdb-7.5.91/sim/mips/../common/dv-sockser.c:44:24: fatal error: 
netinet/in.h: No such file or directory


ATM, I assume this patch to be the cause (sim/mips):

2012-09-03 Joel Sherrill <joel.sherrill@oarcorp.com>

         * configure.ac: Always link against dv-sockser.o.
         * configure: Regenerate.


 From what I can gather from comparing sim/mips/configure.ac to other 
sim/*/configure.ac (e.g. bfin/configure.ac),
dv-sockser.o needs to be suppressed for mingw32-* targets?

Ralf

Re: [gdb-7.5.91] mips sim fails to build on mingw32

[Mike Frysinger]

[-- Attachment #1: Type: Text/Plain, Size: 1977 bytes --]

On Friday 15 March 2013 08:49:28 Ralf Corsepius wrote:
> Hi,
> 
> The mips sim fails to build on mingw:
> ...
> x86_64-w64-mingw32-gcc -DHAVE_CONFIG_H     -DPROFILE=1 -DWITH_PROFILE=-1
>    -DWITH_ALIGNMENT=NONSTRICT_ALIGNMENT -DWITH_TARGET_WORD_BITSIZE=32
> -DWITH_TARGET_WORD_MSB=31 -DWITH_DEFAULT_TARGET_BYTE_ORDER=BIG_ENDIAN
> -DWITH_FLOATING_POINT=HARD_FLOATING_POINT
> -DWITH_TARGET_FLOATING_POINT_BITSIZE=32 -DWITH_HW=0
> -DWITH_HOST_BYTE_ORDER=0 -DDEFAULT_INLINE=0   -DWITH_RESERVED_BITS=1
> -DWITH_SMP=0  -Wall -Wdeclaration-after-statement -Wpointer-arith
> -Wformat-nonliteral -Wno-pointer-sign -Wno-unused -Wunused-value
> -Wunused-function -Wno-switch -Wno-char-subscripts -Wmissing-prototypes
> -Wno-format    -DMIPS_MACH_DEFAULT=bfd_mach_mips8000   -I.
> -I../../../gdb-7.5.91/sim/mips -I../common
> -I../../../gdb-7.5.91/sim/mips/../common -I../../include
> -I../../../gdb-7.5.91/sim/mips/../..//include -I../../bfd
> -I../../../gdb-7.5.91/sim/mips/../..//bfd -I../../opcodes
> -I../../../gdb-7.5.91/sim/mips/../..//opcodes  -O2 -g -pipe -Wall
> -Wp,-D_FORTIFY_SOURCE=2 -fexceptions --param=ssp-buffer-size=4
> -mms-bitfields -D__USE_MINGW_ACCESS -c -o interp.o -MT interp.o -MMD -MP
> -MF .deps/interp.Tpo ../../../gdb-7.5.91/sim/mips/interp.c
> ../../../gdb-7.5.91/sim/mips/../common/dv-sockser.c:44:24: fatal error:
> netinet/in.h: No such file or directory
> 
> 
> ATM, I assume this patch to be the cause (sim/mips):
> 
> 2012-09-03 Joel Sherrill <joel.sherrill@oarcorp.com>
> 
>          * configure.ac: Always link against dv-sockser.o.
>          * configure: Regenerate.
> 
> 
>  From what I can gather from comparing sim/mips/configure.ac to other
> sim/*/configure.ac (e.g. bfin/configure.ac),
> dv-sockser.o needs to be suppressed for mingw32-* targets?

dv-sockser has always been a bit of an ad-hoc hack (at just about every 
layer).  adding a m4 macro to the common/ dir would probably be prudent.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [gdb-7.5.91] mips sim fails to build on mingw32

[Ralf Corsepius]

On 03/15/2013 07:16 PM, Mike Frysinger wrote:
> On Friday 15 March 2013 08:49:28 Ralf Corsepius wrote:
>> Hi,
>>
>> The mips sim fails to build on mingw:
>> ...
>> x86_64-w64-mingw32-gcc -DHAVE_CONFIG_H     -DPROFILE=1 -DWITH_PROFILE=-1
>>     -DWITH_ALIGNMENT=NONSTRICT_ALIGNMENT -DWITH_TARGET_WORD_BITSIZE=32
>> -DWITH_TARGET_WORD_MSB=31 -DWITH_DEFAULT_TARGET_BYTE_ORDER=BIG_ENDIAN
>> -DWITH_FLOATING_POINT=HARD_FLOATING_POINT
>> -DWITH_TARGET_FLOATING_POINT_BITSIZE=32 -DWITH_HW=0
>> -DWITH_HOST_BYTE_ORDER=0 -DDEFAULT_INLINE=0   -DWITH_RESERVED_BITS=1
>> -DWITH_SMP=0  -Wall -Wdeclaration-after-statement -Wpointer-arith
>> -Wformat-nonliteral -Wno-pointer-sign -Wno-unused -Wunused-value
>> -Wunused-function -Wno-switch -Wno-char-subscripts -Wmissing-prototypes
>> -Wno-format    -DMIPS_MACH_DEFAULT=bfd_mach_mips8000   -I.
>> -I../../../gdb-7.5.91/sim/mips -I../common
>> -I../../../gdb-7.5.91/sim/mips/../common -I../../include
>> -I../../../gdb-7.5.91/sim/mips/../..//include -I../../bfd
>> -I../../../gdb-7.5.91/sim/mips/../..//bfd -I../../opcodes
>> -I../../../gdb-7.5.91/sim/mips/../..//opcodes  -O2 -g -pipe -Wall
>> -Wp,-D_FORTIFY_SOURCE=2 -fexceptions --param=ssp-buffer-size=4
>> -mms-bitfields -D__USE_MINGW_ACCESS -c -o interp.o -MT interp.o -MMD -MP
>> -MF .deps/interp.Tpo ../../../gdb-7.5.91/sim/mips/interp.c
>> ../../../gdb-7.5.91/sim/mips/../common/dv-sockser.c:44:24: fatal error:
>> netinet/in.h: No such file or directory
>>
>>
>> ATM, I assume this patch to be the cause (sim/mips):
>>
>> 2012-09-03 Joel Sherrill <joel.sherrill@oarcorp.com>
>>
>>           * configure.ac: Always link against dv-sockser.o.
>>           * configure: Regenerate.
>>
>>
>>   From what I can gather from comparing sim/mips/configure.ac to other
>> sim/*/configure.ac (e.g. bfin/configure.ac),
>> dv-sockser.o needs to be suppressed for mingw32-* targets?
>
> dv-sockser has always been a bit of an ad-hoc hack (at just about every
> layer).  adding a m4 macro to the common/ dir would probably be prudent.
> -mike

Hmm, Joel, which issue did your patch (snippet below) try to address?


diff --git a/sim/mips/configure.ac b/sim/mips/configure.ac
index 5f59637..862e2b5 100644
--- a/sim/mips/configure.ac
+++ b/sim/mips/configure.ac
@@ -409,14 +413,13 @@ case "${target}" in
    mips*tx39*)
         hw_enabled=yes
         hw_extra_devices="tx3904cpu tx3904irc tx3904tmr tx3904sio"
-       mips_extra_objs="dv-sockser.o"
         SIM_SUBTARGET="$SIM_SUBTARGET -DTARGET_TX3904=1"
         ;;
    *)
-       mips_extra_objs=""
         ;;
  esac
  SIM_AC_OPTION_HARDWARE($hw_enabled,$hw_devices,$hw_extra_devices)
+mips_extra_objs="dv-sockser.o"
  AC_SUBST(mips_extra_objs)


IIUC, this lets all mips-sim-variants link against dv-sockser.o, while 
previously only the tx39*-variants did. I don't understand this step.

Ralf