Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965556AbbKDOIS (ORCPT ); Wed, 4 Nov 2015 09:08:18 -0500 Received: from mx1.redhat.com ([209.132.183.28]:50810 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965184AbbKDOIP (ORCPT ); Wed, 4 Nov 2015 09:08:15 -0500 Date: Wed, 4 Nov 2015 15:08:11 +0100 From: Jiri Olsa To: Markus Trippelsdorf Cc: Ingo Molnar , linux-kernel@vger.kernel.org, Peter Zijlstra , Arnaldo Carvalho de Melo , Thomas Gleixner , Andrew Morton Subject: [PATCH] perf tools: Insert split maps correctly into origin group Message-ID: <20151104140811.GA32664@krava.brq.redhat.com> References: <20151104101533.GA332@x4> <20151104104929.GB332@x4> <20151104122740.GH29671@krava.brq.redhat.com> <20151104124137.GC332@x4> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151104124137.GC332@x4> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3411 Lines: 101 On Wed, Nov 04, 2015 at 01:41:37PM +0100, Markus Trippelsdorf wrote: SNIP > > > Apparently it started with: > > > > > > commit cfc5acd4c80b875d2f739d6a93562034aee5563f > > > Author: Arnaldo Carvalho de Melo > > > Date: Wed Sep 23 15:15:54 2015 -0300 > > > > > > perf top: Filter symbols based on __map__is_kernel(map) > > > > I think it's correct to assume map->groups != NULL, > > IMO the change just uncovered another bug > > > > I couldn't reproduce this, but I could find one place where this > > assumption is broken.. could you please try attached patch? > > Your patch fixes the issue. > Thanks. thanks, sending patch jirka --- When new maps are cloned out of split map they are added into origin map's group, but their groups pointer is not updated. This could lead to segfault, because map::groups is expected to be always set as reported by Markus: __map__is_kernel (map=map@entry=0x1abb7a0) at util/map.c:238 238 return __machine__kernel_map(map->groups->machine, map->type) = (gdb) bt #0 __map__is_kernel (map=map@entry=0x1abb7a0) at util/map.c:238 #1 0x00000000004393e4 in symbol_filter (map=map@entry=0x1abb7a0, sym=sym@entry #2 0x00000000004fcd4d in dso__load_sym (dso=dso@entry=0x166dae0, map=map@entry #3 0x00000000004a64e0 in dso__load (dso=0x166dae0, map=map@entry=0x1abb7a0, fi #4 0x00000000004b941f in map__load (filter=0x4393c0 , map= Tested-by: Markus Trippelsdorf Link: http://lkml.kernel.org/n/tip-b66q3nbv0m456f2evlhtuw3u@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/map.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tools/perf/util/map.c b/tools/perf/util/map.c index 4e38c396a897..afc6b56cf749 100644 --- a/tools/perf/util/map.c +++ b/tools/perf/util/map.c @@ -644,6 +644,12 @@ size_t map_groups__fprintf(struct map_groups *mg, FILE *fp) return printed; } +static void __map_groups__insert(struct map_groups *mg, struct map *map) +{ + __maps__insert(&mg->maps[map->type], map); + map->groups = mg; +} + static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp) { struct rb_root *root; @@ -682,7 +688,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp } before->end = map->start; - __maps__insert(maps, before); + __map_groups__insert(pos->groups, before); if (verbose >= 2) map__fprintf(before, fp); } @@ -696,7 +702,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp } after->start = map->end; - __maps__insert(maps, after); + __map_groups__insert(pos->groups, after); if (verbose >= 2) map__fprintf(after, fp); } -- 2.4.3 -- 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/