[PULL for 9.1 00/21] Some fixes for 9.1-rc3 (build, replay, docs, plugins)

Alex Bennée posted 21 patches 3 months, 1 week ago
There is a newer version of this series
docs/about/emulation.rst                  |  49 +++++++--
docs/devel/migration/uadk-compression.rst |   4 +-
docs/devel/tcg-plugins.rst                |  13 ++-
docs/interop/qemu-ga.rst                  |   2 +-
docs/tools/qemu-vmsr-helper.rst           |   4 +-
configure                                 |   8 +-
Makefile                                  |   3 +-
qapi/rocker.json                          |   4 +-
include/exec/memory.h                     |   2 +-
include/sysemu/replay.h                   |   5 -
include/sysemu/runstate.h                 |   1 +
accel/tcg/tcg-accel-ops-rr.c              |   2 +-
chardev/char.c                            |  71 +++++++++----
hw/arm/smmu-common.c                      |   2 +-
hw/net/virtio-net.c                       |  17 +--
migration/savevm.c                        |   1 +
plugins/core.c                            |  43 +++++---
qga/main.c                                |   2 +-
replay/replay.c                           |  23 +---
system/runstate.c                         |  31 +++++-
target/i386/tcg/access.c                  |  27 +++--
plugins/meson.build                       |  50 ++++-----
scripts/checkpatch.pl                     |  24 +++++
scripts/replay-dump.py                    | 167 ++++++++++++++++++++----------
tests/avocado/machine_rx_gdbsim.py        |   2 -
tests/avocado/replay_kernel.py            |  31 +++++-
tests/avocado/replay_linux.py             |  10 ++
27 files changed, 406 insertions(+), 192 deletions(-)
[PULL for 9.1 00/21] Some fixes for 9.1-rc3 (build, replay, docs, plugins)
Posted by Alex Bennée 3 months, 1 week ago
The following changes since commit b0d6c037ea0a825192285f149af72a73a1baaa04:

  Update version for v9.1.0-rc2 release (2024-08-15 20:32:37 +1000)

are available in the Git repository at:

  https://gitlab.com/stsquad/qemu.git tags/pull-maintainer-9.1-rc3-150824-1

for you to fetch changes up to f39cbfc9f8dcb17e8168735acb6fb75638d9904f:

  plugins: fix race condition with scoreboards (2024-08-15 12:10:23 +0100)

----------------------------------------------------------------
Some fixes for 9.1-rc3 (build, replay, docs, plugins)

  - re-enable gdbsim-r5f562n8 test
  - ensure updates to python deps re-trigger configure
  - tweak configure detection of GDB MTE support
  - make checkpatch emit more warnings on updating headers
  - allow i386 access_ptr to force slow path for plugins
  - fixe some replay regressions
  - update the replay-dump tool
  - better handle muxed chardev during replay
  - clean up TCG plugins docs to mention scoreboards
  - fix plugin scoreboard race condition

----------------------------------------------------------------
Alex Bennée (2):
      Makefile: trigger re-configure on updated pythondeps
      target/i386: allow access_ptr to force slow path on failed probe

Gustavo Romero (3):
      configure: Fix arch detection for GDB_HAS_MTE
      configure: Avoid use of param. expansion when using gdb_version
      configure: Fix GDB version detection for GDB_HAS_MTE

Nicholas Piggin (10):
      scripts/replay-dump.py: Update to current rr record format
      scripts/replay-dump.py: rejig decoders in event number order
      tests/avocado: excercise scripts/replay-dump.py in replay tests
      replay: allow runstate shutdown->running when replaying trace
      Revert "replay: stop us hanging in rr_wait_io_event"
      tests/avocado: replay_kernel.py add x86-64 q35 machine test
      chardev: set record/replay on the base device of a muxed device
      virtio-net: Use replay_schedule_bh_event for bhs that affect machine state
      virtio-net: Use virtual time for RSC timers
      savevm: Fix load_snapshot error path crash

