[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/10] disas: Support the Capstone disassembler
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 04/10] disas: Support the Capstone disassembler library |
Date: |
Sat, 16 Sep 2017 19:52:26 +0100 |
On 14 September 2017 at 19:35, Richard Henderson
<address@hidden> wrote:
>
> + /* ??? Capstone requires that we copy the data into a host-addressable
> + buffer first and has no call-back to read more. Therefore we need
> + an estimate of buffer size. This will work for most RISC, but we'll
> + need to figure out something else for variable-length ISAs. */
> + if (s.info.cap_arch >= 0 && cap_disas(&s.info, pc, 4 * nb_insn)) {
> + return;
> + }
Can we do something with gradually adding more to our host buffer
until capstone says it's managed to disassemble the right number
of instructions?
thanks
-- PMM
- [Qemu-devel] [PATCH 05/10] target/i386: Support Capstone in disas_set_info, (continued)
- [Qemu-devel] [PATCH 05/10] target/i386: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 01/10] target/i386: Convert to disas_set_info hook, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 03/10] disas: Remove unused flags arguments, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 07/10] target/ppc: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 04/10] disas: Support the Capstone disassembler library, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 06/10] target/arm: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 08/10] target/s390x: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 10/10] target/mips: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- [Qemu-devel] [PATCH 09/10] target/sparc: Support Capstone in disas_set_info, Richard Henderson, 2017/09/14
- Re: [Qemu-devel] [PATCH 00/10] Support the Capstone disassembler, Philippe Mathieu-Daudé, 2017/09/15