From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33907) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TRDmO-0005CY-HZ for qemu-devel@nongnu.org; Wed, 24 Oct 2012 23:06:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TRDmN-0000yH-GA for qemu-devel@nongnu.org; Wed, 24 Oct 2012 23:06:44 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:39737) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TRDmN-0000xd-AN for qemu-devel@nongnu.org; Wed, 24 Oct 2012 23:06:43 -0400 Received: from eusync4.samsung.com (mailout4.w1.samsung.com [210.118.77.14]) by mailout4.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MCF0004ZHZZ3XB0@mailout4.w1.samsung.com> for qemu-devel@nongnu.org; Thu, 25 Oct 2012 04:07:11 +0100 (BST) Received: from [106.109.8.9] by eusync4.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0MCF008DOHZ1D750@eusync4.samsung.com> for qemu-devel@nongnu.org; Thu, 25 Oct 2012 04:06:38 +0100 (BST) Message-id: <5088ACBD.4040709@samsung.com> Date: Thu, 25 Oct 2012 07:06:37 +0400 From: Evgeny Voevodin MIME-version: 1.0 References: <1350973278-2236-1-git-send-email-e.voevodin@samsung.com> In-reply-to: <1350973278-2236-1-git-send-email-e.voevodin@samsung.com> Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/7] TCG global variables clean-up List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: qemu-devel@nongnu.org, blauwirbel@gmail.com, kyungmin.park@samsung.com, edgar.iglesias@gmail.com, aurelien@aurel32.net, rth@twiddle.net Any other comments on the patches? I didn't get the consensus. Do we need a pointer to tcg context? As I said before, I didn't notice any slow-down with it. On 10/23/2012 10:21 AM, Evgeny Voevodin wrote: > This set of patches moves global variables to tcg_ctx: > gen_opc_ptr > gen_opparam_ptr > gen_opc_buf > gen_opparam_buf > > Build tested for all targets. > Execution tested on ARM. > > I didn't notice any slow-down of kernel boot after this set was applied. > > Changelog: > v1->v2: > Introduced TCGContext *tcg_cur_ctx global to use in those places where we don't > have an interface to pass pointer to tcg_ctx. > Code style clean-up > > Evgeny (2): > tcg/tcg.h: Duplicate global TCG variables in TCGContext > TCG: Remove unused global variables > > Evgeny Voevodin (5): > translate-all.c: Introduce TCGContext *tcg_cur_ctx > TCG: Use gen_opc_ptr from context instead of global variable. > TCG: Use gen_opparam_ptr from context instead of global variable. > TCG: Use gen_opc_buf from context instead of global variable. > TCG: Use gen_opparam_buf from context instead of global variable. > > gen-icount.h | 2 +- > target-alpha/translate.c | 10 +- > target-arm/translate.c | 10 +- > target-cris/translate.c | 13 +- > target-i386/translate.c | 10 +- > target-lm32/translate.c | 13 +- > target-m68k/translate.c | 10 +- > target-microblaze/translate.c | 13 +- > target-mips/translate.c | 11 +- > target-openrisc/translate.c | 13 +- > target-ppc/translate.c | 11 +- > target-s390x/translate.c | 11 +- > target-sh4/translate.c | 10 +- > target-sparc/translate.c | 10 +- > target-unicore32/translate.c | 10 +- > target-xtensa/translate.c | 8 +- > tcg/optimize.c | 62 ++++---- > tcg/tcg-op.h | 324 ++++++++++++++++++++--------------------- > tcg/tcg.c | 85 ++++++----- > tcg/tcg.h | 11 +- > translate-all.c | 4 +- > 21 files changed, 328 insertions(+), 323 deletions(-) > -- Kind regards, Evgeny Voevodin, Technical Leader, Mobile Group, Samsung Moscow Research Center, e-mail: e.voevodin@samsung.com