[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 5/7] disas: microblaze: QOMify target specifi
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH v3 5/7] disas: microblaze: QOMify target specific disas setup |
Date: |
Thu, 28 May 2015 21:43:18 -0700 |
Ping!
On Sun, May 24, 2015 at 3:47 PM, Peter Crosthwaite
<address@hidden> wrote:
> Move the target_disas() MB specifics to the QOM disas_set_info hook
> and delete the MB specific code in disas.c.
>
> This also now adds support for monitor disas to Microblaze.
>
> E.g.
> (qemu) xp 0x90000000
> 0000000090000000: 0x94208001
>
> And before this patch:
> (qemu) xp/i 0x90000000
> 0x90000000: Asm output not supported on this arch
>
> After:
> (qemu) xp/i 0x90000000
> 0x90000000: mfs r1, rmsr
>
> Signed-off-by: Peter Crosthwaite <address@hidden>
> ---
> disas.c | 3 ---
> target-microblaze/cpu.c | 8 ++++++++
> 2 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/disas.c b/disas.c
> index fde5029..937e08b 100644
> --- a/disas.c
> +++ b/disas.c
> @@ -268,9 +268,6 @@ void target_disas(FILE *out, CPUState *cpu, target_ulong
> code,
> #elif defined(TARGET_S390X)
> s.info.mach = bfd_mach_s390_64;
> s.info.print_insn = print_insn_s390;
> -#elif defined(TARGET_MICROBLAZE)
> - s.info.mach = bfd_arch_microblaze;
> - s.info.print_insn = print_insn_microblaze;
> #elif defined(TARGET_MOXIE)
> s.info.mach = bfd_arch_moxie;
> s.info.print_insn = print_insn_moxie;
> diff --git a/target-microblaze/cpu.c b/target-microblaze/cpu.c
> index 67e3182..89b8363 100644
> --- a/target-microblaze/cpu.c
> +++ b/target-microblaze/cpu.c
> @@ -111,6 +111,12 @@ static void mb_cpu_reset(CPUState *s)
> #endif
> }
>
> +static void mb_disas_set_info(CPUState *cpu, disassemble_info *info)
> +{
> + info->mach = bfd_arch_microblaze;
> + info->print_insn = print_insn_microblaze;
> +}
> +
> static void mb_cpu_realizefn(DeviceState *dev, Error **errp)
> {
> CPUState *cs = CPU(dev);
> @@ -183,6 +189,8 @@ static void mb_cpu_class_init(ObjectClass *oc, void *data)
> dc->vmsd = &vmstate_mb_cpu;
> dc->props = mb_properties;
> cc->gdb_num_core_regs = 32 + 5;
> +
> + cc->disas_set_info = mb_disas_set_info;
> }
>
> static const TypeInfo mb_cpu_type_info = {
> --
> 1.9.1
>
>
- [Qemu-devel] [PATCH v3 0/7] Unify and QOMify (target|monitor)_disas, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 1/7] disas: Add print_insn to disassemble info, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 2/7] disas: QOMify target specific setup, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 3/7] disas: arm-a64: Make printfer and stream variable, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 4/7] disas: arm: QOMify target specific disas setup, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 5/7] disas: microblaze: QOMify target specific disas setup, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 6/7] disas: cris: Fix 0 buffer length case, Peter Crosthwaite, 2015/05/24
- [Qemu-devel] [PATCH v3 7/7] disas: cris: QOMify target specific disas setup, Peter Crosthwaite, 2015/05/24