On Wed, Oct 09, 2019 at 08:50:02PM -0400, Richard Henderson wrote: > On 10/7/19 1:06 PM, Andrew Jones wrote: > > +static void kvm_arm_vm_state_change(void *opaque, int running, RunState > > state) > > +{ > > + CPUState *cs = opaque; > > + ARMCPU *cpu = ARM_CPU(cs); > > + > > + if (running) { > > + if (cpu->kvm_adjvtime) { > > + kvm_arm_set_virtual_time(cs); > > + } > > + } > > +} > > Worth putting this in kvm.c too, so you don't have to duplicate it? You can > always split it apart later if you ever do need a different hook for 32 vs 64.
True. I'll send a v1 without the premature duplication of this function and your r-b's Thanks, drew