From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v4 0/4] Fix -icount with iothread
Date: Fri, 15 Apr 2011 09:49:46 +0200 [thread overview]
Message-ID: <20110415074946.GB2423@edde.se.axis.com> (raw)
In-Reply-To: <1302681826-10236-1-git-send-email-pbonzini@redhat.com>
On Wed, Apr 13, 2011 at 10:03:42AM +0200, Paolo Bonzini wrote:
> This series finally fixes -icount with iothread and avoids deadlocks
> due to the vm_clock not making progress when the VM is stopped.
> The crux of the fix is in patch 1, while patch 2 implements the
> "clock warping" that fixes deadlocks in v2. Clock warping uses
> the nanosecond resolution rt_clock timers introduced by my previous
> series.
>
> With this in place, patch 3 can revert the previous attempt(s).
> Finally, patch 4 makes the icount code clearer by finishing the
> bugfix/reorganization of qemu_next_deadline vs. qemu_next_alarm_deadline.
>
> v1->v2:
> reordered patches, renamed qemu_next_deadline
>
> v2->v3:
> introduced warp timer
>
> v3->v4:
> added comments to the code
>
> Paolo Bonzini (4):
> really fix -icount in the iothread case
> enable vm_clock to "warp" in the iothread+icount case
> Revert wrong fixes for -icount in the iothread case
> qemu_next_deadline should not consider host-time timers
>
> cpus.c | 13 ++++-
> qemu-common.h | 1 +
> qemu-timer.c | 169 ++++++++++++++++++++++++++++++++++++++++++++-------------
> qemu-timer.h | 3 +-
> roms/seabios | 2 +-
> 5 files changed, 146 insertions(+), 42 deletions(-)
I've applied this series, thanks alot for working on it.
BTW, I removed this hunk from patch nr 2:
diff --git a/roms/seabios b/roms/seabios
index cc97564..06d0bdd 160000
--- a/roms/seabios
+++ b/roms/seabios
@@ -1 +1 @@
-Subproject commit cc975646af69f279396d4d5e1379ac6af80ee637
+Subproject commit 06d0bdd9e2e20377b3180e4986b14c8549b393e4
--
Cheers
next prev parent reply other threads:[~2011-04-15 7:49 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-13 8:03 [Qemu-devel] [PATCH v4 0/4] Fix -icount with iothread Paolo Bonzini
2011-04-13 8:03 ` [Qemu-devel] [PATCH v4 1/4] really fix -icount in the iothread case Paolo Bonzini
2011-04-13 8:03 ` [Qemu-devel] [PATCH v4 2/4] enable vm_clock to "warp" in the iothread+icount case Paolo Bonzini
2011-04-13 8:03 ` [Qemu-devel] [PATCH v4 3/4] Revert wrong fixes for -icount in the iothread case Paolo Bonzini
2011-04-13 8:03 ` [Qemu-devel] [PATCH v4 4/4] qemu_next_deadline should not consider host-time timers Paolo Bonzini
2011-04-15 7:49 ` Edgar E. Iglesias [this message]
2011-04-15 10:18 ` [Qemu-devel] [PATCH v4 0/4] Fix -icount with iothread Paolo Bonzini
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=20110415074946.GB2423@edde.se.axis.com \
--to=edgar.iglesias@gmail.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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.