Re: Patch: Add arch_string to spu-low.c

["Ulrich Weigand" <uweigand@de.ibm.com>]

Daniel Jakowowitz wrote:
> On Thu, Feb 22, 2007 at 04:21:28PM +0100, Ulrich Weigand wrote:
> > Daniel Jakowobitz wrote:
> > 
> > > OK if it's OK with Ulrich (do you always want "SPU:256K"?  Just "spu"
> > > should work too).
> > 
> > Can you elaborate on what effect that change would have?  Does it
> > then try to automatically select the subarchitecture?  (We just
> > have the one right now anyway ...)
> 
> It will select the default subarchitecture - what may be more
> interesting is that it won't complain if the user or binary file has
> selected a different subarchitecture.  This is a bit tricky; see
> choose_architecture_for_target in arch-utils.c, particularly the
> the_default check.
> 
> So suppose you have in the future an spu:512K, and gdbserver doesn't
> need to know the difference, but it's recorded in the ELF file.  GDB
> would object if gdbserver reported "I have an spu:256K".

Ah, thanks for the explanation.  I agree that "spu" would be the
better choice then.

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com