Re: [PATCH 0/5] arm: set of big endian related fixes for armeb (v7)

[Victor Kamensky <victor.kamensky@linaro.org>]

On 20 October 2014 18:12, Andrew Pinski <pinskia@gmail.com> wrote:
> On Mon, Oct 20, 2014 at 5:56 PM, Victor Kamensky
> <victor.kamensky@linaro.org> wrote:
>> Hi Folks,
>>
>> Please find five patches following this cover letter that address several
>> big endian related issues for ARM V7 target. Problems were discovered by
>> running gdb/testsuite on armeb (big endian) target rootfs/kernel and
>> comparing results from the same source against arm (little endian).
>>
>> Note there are several failures remain that are not addressed by this
>> series. I.e valgrind related test cases all failed, because valgrind does
>> not support armeb target. There are other failures in a bit obscure
>> places like gdb.dwarf2/implptrpiece.exp, gdb.python/py-value-cc.exp,
>> but over all number of passes/failures significantly improved in armeb
>> case.
>>
>> Please note it is my first attempt to contribute into gdb. If I missed
>> something please guide me how I can correct that.
>
> Yes bfd patches need to be also sent to binutils@.

Thanks, Andrew. I posted first patch of this series on suggested binutils
mailing list [1]. Other four patches in gdb directory.

- Victor

[1] https://sourceware.org/ml/binutils/2014-10/msg00175.html

> Thanks,
> Andrew Pinski
>
>
>>
>> Thanks,
>> Victor
>>
>> Victor Kamensky (5):
>>   ARM: plt_size functions need to read instructions in right byte order
>>   ARM: extract_arm_insn function need to read instrs correctly in be8
>>     case
>>   ARM: arm_breakpoint should be little endian form in case for arm BE8
>>   ARM: read_pieced_value do big endian processing only in case of valid
>>     gdb_regnum
>>   ARM: asm-source.exp link options in case of armv7b target
>>
>>  bfd/ChangeLog                        |  9 +++++++
>>  bfd/elf32-arm.c                      | 48 +++++++++++++++++++++++++++++++++---
>>  gdb/ChangeLog                        | 16 ++++++++++++
>>  gdb/arm-linux-tdep.c                 |  2 +-
>>  gdb/arm-tdep.c                       |  2 +-
>>  gdb/dwarf2loc.c                      | 30 +++++++++++-----------
>>  gdb/testsuite/ChangeLog              |  4 +++
>>  gdb/testsuite/gdb.asm/asm-source.exp |  4 +++
>>  8 files changed, 94 insertions(+), 21 deletions(-)
>>
>> --
>> 1.8.1.4
>>