Re: [Qemu-devel] [PULL 00/18] Record/replay core for QEMU 2.4-rc1

[Alex Bennée]

Paolo Bonzini <address@hidden> writes:

> The following changes since commit 7bc8e0c967a4ef77657174d28af775691e18b4ce:
>
>   Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging 
> (2015-10-29 09:49:52 +0000)
>
> are available in the git repository at:
>
>
>   git://github.com/bonzini/qemu.git tags/for-upstream-replay
>
> for you to fetch changes up to e02cf7f4de91174dc322e235da6e1a1198601920:
>
>   replay: recording of the user input (2015-11-04 15:09:54 +0100)
>
> ----------------------------------------------------------------
> So here it is, let's see what happens.
>
> ----------------------------------------------------------------
> Pavel Dovgalyuk (18):
>       replay: global variables and function stubs
>       replay: internal functions for replay log
>       replay: introduce mutex to protect the replay log
>       replay: introduce icount event
>       cpu-exec: allow temporary disabling icount
>       cpu: replay instructions sequence
>       replay: interrupts and exceptions
>       replay: asynchronous events infrastructure
>       replay: recording and replaying clock ticks
>       replay: shutdown event
>       icount: improve counting for record/replay
>       replay: checkpoints
>       bottom halves: introduce bh call function
>       replay: ptimer
>       replay: initialization and deinitialization
>       replay: replay blockers for devices
>       replay: command line options
>       replay: recording of the user input
>
>  Makefile.objs             |   2 +
>  async.c                   |   7 +-
>  cpu-exec.c                |  55 ++++++--
>  cpus.c                    |  64 ++++++---
>  docs/replay.txt           | 168 +++++++++++++++++++++++
>  exec.c                    |   2 +
>  hw/bt/hci.c               |   7 +
>  hw/core/ptimer.c          |   3 +-
>  include/block/aio.h       |   5 +
>  include/exec/exec-all.h   |   1 +
>  include/qapi/qmp/qerror.h |   3 +
>  include/sysemu/replay.h   | 120 ++++++++++++++++
>  include/ui/input.h        |   2 +
>  main-loop.c               |   3 +
>  qapi-schema.json          |  18 +++
>  qemu-options.hx           |   8 +-
>  qemu-timer.c              |  43 +++++-
>  replay/Makefile.objs      |   5 +
>  replay/replay-events.c    | 279 +++++++++++++++++++++++++++++++++++++
>  replay/replay-input.c     | 160 ++++++++++++++++++++++
>  replay/replay-internal.c  | 206 ++++++++++++++++++++++++++++
>  replay/replay-internal.h  | 140 +++++++++++++++++++
>  replay/replay-time.c      |  64 +++++++++
>  replay/replay.c           | 342 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  stubs/Makefile.objs       |   2 +
>  stubs/replay-user.c       |  32 +++++
>  stubs/replay.c            |  31 +++++
>  translate-all.c           |   2 +-
>  ui/input.c                |  27 ++--
>  vl.c                      |  60 ++++++--
>  30 files changed, 1803 insertions(+), 58 deletions(-)
>  create mode 100644 docs/replay.txt
>  create mode 100644 include/sysemu/replay.h
>  create mode 100644 replay/Makefile.objs
>  create mode 100644 replay/replay-events.c
>  create mode 100644 replay/replay-input.c
>  create mode 100644 replay/replay-internal.c
>  create mode 100644 replay/replay-internal.h
>  create mode 100644 replay/replay-time.c
>  create mode 100644 replay/replay.c
>  create mode 100644 stubs/replay-user.c
>  create mode 100644 stubs/replay.c

Are there any tests to protect basic functionality?

I'm just worried about stuff getting broken when MTTCG comes along.

-- 
Alex Bennée