On 11/03/13 18:58, Jason J. Herne wrote: > Make use of new kvm_s390_get_registers_partial() for kvm_handle_css_inst() and > handle_hypercall() since they only need registers from the partial set and > they > are called quite frequently. > > Signed-off-by: Jason J. Herne <jjhe...@us.ibm.com> > --- > target-s390x/kvm.c | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c > index 934757e..36861aa 100644 > --- a/target-s390x/kvm.c > +++ b/target-s390x/kvm.c > @@ -468,12 +468,16 @@ static int kvm_handle_css_inst(S390CPU *cpu, struct > kvm_run *run, > int r = 0; > int no_cc = 0; > CPUS390XState *env = &cpu->env; > + CPUState *cs = ENV_GET_CPU(env); > > if (ipa0 != 0xb2) { > /* Not handled for now. */ > return -1; > } > - cpu_synchronize_state(env); > + > + kvm_s390_get_registers_partial(cs); > + cs->kvm_vcpu_dirty = true; > + > switch (ipa1) { > case PRIV_XSCH: > r = ioinst_handle_xsch(env, env->regs[1]);
Looks good, but can you also do the same for TSCH? This is also related to the main I/O path. > @@ -604,7 +608,10 @@ static int handle_priv(S390CPU *cpu, struct kvm_run *run, > > static int handle_hypercall(CPUS390XState *env, struct kvm_run *run) > { > - cpu_synchronize_state(env); > + CPUState *cs = ENV_GET_CPU(env); > + > + kvm_s390_get_registers_partial(cs); > + cs->kvm_vcpu_dirty = true; > env->regs[2] = s390_virtio_hypercall(env); > > return 0; >