On 10/6/20 9:55 PM, Philippe Mathieu-Daudé wrote:
> On 10/3/20 7:14 PM, Pavel Dovgalyuk wrote:
>> From: Pavel Dovgalyuk <pavel.dovga...@gmail.com>
>>
>> This is a test for GDB reverse debugging commands: reverse step and reverse
>> continue.
>> Every test in this suite consists of two phases: record and replay.
>> Recording saves the execution of some instructions and makes an initial
>> VM snapshot to allow reverse execution.
>> Replay saves the order of the first instructions and then checks that they
>> are executed backwards in the correct order.
>> After that the execution is replayed to the end, and reverse continue
>> command is checked by setting several breakpoints, and asserting
>> that the execution is stopped at the last of them.
>>
>> Signed-off-by: Pavel Dovgalyuk <pavel.dovgal...@ispras.ru>
>> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>> Reviewed-by: Willian Rampazzo <willi...@redhat.com>
>>
>> --
This separator confuses the git tools. Can you stop using it
and use the '---' one instead?
>>
>> v5:
>> - disabled (as some other tests) when running on gitlab
>> due to the unidentified timeout problem
^ this should be stripped automatically but isn't.
>> ---
>> MAINTAINERS | 1
>> tests/acceptance/reverse_debugging.py | 208
>> +++++++++++++++++++++++++++++++++
>> 2 files changed, 209 insertions(+)
>> create mode 100644 tests/acceptance/reverse_debugging.py
>
> Thanks, applied to my acceptance-testing tree.
This patch as already be merged by Paolo as commit be52eca309
("tests/acceptance: add reverse debugging test").
The notes below your '--' separator have been merged too.
Dirty, but not a big deal.
Regards,
Phil.
