SYS-OSS/FRET-qemu
4
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

accel/tcg: Use CPUState.cc instead of CPU_GET_CLASS in cpu-exec.c

Browse Source 
CPU_GET_CLASS does runtime type checking; use the cached
copy of the class instead.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Richard Henderson 2024-01-28 12:57:59 +10:00
parent 2889fb8bd2
commit 991bd65ddd
1 changed files with 56 additions and 53 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

61
accel/tcg/cpu-exec.c
View File

@ -343,9 +343,9 @@ static bool check_for_breakpoints_slow(CPUState *cpu, vaddr pc,
#ifdef CONFIG_USER_ONLY #ifdef CONFIG_USER_ONLY
g_assert_not_reached(); g_assert_not_reached();
#else #else
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
assert(cc->tcg_ops->debug_check_breakpoint); assert(tcg_ops->debug_check_breakpoint);
match_bp = cc->tcg_ops->debug_check_breakpoint(cpu); match_bp = tcg_ops->debug_check_breakpoint(cpu);
#endif #endif
} }
@ -462,10 +462,11 @@ cpu_tb_exec(CPUState *cpu, TranslationBlock *itb, int *tb_exit)
* counter hit zero); we must restore the guest PC to the address * counter hit zero); we must restore the guest PC to the address
* of the start of the TB. * of the start of the TB.
*/ */
CPUClass *cc = CPU_GET_CLASS(cpu); CPUClass *cc = cpu->cc;
const TCGCPUOps *tcg_ops = cc->tcg_ops;
if (cc->tcg_ops->synchronize_from_tb) { if (tcg_ops->synchronize_from_tb) {
cc->tcg_ops->synchronize_from_tb(cpu, last_tb); tcg_ops->synchronize_from_tb(cpu, last_tb);
} else { } else {
tcg_debug_assert(!(tb_cflags(last_tb) & CF_PCREL)); tcg_debug_assert(!(tb_cflags(last_tb) & CF_PCREL));
assert(cc->set_pc); assert(cc->set_pc);
@ -497,19 +498,19 @@ cpu_tb_exec(CPUState *cpu, TranslationBlock *itb, int *tb_exit)
static void cpu_exec_enter(CPUState *cpu) static void cpu_exec_enter(CPUState *cpu)
{ {
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
if (cc->tcg_ops->cpu_exec_enter) { if (tcg_ops->cpu_exec_enter) {
cc->tcg_ops->cpu_exec_enter(cpu); tcg_ops->cpu_exec_enter(cpu);
} }
} }
static void cpu_exec_exit(CPUState *cpu) static void cpu_exec_exit(CPUState *cpu)
{ {
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
if (cc->tcg_ops->cpu_exec_exit) { if (tcg_ops->cpu_exec_exit) {
cc->tcg_ops->cpu_exec_exit(cpu); tcg_ops->cpu_exec_exit(cpu);
} }
} }
@ -685,7 +686,7 @@ static inline bool cpu_handle_halt(CPUState *cpu)
static inline void cpu_handle_debug_exception(CPUState *cpu) static inline void cpu_handle_debug_exception(CPUState *cpu)
{ {
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
CPUWatchpoint *wp; CPUWatchpoint *wp;
if (!cpu->watchpoint_hit) { if (!cpu->watchpoint_hit) {
@ -694,8 +695,8 @@ static inline void cpu_handle_debug_exception(CPUState *cpu)
} }
} }
if (cc->tcg_ops->debug_excp_handler) { if (tcg_ops->debug_excp_handler) {
cc->tcg_ops->debug_excp_handler(cpu); tcg_ops->debug_excp_handler(cpu);
} }
} }
@ -712,6 +713,7 @@ static inline bool cpu_handle_exception(CPUState *cpu, int *ret)
#endif #endif
return false; return false;
} }
if (cpu->exception_index >= EXCP_INTERRUPT) { if (cpu->exception_index >= EXCP_INTERRUPT) {
/* exit request from the cpu execution loop */ /* exit request from the cpu execution loop */
*ret = cpu->exception_index; *ret = cpu->exception_index;
@ -720,23 +722,26 @@ static inline bool cpu_handle_exception(CPUState *cpu, int *ret)
} }
cpu->exception_index = -1; cpu->exception_index = -1;
return true; return true;
} else { }
#if defined(CONFIG_USER_ONLY) #if defined(CONFIG_USER_ONLY)
/* if user mode only, we simulate a fake exception /*
which will be handled outside the cpu execution * If user mode only, we simulate a fake exception which will be
loop */ * handled outside the cpu execution loop.
*/
#if defined(TARGET_I386) #if defined(TARGET_I386)
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
cc->tcg_ops->fake_user_interrupt(cpu); tcg_ops->fake_user_interrupt(cpu);
#endif /* TARGET_I386 */ #endif /* TARGET_I386 */
*ret = cpu->exception_index; *ret = cpu->exception_index;
cpu->exception_index = -1; cpu->exception_index = -1;
return true; return true;
#else #else
if (replay_exception()) { if (replay_exception()) {
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
bql_lock(); bql_lock();
cc->tcg_ops->do_interrupt(cpu); tcg_ops->do_interrupt(cpu);
bql_unlock(); bql_unlock();
cpu->exception_index = -1; cpu->exception_index = -1;
@ -756,7 +761,6 @@ static inline bool cpu_handle_exception(CPUState *cpu, int *ret)
return true; return true;
} }
#endif #endif
}
return false; return false;
} }
@ -856,10 +860,10 @@ static inline bool cpu_handle_interrupt(CPUState *cpu,
True when it is, and we should restart on a new TB, True when it is, and we should restart on a new TB,
and via longjmp via cpu_loop_exit. */ and via longjmp via cpu_loop_exit. */
else { else {
CPUClass *cc = CPU_GET_CLASS(cpu); const TCGCPUOps *tcg_ops = cpu->cc->tcg_ops;
if (cc->tcg_ops->cpu_exec_interrupt && if (tcg_ops->cpu_exec_interrupt &&
cc->tcg_ops->cpu_exec_interrupt(cpu, interrupt_request)) { tcg_ops->cpu_exec_interrupt(cpu, interrupt_request)) {
if (need_replay_interrupt(interrupt_request)) { if (need_replay_interrupt(interrupt_request)) {
replay_interrupt(); replay_interrupt();
} }
@ -1078,10 +1082,9 @@ int cpu_exec(CPUState *cpu)
bool tcg_exec_realizefn(CPUState *cpu, Error **errp) bool tcg_exec_realizefn(CPUState *cpu, Error **errp)
{ {
static bool tcg_target_initialized; static bool tcg_target_initialized;
CPUClass *cc = CPU_GET_CLASS(cpu);
if (!tcg_target_initialized) { if (!tcg_target_initialized) {
cc->tcg_ops->initialize(); cpu->cc->tcg_ops->initialize();
tcg_target_initialized = true; tcg_target_initialized = true;
} }