[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YcDXrwXDw7nI6u2b@bombadil.infradead.org>
Date: Mon, 20 Dec 2021 11:21:19 -0800
From: Luis Chamberlain <mcgrof@...nel.org>
To: Vimal Agrawal <Vimal.Agrawal@...hos.com>
Cc: Masahiro Yamada <masahiroy@...nel.org>,
Michal Marek <michal.lkml@...kovi.net>,
Nick Desaulniers <ndesaulniers@...gle.com>,
Jan Beulich <JBeulich@...e.com>, Jeff Mahoney <jeffm@...e.com>,
Sam Ravnborg <sam@...nborg.org>,
"linux-kbuild@...r.kernel.org" <linux-kbuild@...r.kernel.org>,
"jeyu@...nel.org" <jeyu@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Vimal Agrawal <avimalin@...il.com>
Subject: Re: [PATCH] kernel/module.c: fix for symbol decode in stack trace
for stripped modules
On Mon, Dec 20, 2021 at 08:57:46AM +0000, Vimal Agrawal wrote:
> Hi Luis,
>
> Sorry for goof up with inline replies. I found that gmail supports bottom-posting so I will be replying inline from gmail next time. I will send the next patch using git send-email.
>
> Looks like it has been there in crash source for very long.
>
> store_module_symbols_v2
> sprintf(buf2, "%s%s", "_MODULE_START_", mod_name);
> sprintf(buf3, "%s%s", "_MODULE_INIT_START_", mod_name);
Can you point to the commit that added it? Preferably if you can have
a URL I can just use to see the change?
> I will test it first on latest ubuntu which has kernel version 5.13.0-22.
No, that's not sufficient, I really want you to use either Linus' latest
tree or linux-next.
Luis
Powered by blists - more mailing lists