Philippe Mathieu-Daudé (2):
      tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot
      buildsys: Fix building without plugins on Darwin

Pierrick Bouvier (2):
      docs/devel: update tcg-plugins page
      plugins: fix race condition with scoreboards

Stefan Weil (1):
      docs: Fix some typos (found by typos) and grammar issues

Stefano Garzarella (1):
      scripts/checkpatch: more checks on files imported from Linux

 docs/about/emulation.rst                  |  49 +++++++--
 docs/devel/migration/uadk-compression.rst |   4 +-
 docs/devel/tcg-plugins.rst                |  13 ++-
 docs/interop/qemu-ga.rst                  |   2 +-
 docs/tools/qemu-vmsr-helper.rst           |   4 +-
 configure                                 |   8 +-
 Makefile                                  |   3 +-
 qapi/rocker.json                          |   4 +-
 include/exec/memory.h                     |   2 +-
 include/sysemu/replay.h                   |   5 -
 include/sysemu/runstate.h                 |   1 +
 accel/tcg/tcg-accel-ops-rr.c              |   2 +-
 chardev/char.c                            |  71 +++++++++----
 hw/arm/smmu-common.c                      |   2 +-
 hw/net/virtio-net.c                       |  17 +--
 migration/savevm.c                        |   1 +
 plugins/core.c                            |  43 +++++---
 qga/main.c                                |   2 +-
 replay/replay.c                           |  23 +---
 system/runstate.c                         |  31 +++++-
 target/i386/tcg/access.c                  |  27 +++--
 plugins/meson.build                       |  50 ++++-----
 scripts/checkpatch.pl                     |  24 +++++
 scripts/replay-dump.py                    | 167 ++++++++++++++++++++----------
 tests/avocado/machine_rx_gdbsim.py        |   2 -
 tests/avocado/replay_kernel.py            |  31 +++++-
 tests/avocado/replay_linux.py             |  10 ++
 27 files changed, 406 insertions(+), 192 deletions(-)

-- 
2.39.2


Re: [PULL for 9.1 00/21] Some fixes for 9.1-rc3 (build, replay, docs, plugins)
Posted by Richard Henderson 3 months, 1 week ago
On 8/16/24 00:48, Alex Bennée wrote:
> The following changes since commit b0d6c037ea0a825192285f149af72a73a1baaa04:
> 
>    Update version for v9.1.0-rc2 release (2024-08-15 20:32:37 +1000)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/stsquad/qemu.git tags/pull-maintainer-9.1-rc3-150824-1
> 
> for you to fetch changes up to f39cbfc9f8dcb17e8168735acb6fb75638d9904f:
> 
>    plugins: fix race condition with scoreboards (2024-08-15 12:10:23 +0100)
> 
> ----------------------------------------------------------------
> Some fixes for 9.1-rc3 (build, replay, docs, plugins)
> 
>    - re-enable gdbsim-r5f562n8 test
>    - ensure updates to python deps re-trigger configure
>    - tweak configure detection of GDB MTE support
>    - make checkpatch emit more warnings on updating headers
>    - allow i386 access_ptr to force slow path for plugins
>    - fixe some replay regressions
>    - update the replay-dump tool
>    - better handle muxed chardev during replay
>    - clean up TCG plugins docs to mention scoreboards
>    - fix plugin scoreboard race condition
> 
> ----------------------------------------------------------------
> Alex Bennée (2):
>        Makefile: trigger re-configure on updated pythondeps
>        target/i386: allow access_ptr to force slow path on failed probe
> 
> Gustavo Romero (3):
>        configure: Fix arch detection for GDB_HAS_MTE
>        configure: Avoid use of param. expansion when using gdb_version
>        configure: Fix GDB version detection for GDB_HAS_MTE
> 
> Nicholas Piggin (10):
>        scripts/replay-dump.py: Update to current rr record format
>        scripts/replay-dump.py: rejig decoders in event number order
>        tests/avocado: excercise scripts/replay-dump.py in replay tests
>        replay: allow runstate shutdown->running when replaying trace
>        Revert "replay: stop us hanging in rr_wait_io_event"
>        tests/avocado: replay_kernel.py add x86-64 q35 machine test
>        chardev: set record/replay on the base device of a muxed device
>        virtio-net: Use replay_schedule_bh_event for bhs that affect machine state
>        virtio-net: Use virtual time for RSC timers
>        savevm: Fix load_snapshot error path crash
> 
> Philippe Mathieu-Daudé (2):
>        tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot
>        buildsys: Fix building without plugins on Darwin
> 
> Pierrick Bouvier (2):
>        docs/devel: update tcg-plugins page
>        plugins: fix race condition with scoreboards
> 
> Stefan Weil (1):
>        docs: Fix some typos (found by typos) and grammar issues

