On Thu, Jul 26, 2012 at 5:30 PM, Marc Zyngier <marc.zyngier at arm.com> wrote: > 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. > you want to take them all out then? -Christoffer
