Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling

From	xiakaixu <xiakaixu@huawei.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling
Date	2015-10-21 12:40 +0200
Message-ID	<qlYYi-hR-7@gated-at.bofh.it> (permalink)
References	<qlzwT-5dK-41@gated-at.bofh.it> <qlzwT-5dK-39@gated-at.bofh.it> <qlO2T-10k-21@gated-at.bofh.it> <qlXIS-70g-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

于 2015/10/21 17:12, Peter Zijlstra 写道:
> On Tue, Oct 20, 2015 at 03:53:02PM -0700, Alexei Starovoitov wrote:
>> On 10/20/15 12:22 AM, Kaixu Xia wrote:
>>> diff --git a/kernel/events/core.c b/kernel/events/core.c
>>> index b11756f..5219635 100644
>>> --- a/kernel/events/core.c
>>> +++ b/kernel/events/core.c
>>> @@ -6337,6 +6337,9 @@ static int __perf_event_overflow(struct perf_event *event,
>>>  		irq_work_queue(&event->pending);
>>>  	}
>>>
>>> +	if (unlikely(!atomic_read(&event->soft_enable)))
>>> +		return 0;
>>> +
>>>  	if (event->overflow_handler)
>>>  		event->overflow_handler(event, data, regs);
>>>  	else
>>
>> Peter,
>> does this part look right or it should be moved right after
>> if (unlikely(!is_sampling_event(event)))
>>                 return 0;
>> or even to other function?
>>
>> It feels to me that it should be moved, since we probably don't
>> want to active throttling, period adjust and event_limit for events
>> that are in soft_disabled state.
> 
> Depends on what its meant to do. As long as you let the interrupt
> happen, I think we should in fact do those things (maybe not the
> event_limit), but period adjustment and esp. throttling are important
> when the event is enabled.
> 
> If you want to actually disable the event: pmu->stop() will make it
> stop, and you can restart using pmu->start().
> 
> And I suppose you can wrap that with a counter if you need nesting.
> 
> I'm not sure if any of that is a viable solution, because the patch
> description is somewhat short on the problem statement.
> 
> As is, I'm not too charmed with the patch, but lacking a better
> understanding of what exactly we're trying to achieve I'm struggling
> with proposing alternatives.

Thanks for your comments!
The RFC patch set contains the necessary commit log [1].

In some scenarios we don't want to output trace data when perf sampling
in order to reduce overhead. For example, perf can be run as daemon to
dump trace data when necessary, such as the system performance goes down.
Just like the example given in the cover letter, we only receive the
samples within sys_write() syscall.

The helper bpf_perf_event_control() in this patch set can control the
data output process and get the samples we are most interested in.
The cpu_function_call is probably too much to do from bpf program, so
I choose current design that like 'soft_disable'.

[1] https://lkml.org/lkml/2015/10/12/135
> 
> .
> 


--
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/

Thread

[PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling Kaixu Xia <xiakaixu@huawei.com> - 2015-10-20 09:30 +0200
  Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Alexei Starovoitov <ast@plumgrid.com> - 2015-10-21 01:00 +0200
    Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 11:20 +0200
      Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling xiakaixu <xiakaixu@huawei.com> - 2015-10-21 12:40 +0200
        Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 13:40 +0200
          Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-21 14:00 +0200
            Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 14:20 +0200
              Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-21 15:50 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 16:00 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling pi3orama <pi3orama@163.com> - 2015-10-21 16:10 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 16:10 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY maps trace data output when perf sampling pi3orama <pi3orama@163.com> - 2015-10-21 17:20 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 19:00 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Alexei Starovoitov <ast@plumgrid.com> - 2015-10-21 23:30 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-22 11:10 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-22 12:40 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-23 15:00 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-23 17:20 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling xiakaixu <xiakaixu@huawei.com> - 2015-10-27 07:50 +0100
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-22 05:00 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Ingo Molnar <mingo@kernel.org> - 2015-10-22 09:50 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-22 10:00 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-22 11:30 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-22 04:10 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Alexei Starovoitov <ast@plumgrid.com> - 2015-10-22 05:10 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-22 05:20 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Alexei Starovoitov <ast@plumgrid.com> - 2015-10-22 05:30 +0200
                Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-22 11:50 +0200
      Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-21 13:40 +0200
        Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling Peter Zijlstra <peterz@infradead.org> - 2015-10-21 14:00 +0200
          Re: [PATCH V5 1/1] bpf: control events stored in PERF_EVENT_ARRAY  maps trace data output when perf sampling "Wangnan (F)" <wangnan0@huawei.com> - 2015-10-21 14:10 +0200

csiph-web