* [PATCH] perf tools: intel-pt: fix endless record after being terminated
@ 2020-01-02 7:42 Wei Li
2020-01-06 12:02 ` Jiri Olsa
2020-01-13 14:47 ` Adrian Hunter
0 siblings, 2 replies; 4+ messages in thread
From: Wei Li @ 2020-01-02 7:42 UTC (permalink / raw)
To: acme, mark.rutland, alexander.shishkin, jolsa, namhyung, adrian.hunter
Cc: peterz, mingo, linux-kernel, huawei.libin
In __cmd_record(), when receiving SIGINT(ctrl + c), a done flag will
be set and the event list will be disabled by evlist__disable() once.
While in auxtrace_record.read_finish(), the related events will be
enabled again, if they are continuous, the recording seems to be endless.
If the intel_pt event is disabled, we don't enable it again here.
Before the patch:
huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
intel_pt//u -p 46803
^C^C^C^C^C^C
After the patch:
huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
intel_pt//u -p 48591
^C[ perf record: Woken up 0 times to write data ]
Warning:
AUX data lost 504 times out of 4816!
[ perf record: Captured and wrote 2024.405 MB perf.data ]
Signed-off-by: Wei Li <liwei391@huawei.com>
---
tools/perf/arch/x86/util/intel-pt.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/tools/perf/arch/x86/util/intel-pt.c b/tools/perf/arch/x86/util/intel-pt.c
index 20df442fdf36..1e96afcd8646 100644
--- a/tools/perf/arch/x86/util/intel-pt.c
+++ b/tools/perf/arch/x86/util/intel-pt.c
@@ -1173,9 +1173,13 @@ static int intel_pt_read_finish(struct auxtrace_record *itr, int idx)
struct evsel *evsel;
evlist__for_each_entry(ptr->evlist, evsel) {
- if (evsel->core.attr.type == ptr->intel_pt_pmu->type)
- return perf_evlist__enable_event_idx(ptr->evlist, evsel,
- idx);
+ if (evsel->core.attr.type == ptr->intel_pt_pmu->type) {
+ if (evsel->disabled)
+ return 0;
+ else
+ return perf_evlist__enable_event_idx(
+ ptr->evlist, evsel, idx);
+ }
}
return -EINVAL;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] perf tools: intel-pt: fix endless record after being terminated
2020-01-02 7:42 [PATCH] perf tools: intel-pt: fix endless record after being terminated Wei Li
@ 2020-01-06 12:02 ` Jiri Olsa
2020-01-14 13:45 ` Adrian Hunter
2020-01-13 14:47 ` Adrian Hunter
1 sibling, 1 reply; 4+ messages in thread
From: Jiri Olsa @ 2020-01-06 12:02 UTC (permalink / raw)
To: Wei Li
Cc: acme, mark.rutland, alexander.shishkin, namhyung, adrian.hunter,
peterz, mingo, linux-kernel, huawei.libin
On Thu, Jan 02, 2020 at 03:42:11PM +0800, Wei Li wrote:
> In __cmd_record(), when receiving SIGINT(ctrl + c), a done flag will
> be set and the event list will be disabled by evlist__disable() once.
>
> While in auxtrace_record.read_finish(), the related events will be
> enabled again, if they are continuous, the recording seems to be endless.
>
> If the intel_pt event is disabled, we don't enable it again here.
>
> Before the patch:
> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
> intel_pt//u -p 46803
> ^C^C^C^C^C^C
>
> After the patch:
> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
> intel_pt//u -p 48591
> ^C[ perf record: Woken up 0 times to write data ]
> Warning:
> AUX data lost 504 times out of 4816!
>
> [ perf record: Captured and wrote 2024.405 MB perf.data ]
>
> Signed-off-by: Wei Li <liwei391@huawei.com>
> ---
> tools/perf/arch/x86/util/intel-pt.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/tools/perf/arch/x86/util/intel-pt.c b/tools/perf/arch/x86/util/intel-pt.c
> index 20df442fdf36..1e96afcd8646 100644
> --- a/tools/perf/arch/x86/util/intel-pt.c
> +++ b/tools/perf/arch/x86/util/intel-pt.c
> @@ -1173,9 +1173,13 @@ static int intel_pt_read_finish(struct auxtrace_record *itr, int idx)
> struct evsel *evsel;
>
> evlist__for_each_entry(ptr->evlist, evsel) {
> - if (evsel->core.attr.type == ptr->intel_pt_pmu->type)
> - return perf_evlist__enable_event_idx(ptr->evlist, evsel,
> - idx);
> + if (evsel->core.attr.type == ptr->intel_pt_pmu->type) {
> + if (evsel->disabled)
> + return 0;
> + else
> + return perf_evlist__enable_event_idx(
> + ptr->evlist, evsel, idx);
what's the logic behind enabling the event in here?
thanks,
jirka
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] perf tools: intel-pt: fix endless record after being terminated
2020-01-02 7:42 [PATCH] perf tools: intel-pt: fix endless record after being terminated Wei Li
2020-01-06 12:02 ` Jiri Olsa
@ 2020-01-13 14:47 ` Adrian Hunter
1 sibling, 0 replies; 4+ messages in thread
From: Adrian Hunter @ 2020-01-13 14:47 UTC (permalink / raw)
To: Wei Li, acme, mark.rutland, alexander.shishkin, jolsa, namhyung
Cc: peterz, mingo, linux-kernel, huawei.libin
On 2/01/20 9:42 am, Wei Li wrote:
> In __cmd_record(), when receiving SIGINT(ctrl + c), a done flag will
> be set and the event list will be disabled by evlist__disable() once.
>
> While in auxtrace_record.read_finish(), the related events will be
> enabled again, if they are continuous, the recording seems to be endless.
>
> If the intel_pt event is disabled, we don't enable it again here.
>
> Before the patch:
> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
> intel_pt//u -p 46803
> ^C^C^C^C^C^C
>
> After the patch:
> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
> intel_pt//u -p 48591
> ^C[ perf record: Woken up 0 times to write data ]
> Warning:
> AUX data lost 504 times out of 4816!
>
> [ perf record: Captured and wrote 2024.405 MB perf.data ]
>
> Signed-off-by: Wei Li <liwei391@huawei.com>
All ->read_finish() implementations seem the same, so can we make a common
helper auxtrace_read_finish() and set all .read_finish = to that.
> ---
> tools/perf/arch/x86/util/intel-pt.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/tools/perf/arch/x86/util/intel-pt.c b/tools/perf/arch/x86/util/intel-pt.c
> index 20df442fdf36..1e96afcd8646 100644
> --- a/tools/perf/arch/x86/util/intel-pt.c
> +++ b/tools/perf/arch/x86/util/intel-pt.c
> @@ -1173,9 +1173,13 @@ static int intel_pt_read_finish(struct auxtrace_record *itr, int idx)
> struct evsel *evsel;
>
> evlist__for_each_entry(ptr->evlist, evsel) {
> - if (evsel->core.attr.type == ptr->intel_pt_pmu->type)
> - return perf_evlist__enable_event_idx(ptr->evlist, evsel,
> - idx);
> + if (evsel->core.attr.type == ptr->intel_pt_pmu->type) {
> + if (evsel->disabled)
> + return 0;
> + else
> + return perf_evlist__enable_event_idx(
> + ptr->evlist, evsel, idx);
> + }
> }
> return -EINVAL;
> }
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] perf tools: intel-pt: fix endless record after being terminated
2020-01-06 12:02 ` Jiri Olsa
@ 2020-01-14 13:45 ` Adrian Hunter
0 siblings, 0 replies; 4+ messages in thread
From: Adrian Hunter @ 2020-01-14 13:45 UTC (permalink / raw)
To: Jiri Olsa, Wei Li
Cc: acme, mark.rutland, alexander.shishkin, namhyung, peterz, mingo,
linux-kernel, huawei.libin
On 6/01/20 2:02 pm, Jiri Olsa wrote:
> On Thu, Jan 02, 2020 at 03:42:11PM +0800, Wei Li wrote:
>> In __cmd_record(), when receiving SIGINT(ctrl + c), a done flag will
>> be set and the event list will be disabled by evlist__disable() once.
>>
>> While in auxtrace_record.read_finish(), the related events will be
>> enabled again, if they are continuous, the recording seems to be endless.
>>
>> If the intel_pt event is disabled, we don't enable it again here.
>>
>> Before the patch:
>> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
>> intel_pt//u -p 46803
>> ^C^C^C^C^C^C
>>
>> After the patch:
>> huawei@huawei-2288H-V5:~/linux-5.5-rc4/tools/perf$ ./perf record -e \
>> intel_pt//u -p 48591
>> ^C[ perf record: Woken up 0 times to write data ]
>> Warning:
>> AUX data lost 504 times out of 4816!
>>
>> [ perf record: Captured and wrote 2024.405 MB perf.data ]
>>
>> Signed-off-by: Wei Li <liwei391@huawei.com>
>> ---
>> tools/perf/arch/x86/util/intel-pt.c | 10 +++++++---
>> 1 file changed, 7 insertions(+), 3 deletions(-)
>>
>> diff --git a/tools/perf/arch/x86/util/intel-pt.c b/tools/perf/arch/x86/util/intel-pt.c
>> index 20df442fdf36..1e96afcd8646 100644
>> --- a/tools/perf/arch/x86/util/intel-pt.c
>> +++ b/tools/perf/arch/x86/util/intel-pt.c
>> @@ -1173,9 +1173,13 @@ static int intel_pt_read_finish(struct auxtrace_record *itr, int idx)
>> struct evsel *evsel;
>>
>> evlist__for_each_entry(ptr->evlist, evsel) {
>> - if (evsel->core.attr.type == ptr->intel_pt_pmu->type)
>> - return perf_evlist__enable_event_idx(ptr->evlist, evsel,
>> - idx);
>> + if (evsel->core.attr.type == ptr->intel_pt_pmu->type) {
>> + if (evsel->disabled)
>> + return 0;
>> + else
>> + return perf_evlist__enable_event_idx(
>> + ptr->evlist, evsel, idx);
>
> what's the logic behind enabling the event in here?
Tracing stops when the auxtrace buffer is full and won't start again until
the event is scheduled in (which is never for system-wide events) but
enabling here will start tracing again immediately if possible.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-14 13:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-02 7:42 [PATCH] perf tools: intel-pt: fix endless record after being terminated Wei Li
2020-01-06 12:02 ` Jiri Olsa
2020-01-14 13:45 ` Adrian Hunter
2020-01-13 14:47 ` Adrian Hunter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).