cpus: Pass CPUState to [qemu_]cpu_has_work()

For target-mips also change the return type to bool.

Make include paths for cpu-qom.h consistent for alpha and unicore32.

Signed-off-by: Andreas Färber <afaerber@suse.de>
[AF: Updated new target-openrisc function accordingly]
Acked-by: Richard Henderson <rth@twiddle.net> (for alpha)
This commit is contained in:
Andreas Färber 2012-05-03 06:43:49 +02:00
parent b13ce26d3e
commit 3993c6bddf
23 changed files with 66 additions and 31 deletions

View File

@ -438,8 +438,6 @@ void cpu_reset_interrupt(CPUArchState *env, int mask);
void cpu_exit(CPUArchState *s); void cpu_exit(CPUArchState *s);
bool qemu_cpu_has_work(CPUArchState *env);
/* Breakpoint/watchpoint flags */ /* Breakpoint/watchpoint flags */
#define BP_MEM_READ 0x01 #define BP_MEM_READ 0x01
#define BP_MEM_WRITE 0x02 #define BP_MEM_WRITE 0x02

View File

@ -27,9 +27,9 @@ int tb_invalidated_flag;
//#define CONFIG_DEBUG_EXEC //#define CONFIG_DEBUG_EXEC
bool qemu_cpu_has_work(CPUArchState *env) bool qemu_cpu_has_work(CPUState *cpu)
{ {
return cpu_has_work(env); return cpu_has_work(cpu);
} }
void cpu_loop_exit(CPUArchState *env) void cpu_loop_exit(CPUArchState *env)
@ -181,16 +181,14 @@ volatile sig_atomic_t exit_request;
int cpu_exec(CPUArchState *env) int cpu_exec(CPUArchState *env)
{ {
#ifdef TARGET_PPC
CPUState *cpu = ENV_GET_CPU(env); CPUState *cpu = ENV_GET_CPU(env);
#endif
int ret, interrupt_request; int ret, interrupt_request;
TranslationBlock *tb; TranslationBlock *tb;
uint8_t *tc_ptr; uint8_t *tc_ptr;
tcg_target_ulong next_tb; tcg_target_ulong next_tb;
if (env->halted) { if (env->halted) {
if (!cpu_has_work(env)) { if (!cpu_has_work(cpu)) {
return EXCP_HALTED; return EXCP_HALTED;
} }

2
cpus.c
View File

@ -72,7 +72,7 @@ static bool cpu_thread_is_idle(CPUArchState *env)
if (cpu->stopped || !runstate_is_running()) { if (cpu->stopped || !runstate_is_running()) {
return true; return true;
} }
if (!env->halted || qemu_cpu_has_work(env) || if (!env->halted || qemu_cpu_has_work(cpu) ||
kvm_async_interrupts_enabled()) { kvm_async_interrupts_enabled()) {
return false; return false;
} }

View File

@ -516,7 +516,7 @@ static target_ulong h_cede(PowerPCCPU *cpu, sPAPREnvironment *spapr,
env->msr |= (1ULL << MSR_EE); env->msr |= (1ULL << MSR_EE);
hreg_compute_hflags(env); hreg_compute_hflags(env);
if (!cpu_has_work(env)) { if (!cpu_has_work(CPU(cpu))) {
env->halted = 1; env->halted = 1;
env->exception_index = EXCP_HLT; env->exception_index = EXCP_HLT;
env->exit_request = 1; env->exit_request = 1;

View File

@ -131,7 +131,7 @@ static void xtensa_ccompare_cb(void *opaque)
if (env->halted) { if (env->halted) {
env->halt_clock = qemu_get_clock_ns(vm_clock); env->halt_clock = qemu_get_clock_ns(vm_clock);
xtensa_advance_ccount(env, env->wake_ccount - env->sregs[CCOUNT]); xtensa_advance_ccount(env, env->wake_ccount - env->sregs[CCOUNT]);
if (!cpu_has_work(env)) { if (!cpu_has_work(CPU(cpu))) {
env->sregs[CCOUNT] = env->wake_ccount + 1; env->sregs[CCOUNT] = env->wake_ccount + 1;
xtensa_rearm_ccompare_timer(env); xtensa_rearm_ccompare_timer(env);
} }

View File

@ -86,6 +86,16 @@ struct CPUState {
*/ */
void cpu_reset(CPUState *cpu); void cpu_reset(CPUState *cpu);
/**
* qemu_cpu_has_work:
* @cpu: The vCPU to check.
*
* Checks whether the CPU has work to do.
*
* Returns: %true if the CPU has work, %false otherwise.
*/
bool qemu_cpu_has_work(CPUState *cpu);
/** /**
* qemu_cpu_is_self: * qemu_cpu_is_self:
* @cpu: The vCPU to check against. * @cpu: The vCPU to check against.

View File

@ -19,7 +19,7 @@
* <http://www.gnu.org/licenses/lgpl-2.1.html> * <http://www.gnu.org/licenses/lgpl-2.1.html>
*/ */
#include "cpu-qom.h" #include "cpu.h"
#include "qemu-common.h" #include "qemu-common.h"

View File

@ -510,8 +510,10 @@ static inline void cpu_set_tls(CPUAlphaState *env, target_ulong newtls)
} }
#endif #endif
static inline bool cpu_has_work(CPUAlphaState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUAlphaState *env = &ALPHA_CPU(cpu)->env;
/* Here we are checking to see if the CPU should wake up from HALT. /* Here we are checking to see if the CPU should wake up from HALT.
We will have gotten into this state only for WTINT from PALmode. */ We will have gotten into this state only for WTINT from PALmode. */
/* ??? I'm not sure how the IPL state works with WTINT to keep a CPU /* ??? I'm not sure how the IPL state works with WTINT to keep a CPU

View File

@ -718,8 +718,10 @@ static inline void cpu_get_tb_cpu_state(CPUARMState *env, target_ulong *pc,
} }
} }
static inline bool cpu_has_work(CPUARMState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUARMState *env = &ARM_CPU(cpu)->env;
return env->interrupt_request & return env->interrupt_request &
(CPU_INTERRUPT_FIQ | CPU_INTERRUPT_HARD | CPU_INTERRUPT_EXITTB); (CPU_INTERRUPT_FIQ | CPU_INTERRUPT_HARD | CPU_INTERRUPT_EXITTB);
} }

View File

@ -285,8 +285,10 @@ static inline void cpu_get_tb_cpu_state(CPUCRISState *env, target_ulong *pc,
#define cpu_list cris_cpu_list #define cpu_list cris_cpu_list
void cris_cpu_list(FILE *f, fprintf_function cpu_fprintf); void cris_cpu_list(FILE *f, fprintf_function cpu_fprintf);
static inline bool cpu_has_work(CPUCRISState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUCRISState *env = &CRIS_CPU(cpu)->env;
return env->interrupt_request & (CPU_INTERRUPT_HARD | CPU_INTERRUPT_NMI); return env->interrupt_request & (CPU_INTERRUPT_HARD | CPU_INTERRUPT_NMI);
} }

View File

@ -1100,8 +1100,10 @@ static inline void cpu_clone_regs(CPUX86State *env, target_ulong newsp)
#include "hw/apic.h" #include "hw/apic.h"
#endif #endif
static inline bool cpu_has_work(CPUX86State *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUX86State *env = &X86_CPU(cpu)->env;
return ((env->interrupt_request & (CPU_INTERRUPT_HARD | return ((env->interrupt_request & (CPU_INTERRUPT_HARD |
CPU_INTERRUPT_POLL)) && CPU_INTERRUPT_POLL)) &&
(env->eflags & IF_MASK)) || (env->eflags & IF_MASK)) ||

View File

@ -253,8 +253,10 @@ static inline void cpu_get_tb_cpu_state(CPULM32State *env, target_ulong *pc,
*flags = 0; *flags = 0;
} }
static inline bool cpu_has_work(CPULM32State *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPULM32State *env = &LM32_CPU(cpu)->env;
return env->interrupt_request & CPU_INTERRUPT_HARD; return env->interrupt_request & CPU_INTERRUPT_HARD;
} }

View File

@ -257,8 +257,10 @@ static inline void cpu_get_tb_cpu_state(CPUM68KState *env, target_ulong *pc,
| ((env->macsr >> 4) & 0xf); /* Bits 0-3 */ | ((env->macsr >> 4) & 0xf); /* Bits 0-3 */
} }
static inline bool cpu_has_work(CPUM68KState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUM68KState *env = &M68K_CPU(cpu)->env;
return env->interrupt_request & CPU_INTERRUPT_HARD; return env->interrupt_request & CPU_INTERRUPT_HARD;
} }

View File

@ -374,8 +374,10 @@ void cpu_unassigned_access(CPUMBState *env1, hwaddr addr,
int is_write, int is_exec, int is_asi, int size); int is_write, int is_exec, int is_asi, int size);
#endif #endif
static inline bool cpu_has_work(CPUMBState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUMBState *env = &MICROBLAZE_CPU(cpu)->env;
return env->interrupt_request & (CPU_INTERRUPT_HARD | CPU_INTERRUPT_NMI); return env->interrupt_request & (CPU_INTERRUPT_HARD | CPU_INTERRUPT_NMI);
} }

View File

@ -706,16 +706,17 @@ static inline int mips_vpe_active(CPUMIPSState *env)
return active; return active;
} }
static inline int cpu_has_work(CPUMIPSState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
int has_work = 0; CPUMIPSState *env = &MIPS_CPU(cpu)->env;
bool has_work = false;
/* It is implementation dependent if non-enabled interrupts /* It is implementation dependent if non-enabled interrupts
wake-up the CPU, however most of the implementations only wake-up the CPU, however most of the implementations only
check for interrupts that can be taken. */ check for interrupts that can be taken. */
if ((env->interrupt_request & CPU_INTERRUPT_HARD) && if ((env->interrupt_request & CPU_INTERRUPT_HARD) &&
cpu_mips_hw_interrupts_pending(env)) { cpu_mips_hw_interrupts_pending(env)) {
has_work = 1; has_work = true;
} }
/* MIPS-MT has the ability to halt the CPU. */ /* MIPS-MT has the ability to halt the CPU. */
@ -723,11 +724,11 @@ static inline int cpu_has_work(CPUMIPSState *env)
/* The QEMU model will issue an _WAKE request whenever the CPUs /* The QEMU model will issue an _WAKE request whenever the CPUs
should be woken up. */ should be woken up. */
if (env->interrupt_request & CPU_INTERRUPT_WAKE) { if (env->interrupt_request & CPU_INTERRUPT_WAKE) {
has_work = 1; has_work = true;
} }
if (!mips_vpe_active(env)) { if (!mips_vpe_active(env)) {
has_work = 0; has_work = false;
} }
} }
return has_work; return has_work;

View File

@ -437,8 +437,10 @@ static inline int cpu_mmu_index(CPUOpenRISCState *env)
} }
#define CPU_INTERRUPT_TIMER CPU_INTERRUPT_TGT_INT_0 #define CPU_INTERRUPT_TIMER CPU_INTERRUPT_TGT_INT_0
static inline bool cpu_has_work(CPUOpenRISCState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUOpenRISCState *env = &OPENRISC_CPU(cpu)->env;
return env->interrupt_request & (CPU_INTERRUPT_HARD | return env->interrupt_request & (CPU_INTERRUPT_HARD |
CPU_INTERRUPT_TIMER); CPU_INTERRUPT_TIMER);
} }

