qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH 0/3] tci rewrite
@ 2014-05-13 21:22 Richard Henderson
  2014-05-13 21:22 ` [Qemu-devel] [PATCH 1/3] ppc: Disable cacheutils for the interpreter Richard Henderson
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Richard Henderson @ 2014-05-13 21:22 UTC (permalink / raw)
  To: qemu-devel; +Cc: sw

There are a number of problems with the current interpreter:

  (1) It uses direct unaligned accesses as part of the bytecode stream.
      This fails with SIGBUS for strict alignment hosts.  This could of
      course be fixed to use "proper" unaligned accesses, but this would
      just be slow.

  (2) The method by which it implements calls is non-portable, and really
      only works for x86.

  (3) The code is full of ifdefs and TODOs that never got completed.

  (4) The "registers" are in a global "tci_reg" array that implies that
      if threads are ever used, we'll immediately get corruption.  Fixing
      this is complicated by the structure of the interpreter.

  (5) It hasn't been updated to the "new" ldst opcodes.

To me, all of this adds up to a complete rewrite.  While it might just
be possible to stage such a thing in, I really don't see a way to do
that cleanly.  Nor am I certain it's worth the effort.

This rewrite fixes all three problems in one go:

  (1) The bytecodes are encoded into a stream of uint32_t, so there are
      never any unaligned accesses.

  (2) Use libffi.  This requires a few adjustments to the generic call
      code, so that the parameters get layed out on the "stack" as we wish.

  (3) Minimial ifdefage.  My goal was to make sure that nearly all code
      paths are compiled, and then removed by an optimizing compiler if
      they're not reachable.

  (4) Don't do that.

  (5) Do that.

The result works on 4 hosts that I could try: x86, arm, sparc, ppc64.

The speed of emulation appears to be about the same.  Measurement there
seems to be within noise.

The size of both the interpreter and the bytecodes are greatly reduced.

   text	   data	    bss	    dec	    hex	filename
   7563	      0	    160	   7723	   1e2b	bld-old/ppc64-softmmu/tci.o
   4313	      0	      8	   4321	   10e1	bld-new/ppc64-softmmu/tci.o

  Alpha rom boot
  -gen code size       64208/33430528
  +gen code size       36656/33430528

  PPC rom boot
  -gen code size       188144/33430528
  +gen code size       99776/33430528

  Sparc rom boot
  -gen code size       432784/33430528
  +gen code size       271104/33430528

Since the patch is essentially illegible, please have a look at

  git://github.com/rth7680/qemu.git tci-6


r~


Richard Henderson (3):
  ppc: Disable cacheutils for the interpreter
  tci: Build ffi data structures for helpers
  tci: Rewrite from scratch

 configure                    |   12 +
 disas/tci.c                  |  415 +++++++++-
 include/exec/cpu-all.h       |    6 +-
 include/exec/exec-all.h      |    5 +-
 include/exec/helper-ffi.h    |   83 ++
 include/exec/helper-tcg.h    |   18 +-
 include/qemu/cache-utils.h   |    2 +-
 include/qemu/tci.h           |  171 +++++
 target-i386/ops_sse_header.h |    6 +
 target-ppc/helper.h          |    1 +
 tcg/tcg.c                    |   59 +-
 tcg/tci/README               |  113 +--
 tcg/tci/tcg-target.c         | 1303 ++++++++++++++-----------------
 tcg/tci/tcg-target.h         |  156 ++--
 tci.c                        | 1752 ++++++++++++++----------------------------
 translate-all.c              |    5 +-
 util/cache-utils.c           |    2 +-
 17 files changed, 1976 insertions(+), 2133 deletions(-)
 create mode 100644 include/exec/helper-ffi.h
 create mode 100644 include/qemu/tci.h

-- 
1.9.0

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2014-05-13 22:51 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-13 21:22 [Qemu-devel] [PATCH 0/3] tci rewrite Richard Henderson
2014-05-13 21:22 ` [Qemu-devel] [PATCH 1/3] ppc: Disable cacheutils for the interpreter Richard Henderson
2014-05-13 21:22 ` [Qemu-devel] [PATCH 2/3] tci: Build ffi data structures for helpers Richard Henderson
2014-05-13 21:22 ` [Qemu-devel] [PATCH 3/3] tci: Rewrite from scratch Richard Henderson
2014-05-13 21:49 ` [Qemu-devel] [PATCH 0/3] tci rewrite Peter Maydell
2014-05-13 21:52   ` Richard Henderson
2014-05-13 22:12     ` Peter Maydell
2014-05-13 22:51       ` Richard Henderson

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).