From: "Alex Bennée" <alex.bennee@linaro.org>
To: Pavel Dovgalyuk <dovgaluk@ispras.ru>
Cc: peter.maydell@linaro.org, rth@twiddle.net, pbonzini@redhat.com,
qemu-devel@nongnu.org, mttcg@listserver.greensocs.com,
fred.konrad@greensocs.com, a.rigo@virtualopensystems.com,
cota@braap.org, bobby.prani@gmail.com, nikunj@linux.vnet.ibm.com
Subject: Re: [Qemu-devel] [PATCH v3 00/11] MTTCG fix-ups for 2.9
Date: Wed, 22 Mar 2017 14:17:41 +0000 [thread overview]
Message-ID: <87inn1xunu.fsf@linaro.org> (raw)
In-Reply-To: <002501d29e64$27b383c0$771a8b40$@ru>
Pavel Dovgalyuk <dovgaluk@ispras.ru> writes:
>> From: Alex Bennée [mailto:alex.bennee@linaro.org]
>> Pavel Dovgalyuk <dovgaluk@ispras.ru> writes:
>> >> From: Alex Bennée [mailto:alex.bennee@linaro.org]
>> >> Pavel Dovgalyuk <dovgaluk@ispras.ru> writes:
>> >
>> >> I ran the following test on both pre-mttcg-merge and my current HEAD
>> >> which includes Paolo's fix series:
>> >>
>> >> ./arm-softmmu/qemu-system-arm -machine type=virt \
>> >> -display none -smp 1 -m 4096 -cpu cortex-a15 \
>> >> -kernel ../images/aarch32-current-linux-initrd-guest.img
>> >> -append "console=ttyAMA0" -serial mon:stdio \
>> >> -net none \
>> >> -icount shift=7,rr=record,rrfile=replay.bin
>> >>
>> >> And then:
>> >>
>> >> ./arm-softmmu/qemu-system-arm -machine type=virt \
>> >> -display none -smp 1 -m 4096 -cpu cortex-a15 \
>> >> -kernel ../images/aarch32-current-linux-initrd-guest.img
>> >> -append "console=ttyAMA0" -serial mon:stdio \
>> >> -net none \
>> >> -icount shift=7,rr=replay,rrfile=replay.bin
>> >>
>> >> And they both ran the same. However I kept running into:
>> >>
>> >> [ 3.542408] RPC: Registered tcp NFSv4.1 backchannel transport module.
>> >> qemu-system-arm: Missing character write event in the replay log
>> >>
>> >> This seems to be a pre-existing
>> >
>> > Does it mean that qemu-arm platform includes some serial devices that were
>> > not detected by the replay?
>>
>> It's the standard ARM platform serial port. When I read replay.txt is
>> said:
>>
>> * Supports i386, x86_64, and ARM hardware platforms.
>>
>> Should we add some qualifications about which machine types are
>> supported? What is you ARM test case for record/replay?
>
> I tested on vexpress-a9 platform with Debian wheezy.
Thanks for that. I now have a test case that I can reproduce failures on
without needing graphics.
I've been investigating if there are any problems with the timer
processing now they have been moved into the TCG thread. The record
stage seems to work fine but I'm having difficulty figuring out why
playback freezes. It seems we get to a point where we are stuck waiting
for a suspiciously exact timer deadline:
replay_account_executed_instructions: instructions_count reached zero
handle_icount_deadline: deadline=471077745280
replay_account_executed_instructions: instructions_count reached zero
handle_icount_deadline: deadline=10000000
[Switching to Thread 0x7fff7a7be700 (LWP 733)]
Thread 3 "qemu-system-arm" hit Breakpoint 1, handle_icount_deadline () at /home/alex/lsrc/qemu/qemu.git/cpus.c:1184
1184 fprintf(stderr,"%s: no change to deadline=%ld for %ld\n",
(gdb) c
Continuing.
handle_icount_deadline: no change to deadline=10000000 for 11
Thread 3 "qemu-system-arm" hit Breakpoint 1, handle_icount_deadline () at /home/alex/lsrc/qemu/qemu.git/cpus.c:1184
1184 fprintf(stderr,"%s: no change to deadline=%ld for %ld\n",
(gdb) c 10
Will ignore next 9 crossings of breakpoint 1. Continuing.
handle_icount_deadline: no change to deadline=10000000 for 12
handle_icount_deadline: no change to deadline=10000000 for 13
handle_icount_deadline: no change to deadline=10000000 for 14
handle_icount_deadline: no change to deadline=10000000 for 15
handle_icount_deadline: no change to deadline=10000000 for 16
handle_icount_deadline: no change to deadline=10000000 for 17
handle_icount_deadline: no change to deadline=10000000 for 18
handle_icount_deadline: no change to deadline=10000000 for 19
handle_icount_deadline: no change to deadline=10000000 for 20
handle_icount_deadline: no change to deadline=10000000 for 21
(gdb) p replay_state
$1 = {cached_clock = {1490191319270134000, 0}, current_step = 267869922, instructions_count = 0, data_kind = 12, has_unread_data = 1, file_offset = 0, block_request_id = 0}
But the timers are all enabled:
(gdb) qemu timers
Processing Realtime timers
clock QEMU_CLOCK_REALTIME is enabled:true, last:-9223372036854775808
Processing Virtual timers
clock QEMU_CLOCK_VIRTUAL is enabled:true, last:-9223372036854775808
timer 34297350016/1 (cb:0x555555a2e952 <ptimer_tick>)
timer 503290000000/1000000 (cb:0x555555bd4d1d <ra_timer_handler>)
Processing Host timers
clock QEMU_CLOCK_HOST is enabled:true, last:1490191319270134000
Processing Virtual RT timers
clock QEMU_CLOCK_VIRTUAL_RT is enabled:true, last:-9223372036854775808
One area I wanted to look back at was comparing the record trace from
pre-mttcg-merge to now to see if any information was missing. However
the bin file has quite a lot of noise in it from changing fields so I
was wondering do you have any sort of dumper tool for comparing the
traces? If not is the format of the trace file specified anywhere?
I found:
#define REPLAY_VERSION 0xe02005
but couldn't find any more than that. Is the format basically just
described by the calls to replay_put_byte?
--
Alex Bennée
next prev parent reply other threads:[~2017-03-22 14:17 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-07 15:50 [Qemu-devel] [PATCH v3 00/11] MTTCG fix-ups for 2.9 Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 01/11] vl/cpus: be smarter with icount and MTTCG Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 02/11] target/i386/cpu.h: declare TCG_GUEST_DEFAULT_MO Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 03/11] cpus.c: add additional error_report when !TARGET_SUPPORT_MTTCG Alex Bennée
2017-03-07 17:48 ` Philippe Mathieu-Daudé
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 04/11] sparc/sparc64: grab BQL before calling cpu_check_irqs Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 05/11] s390x/misc_helper.c: wrap IO instructions in BQL Alex Bennée
2017-03-07 17:46 ` Philippe Mathieu-Daudé
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 06/11] target/xtensa: hold BQL for interrupt processing Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 07/11] translate-all: exit cpu_restore_state early if translating Alex Bennée
2017-03-07 19:20 ` Richard Henderson
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 08/11] target/mips: hold BQL for timer interrupts Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] [PATCH v3 09/11] target-i386: defer VMEXIT to do_interrupt Alex Bennée
2017-03-07 19:23 ` Richard Henderson
2017-03-07 15:50 ` [PATCH v3 10/11] target/arm/helper: make it clear the EC field is also in hex Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] " Alex Bennée
2017-03-07 17:49 ` [Qemu-arm] " Philippe Mathieu-Daudé
2017-03-07 17:49 ` [Qemu-devel] " Philippe Mathieu-Daudé
2017-03-07 15:50 ` [PATCH v3 11/11] hw/intc/arm_gic: modernise the DPRINTF Alex Bennée
2017-03-07 15:50 ` [Qemu-devel] " Alex Bennée
2017-03-07 17:53 ` [Qemu-arm] " Philippe Mathieu-Daudé
2017-03-07 17:53 ` [Qemu-devel] " Philippe Mathieu-Daudé
2017-03-07 20:25 ` [Qemu-devel] [PATCH v3 00/11] MTTCG fix-ups for 2.9 Pranith Kumar
[not found] ` <877f40i5e3.fsf@linaro.org>
2017-03-08 14:20 ` Pranith Kumar
2017-03-13 12:32 ` Pavel Dovgalyuk
2017-03-13 13:16 ` Alex Bennée
2017-03-14 12:15 ` Pavel Dovgalyuk
2017-03-14 15:18 ` Alex Bennée
2017-03-16 8:34 ` Pavel Dovgalyuk
2017-03-16 13:06 ` Alex Bennée
2017-03-16 14:46 ` Pavel Dovgalyuk
2017-03-22 14:17 ` Alex Bennée [this message]
2017-03-29 6:06 ` Pavel Dovgalyuk
2017-03-29 9:42 ` Alex Bennée
2017-03-30 11:44 ` Pavel Dovgalyuk
2017-03-30 12:42 ` Alex Bennée
2017-03-31 9:16 ` Pavel Dovgalyuk
2017-03-31 10:16 ` Paolo Bonzini
2017-03-31 11:21 ` Alex Bennée
2017-03-31 11:31 ` Paolo Bonzini
2017-03-31 19:49 ` Alex Bennée
2017-03-31 13:14 ` Alex Bennée
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87inn1xunu.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=a.rigo@virtualopensystems.com \
--cc=bobby.prani@gmail.com \
--cc=cota@braap.org \
--cc=dovgaluk@ispras.ru \
--cc=fred.konrad@greensocs.com \
--cc=mttcg@listserver.greensocs.com \
--cc=nikunj@linux.vnet.ibm.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.