[Ltrace-devel] [PATCH] Fix SIGSEGV on MIPS big endian

Oliver Spornitz ospornit at rz-online.de
Tue Jan 29 19:29:49 UTC 2013


Am 28.01.2013 um 18:49 schrieb Petr Machata:

> Thanks for hunting this down.  Is there a compiler-provided #define that
> we could use instead of introducing our own?  (It's of course OK to
> introduce one like you did if there is nothing that we can portably rely
> on.)

Yes, there are #define's in all my linux toolchains for mips.
They all define:
__MIPSBE__, __MIPSBE, _MIPSBE, MIPSBE for big endian and
__MIPSEL__, __MIPSEL, _MIPSEL, MIPSEL for little endian.

I think we can risk to use these defines directly instead of introducing 
our own. A reworked patch will follow the next days.

> It also seems like we could drop the "sysdeps/linux-gnu/mipsel" symlink
> and just always use ".../mips".  The two cases would be differentiated
> in arch.h like they are now.  If you are interested in preparing such a
> patch, I'll be happy to accept it.

Yes I agree. I am happy to prepare a separate patch this weekend. It is 
a bit wired, but manageable.

Regards
OS



More information about the Ltrace-devel mailing list