From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42419) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z4lVg-0001vZ-E8 for qemu-devel@nongnu.org; Tue, 16 Jun 2015 03:42:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z4lVb-0006Pm-Ds for qemu-devel@nongnu.org; Tue, 16 Jun 2015 03:42:16 -0400 Received: from mail-pa0-x236.google.com ([2607:f8b0:400e:c03::236]:35974) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z4lVb-0006Pg-67 for qemu-devel@nongnu.org; Tue, 16 Jun 2015 03:42:11 -0400 Received: by pabqy3 with SMTP id qy3so7600825pab.3 for ; Tue, 16 Jun 2015 00:42:10 -0700 (PDT) From: Peter Crosthwaite Date: Tue, 16 Jun 2015 00:41:56 -0700 Message-Id: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH v2 0/5] More core code ENV_GET_CPU() removals List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Peter Crosthwaite , afaerber@suse.de, rth@twiddle.net Hi Andreas, Richard and all, I'm moving towards the goal of having no core code usages of ENV_GET_CPU. This has two advantages: 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 specific CPUState. This allows continuing placement of arch specifics before the env in the CPU container (which has TCG perf advantages). Due to point 1, I'm sending this ahead as I think it has standalone value, rather than send as part of multi-arch. Regards, Peter changed since v1 (AF, Eduardo review): Rebase of Bharata's patches Add () to fn's in commit messages Remove uses of ENV_GET_CPU() in target code Include and complete cpu-exec patch (formerly in multi-arch RFC v2) Peter Crosthwaite (5): 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 cpu-exec: Purge all uses of CPU_GET_ENV() bsd-user/main.c | 5 +++-- cpu-exec.c | 28 +++++++++++++--------------- cpus.c | 8 +++----- dtc | 2 +- exec.c | 7 +++---- gdbstub.c | 9 +++------ include/exec/exec-all.h | 4 ++-- include/exec/gdbstub.h | 2 +- linux-user/main.c | 30 +++++++++++++++--------------- linux-user/signal.c | 2 +- target-alpha/cpu.c | 2 +- target-alpha/cpu.h | 2 +- target-alpha/sys_helper.c | 2 +- target-arm/cpu.c | 2 +- target-arm/cpu.h | 2 +- target-cris/cpu.c | 2 +- target-cris/cpu.h | 2 +- target-i386/cpu.c | 2 +- target-i386/cpu.h | 2 +- target-i386/translate.c | 2 +- target-lm32/cpu.c | 2 +- target-lm32/cpu.h | 2 +- target-m68k/cpu.c | 2 +- target-m68k/cpu.h | 2 +- target-microblaze/cpu.c | 2 +- target-microblaze/cpu.h | 2 +- target-mips/cpu.c | 2 +- target-mips/cpu.h | 2 +- target-moxie/cpu.c | 2 +- target-moxie/cpu.h | 2 +- target-openrisc/cpu.c | 2 +- target-openrisc/cpu.h | 2 +- target-ppc/cpu.h | 2 +- target-ppc/translate_init.c | 2 +- target-s390x/cpu.c | 2 +- target-s390x/cpu.h | 2 +- target-sh4/cpu.c | 2 +- target-sh4/cpu.h | 2 +- target-sparc/cpu.c | 2 +- target-sparc/cpu.h | 2 +- target-tricore/cpu.c | 2 +- target-tricore/cpu.h | 2 +- target-unicore32/cpu.c | 2 +- target-unicore32/cpu.h | 3 ++- target-xtensa/cpu.c | 2 +- target-xtensa/cpu.h | 2 +- translate-all.c | 6 ++---- 47 files changed, 84 insertions(+), 92 deletions(-) -- 1.9.1