[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20230220171608.GA1993@templeofstupid.com>
Date: Mon, 20 Feb 2023 09:16:08 -0800
From: Krister Johansen <kjlx@...pleofstupid.com>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
Cc: Adrian Hunter <adrian.hunter@...el.com>,
Krister Johansen <kjlx@...pleofstupid.com>,
linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org,
Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
Ian Rogers <irogers@...gle.com>,
Michael Petlan <mpetlan@...hat.com>,
David Reaver <me@...idreaver.com>
Subject: Re: [PATCH v2] perf/util: Symbol lookup with kcore can fail if
multiple segments match stext
On Wed, Feb 01, 2023 at 10:00:51PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Jan 30, 2023 at 03:29:54PM +0200, Adrian Hunter escreveu:
> > On 25/01/23 20:34, Krister Johansen wrote:
> > > This problem was encountered on an arm64 system with a lot of memory.
> > > Without kernel debug symbols installed, and with both kcore and kallsyms
> > > available, perf managed to get confused and returned "unknown" for all
> > > of the kernel symbols that it tried to look up.
> > >
> > > On this system, stext fell within the vmalloc segment. The kcore symbol
> > > matching code tries to find the first segment that contains stext and
> > > uses that to replace the segment generated from just the kallsyms
> > > information. In this case, however, there were two: a very large
> > > vmalloc segment, and the text segment. This caused perf to get confused
> > > because multiple overlapping segments were inserted into the RB tree
> > > that holds the discovered segments. However, that alone wasn't
> > > sufficient to cause the problem. Even when we could find the segment,
> > > the offsets were adjusted in such a way that the newly generated symbols
> > > didn't line up with the instruction addresses in the trace. The most
> > > obvious solution would be to consult which segment type is text from
> > > kcore, but this information is not exposed to users.
> > >
> > > Instead, select the smallest matching segment that contains stext
> > > instead of the first matching segment. This allows us to match the text
> > > segment instead of vmalloc, if one is contained within the other.
> > >
> > > Signed-off-by: Krister Johansen <kjlx@...pleofstupid.com>
> >
> > Reviewed-by: Adrian Hunter <adrian.hunter@...el.com>
>
> Thanks, applied.
Thanks, Arnaldo. If it's not too late, would you be willing to tag this
one as:
Fixes: 5654997838c2 ("perf tools: Use the "_stest" symbol to identify the kernel map when loading kcore")
Cc: <stable@...r.kernel.org>
So that it makes it to the right stable branches? I meant to do this
prior to sending it along.
-K
Powered by blists - more mailing lists