qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: qemu-devel@nongnu.org
Cc: pavel.dovgaluk@ispras.ru
Subject: [Qemu-devel] [RFH PATCH 0/4] record/replay fixups and doubts
Date: Tue,  6 Oct 2015 22:00:54 +0200	[thread overview]
Message-ID: <1444161658-15038-1-git-send-email-pbonzini@redhat.com> (raw)

These are some comments I have about the record/replay code.  I can
integrate these in your patches myself, but I need an ack/tested-by and
in some case more answers...  Please take a look.

Paolo Bonzini (4):
  replay: generalize ptimer event to bottom halves
  more replay fixes
  why is runstate_is_running needed?
  events doubts

 Makefile.objs                       |  2 ++
 Makefile.target                     |  1 -
 cpu-exec.c                          |  2 +-
 cpus.c                              |  2 +-
 exec.c                              |  2 +-
 hw/bt/hci.c                         |  4 ++--
 hw/core/ptimer.c                    |  8 ++------
 include/qapi/qmp/qerror.h           |  2 +-
 {replay => include/sysemu}/replay.h |  4 ++--
 qapi/common.json                    |  6 +-----
 qemu-timer.c                        | 23 +++++++++--------------
 replay/Makefile.objs                | 11 +++++------
 replay/replay-events.c              | 24 +++++++++++++++---------
 replay/replay-input.c               |  2 +-
 replay/replay-internal.c            |  4 ++--
 replay/replay-internal.h            |  2 +-
 replay/replay-time.c                |  2 +-
 replay/replay.c                     |  2 +-
 stubs/Makefile.objs                 |  1 +
 {replay => stubs}/replay-user.c     |  6 +-----
 stubs/replay.c                      | 10 +++++-----
 ui/input.c                          |  2 +-
 vl.c                                |  6 +++---
 23 files changed, 59 insertions(+), 69 deletions(-)
 rename {replay => include/sysemu}/replay.h (97%)
 mode change 100755 => 100644 replay/Makefile.objs
 mode change 100755 => 100644 replay/replay-events.c
 mode change 100755 => 100644 replay/replay-input.c
 mode change 100755 => 100644 replay/replay-internal.c
 mode change 100755 => 100644 replay/replay-internal.h
 mode change 100755 => 100644 replay/replay-time.c
 mode change 100755 => 100644 replay/replay.c
 rename {replay => stubs}/replay-user.c (90%)

-- 
2.5.0

             reply	other threads:[~2015-10-06 20:01 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-06 20:00 Paolo Bonzini [this message]
2015-10-06 20:00 ` [Qemu-devel] [PATCH 1/4] replay: generalize ptimer event to bottom halves Paolo Bonzini
2015-10-07  7:53   ` Pavel Dovgaluk
2015-10-06 20:00 ` [Qemu-devel] [PATCH 2/4] more replay fixes Paolo Bonzini
2015-10-06 20:13   ` Eric Blake
2015-10-07  8:11   ` Pavel Dovgaluk
2015-10-06 20:00 ` [Qemu-devel] [PATCH 3/4] why is runstate_is_running needed? Paolo Bonzini
2015-10-07  8:14   ` Pavel Dovgaluk
     [not found]   ` <22126.3941414238$1444205724@news.gmane.org>
2015-10-07  8:46     ` Paolo Bonzini
2015-10-07  9:37   ` Pavel Dovgaluk
2015-10-06 20:00 ` [Qemu-devel] [PATCH 4/4] events doubts Paolo Bonzini
2015-10-07  8:21   ` Pavel Dovgaluk
     [not found]   ` <35633.6639299572$1444206177@news.gmane.org>
2015-10-07  8:52     ` Paolo Bonzini
2015-10-07  9:50       ` Pavel Dovgaluk
2015-10-07 10:23         ` Paolo Bonzini
2015-10-07 10:42           ` Pavel Dovgaluk
2015-10-07 10:48             ` Paolo Bonzini
2015-10-07 10:51               ` Pavel Dovgaluk
2015-10-07 14:59                 ` Paolo Bonzini
2015-10-13  8:10 ` [Qemu-devel] [RFH PATCH 0/4] record/replay fixups and doubts Pavel Dovgaluk
2015-10-13 16:39   ` Paolo Bonzini
2015-10-23  7:28 ` Pavel Dovgaluk
2015-10-23  8:12   ` 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=1444161658-15038-1-git-send-email-pbonzini@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=pavel.dovgaluk@ispras.ru \
    --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 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).