Re: [PATCH v3] scripts: add script for translating stack dump function

From: Linus Torvalds
Date: Mon Sep 19 2016 - 15:15:47 EST


On Mon, Sep 19, 2016 at 11:56 AM, Linus Torvalds
<torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
>>
>> The only working (and fast) approach I could come up with was an ugly
>> hack. It assumes that start_kernel() is in init/main.c.
>
> That sounds entirely reasonable. Maybe somebody can come up with a
> better and more general approach, but that sounds like a fine starting
> point, and any incremental improvents can happen in the tree. So I'll
> apply your patch (assuming it passes my basic testing, which I expect
> it will).

Hmm. Would you mind if I change the

addr2line -fpie $objfile $hexaddr | sed "s;$dir_prefix;;"

into

addr2line -fpie $objfile $hexaddr |
sed "s; at $dir_prefix\(\./\)*; at ;"

instead? There's two changes there: matching the " at " part (to just
make the match stricter) but also matching any following "./" thing
(which shows up for our include tree files, at least for me).

With that, all the cases I threw at it looked pretty good.

But the stricter matching might not matter, of course, and maybe there
is some addr2line version that doesn't do that? So I'm checking here..

Linus