July 31 bfd change breaks mips gdb

July 31 bfd change breaks mips gdb

[Michael Snyder]

Dear binutils-folks, 

This change:

2002-07-31  Thiemo Seufer <seufer@csv.ica.uni-stuttgart.de>

	[...]
	* elf32-mips.c (mips_elf32_object_p): Reject n32 ABI files.

breaks mips-gdb, since gdb can no longer recognize n32 mips files
as executable binaries.

This problem needs to be addressed.

Thanks, 
Michael Snyder

Re: July 31 bfd change breaks mips gdb

[Thiemo Seufer]

Michael Snyder wrote:
> Dear binutils-folks, 
> 
> This change:
> 
> 2002-07-31  Thiemo Seufer <seufer@csv.ica.uni-stuttgart.de>
> 
> 	[...]
> 	* elf32-mips.c (mips_elf32_object_p): Reject n32 ABI files.
> 
> breaks mips-gdb, since gdb can no longer recognize n32 mips files
> as executable binaries.

I have used gdb including this patch on IRIX6. n32 debugging worked
at this time.

> This problem needs to be addressed.

I'll look into it tomorrow.


Thiemo

Re: July 31 bfd change breaks mips gdb

[Kevin Buettner]

On Sep 3,  3:51pm, Michael Snyder wrote:

> This change:
> 
> 2002-07-31  Thiemo Seufer <seufer@csv.ica.uni-stuttgart.de>
> 
> 	[...]
> 	* elf32-mips.c (mips_elf32_object_p): Reject n32 ABI files.
> 
> breaks mips-gdb, since gdb can no longer recognize n32 mips files
> as executable binaries.

The patch below fixes the problem for me...

	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
	vector.

Index: config.bfd
===================================================================
RCS file: /cvs/src/src/bfd/config.bfd,v
retrieving revision 1.116
diff -u -p -r1.116 config.bfd
--- config.bfd	28 Aug 2002 10:38:44 -0000	1.116
+++ config.bfd	4 Sep 2002 00:03:07 -0000
@@ -701,8 +701,8 @@ case "${targ}" in
     ;;
 #ifdef BFD64
   mips*-*-irix6*)
-    targ_defvec=bfd_elf32_bigmips_vec
-    targ_selvecs="bfd_elf32_littlemips_vec bfd_elf64_bigmips_vec bfd_elf64_littlemips_vec"
+    targ_defvec=bfd_elf32_nbigmips_vec
+    targ_selvecs="bfd_elf32_nlittlemips_vec bfd_elf32_bigmips_vec bfd_elf32_littlemips_vec bfd_elf64_bigmips_vec bfd_elf64_littlemips_vec"
     ;;
 #endif
   mips*-*-irix5*)

Re: July 31 bfd change breaks mips gdb

[Thiemo Seufer]

Kevin Buettner wrote:
> On Sep 3,  3:51pm, Michael Snyder wrote:
> 
> > This change:
> > 
> > 2002-07-31  Thiemo Seufer <seufer@csv.ica.uni-stuttgart.de>
> > 
> > 	[...]
> > 	* elf32-mips.c (mips_elf32_object_p): Reject n32 ABI files.
> > 
> > breaks mips-gdb, since gdb can no longer recognize n32 mips files
> > as executable binaries.
> 
> The patch below fixes the problem for me...
> 
> 	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
> 	vector.

I had the same in my local tree for a while, which explains why it
worked for me. :-) Thanks for looking at it.


Thiemo

Re: July 31 bfd change breaks mips gdb

[Eric Christopher]

> The patch below fixes the problem for me...
> 
> 	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
> 	vector.
> 

OK.

-eric

-- 
I don't want a pony, I want a rocket
powered jetpack!

Re: July 31 bfd change breaks mips gdb

[Kevin Buettner]

On Sep 4,  1:52pm, Eric Christopher wrote:

> > 	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
> > 	vector.
> 
> OK.

Committed.

Re: July 31 bfd change breaks mips gdb

[Andrew Cagney]

> On Sep 4,  1:52pm, Eric Christopher wrote:
> 
> 
>> > 	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
>> > 	vector.
> 
>> 
>> OK.
> 
> 
> Committed.

Hmm, can you pull this into gdb_5_3-branch?

Andrew

Re: July 31 bfd change breaks mips gdb

[Kevin Buettner]

On Sep 5,  3:42pm, Andrew Cagney wrote:

> >> > 	* config.bfd (mips*-*-irix6*): Add new ABI vectors.  Make n32 default
> >> > 	vector.
> 
> Hmm, can you pull this into gdb_5_3-branch?

Done.