Re: [PATCH] perf tools: Don't set leader if parser doesn't collect an evsel

2015-09-01 Thread Wangnan (F)
On 2015/9/2 13:57, 平松雅巳 / HIRAMATU，MASAMI wrote: From: Wang Nan [mailto:wangn...@huawei.com] Similar to patch 'perf tools: Don't set cmdline_group_boundary if no evsel is collected', in case when parser collects no evsel (at this point it shouldn't happen), parse_events__set_leader() is not sa

RE: [PATCH] perf tools: Don't set leader if parser doesn't collect an evsel

2015-09-01 Thread 平松雅巳 / HIRAMATU，MASAMI
> From: Wang Nan [mailto:wangn...@huawei.com] > > Similar to patch 'perf tools: Don't set cmdline_group_boundary if no > evsel is collected', in case when parser collects no evsel (at this > point it shouldn't happen), parse_events__set_leader() is not safe. > > This patch checks list_empty becor

Re: [PATCH] perf tools: Don't set leader if parser doesn't collect an evsel

2015-09-01 Thread Wangnan (F)
On 2015/9/2 10:53, Wang Nan wrote: Similar to patch 'perf tools: Don't set cmdline_group_boundary if no evsel is collected', in case when parser collects no evsel (at this point it shouldn't happen), parse_events__set_leader() is not safe. This patch checks list_empty becore calling __perf_evl

[PATCH] perf tools: Don't set leader if parser doesn't collect an evsel

2015-09-01 Thread Wang Nan
Similar to patch 'perf tools: Don't set cmdline_group_boundary if no evsel is collected', in case when parser collects no evsel (at this point it shouldn't happen), parse_events__set_leader() is not safe. This patch checks list_empty becore calling __perf_evlist__set_leader() for safty reason. Si