From: "Pavel Dovgaluk" <Pavel.Dovgaluk@ispras.ru>
To: 'Paolo Bonzini' <pbonzini@redhat.com>, qemu-devel@nongnu.org
Cc: edgar.iglesias@xilinx.com, peter.maydell@linaro.org,
igor.rubinov@gmail.com, mark.burton@greensocs.com,
real@ispras.ru, batuzovk@ispras.ru,
maria.klimushenkova@ispras.ru, hines@cert.org,
alex.bennee@linaro.org, fred.konrad@greensocs.com
Subject: Re: [Qemu-devel] [PATCH v18 13/21] icount: improve counting for record/replay
Date: Wed, 23 Sep 2015 11:42:17 +0300 [thread overview]
Message-ID: <000e01d0f5db$c10ea5f0$432bf1d0$@Dovgaluk@ispras.ru> (raw)
In-Reply-To: <56025DE5.5040600@redhat.com>
> From: Paolo Bonzini [mailto:paolo.bonzini@gmail.com] On Behalf Of Paolo Bonzini
> On 23/09/2015 09:22, Pavel Dovgaluk wrote:
> > Sometimes tcg thread halts in qemu_tcg_wait_io_event function,
> > waiting for any external event. Virtual clock does not run, because
> > warp is not called. warp call in main_loop_wait proceeds virtual
> > clock and allows tcg thread to run further.
>
> Ok, this makes sense!
>
> Would this work too as a replacement for this patch?
No, it doesn't help.
It seems that tcg is waiting within qemu_cond_wait function without leaving it.
>
> diff --git a/cpus.c b/cpus.c
> index fbbd17f..9480acc 100644
> --- a/cpus.c
> +++ b/cpus.c
> @@ -926,6 +926,7 @@ static void qemu_tcg_wait_io_event(CPUState *cpu)
> }
>
> while (iothread_requesting_mutex) {
> + qemu_clock_warp(QEMU_CLOCK_VIRTUAL);
> qemu_cond_wait(&qemu_io_proceeded_cond, &qemu_global_mutex);
> }
Pavel Dovgalyuk
next prev parent reply other threads:[~2015-09-23 8:42 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-17 16:23 [Qemu-devel] [PATCH v18 00/21] Deterministic replay core Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 01/21] i386: partial revert of interrupt poll fix Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 02/21] replay: global variables and function stubs Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 03/21] replay: internal functions for replay log Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 04/21] replay: introduce mutex to protect the " Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 05/21] replay: introduce icount event Pavel Dovgalyuk
2015-09-17 16:23 ` [Qemu-devel] [PATCH v18 06/21] cpu-exec: allow temporary disabling icount Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 07/21] cpu: replay instructions sequence Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 08/21] i386: interrupt poll processing Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 09/21] replay: interrupts and exceptions Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 10/21] replay: asynchronous events infrastructure Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 11/21] replay: recording and replaying clock ticks Pavel Dovgalyuk
2015-09-22 13:15 ` Paolo Bonzini
2015-09-23 7:09 ` Pavel Dovgaluk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 12/21] replay: shutdown event Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 13/21] icount: improve counting for record/replay Pavel Dovgalyuk
2015-09-22 13:15 ` Paolo Bonzini
2015-09-23 7:22 ` Pavel Dovgaluk
[not found] ` <1946.37127356163$1442992987@news.gmane.org>
2015-09-23 8:08 ` Paolo Bonzini
2015-09-23 8:42 ` Pavel Dovgaluk [this message]
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 14/21] replay: checkpoints Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 15/21] bottom halves: introduce bh call function Pavel Dovgalyuk
2015-09-17 16:24 ` [Qemu-devel] [PATCH v18 16/21] replay: ptimer Pavel Dovgalyuk
2015-09-17 16:25 ` [Qemu-devel] [PATCH v18 17/21] typedef: add typedef for QemuOpts Pavel Dovgalyuk
2015-09-17 16:25 ` [Qemu-devel] [PATCH v18 18/21] replay: initialization and deinitialization Pavel Dovgalyuk
2015-09-17 16:25 ` [Qemu-devel] [PATCH v18 19/21] replay: replay blockers for devices Pavel Dovgalyuk
2015-10-06 20:15 ` Eric Blake
2015-09-17 16:25 ` [Qemu-devel] [PATCH v18 20/21] replay: command line options Pavel Dovgalyuk
2015-09-17 16:25 ` [Qemu-devel] [PATCH v18 21/21] replay: recording of the user input Pavel Dovgalyuk
2015-09-21 7:12 ` [Qemu-devel] [PATCH v18 00/21] Deterministic replay core Pavel Dovgaluk
2015-09-22 13:13 ` Paolo Bonzini
[not found] ` <13403.5645904534$1442819578@news.gmane.org>
2015-10-06 15:09 ` Paolo Bonzini
2015-10-06 16:38 ` Paolo Bonzini
[not found] ` <CAPnv1P+PMm7stf-TUoWBX9+frHHOWzz=LB-Errn6_=fBO_QxNQ@mail.gmail.com>
[not found] ` <CAPnv1PKaYDZQeMxK33e1U3mWcsNQiufPnDxLZgiFxXAktxzktA@mail.gmail.com>
2015-11-04 10:57 ` Igor R
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='000e01d0f5db$c10ea5f0$432bf1d0$@Dovgaluk@ispras.ru' \
--to=pavel.dovgaluk@ispras.ru \
--cc=alex.bennee@linaro.org \
--cc=batuzovk@ispras.ru \
--cc=edgar.iglesias@xilinx.com \
--cc=fred.konrad@greensocs.com \
--cc=hines@cert.org \
--cc=igor.rubinov@gmail.com \
--cc=maria.klimushenkova@ispras.ru \
--cc=mark.burton@greensocs.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=real@ispras.ru \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).