[v2] Some record/replay fixes

[Qemu-devel] [for-4.2 PATCH v2 0/8] Some record/replay fixes

Pavel Dovgalyuk posted 8 patches 4 years, 9 months ago

Diff against v1
Download series mbox

Test docker-clang@ubuntu passed

Test s390x passed

Test asan passed

Test docker-mingw@fedora passed

Test FreeBSD passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/156404424989.18669.12696116045723475173.stgit@pasha-Precision-3630-Tower

Maintainers: Laurent Vivier <lvivier@redhat.com>, Alistair Francis <Alistair.Francis@wdc.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Aurelien Jarno <aurelien@aurel32.net>, Michael Walle <michael@walle.cc>, Thomas Huth <thuth@redhat.com>, Aleksandar Rikalo <arikalo@wavecomp.com>, Chris Wulff <crwulff@gmail.com>, Max Filippov <jcmvbkbc@gmail.com>, Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru>, Guan Xuetao <gxt@mprc.pku.edu.cn>, Marek Vasut <marex@denx.de>, Artyom Tarasenko <atar4qemu@gmail.com>, Palmer Dabbelt <palmer@sifive.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Sagar Karandikar <sagark@eecs.berkeley.edu>, Peter Maydell <peter.maydell@linaro.org>, Aleksandar Markovic <amarkovic@wavecomp.com>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, Eduardo Habkost <ehabkost@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <rth@twiddle.net>, David Gibson <david@gibson.dropbear.id.au>

accel/tcg/cpu-exec.c                    |    1 -
accel/tcg/translator.c                  |    2 +
cpus.c                                  |   17 ++++++++---
docs/devel/replay.txt                   |   46 +++++++++++++++++++++++++++++++
include/exec/gen-icount.h               |   38 +++++++++++++-------------
include/qemu/timer.h                    |    8 ++++-
include/sysemu/replay.h                 |    2 +
qtest.c                                 |    3 +-
replay/replay-events.c                  |    2 +
replay/replay-internal.c                |   10 +++----
replay/replay-internal.h                |   10 +++----
replay/replay-snapshot.c                |    6 ++--
replay/replay-time.c                    |   36 +++++++++++-------------
replay/replay.c                         |   30 ++++++++++----------
target/alpha/translate.c                |    4 +--
target/arm/translate-a64.c              |    6 +---
target/arm/translate.c                  |   10 ++-----
target/cris/translate.c                 |    4 +--
target/hppa/translate.c                 |    2 +
target/i386/translate.c                 |   18 +++++-------
target/lm32/translate.c                 |   12 ++------
target/microblaze/translate.c           |    4 +--
target/mips/translate.c                 |   16 +++--------
target/nios2/translate.c                |    4 +--
target/ppc/translate.c                  |   18 +++---------
target/ppc/translate_init.inc.c         |    4 +--
target/riscv/insn_trans/trans_rvi.inc.c |    2 +
target/sparc/translate.c                |   32 +++++++++++-----------
target/unicore32/translate.c            |    2 +
target/xtensa/translate.c               |   20 +++----------
tests/ptimer-test-stubs.c               |    4 +--
tests/ptimer-test.c                     |    6 +++-
util/qemu-timer.c                       |   30 ++++++++++++++++++--
33 files changed, 225 insertions(+), 184 deletions(-)
create mode 100644 docs/devel/replay.txt

Expand all Fold all

[Qemu-devel] [for-4.2 PATCH v2 0/8] Some record/replay fixes

Posted by Pavel Dovgalyuk 4 years, 9 months ago

The set of patches include the latest fixes for record/replay icount function:
 - fix for icount for the case when translation blocks are chained
 - development documentation update
 - some refactoring

v2 changes (suggested by Paolo Bonzini):
 - allow fixed qemu_clock_deadline_ns_all to be used with any timers
 - clean up can_do_io at the start of every TB
 - remove unnecessary gen_io_end calls due to the previous change

---

Pavel Dovgalyuk (7):
      replay: document development rules
      util/qemu-timer: refactor deadline calculation for external timers
      replay: fix replay shutdown
      replay: refine replay-time module
      replay: rename step-related variables and functions
      icount: clean up cpu_can_io at the entry to the block
      icount: remove unnecessary gen_io_end calls

pbonzini@redhat.com (1):
      replay: add missing fix for internal function


 accel/tcg/cpu-exec.c                    |    1 -
 accel/tcg/translator.c                  |    2 +
 cpus.c                                  |   17 ++++++++---
 docs/devel/replay.txt                   |   46 +++++++++++++++++++++++++++++++
 include/exec/gen-icount.h               |   38 +++++++++++++-------------
 include/qemu/timer.h                    |    8 ++++-
 include/sysemu/replay.h                 |    2 +
 qtest.c                                 |    3 +-
 replay/replay-events.c                  |    2 +
 replay/replay-internal.c                |   10 +++----
 replay/replay-internal.h                |   10 +++----
 replay/replay-snapshot.c                |    6 ++--
 replay/replay-time.c                    |   36 +++++++++++-------------
 replay/replay.c                         |   30 ++++++++++----------
 target/alpha/translate.c                |    4 +--
 target/arm/translate-a64.c              |    6 +---
 target/arm/translate.c                  |   10 ++-----
 target/cris/translate.c                 |    4 +--
 target/hppa/translate.c                 |    2 +
 target/i386/translate.c                 |   18 +++++-------
 target/lm32/translate.c                 |   12 ++------
 target/microblaze/translate.c           |    4 +--
 target/mips/translate.c                 |   16 +++--------
 target/nios2/translate.c                |    4 +--
 target/ppc/translate.c                  |   18 +++---------
 target/ppc/translate_init.inc.c         |    4 +--
 target/riscv/insn_trans/trans_rvi.inc.c |    2 +
 target/sparc/translate.c                |   32 +++++++++++-----------
 target/unicore32/translate.c            |    2 +
 target/xtensa/translate.c               |   20 +++----------
 tests/ptimer-test-stubs.c               |    4 +--
 tests/ptimer-test.c                     |    6 +++-
 util/qemu-timer.c                       |   30 ++++++++++++++++++--
 33 files changed, 225 insertions(+), 184 deletions(-)
 create mode 100644 docs/devel/replay.txt

-- 
Pavel Dovgalyuk