From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44016) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7oFS-0001OI-Ed for qemu-devel@nongnu.org; Wed, 24 Jun 2015 13:14:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z7oFO-0000DK-8G for qemu-devel@nongnu.org; Wed, 24 Jun 2015 13:14:06 -0400 Received: from cantor2.suse.de ([195.135.220.15]:46898 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7oFN-00009o-Vo for qemu-devel@nongnu.org; Wed, 24 Jun 2015 13:14:02 -0400 Message-ID: <558AE557.60207@suse.de> Date: Wed, 24 Jun 2015 19:13:59 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH qom v4 0/7] More core code ENV_GET_CPU removals List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Crosthwaite , qemu-devel@nongnu.org Cc: ehabkost@redhat.com, Peter Crosthwaite , Riku Voipio , Paolo Bonzini , aurelien@aurel32.net, rth@twiddle.net Hi, Am 24.06.2015 um 04:31 schrieb Peter Crosthwaite: > Hi All, >=20 > I'm moving towards the goal of having no core code usages of ENV_GET_CP= U. > This has two advantages: >=20 > 1: It means we are closer to common-obj'ing core code like exec.c, cpus= .c > and friends. > 2: Multi arch is easier if ENV_GET_CPU() stays arch specific. It means = I > don't need those patches where I reorder the env within the arch specif= ic > CPUState. This allows continuing placement of arch specifics before the > env in the CPU container (which has TCG perf advantages). >=20 > Due to point 1, I'm sending this ahead as I think it has standalone val= ue, > rather than send as part of multi-arch. Thanks for rebasing and combining this so handily, rebased onto qom-cpu with the dtc change stripped and a few massages to the messages: https://github.com/afaerber/qemu-cpu/commits/qom-cpu I considered modifying the ppc patch to have a local CPUState variable for the future, but left it as is for now. Most changes seemed mechanical. Alex gave a go-ahead; a few CCs haven't replied yet, please speak up within the next few days if we should wait (in particular Paolo and Riku). I still need to test on BSD - if someone has a Tested-by there that would be helpful. Regards, Andreas > Bharata B Rao (3): > cpus: Add Error argument to cpu_exec_init() > cpus: Convert cpu_index into a bitmap > ppc: Move cpu_exec_init() call to realize function >=20 > Peter Crosthwaite (4): > translate-all: Change tb_flush() env argument to cpu > gdbserver: _fork: Change fn to accept cpu instead of env > cpus: Change tcg_cpu_exec() arg to cpu, not env > cpus: Change exec_init() arg to cpu, not env >=20 > bsd-user/main.c | 2 +- > cpus.c | 7 +++-- > dtc | 2 +- > exec.c | 63 +++++++++++++++++++++++++++++++++++++= +------- > gdbstub.c | 9 +++---- > include/exec/exec-all.h | 4 +-- > include/exec/gdbstub.h | 2 +- > include/qom/cpu.h | 1 + > linux-user/main.c | 2 +- > linux-user/signal.c | 2 +- > qom/cpu.c | 7 +++++ > target-alpha/cpu.c | 2 +- > target-alpha/sys_helper.c | 2 +- > target-arm/cpu.c | 2 +- > target-cris/cpu.c | 2 +- > target-i386/cpu.c | 2 +- > target-i386/translate.c | 2 +- > target-lm32/cpu.c | 2 +- > target-m68k/cpu.c | 2 +- > target-microblaze/cpu.c | 2 +- > target-mips/cpu.c | 2 +- > target-moxie/cpu.c | 2 +- > target-openrisc/cpu.c | 2 +- > target-ppc/translate_init.c | 9 +++++-- > target-s390x/cpu.c | 2 +- > target-sh4/cpu.c | 2 +- > target-sparc/cpu.c | 2 +- > target-tricore/cpu.c | 2 +- > target-unicore32/cpu.c | 2 +- > target-xtensa/cpu.c | 2 +- > translate-all.c | 6 ++--- > 31 files changed, 101 insertions(+), 51 deletions(-) >=20 --=20 SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Felix Imend=C3=B6rffer, Jane Smithard, Dilip Upmanyu, Graham Norton; = HRB 21284 (AG N=C3=BCrnberg)