Patch 19:

ERROR: pull request includes commits attributed to list

Please re-spin.


r~

Re: [PULL for 9.1 00/21] Some fixes for 9.1-rc3 (build, replay, docs, plugins)
Posted by Alex Bennée 3 months, 1 week ago
Richard Henderson <richard.henderson@linaro.org> writes:

> On 8/16/24 00:48, Alex Bennée wrote:
>> The following changes since commit b0d6c037ea0a825192285f149af72a73a1baaa04:
>>    Update version for v9.1.0-rc2 release (2024-08-15 20:32:37 +1000)
>> are available in the Git repository at:
>>    https://gitlab.com/stsquad/qemu.git
>> tags/pull-maintainer-9.1-rc3-150824-1
>> for you to fetch changes up to
>> f39cbfc9f8dcb17e8168735acb6fb75638d9904f:
>>    plugins: fix race condition with scoreboards (2024-08-15 12:10:23
>> +0100)
>> ----------------------------------------------------------------
>> Some fixes for 9.1-rc3 (build, replay, docs, plugins)
>>    - re-enable gdbsim-r5f562n8 test
>>    - ensure updates to python deps re-trigger configure
>>    - tweak configure detection of GDB MTE support
>>    - make checkpatch emit more warnings on updating headers
>>    - allow i386 access_ptr to force slow path for plugins
>>    - fixe some replay regressions
>>    - update the replay-dump tool
>>    - better handle muxed chardev during replay
>>    - clean up TCG plugins docs to mention scoreboards
>>    - fix plugin scoreboard race condition
>> ----------------------------------------------------------------
>> Alex Bennée (2):
>>        Makefile: trigger re-configure on updated pythondeps
>>        target/i386: allow access_ptr to force slow path on failed probe
>> Gustavo Romero (3):
>>        configure: Fix arch detection for GDB_HAS_MTE
>>        configure: Avoid use of param. expansion when using gdb_version
>>        configure: Fix GDB version detection for GDB_HAS_MTE
>> Nicholas Piggin (10):
>>        scripts/replay-dump.py: Update to current rr record format
>>        scripts/replay-dump.py: rejig decoders in event number order
>>        tests/avocado: excercise scripts/replay-dump.py in replay tests
>>        replay: allow runstate shutdown->running when replaying trace
>>        Revert "replay: stop us hanging in rr_wait_io_event"
>>        tests/avocado: replay_kernel.py add x86-64 q35 machine test
>>        chardev: set record/replay on the base device of a muxed device
>>        virtio-net: Use replay_schedule_bh_event for bhs that affect machine state
>>        virtio-net: Use virtual time for RSC timers
>>        savevm: Fix load_snapshot error path crash
>> Philippe Mathieu-Daudé (2):
>>        tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot
>>        buildsys: Fix building without plugins on Darwin
>> Pierrick Bouvier (2):
>>        docs/devel: update tcg-plugins page
>>        plugins: fix race condition with scoreboards
>> Stefan Weil (1):
>>        docs: Fix some typos (found by typos) and grammar issues
>
> Patch 19:
>
> ERROR: pull request includes commits attributed to list

