Re: [GIT PULL] objtool fixes and updates

From: Ingo Molnar
Date: Mon Mar 31 2025 - 12:07:19 EST



* Josh Poimboeuf <jpoimboe@xxxxxxxxxx> wrote:

> On Mon, Mar 31, 2025 at 11:39:13AM +0200, Ingo Molnar wrote:
> > I'm wondering why LKP didn't pick up the primary warning, only the
> > final warning that skips duplicate warnings, which is kinda totally
> > useless in isolation:
>
> At least it alerted us to the other loongarch warnings we hadn't seen ;-)

There's that :-)

> > `-- arch-loongarch-kernel-traps.o:warning:objtool:show_stack:skipping-duplicate-warning(s)
> >
> > Maybe we should just do what is below - output a single warning, a bit
> > like lockdep.
>
> Yeah, "skipping duplicate warnings" isn't technically a warning so it
> probably shouldn't be presented as one.
>
> I'll just revert 0a7fb6f07e3a and go back to the simple "one warning
> per function" which worked fine before.

Thanks! Our experience with lockdep is that the fewer warnings per
build/bootup, the better: the value of secondary warnings is
diminishing, sometimes even negative.

It might even be OK to generate just one objtool warning per .o file.

Ingo