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

Ulrich Weigand
2007-02-22T15:34:22Z

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