Skip to content

Disas QOMification, round 2

Peter Crosthwaite qemu-devel 1 version Applied

Status from this message: I have queued them on qom-cpu-next now (subjects to be adjusted), but I

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 16 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/8disas: s390x: QOMify target specific disas setupdisas.c target-s390x/cpu.c+8 -61 reply
2/8disas: moxie: QOMify target specific disas setupdisas.c target-moxie/cpu.c+7 -60 replies
3/8disas: m68k: QOMify target specific disas setupdisas.c target-m68k/cpu.c+7 -42 replies
4/8disas: sparc: QOMify target specific disas setupdisas.c target-sparc/cpu.c+9 -100 replies
5/8disas: lm32: QOMify target specific disas setupdisas.c target-lm32/cpu.c+8 -61 reply
6/8disas: sh4: QOMify target specific disas setupdisas.c target-sh4/cpu.c+8 -61 reply
7/8disas: mips: QOMify target specific disas setupdisas.c target-mips/cpu.c+10 -121 reply
8/8disas: alpha: QOMify target specific disas setupdisas.c target-alpha/cpu.c+8 -51 reply

Reviews trailers per patch

4 of 8 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchLaurent VivierMichael WalleGreg UngererLeon AlraeAurelien JarnoRichard Henderson
SeriesA-b
1/8
2/8
3/8R-b R-b
4/8
5/8A-b
6/8A-b
7/8A-b
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/8] Disas QOMification, round 2

Continue QOMifying target-specific disassembly. Convert all arches except for X86 and PPC. They will be round 3 and are non-trivial.

This brings us close to no arch-specific code in disas.c allow conversion to common-obj and preparing these arches for inclusion in multi-arch.

Tested to various degrees depending on image availability. Arches with bios's are the easiest, others have sample test images available. Some that accept binary blobs as code I half-tested with random data as code. Moxie has no testing. Status:

s390: Tested with bios
moxie: No testing (requires an elf)
m68k: Tested with random code
sparc,sparc64: Tested with bios
lm32: Tested with random code
sh4: Tested with sample image
mips: Tested with sample image
alpha: Tested with bios

Peter Crosthwaite (8):
disas: s390x: QOMify target specific disas setup
disas: moxie: QOMify target specific disas setup
disas: m68k: QOMify target specific disas setup
disas: sparc: QOMify target specific disas setup
disas: lm32: QOMify target specific disas setup
disas: sh4: QOMify target specific disas setup
disas: mips: QOMify target specific disas setup
disas: alpha: QOMify target specific disas setup

 disas.c            | 55 ------------------------------------------------------
 target-alpha/cpu.c |  8 ++++++++
 target-lm32/cpu.c  |  8 ++++++++
 target-m68k/cpu.c  |  7 +++++++
 target-mips/cpu.c  | 10 ++++++++++
 target-moxie/cpu.c |  7 +++++++
 target-s390x/cpu.c |  8 ++++++++
 target-sh4/cpu.c   |  8 ++++++++
 target-sparc/cpu.c |  9 +++++++++
 9 files changed, 65 insertions(+), 55 deletions(-)
Signature · 1 line
1.9.1