Re: [PATCH RESEND] kprobe: Use percpu counter to collect nhit statistics

From: Ingo Molnar
Date: Tue Feb 09 2016 - 05:09:54 EST



* Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Thu, 4 Feb 2016 10:24:12 +0000
> åæéå / HIRAMATUïMASAMI <masami.hiramatsu.pt@xxxxxxxxxxx> wrote:
>
> > From: Martin KaFai Lau [mailto:kafai@xxxxxx]
> > >
> > >When doing ebpf+kprobe on some hot TCP functions (e.g.
> > >tcp_rcv_established), the kprobe_dispatcher shows up in 'perf report'.
> > >
> > >In kprobe_dispatcher(), there is a lot of cache bouncing
> > >in 'tk->nhit++'. 'tk->nhit' and 'tk->tp.flags' also share
> > >the same cacheline.
> > >
> >
> > Ingo, could you consider to merge this?
> >
>
> Ingo, you can also add my
>
> Acked-by: Steven Rostedt <rostedt@xxxxxxxxxxx>

Yeah, nice optimization, picked it up - thanks guys!

Ingo