From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KqCWE-0001Oq-Gf for qemu-devel@nongnu.org; Wed, 15 Oct 2008 15:58:54 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KqCW9-0001MA-1m for qemu-devel@nongnu.org; Wed, 15 Oct 2008 15:58:51 -0400 Received: from [199.232.76.173] (port=36777 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KqCW8-0001Lo-3b for qemu-devel@nongnu.org; Wed, 15 Oct 2008 15:58:48 -0400 Received: from mx2.redhat.com ([66.187.237.31]:49429) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KqCW6-0001H6-R0 for qemu-devel@nongnu.org; Wed, 15 Oct 2008 15:58:47 -0400 From: Glauber Costa Date: Wed, 15 Oct 2008 19:54:59 -0200 Message-Id: <1224107718-19128-3-git-send-email-glommer@redhat.com> In-Reply-To: <1224107718-19128-1-git-send-email-glommer@redhat.com> References: <1224107718-19128-1-git-send-email-glommer@redhat.com> Subject: [Qemu-devel] [PATCH 02/21] introduce QEMUAccel and fill it with interrupt specific driver Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: aliguori@us.ibm.com, jan.kiszka@siemens.com, jes@sgi.com, avi@qumranet.com, Glauber Costa , dmitry.baryshkov@siemens.com From: Glauber Costa This patch introduces QEMUAccel, a placeholder for function pointers that aims at helping qemu to abstract accelerators such as kqemu and kvm (actually, the 'accelerator' name was proposed by avi kivity, since he loves referring to kvm that way). To begin with, the accelerator is given the opportunity to register a cpu_interrupt function, to be called after the raw cpu_interrupt. This has the side effect of, for the kqemu accelerator, calling kqemu_cpu_interrupt everytime, which didn't use to happen. But looking at the code, this seems safe to me. This patch applies on raw qemu. Signed-off-by: Glauber Costa Signed-off-by: Dmitry Baryshkov --- Makefile.target | 2 +- accel.c | 17 +++++++++++++++++ accel.h | 20 ++++++++++++++++++++ exec.c | 3 +++ kqemu.c | 9 +++++++++ vl.c | 17 +++++------------ 6 files changed, 55 insertions(+), 13 deletions(-) create mode 100644 accel.c create mode 100644 accel.h diff --git a/Makefile.target b/Makefile.target index e2edf9d..623ecd8 100644 --- a/Makefile.target +++ b/Makefile.target @@ -188,7 +188,7 @@ all: $(PROGS) ######################################################### # cpu emulator library LIBOBJS=exec.o kqemu.o translate-all.o cpu-exec.o\ - translate.o host-utils.o + translate.o host-utils.o accel.o ifdef CONFIG_DYNGEN_OP exec.o: dyngen-opc.h LIBOBJS+=op.o diff --git a/accel.c b/accel.c new file mode 100644 index 0000000..d30460d --- /dev/null +++ b/accel.c @@ -0,0 +1,17 @@ +#include "hw/hw.h" +#include "accel.h" + +QEMUAccel *current_accel; + +int _accel_nop(void) +{ + return 0; +} + +#define accel_nop ((void *)_accel_nop) + +/* Accelerator wrapper for the no-accel (raw qemu) case */ +QEMUAccel noaccel = { + .cpu_interrupt = accel_nop, +}; + diff --git a/accel.h b/accel.h new file mode 100644 index 0000000..8e5ddc6 --- /dev/null +++ b/accel.h @@ -0,0 +1,20 @@ +#ifndef _ACCEL_H_ +#define _ACCEL_H_ + +typedef struct QEMUAccel { + void (*cpu_interrupt)(CPUState *env); +} QEMUAccel; + +extern QEMUAccel *current_accel; +extern QEMUAccel noaccel; + +static inline void register_qemu_accel(QEMUAccel *accel) +{ + current_accel = accel; +} + +static inline void accel_cpu_interrupt(CPUState *env) +{ + current_accel->cpu_interrupt(env); +} +#endif diff --git a/exec.c b/exec.c index 1cad0be..21253cc 100644 --- a/exec.c +++ b/exec.c @@ -43,6 +43,8 @@ #include #endif +#include "accel.h" + //#define DEBUG_TB_INVALIDATE //#define DEBUG_FLUSH //#define DEBUG_TLB @@ -1430,6 +1432,7 @@ void cpu_single_step(CPUState *env, int enabled) tb_flush(env); } #endif + accel_cpu_interrupt(env); } /* enable or disable low levels log */ diff --git a/kqemu.c b/kqemu.c index 9b52237..87c06cd 100644 --- a/kqemu.c +++ b/kqemu.c @@ -50,6 +50,7 @@ #include #include #include "kqemu.h" +#include "accel.h" #ifdef _WIN32 #define KQEMU_DEVICE "\\\\.\\kqemu" @@ -150,6 +151,8 @@ static void kqemu_update_cpuid(CPUState *env) accelerated code */ } +QEMUAccel kqemu_accel; + int kqemu_start(void) { struct kqemu_init kinit; @@ -232,6 +235,7 @@ int kqemu_start(void) } nb_pages_to_flush = 0; nb_ram_pages_to_update = 0; + register_qemu_accel(&kqemu_accel); qpi_init(); return 0; @@ -243,6 +247,11 @@ void kqemu_init_env(CPUState *env) env->kqemu_enabled = kqemu_allowed; } +QEMUAccel kqemu_accel = { + .cpu_interrupt = kqemu_cpu_interrupt, +}; + + void kqemu_flush_page(CPUState *env, target_ulong addr) { #if defined(DEBUG) diff --git a/vl.c b/vl.c index 97aca75..f72172f 100644 --- a/vl.c +++ b/vl.c @@ -149,6 +149,8 @@ #define SMBD_COMMAND "/usr/sbin/smbd" #endif +#include "accel.h" + //#define DEBUG_UNUSED_IOPORT //#define DEBUG_IOPORT //#define DEBUG_NET @@ -1317,11 +1319,6 @@ static void host_alarm_handler(int host_signum) if (env) { /* stop the currently executing cpu because a timer occured */ cpu_interrupt(env, CPU_INTERRUPT_EXIT); -#ifdef USE_KQEMU - if (env->kqemu_enabled) { - kqemu_cpu_interrupt(env); - } -#endif } event_pending = 1; } @@ -7554,14 +7551,8 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id) void qemu_service_io(void) { CPUState *env = cpu_single_env; - if (env) { + if (env) cpu_interrupt(env, CPU_INTERRUPT_EXIT); -#ifdef USE_KQEMU - if (env->kqemu_enabled) { - kqemu_cpu_interrupt(env); - } -#endif - } } /***********************************************************/ @@ -8818,6 +8809,8 @@ int main(int argc, char **argv) } #endif + register_qemu_accel(&noaccel); + register_machines(); machine = first_machine; cpu_model = NULL; -- 1.5.5.1