What does that actually mean?

> Please re-spin.

Should I just drop Stefan's doc tweaks?

>
>
> r~

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro
Re: [PULL for 9.1 00/21] Some fixes for 9.1-rc3 (build, replay, docs, plugins)
Posted by Thomas Huth 3 months, 1 week ago
On 16/08/2024 14.54, Alex Bennée wrote:
> Richard Henderson <richard.henderson@linaro.org> writes:
> 
>> On 8/16/24 00:48, Alex Bennée wrote:
>>> The following changes since commit b0d6c037ea0a825192285f149af72a73a1baaa04:
>>>     Update version for v9.1.0-rc2 release (2024-08-15 20:32:37 +1000)
>>> are available in the Git repository at:
>>>     https://gitlab.com/stsquad/qemu.git
>>> tags/pull-maintainer-9.1-rc3-150824-1
>>> for you to fetch changes up to
>>> f39cbfc9f8dcb17e8168735acb6fb75638d9904f:
>>>     plugins: fix race condition with scoreboards (2024-08-15 12:10:23
>>> +0100)
>>> ----------------------------------------------------------------
>>> Some fixes for 9.1-rc3 (build, replay, docs, plugins)
>>>     - re-enable gdbsim-r5f562n8 test
>>>     - ensure updates to python deps re-trigger configure
>>>     - tweak configure detection of GDB MTE support
>>>     - make checkpatch emit more warnings on updating headers
>>>     - allow i386 access_ptr to force slow path for plugins
>>>     - fixe some replay regressions
>>>     - update the replay-dump tool
>>>     - better handle muxed chardev during replay
>>>     - clean up TCG plugins docs to mention scoreboards
>>>     - fix plugin scoreboard race condition
>>> ----------------------------------------------------------------
>>> Alex Bennée (2):
>>>         Makefile: trigger re-configure on updated pythondeps
>>>         target/i386: allow access_ptr to force slow path on failed probe
>>> Gustavo Romero (3):
>>>         configure: Fix arch detection for GDB_HAS_MTE
>>>         configure: Avoid use of param. expansion when using gdb_version
>>>         configure: Fix GDB version detection for GDB_HAS_MTE
>>> Nicholas Piggin (10):
>>>         scripts/replay-dump.py: Update to current rr record format
>>>         scripts/replay-dump.py: rejig decoders in event number order
>>>         tests/avocado: excercise scripts/replay-dump.py in replay tests
>>>         replay: allow runstate shutdown->running when replaying trace
>>>         Revert "replay: stop us hanging in rr_wait_io_event"
>>>         tests/avocado: replay_kernel.py add x86-64 q35 machine test
>>>         chardev: set record/replay on the base device of a muxed device
>>>         virtio-net: Use replay_schedule_bh_event for bhs that affect machine state
>>>         virtio-net: Use virtual time for RSC timers
>>>         savevm: Fix load_snapshot error path crash
>>> Philippe Mathieu-Daudé (2):
>>>         tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot
>>>         buildsys: Fix building without plugins on Darwin
>>> Pierrick Bouvier (2):
>>>         docs/devel: update tcg-plugins page
>>>         plugins: fix race condition with scoreboards
>>> Stefan Weil (1):
>>>         docs: Fix some typos (found by typos) and grammar issues
>>
>> Patch 19:
>>
>> ERROR: pull request includes commits attributed to list
> 
> What does that actually mean?

Stefan's patch has:

From: Stefan Weil via <qemu-devel@nongnu.org>

>> Please re-spin.
> 
> Should I just drop Stefan's doc tweaks?

No, IIRC, you can fix the commit by doing

  git commit --amend --reset-author="..."

  HTH,
   Thomas