On Sat, Oct 12, 2019 at 03:42:56PM -0400, Chuck Lever wrote: > +static int __ib_poll_cq(struct ib_cq *cq, int num_entries, struct ib_wc *wc) > +{ > + int rc; > + > + rc = ib_poll_cq(cq, num_entries, wc); > + trace_cq_poll(cq, num_entries, rc); > + return rc; > +} Why not put the trace point in ib_poll_cq directly? What is the overhead of these things if you don't use them? Jason