All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stuart Brady <sdbrady@ntlworld.com>
To: Stefan Weil <weil@mail.berlios.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] tcg, tci: Add TCG and interpreter for bytecode (virtual machine)
Date: Mon, 12 Oct 2009 00:17:36 +0100	[thread overview]
Message-ID: <20091011231735.GA10123@miranda.arrow> (raw)
In-Reply-To: <4AD208F5.9000203@mail.berlios.de>

On Sun, Oct 11, 2009 at 06:33:57PM +0200, Stefan Weil wrote:
> Stuart Brady schrieb:
> > exec-all.h defines 'USE_DIRECT_JUMP' based on the host architecture,
> > which is not relevant under TCI. Also, a tb_set_jmp_target1()
> > implementation is needed for TCI.

> Done. Tested with mips / mipsel host.

I think the #ifdef CONFIG_TCG_INTERPRETER sections belong before
the arch-specific sections. :-)

> Alignment is still to be done (my test platforms don't need it).

I will do this at some point if nobody beats me to it. :-)

BTW, I'm wondering about certain places that require host-specific
code... for example:

 * qemu-lock.h, which is used by cpu_exec() for the tb lock and by
   cpu_unlink_tb() for non-NPTL hosts.

 * The generic cpu_get_real_ticks() doesn't seem great... I suppose
   the MIPS version using clock() might be more appropriate?

Apart from this, I suppose user mode emulation simply has to depend
on host-specific code for signal handling... :-(

I gather INDEX_op_call handling could be improved?  Presumably that's
really just a matter of casting to the right type of function pointer
when making each call?

Cheers,
-- 
Stuart Brady

  parent reply	other threads:[~2009-10-11 23:16 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-28 16:50 [Qemu-devel] [RFC] tcg/interpreter: Add TCG + interpreter for bytecode (virtual machine) Stefan Weil
2009-09-28 16:50 ` [Qemu-devel] [PATCH] tcg, tci: Add TCG and " Stefan Weil
2009-09-28 18:42   ` Blue Swirl
2009-09-28 21:33     ` Stefan Weil
2009-10-04 18:27   ` Stuart Brady
2009-10-11 16:33     ` Stefan Weil
2009-10-11 16:55       ` malc
2009-10-11 21:04       ` malc
2009-10-26 15:26         ` Stefan Weil
2009-10-26 15:52           ` malc
2009-10-11 23:17       ` Stuart Brady [this message]
2009-10-11 23:50         ` malc
2009-10-12  6:46           ` Stuart Brady
2009-10-12 12:33             ` malc
2009-10-12 12:55               ` malc
2009-10-12 17:39               ` Stuart Brady
2009-10-12 20:07                 ` malc
2009-10-12 20:30                   ` Stuart Brady
2010-05-12 19:06 ` [Qemu-devel] [RFC] tcg/interpreter: Add TCG + " Stefan Weil
2010-05-13  2:01 ` Jun Koi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20091011231735.GA10123@miranda.arrow \
    --to=sdbrady@ntlworld.com \
    --cc=qemu-devel@nongnu.org \
    --cc=weil@mail.berlios.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.