Re: [PATCH] arm reversible : <phase_2_complete>

[oza Pawandeep <oza dot pawandeep at gmail dot com>]

Hi Yao,

first of all thank you for your comments, will be sending the patch
soon with your comments incorporated as much as possible.
thank you again for sending test results;

I suppose failed test case might be including

1) system call support
2) signal support
3) any other linux ABI support
4) there are some programs on x86 assembly which needs to be written
for ARM and put separately.

Thanks Yao,
Oza.


On Sat, Oct 15, 2011 at 6:02 PM, Yao Qi <yao@codesourcery.com> wrote:
> On 10/15/2011 05:31 PM, Yao Qi wrote:
>> I noticed that test cases in gdb.reverse are *not* enabled in default.
>> You need to add the following two lines in your board file to turn them on,
>>
>> Â set_board_info gdb,use_precord 1
>> Â set_board_info gdb,can_reverse 1
>>
>> It is good if you can post the test result in gdb.reverse so that we are
>> more confidient to your patch.
> Â Â Â ^^^^^^^^^^ typo "confident".
>
> I run test cases in gdb.reverse with my own board file [1], and get
> following result on x86:
>
> Â Â Â Â Â Â Â Â=== gdb Summary ===
>
> # of expected passes      Â2774
> # of unexpected failures    Â22
>
> I also applied your patch, and run gdb.reverse test cases on arm natively:
>
> Â Â Â Â Â Â Â Â=== gdb Summary ===
>
> # of expected passes      Â1180
> # of unexpected failures    Â1090
> # of expected failures     Â142
>
> I don't think these fails are all related to your arm-reverse patch.
> IMO, it means there are still some work to do, not only in arm bits, but
> also in gdb general parts and test cases. ÂSorry that I can't give any
> useful suggestion here.
>
> --
> Yao (éå)
>
> [1] My board file reverse.exp
>
> load_generic_config "unix"
> process_multilib_options ""
>
> # The default compiler for this target.
> set_board_info compiler "[find_gcc]"
>
> set_board_info gdb,can_reverse 1
> set_board_info gdb,use_precord 1
>