From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, qemu-ppc@nongnu.org
Subject: [Qemu-devel] [PATCH v3 0/9] Support the Capstone disassembler
Date: Tue, 26 Sep 2017 13:14:18 -0700 [thread overview]
Message-ID: <20170926201427.2833-1-richard.henderson@linaro.org> (raw)
Changes since v2:
* Fix a minor buglet in the monitor disassembly
* Add capstone as a submodule, preferring that when present.
The submodule idea came from discussion with PMM. The idea is that
we would potentially remove the old binutils code, and libvxil, and
rely on capstone entirely. Which would then of course require capstone
to be present in order to get reasonable debugging.
I have not done the actual removal yet, but that is an easy next step.
r~
PS: Comments from the ppc folk? Can I encourage you to try this and
sanity check what capstone is emitting? Unlike for s390x, it doesn't
look obviously wrong to me...
Richard Henderson (9):
target/i386: Convert to disas_set_info hook
target/ppc: Convert to disas_set_info hook
disas: Remove unused flags arguments
disas: Support the Capstone disassembler library
i386: Support Capstone in disas_set_info
arm: Support Capstone in disas_set_info
ppc: Support Capstone in disas_set_info
disas: Remove monitor_disas_is_physical
disas: Add capstone as submodule
Makefile | 4 +
include/disas/bfd.h | 4 +
include/disas/capstone.h | 38 ++++++
include/disas/disas.h | 4 +-
include/exec/log.h | 4 +-
disas.c | 308 ++++++++++++++++++++++++++++++------------
monitor.c | 29 +---
target/alpha/translate.c | 2 +-
target/arm/cpu.c | 21 ++-
target/arm/translate-a64.c | 3 +-
target/arm/translate.c | 3 +-
target/cris/translate.c | 3 +-
target/hppa/translate.c | 2 +-
target/i386/cpu.c | 19 +++
target/i386/translate.c | 8 +-
target/lm32/translate.c | 2 +-
target/m68k/translate.c | 2 +-
target/microblaze/translate.c | 2 +-
target/mips/translate.c | 2 +-
target/nios2/translate.c | 2 +-
target/openrisc/translate.c | 2 +-
target/ppc/translate.c | 5 +-
target/ppc/translate_init.c | 27 ++++
target/s390x/translate.c | 2 +-
target/sh4/translate.c | 2 +-
target/sparc/translate.c | 2 +-
target/tricore/translate.c | 2 +-
target/unicore32/translate.c | 2 +-
target/xtensa/translate.c | 2 +-
.gitmodules | 3 +
capstone | 1 +
configure | 36 +++++
32 files changed, 397 insertions(+), 151 deletions(-)
create mode 100644 include/disas/capstone.h
create mode 160000 capstone
--
2.13.5
next reply other threads:[~2017-09-26 20:14 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-26 20:14 Richard Henderson [this message]
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 1/9] target/i386: Convert to disas_set_info hook Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 2/9] target/ppc: " Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 3/9] disas: Remove unused flags arguments Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 4/9] disas: Support the Capstone disassembler library Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 5/9] i386: Support Capstone in disas_set_info Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 6/9] arm: " Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 7/9] ppc: " Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 8/9] disas: Remove monitor_disas_is_physical Richard Henderson
2017-09-26 20:14 ` [Qemu-devel] [PATCH v3 9/9] disas: Add capstone as submodule Richard Henderson
2017-09-26 20:27 ` [Qemu-devel] [PATCH v3 0/9] Support the Capstone disassembler no-reply
2017-09-26 20:29 ` no-reply
2017-09-27 19:06 ` Richard Henderson
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=20170926201427.2833-1-richard.henderson@linaro.org \
--to=richard.henderson@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@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).