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