>On Sat, Feb 05, 2011 at 10:49:51PM +0100, Laurent Vivier wrote:
>> >On 02/03/2011 12:16 PM, Laurent Vivier wrote:
>> >> And this one ?
>> >>
>> >> linux-user: correct core dump format
>> >>
>> >> http://patchwork.ozlabs.org/patch/78464/
>
>Actually that patch is not ok. The issues you fix on m68k appear
>on arm/eabi after applying your patch. bswap part appears ok, but
>the padding is needed atleast on arm.
It is strange as we have in gdb, if I remember correctly, an explicit check of
the size of prstatus:
gdb/bfd/elf.c:
if defined (HAVE_PRSTATUS_T)
static bfd_boolean
elfcore_grok_prstatus (bfd *abfd, Elf_Internal_Note *note)
...
if (note->descsz == sizeof (prstatus_t))
...
How do you test this patch ? Do you use native gdb on ARM ? Because, for m68k,
cross-compiled gdb does not work (it cannot have sizeof(prstatus_t) for m68k).
Regards,
Laurent
--
--------------------- [email protected] ---------------------
"Tout ce qui est impossible reste à accomplir" Jules Verne
"Things are only impossible until they're not" Jean-Luc Picard