On Fri, Jan 23, 2015 at 11:45:06AM +0000, David Vrabel wrote: > > + */ > > +void xen_end_upcall(struct pt_regs *regs) > > +{ > > + if (xen_is_preemptible_hypercall(regs)) { > > + int cpuid = smp_processor_id(); > > + if (_cond_resched()) > > + trace_xen_hypercall_preemption(cpuid); > > I don't think a tracepoint here is useful. OK.. I'll remove. > > + } > > +} > > +NOKPROBE_SYMBOL(xen_end_upcall); > > Do we need this is this function is no longer notrace? Stephen and Andy were going down some corner case rabbit hole and it seemed to me that the conclusion was not settled so to be safe I kept it. I'll let them decide. I did remove the notrace junk. Luis -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html