From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:47458) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R31eQ-000495-E3 for qemu-devel@nongnu.org; Mon, 12 Sep 2011 04:13:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1R31eP-0001Oq-CI for qemu-devel@nongnu.org; Mon, 12 Sep 2011 04:13:58 -0400 Received: from mx1.redhat.com ([209.132.183.28]:3051) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R31eP-0001Oi-5J for qemu-devel@nongnu.org; Mon, 12 Sep 2011 04:13:57 -0400 Message-ID: <4E6DBF40.5020702@redhat.com> Date: Mon, 12 Sep 2011 10:13:52 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <4E6DA603.7090309@redhat.com> <4E6DBC4E.5030904@twiddle.net> In-Reply-To: <4E6DBC4E.5030904@twiddle.net> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 00/15] Sparc AREG0 conversion List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson Cc: Blue Swirl , qemu-devel On 09/12/2011 10:01 AM, Richard Henderson wrote: > > > After this patch set, only load and store op helpers remain in > > > op_helper.c. I have some patches for those but they need more thought. > > > > Have you benchmarked it? > > Asking for a benchmark without full conversion is pointless. Agreed. But I would not push these patches without having tried them out on a prototype of a full conversion (i.e. with the load/store helpers converted, for which Blue Swirl said he has patches, and with the environment not pinned to AREG0 in TCG code). So I hoped that he did have such a prototype, or alternatively that he benchmarked them and showed only minor degradations. Paolo