Re: [Qemu-devel] [PULL] tcg/interpreter: Add TCG + interpreter for bytecode

[Stefan Weil]

Am 31.10.2011 22:38, schrieb Anthony Liguori:
> On 10/31/2011 04:31 PM, Stefan Weil wrote:
> > Hi,
> >
> > this is a 3rd version of the patch series which adds support for
> > QEMU on any host by using a TCG interpreter (TCI).
> >
> > Version 2 was sent to the list and is available here:
> > http://lists.gnu.org/archive/html/qemu-devel/2011-10/msg02504.html
> >
> > The new version removed the TRACE macros which I used during 
> > development.
> > They are no longer needed. This fixes malc's feedback.
> >
> > Patches 1, 2, 5 and 6 are only rebased.
> >
> > Patch 1 is useful even without TCI and improves the handling of a
> > special case needed for PPC hosts.
> >
> > Please tell me if I should send the complete patches to qemu-devel.
> >
> > I'd appreciate if the series could be pulled before QEMU 1.0 to form
> > a base for further development of TCI.
>
> I'll defer to Blue and/or Aurelien here as I don't know enough about 
> TCG to really review this series.
>
> But I'd recommend waiting to pull this until after 1.1 opens up.  This 
> is the sort of thing that 1) people are likely to actually use and 2) 
> could use a full development cycle of testing to get hardened before 
> showing up in a release.
>
> I'd feel better about pulling this into 1.0 if it was guarded by a 
> configure option and disabled by default just to enforce that it's an 
> experimental feature.
>
> Regards,
>
> Anthony Liguori 


The current version of TCI is only activated with configure 
--enable-tcg-interpreter
(see http://lists.gnu.org/archive/html/qemu-devel/2011-10/msg02509.html),
so this should be safe for 1.0. And yes, it's still an experimental feature.

It's up to the maintainers of (Linux) distributions when they will add 
QEMU with TCI
for new hosts which were unsupported by native TCG. It won't work 
automatically -
they have to add the configure option for it. I have no access to such 
hosts for testing,
but offer any help I can give.

Regards,

Stefan Weil