Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754401AbaDWPmz (ORCPT ); Wed, 23 Apr 2014 11:42:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:22666 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753798AbaDWPmw (ORCPT ); Wed, 23 Apr 2014 11:42:52 -0400 Date: Wed, 23 Apr 2014 17:42:11 +0200 From: Jiri Olsa To: Stephane Eranian Cc: Don Zickus , LKML , Arnaldo Carvalho de Melo , Peter Zijlstra , "mingo@elte.hu" , Namhyung Kim , David Ahern Subject: Re: [PATCH] perf tools: fix processing of pid/tid for mmap records Message-ID: <20140423154211.GB2230@krava.brq.redhat.com> References: <20140421160655.GA4201@quad> <20140422194031.GM8488@redhat.com> <20140422204522.GN8488@redhat.com> <20140423133000.GQ8488@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Apr 23, 2014 at 05:06:21PM +0200, Stephane Eranian wrote: SNIP > > That is probably the behaviour we both expect. > > > > Jiri's patch takes all mmap events as pid maps and creates an initial map > > for them. All future threads that don't have mappings will trying to use > > their pid's mapping as a fallback option. > > > > Because it is all pointer based now, updating the pid mappings > > automatically update all the thread mappings too. > > > > I believe this approach will is the right one and makes sense. Do you > > think that works for you? > > > I just tried Jiri's 5 patches and my JIT patches, and it does work correctly. > My Java samples are correctly symbolized. So looks like we have a solution > that works for both of us though it seems complicated. But I can live with it. > Thanks Jiri. can I take this for your Tested-by? thanks, jirka -- 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/