Em Wed, 13 Dec 2017 10:03:56 +0100
Geert Uytterhoeven <ge...@linux-m68k.org> escreveu:
> Hi Mauro,
>
> On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab
> <mche...@osg.samsung.com> wrote:
> > Without this series, I was getting 809 lines of bogus warnings (see below),
> > with was preventing me to see new warnings on my incremental builds
> > while applying new patches at the media tree.
>
> $ linux-log-diff build.log{.old,}
>
> (from https://github.com/geertu/linux-scripts)
That's nice!
Yet, it is producing some noise. I did a clean build with:
$ make ARCH=i386 CF=-D__CHECK_ENDIAN__ CONFIG_DEBUG_SECTION_MISMATCH=y W=1
CHECK='' M=drivers/staging/media | grep -v -e " CC " -e " LD " -e " AR " -e "
CHK " -e " CALL " -e " UPD " -e "scripts/kconfig/conf " -e " CHECK " >old.log
$ make ARCH=i386 CF=-D__CHECK_ENDIAN__ CONFIG_DEBUG_SECTION_MISMATCH=y W=1
CHECK='' M=drivers/media| grep -v -e " CC " -e " LD " -e " AR " -e " CHK " -e "
CALL " -e " UPD " -e "scripts/kconfig/conf " -e " CHECK " >>old.log
and added a new uninitialized "foo" var to a random driver, doing an
incremental build with:
$ make ARCH=i386 CF=-D__CHECK_ENDIAN__ CONFIG_DEBUG_SECTION_MISMATCH=y W=1
CHECK='' | grep -v -e " CC " -e " LD " -e " AR " -e " CHK " -e " CALL " -e "
UPD " -e "scripts/kconfig/conf " -e " CHECK " M=drivers/staging/media >new.log
$ make ARCH=i386 CF=-D__CHECK_ENDIAN__ CONFIG_DEBUG_SECTION_MISMATCH=y W=1
CHECK='' | grep -v -e " CC " -e " LD " -e " AR " -e " CHK " -e " CALL " -e "
UPD " -e "scripts/kconfig/conf " -e " CHECK " M=drivers/media >new.log
Then, I ran the script:
$ linux-log-diff old.log new.log
*** ERRORS ***
*** WARNINGS ***
1 warning regressions:
+ drivers/media/dvb-frontends/dibx000_common.c: warning: unused variable
'foo' [-Wunused-variable]: => 22:5
3 warning improvements:
- ./arch/x86/include/asm/bitops.h: warning: asm output is not an lvalue:
430:22 =>
-
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_common/host/mmu_private.h:
warning: function 'mmu_reg_load' with external linkage has definition: 35:30
=>
-
drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_common/host/mmu_private.h:
warning: function 'mmu_reg_store' with external linkage has definition: 24:26
=>
It detected the "foo" var warning, but it outputs 3 warning improvements
on files that were not even built the second time.
--
Thanks,
Mauro
