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

Groups > linux.kernel > #1685558 > unrolled thread

Re: [PATCH 0/2] perf stat: fix segfault when closing events

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 0/2] perf stat: fix segfault when closing events Ganapatrao Kulkarni <gpkulkarni@gmail.com> - 2017-07-12 09:20 +0200

#1685558 — Re: [PATCH 0/2] perf stat: fix segfault when closing events

The perf tool segmentation fault which we were observing and this
patch series intended to fix it, has been fixed due to change in
default behaviour of perf stat[1][2].

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.12&id=0d79f8b93187c771b6971acfaba67f4e2f1e0710
[2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/commit/?h=linux-4.11.y&id=0d79f8b93187c771b6971acfaba67f4e2f1e0710


On Fri, Jun 9, 2017 at 10:03 AM, Ganapatrao Kulkarni
<gpkulkarni@gmail.com> wrote:
> ping?
>
> On Tue, May 23, 2017 at 8:44 PM, Ganapatrao Kulkarni
> <gpkulkarni@gmail.com> wrote:
>> On Tue, May 23, 2017 at 3:34 PM, Mark Rutland <mark.rutland@arm.com> wrote:
>>> Hi,
>>>
>>> Does anyone have any comments on these?
>>>
>>> I'm happy to rebase/resend if necessary.
>>>
>>> I'd very much like to see this fixed.
>>
>> arm64 platforms have dependency on this patch set to use latest perf tool.
>> Earliest merge/review is much appreciated!!
>>
>>>
>>> Thanks,
>>> Mark.
>>>
>>> On Fri, Apr 28, 2017 at 12:47:08PM +0100, Mark Rutland wrote:
>>>> Hi,
>>>>
>>>> These patches fix a segfault seen in some cases when perf stat is exiting.
>>>>
>>>> We don't balance opening/closing of events in all cases, and go out-of-bounds
>>>> when we close events. Full details in the patch 2 commit message.
>>>>
>>>> Thanks,
>>>> Mark.
>>>>
>>>> Mark Rutland (2):
>>>>   perf evsel: add per{cpu,thread} close helpers
>>>>   perf stat: balance opening/closing of events
>>>>
>>>>  tools/perf/builtin-stat.c | 24 +++++++++++++++++++++++-
>>>>  tools/perf/util/evsel.c   | 26 ++++++++++++++++++++------
>>>>  tools/perf/util/evsel.h   |  4 ++++
>>>>  3 files changed, 47 insertions(+), 7 deletions(-)
>>>>
>>>> --
>>>> 1.9.1
>>>>
>>
>> thanks
>> Ganapat

thanks
Ganapat

[toc] | [standalone]

Back to top | Article view | linux.kernel

csiph-web