From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:34957) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RKzxR-0003ZV-SM for qemu-devel@nongnu.org; Mon, 31 Oct 2011 18:03:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RKzxQ-000840-Dv for qemu-devel@nongnu.org; Mon, 31 Oct 2011 18:03:53 -0400 Received: from v220110690675601.yourvserver.net ([78.47.199.172]:36723) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RKzxQ-00082t-1D for qemu-devel@nongnu.org; Mon, 31 Oct 2011 18:03:52 -0400 Message-ID: <4EAF1B0F.90500@weilnetz.de> Date: Mon, 31 Oct 2011 23:02:55 +0100 From: Stefan Weil MIME-Version: 1.0 References: <4EAF13A8.5020303@weilnetz.de> <4EAF1550.9080107@us.ibm.com> In-Reply-To: <4EAF1550.9080107@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL] tcg/interpreter: Add TCG + interpreter for bytecode List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Blue Swirl , QEMU Developers , Aurelien Jarno 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