smatch output
Artem Bityutskiy
dedekind1 at gmail.com
Wed Feb 5 01:35:48 PST 2014
On Tue, 2014-02-04 at 11:28 +0300, Dan Carpenter wrote:
> I can't log into that. But I think I know what you're talking about.
> It's not a multi-line warning, it's just one really long line. I need
> to fix it though because it's ugly and confusing to everyone except
> me...
Sorry, I thought this gerrit is available for everyone in R/O mode.
And another sorry for thinking this is smatch. Actually the "offender"
is gcc 4.8. It improved the output and now prints the entire line where
the problem is. E.g.:
arch/x86/kernel/e820.c: In function ‘e820_setup_gap’:
arch/x86/kernel/e820.c:630:6: warning: variable ‘found’ set but not used [-Wunused-but-set-variable]
int found;
^
arch/x86/kernel/tsc.c:1029:15: warning: no previous prototype for ‘calibrate_delay_is_known’ [-Wmissing-prototypes]
unsigned long calibrate_delay_is_known(void)
^
arch/x86/kernel/process.c:197:6: warning: no previous prototype for ‘__switch_to_xtra’ [-Wmissing-prototypes]
void __switch_to_xtra(struct task_struct *prev_p, struct task_struct *next_p,
^
arch/x86/kernel/acpi/boot.c: In function ‘acpi_parse_x2apic’:
arch/x86/kernel/acpi/boot.c:217:5: warning: variable ‘enabled’ set but not used [-Wunused-but-set-variable]
u8 enabled;
^
I need to adapt Aiaiai's log parser to handle these.
I'll send you a note when I meed the problematic smatch output which
contained a lot of line numbers for mutexes.
Thanks!
--
Best Regards,
Artem Bityutskiy
More information about the aiaiai
mailing list