On 25/07/12 15:17, Christoffer Dall wrote: > list rename, shot myself in the foot. > > >>> + } >>> + >>> + /************************************************************** >>> + * Enter the guest >>> + */ >>> + trace_kvm_entry(vcpu->arch.regs.pc); >>> + kvm_guest_enter(); >>> + vcpu->mode = IN_GUEST_MODE; >>> + >>> + ret = __kvm_vcpu_run(vcpu); >>> + >>> + vcpu->mode = OUTSIDE_GUEST_MODE; >>> + vcpu->stat.exits++; >> >> The tracepoint above should be sufficient for statistics. >> > > Marc added this one, so I assume there's a valid need for the separate > counter. Marc? Probably because I've been too lazy to setup a trace infrastructure on my test boxes. Stats are easier to access, but seeing now that they've been deprecated and scheduled for removal, we probably shouldn't bother with them. M. -- Jazz is not dead. It just smells funny...