Bug 207243
Summary: | `perf stat --timeout` does not return after timeout expires | ||
---|---|---|---|
Product: | Tracing/Profiling | Reporter: | Konstantin Kharlamov (hi-angel) |
Component: | Perf tool | Assignee: | Arnaldo Carvalho de Melo (acme) |
Status: | ASSIGNED --- | ||
Severity: | normal | CC: | jolsa |
Priority: | P1 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | 5.6.2 | Subsystem: | |
Regression: | No | Bisected commit-id: | |
Attachments: | Fix for 'perf stat --timeout N workload-to-start' |
Description
Konstantin Kharlamov
2020-04-14 22:22:34 UTC
So I reproduced the problem, in my testing only system wide mode is working: [root@five ~]# perf stat -e cycles --timeout 1000 -a Performance counter stats for 'system wide': 7,762,164,845 cycles 1.000529516 seconds time elapsed [root@five ~]# perf stat -e cycles --timeout 150 -a Performance counter stats for 'system wide': 1,291,118,172 cycles 0.150428442 seconds time elapsed [root@five ~]# I'll try to investigate and fix it, thanks for the report. Created attachment 288499 [details]
Fix for 'perf stat --timeout N workload-to-start'
Wow that was quick! Thank you! :-) Can I take that as a: Tested-by: hi-angel@yandex.ru And add that to the patch, in addition to: Reported-by: hi-angel@yandex.ru And also your name to add there as well if possible? - Arnaldo (In reply to Arnaldo Carvalho de Melo from comment #4) > :-) > > Can I take that as a: > > Tested-by: hi-angel@yandex.ru > > And add that to the patch, in addition to: > > Reported-by: hi-angel@yandex.ru > > And also your name to add there as well if possible? > > - Arnaldo Sounds great, though I gotta need some time. I will test it in a few hours! (In reply to Arnaldo Carvalho de Melo from comment #4) > :-) > > Can I take that as a: > > Tested-by: hi-angel@yandex.ru > > And add that to the patch, in addition to: > > Reported-by: hi-angel@yandex.ru > > And also your name to add there as well if possible? > > - Arnaldo Thank you, I tested it, works for me. I replied on the ML |