From: Vimal Agrawal <avimalin@gmail.com>
To: Luis Chamberlain <mcgrof@kernel.org>
Cc: Vimal Agrawal <Vimal.Agrawal@sophos.com>,
Masahiro Yamada <masahiroy@kernel.org>,
Michal Marek <michal.lkml@markovi.net>,
Nick Desaulniers <ndesaulniers@google.com>,
Jan Beulich <JBeulich@suse.com>, Jeff Mahoney <jeffm@suse.com>,
Sam Ravnborg <sam@ravnborg.org>,
"linux-kbuild@vger.kernel.org" <linux-kbuild@vger.kernel.org>,
"jeyu@kernel.org" <jeyu@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] kernel/module.c: fix for symbol decode in stack trace for stripped modules
Date: Tue, 21 Dec 2021 14:36:48 +0530 [thread overview]
Message-ID: <CALkUMdSPZ2Qr8CYMpckRsjizyPapcOcd77_JOcj=73nervwOEg@mail.gmail.com> (raw)
In-Reply-To: <YcDXrwXDw7nI6u2b@bombadil.infradead.org>
Hi Luis,
Please see https://github.com/crash-utility/crash/commit/03e3937ec7d1b356039433137cc6e531379ca454
( function store_module_symbols_v2 in file symbols.c). This was one
of the initial commit for crash utility.
I will work on linux-next and update you.
Vimal
On Tue, Dec 21, 2021 at 12:51 AM Luis Chamberlain <mcgrof@kernel.org> wrote:
>
> 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
next prev parent reply other threads:[~2021-12-21 9:07 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <LO2P265MB2671DF8D82C0C6A1504D85D6939F9@LO2P265MB2671.GBRP265.PROD.OUTLOOK.COM>
[not found] ` <LO2P265MB267173F563B0A2CA5995FA2C939F9@LO2P265MB2671.GBRP265.PROD.OUTLOOK.COM>
2021-11-22 14:02 ` [PATCH] kernel/module.c: fix for symbol decode in stack trace for stripped modules Vimal Agrawal
2021-12-08 19:33 ` Luis Chamberlain
2021-12-09 5:37 ` Vimal Agrawal
2021-12-09 20:40 ` Luis Chamberlain
2021-12-20 8:57 ` Vimal Agrawal
2021-12-20 19:21 ` Luis Chamberlain
2021-12-21 9:06 ` Vimal Agrawal [this message]
2021-12-21 17:16 ` Vimal Agrawal
2021-12-21 22:45 ` Luis Chamberlain
2021-12-21 22:46 ` Luis Chamberlain
2021-12-22 13:23 ` [PATCH v2] kernel/module.c: heuristic enhancement when INSTALL_MOD_STRIP= "--strip-unneeded" is used Vimal Agrawal
2021-12-23 10:36 ` Christoph Hellwig
2021-12-23 11:09 ` Vimal Agrawal
2021-12-24 6:47 ` Christoph Hellwig
2021-12-25 1:08 ` Vimal Agrawal
2022-01-11 15:49 ` Luis Chamberlain
2022-01-12 8:36 ` Vimal Agrawal
2022-01-13 15:23 ` Luis Chamberlain
2022-01-17 6:54 ` [PATCH v3] kernel/module.c: heuristic enhancement in case symbols are missing e.g. " Vimal Agrawal
2022-02-02 20:20 ` Luis Chamberlain
2022-02-03 5:54 ` Vimal Agrawal
2022-02-03 6:06 ` [PATCH v4] modules: add heuristic when stripping unneeded symbols Vimal Agrawal
2022-02-04 8:39 ` [PATCH v5] " Vimal Agrawal
2022-02-04 21:47 ` Luis Chamberlain
2022-02-07 13:21 ` Vimal Agrawal
2022-02-07 22:07 ` Luis Chamberlain
2022-02-08 4:52 ` Vimal Agrawal
2022-02-08 11:02 ` [PATCH v6] " Vimal Agrawal
2022-02-08 11:13 ` Vimal Agrawal
2022-02-08 18:10 ` Luis Chamberlain
2022-02-08 18:25 ` Vimal Agrawal
2022-02-25 7:59 ` Vimal Agrawal
2022-02-25 13:40 ` Vimal Agrawal
2022-02-08 17:55 ` [PATCH v5] " Luis Chamberlain
2022-02-08 18:12 ` Vimal Agrawal
2022-01-17 7:34 ` [PATCH v2] kernel/module.c: heuristic enhancement when INSTALL_MOD_STRIP= "--strip-unneeded" is used Vimal Agrawal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CALkUMdSPZ2Qr8CYMpckRsjizyPapcOcd77_JOcj=73nervwOEg@mail.gmail.com' \
--to=avimalin@gmail.com \
--cc=JBeulich@suse.com \
--cc=Vimal.Agrawal@sophos.com \
--cc=jeffm@suse.com \
--cc=jeyu@kernel.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=mcgrof@kernel.org \
--cc=michal.lkml@markovi.net \
--cc=ndesaulniers@google.com \
--cc=sam@ravnborg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).