Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1215627

Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf event

From Alexei Starovoitov <ast@plumgrid.com>
Newsgroups linux.kernel
Subject Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf event
Date 2015-08-29 02:50 +0200
Message-ID <q2Cvf-2JO-3@gated-at.bofh.it> (permalink)
References <q2lXr-4sW-3@gated-at.bofh.it> <q2lXs-4sW-23@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 8/28/15 12:06 AM, Wang Nan wrote:
> his patch adds a new trace event to establish infrastruction for bpf to
> output data to perf. Userspace perf tools can detect and use this event
> as using the existing tracepoint events.
>
> New bpf trace event entry in debugfs:
>
>       /sys/kernel/debug/tracing/events/bpf/bpf_output_data
>
> Userspace perf tools detect the new tracepoint event as:
>
>       bpf:bpf_output_data                          [Tracepoint event]
>
> Data in ring-buffer of perf events added to this event will be polled
> out, sample types and other attributes can be adjusted to those events
> directly without touching the original kprobe events.

Wang,
I have 2nd thoughts on this.
I've played with it, but global bpf:bpf_output_data event is limiting.
I'd like to use this bpf_output_trace_data() helper for tcp estats
gathering, but global collector will prevent other similar bpf programs
running in parallel.
So as a concept I think it's very useful, but we need a way to select
which ring-buffer to output data to.
proposal A:
Can we use ftrace:instances concept and make bpf_output_trace_data()
into that particular trace_pipe ?
proposal B:
bpf_perf_event_read() model is using nice concept of an array of
perf_events. Can we perf_event_open a 'new' event that can be mmaped
in user space and bpf_output_trace_data(idx, buf, buf_size) into it.
Where 'idx' will be an index of FD from perf_even_open of such
new event?

Thanks!

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 32/32] bpf: Introduce function for outputing data to perf event Wang Nan <wangnan0@huawei.com> - 2015-08-28 09:10 +0200
  Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event Alexei Starovoitov <ast@plumgrid.com> - 2015-08-29 02:50 +0200
    Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event "Wangnan (F)" <wangnan0@huawei.com> - 2015-08-29 03:30 +0200
      Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event Alexei Starovoitov <ast@plumgrid.com> - 2015-08-29 03:40 +0200
        Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event "Wangnan (F)" <wangnan0@huawei.com> - 2015-08-29 04:20 +0200
          Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event Alexei Starovoitov <ast@plumgrid.com> - 2015-08-29 04:30 +0200
            Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event "Wangnan (F)" <wangnan0@huawei.com> - 2015-08-29 04:40 +0200
              Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event Alexei Starovoitov <ast@plumgrid.com> - 2015-08-29 05:00 +0200
                Re: [PATCH 32/32] bpf: Introduce function for outputing data to perf  event "Wangnan (F)" <wangnan0@huawei.com> - 2015-08-29 05:00 +0200

csiph-web