From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34172) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z4jhZ-0004jw-7H for qemu-devel@nongnu.org; Tue, 16 Jun 2015 01:46:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z4jhT-0006c4-IY for qemu-devel@nongnu.org; Tue, 16 Jun 2015 01:46:25 -0400 Received: from mail-pa0-x232.google.com ([2607:f8b0:400e:c03::232]:36121) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z4jhT-0006bL-Az for qemu-devel@nongnu.org; Tue, 16 Jun 2015 01:46:19 -0400 Received: by pabqy3 with SMTP id qy3so5734606pab.3 for ; Mon, 15 Jun 2015 22:46:18 -0700 (PDT) From: Peter Crosthwaite Date: Mon, 15 Jun 2015 22:46:04 -0700 Message-Id: Subject: [Qemu-devel] [PATCH v2 0/4] qom-cpu: Wrap set_pc hook and use in bootloaders List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, Peter Crosthwaite , afaerber@suse.de, edgar.iglesias@gmail.com Wrap the CPUClass::set_pc fn hook in a caller helper to reduce verbosity of calls. Simplify the call from the gdbstub. Then use the call to abstract away the PC env fields from the ARM and Microblaze bootloaders. This moves towards the goal of minimising system level code of the CPU env (and one step closer to common-obj'ing the bootloaders). There's a long way to go (at least for ARM, not so far for MB), but this is a small win in that direction. This helps with multi-arch where the current thinking is to compile out the maximum content possible from cpu.h. This removes program counter definitions from the multi-arch cpu.h compile-in list. changed since v1: Remove thumb changes Peter Crosthwaite (4): qom: cpu: Add wrapper to the set-pc hook gdbstub: Use cpu_set_pc helper arm: boot: Use cpu_set_pc microblaze: boot: Use cpu_set_pc gdbstub.c | 5 +---- hw/arm/boot.c | 19 +++++++------------ hw/microblaze/boot.c | 2 +- include/qom/cpu.h | 21 +++++++++++++++++++++ 4 files changed, 30 insertions(+), 17 deletions(-) -- 1.9.1