View File

@ -2222,8 +2222,10 @@ static inline bool msr_is_64bit(CPUPPCState *env, target_ulong msr)
extern void (*cpu_ppc_hypercall)(PowerPCCPU *); extern void (*cpu_ppc_hypercall)(PowerPCCPU *);
static inline bool cpu_has_work(CPUPPCState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUPPCState *env = &POWERPC_CPU(cpu)->env;
return msr_ee && (env->interrupt_request & CPU_INTERRUPT_HARD); return msr_ee && (env->interrupt_request & CPU_INTERRUPT_HARD);
} }

View File

@ -977,8 +977,10 @@ static inline void cpu_inject_ext(CPUS390XState *env, uint32_t code, uint32_t pa
cpu_interrupt(env, CPU_INTERRUPT_HARD); cpu_interrupt(env, CPU_INTERRUPT_HARD);
} }
static inline bool cpu_has_work(CPUS390XState *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUS390XState *env = &S390_CPU(cpu)->env;
return (env->interrupt_request & CPU_INTERRUPT_HARD) && return (env->interrupt_request & CPU_INTERRUPT_HARD) &&
(env->psw.mask & PSW_MASK_EXT); (env->psw.mask & PSW_MASK_EXT);
} }

View File

@ -371,8 +371,10 @@ static inline void cpu_get_tb_cpu_state(CPUSH4State *env, target_ulong *pc,
| (env->movcal_backup ? TB_FLAG_PENDING_MOVCA : 0); /* Bit 4 */ | (env->movcal_backup ? TB_FLAG_PENDING_MOVCA : 0); /* Bit 4 */
} }
static inline bool cpu_has_work(CPUSH4State *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUSH4State *env = &SUPERH_CPU(cpu)->env;
return env->interrupt_request & CPU_INTERRUPT_HARD; return env->interrupt_request & CPU_INTERRUPT_HARD;
} }

