Message ID | d99c7edf85168916b973679f34e75920d2a6b8d6.1430798763.git.crosthwaite.peter@gmail.com |
---|---|
State | New |
Headers | show |
On 05.05.2015 06:45, Peter Crosthwaite wrote: > In a normal disassembly flow, the printf and stream being used varies > from disas job to job. In particular it varies if mixing monitor_disas > and target_disas. > > Make both the printfer function and target stream settable in the > QEMUDisassmbler class. Remove the stream_ initialisation from the > constructor as it will now runtime vary (and an initial value won't > mean very much). > > Signed-off-by: Peter Crosthwaite <crosthwaite.peter@gmail.com> > --- > disas/arm-a64.cc | 22 +++++++++++++++++----- > 1 file changed, 17 insertions(+), 5 deletions(-) > > diff --git a/disas/arm-a64.cc b/disas/arm-a64.cc > index e04f946..d725e75 100644 > --- a/disas/arm-a64.cc > +++ b/disas/arm-a64.cc > @@ -35,16 +35,25 @@ static Disassembler *vixl_disasm = NULL; > */ > class QEMUDisassembler : public Disassembler { > public: > - explicit QEMUDisassembler(FILE *stream) : stream_(stream) { } > + explicit QEMUDisassembler() { } > ~QEMUDisassembler() { } > > + void SetStream(FILE *stream) { > + stream_ = stream; > + } > + > + void SetPrintf(int (*printf_fn)(FILE *, const char *, ...)) { > + printf_ = printf_fn; > + } > + > protected: > virtual void ProcessOutput(const Instruction *instr) { > - fprintf(stream_, "%08" PRIx32 " %s", > + printf_(stream_, "%08" PRIx32 " %s", > instr->InstructionBits(), GetOutput()); > } > > private: > + int (*printf_)(FILE *, const char *, ...); > FILE *stream_; > }; > > @@ -53,9 +62,9 @@ static int vixl_is_initialized(void) > return vixl_decoder != NULL; > } > > -static void vixl_init(FILE *f) { > +static void vixl_init() { > vixl_decoder = new Decoder(); > - vixl_disasm = new QEMUDisassembler(f); > + vixl_disasm = new QEMUDisassembler(); > vixl_decoder->AppendVisitor(vixl_disasm); > } > > @@ -78,9 +87,12 @@ int print_insn_arm_a64(uint64_t addr, disassemble_info *info) > } > > if (!vixl_is_initialized()) { > - vixl_init(info->stream); > + vixl_init(); > } > > + ((QEMUDisassembler *)vixl_disasm)->SetPrintf(info->fprintf_func); > + ((QEMUDisassembler *)vixl_disasm)->SetStream(info->stream); > + > instrval = bytes[0] | bytes[1] << 8 | bytes[2] << 16 | bytes[3] << 24; > instr = reinterpret_cast<const Instruction *>(&instrval); > vixl_disasm->MapCodeAddress(addr, instr); > Reviewed-by: Claudio Fontana <claudio.fontana@huawei.com> Tested-by: Claudio Fontana <claudio.fontana@huawei.com>
On 05/04/2015 09:45 PM, Peter Crosthwaite wrote: > - explicit QEMUDisassembler(FILE *stream) : stream_(stream) { } > + explicit QEMUDisassembler() { } The explicit is no longer helpful, and I'd still initialize stream_ to null. r~
diff --git a/disas/arm-a64.cc b/disas/arm-a64.cc index e04f946..d725e75 100644 --- a/disas/arm-a64.cc +++ b/disas/arm-a64.cc @@ -35,16 +35,25 @@ static Disassembler *vixl_disasm = NULL; */ class QEMUDisassembler : public Disassembler { public: - explicit QEMUDisassembler(FILE *stream) : stream_(stream) { } + explicit QEMUDisassembler() { } ~QEMUDisassembler() { } + void SetStream(FILE *stream) { + stream_ = stream; + } + + void SetPrintf(int (*printf_fn)(FILE *, const char *, ...)) { + printf_ = printf_fn; + } + protected: virtual void ProcessOutput(const Instruction *instr) { - fprintf(stream_, "%08" PRIx32 " %s", + printf_(stream_, "%08" PRIx32 " %s", instr->InstructionBits(), GetOutput()); } private: + int (*printf_)(FILE *, const char *, ...); FILE *stream_; }; @@ -53,9 +62,9 @@ static int vixl_is_initialized(void) return vixl_decoder != NULL; } -static void vixl_init(FILE *f) { +static void vixl_init() { vixl_decoder = new Decoder(); - vixl_disasm = new QEMUDisassembler(f); + vixl_disasm = new QEMUDisassembler(); vixl_decoder->AppendVisitor(vixl_disasm); } @@ -78,9 +87,12 @@ int print_insn_arm_a64(uint64_t addr, disassemble_info *info) } if (!vixl_is_initialized()) { - vixl_init(info->stream); + vixl_init(); } + ((QEMUDisassembler *)vixl_disasm)->SetPrintf(info->fprintf_func); + ((QEMUDisassembler *)vixl_disasm)->SetStream(info->stream); + instrval = bytes[0] | bytes[1] << 8 | bytes[2] << 16 | bytes[3] << 24; instr = reinterpret_cast<const Instruction *>(&instrval); vixl_disasm->MapCodeAddress(addr, instr);
In a normal disassembly flow, the printf and stream being used varies from disas job to job. In particular it varies if mixing monitor_disas and target_disas. Make both the printfer function and target stream settable in the QEMUDisassmbler class. Remove the stream_ initialisation from the constructor as it will now runtime vary (and an initial value won't mean very much). Signed-off-by: Peter Crosthwaite <crosthwaite.peter@gmail.com> --- disas/arm-a64.cc | 22 +++++++++++++++++----- 1 file changed, 17 insertions(+), 5 deletions(-)