On 07/07/20 15:58, Claudio Fontana wrote: > + > +CpusAccel tcg_cpus = { > + .create_vcpu_thread = tcg_start_vcpu_thread, > + .kick_vcpu_thread = tcg_kick_vcpu_thread, > + > + .synchronize_post_reset = tcg_cpu_synchronize_noop, > + .synchronize_post_init = tcg_cpu_synchronize_noop, > + .synchronize_state = tcg_cpu_synchronize_noop, > + .synchronize_pre_loadvm = tcg_cpu_synchronize_noop, > +};
Could this struct reside in AccelClass instead, so that there's no need to register the operations explicitly? We could still cache it in a global variable in accel_init_machine, in order to avoid pointer chasing. Thanks, Paolo