View File

@ -764,8 +764,10 @@ static inline bool tb_am_enabled(int tb_flags)
#endif #endif
} }
static inline bool cpu_has_work(CPUSPARCState *env1) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUSPARCState *env1 = &SPARC_CPU(cpu)->env;
return (env1->interrupt_request & CPU_INTERRUPT_HARD) && return (env1->interrupt_request & CPU_INTERRUPT_HARD) &&
cpu_interrupts_enabled(env1); cpu_interrupts_enabled(env1);
} }

View File

@ -12,7 +12,7 @@
* or (at your option) any later version. * or (at your option) any later version.
*/ */
#include "cpu-qom.h" #include "cpu.h"
#include "qemu-common.h" #include "qemu-common.h"
static inline void set_feature(CPUUniCore32State *env, int feature) static inline void set_feature(CPUUniCore32State *env, int feature)

View File

@ -181,8 +181,10 @@ void uc32_translate_init(void);
void do_interrupt(CPUUniCore32State *); void do_interrupt(CPUUniCore32State *);
void switch_mode(CPUUniCore32State *, int); void switch_mode(CPUUniCore32State *, int);
static inline bool cpu_has_work(CPUUniCore32State *env) static inline bool cpu_has_work(CPUState *cpu)
{ {
CPUUniCore32State *env = &UNICORE32_CPU(cpu)->env;
return env->interrupt_request & return env->interrupt_request &
(CPU_INTERRUPT_HARD | CPU_INTERRUPT_EXITTB); (CPU_INTERRUPT_HARD | CPU_INTERRUPT_EXITTB);
} }

View File

@ -501,8 +501,10 @@ static inline void cpu_get_tb_cpu_state(CPUXtensaState *env, target_ulong *pc,
#include "cpu-all.h" #include "cpu-all.h"
#include "exec-all.h" #include "exec-all.h"
static inline int cpu_has_work(CPUXtensaState *env) static inline int cpu_has_work(CPUState *cpu)
{ {
CPUXtensaState *env = &XTENSA_CPU(cpu)->env;
return env->pending_irq_level; return env->pending_irq_level;
} }