qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v1 20/28] plugins: try and make plugin_insn_append more ergon


From: Richard Henderson
Subject: Re: [PATCH v1 20/28] plugins: try and make plugin_insn_append more ergonomic
Date: Tue, 26 Oct 2021 13:09:32 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0

On 10/26/21 3:22 AM, Alex Bennée wrote:
Currently we make the assumption that the guest frontend loads all
op code bytes sequentially. This mostly holds up for regular fixed
encodings but some architectures like s390x like to re-read the
instruction which causes weirdness to occur. Rather than changing the
frontends make the plugin API a little more ergonomic and able to
handle the re-read case.

Stuff will still get strange if we read ahead of the opcode but so far
no front ends have done that and this patch asserts the case so we can
catch it early if they do.

Signed-off-by: Alex Bennée<alex.bennee@linaro.org>
Suggested-by: Richard Henderson<richard.henderson@linaro.org>
---
  include/exec/plugin-gen.h | 12 ++++++++++--
  include/qemu/plugin.h     |  7 +++++--
  accel/tcg/plugin-gen.c    |  3 +--
  accel/tcg/translator.c    |  2 +-
  4 files changed, 17 insertions(+), 7 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~



reply via email to

[Prev in Thread] Current Thread [Next in Thread]