* [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-17 23:24 ` kbuild test robot
0 siblings, 0 replies; 14+ messages in thread
From: kbuild test robot @ 2020-03-17 23:24 UTC (permalink / raw)
To: Kaaira Gupta; +Cc: devel, Greg Kroah-Hartman, kbuild-all, Stefano Brivio
[-- Attachment #1: Type: text/plain, Size: 3063 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
head: ba839b32d6f76a68919ed838e9375c47ca05a91a
commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
reproduce:
# apt-get install sparse
# sparse version:
git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
make ARCH=x86_64 allmodconfig
make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
drivers/ide/ide-gd.c:364:10: sparse: also defined here
--
>> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
drivers/scsi/sr.c:691:10: sparse: also defined here
--
>> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
drivers/block/paride/pcd.c:279:10: sparse: also defined here
--
>> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
>> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
--
>> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
vim +362 drivers/ide/ide-gd.c
c103d6ee69f93e Arnd Bergmann 2019-03-15 357
83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
:::::: The code at line 362 was first introduced by commit
:::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
:::::: TO: Arnd Bergmann <arnd@arndb.de>
:::::: CC: Jens Axboe <jaxboe@fusionio.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 72139 bytes --]
[-- Attachment #3: Type: text/plain, Size: 169 bytes --]
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-17 23:24 ` kbuild test robot
0 siblings, 0 replies; 14+ messages in thread
From: kbuild test robot @ 2020-03-17 23:24 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3121 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
head: ba839b32d6f76a68919ed838e9375c47ca05a91a
commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
reproduce:
# apt-get install sparse
# sparse version:
git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
make ARCH=x86_64 allmodconfig
make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
drivers/ide/ide-gd.c:364:10: sparse: also defined here
--
>> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
drivers/scsi/sr.c:691:10: sparse: also defined here
--
>> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
drivers/block/paride/pcd.c:279:10: sparse: also defined here
--
>> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
>> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
--
>> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
vim +362 drivers/ide/ide-gd.c
c103d6ee69f93e Arnd Bergmann 2019-03-15 357
83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
:::::: The code at line 362 was first introduced by commit
:::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
:::::: TO: Arnd Bergmann <arnd@arndb.de>
:::::: CC: Jens Axboe <jaxboe@fusionio.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 72139 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-17 23:24 ` kbuild test robot
@ 2020-03-18 6:08 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:08 UTC (permalink / raw)
To: kbuild test robot; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> reproduce:
> # apt-get install sparse
> # sparse version:
> git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> make ARCH=x86_64 allmodconfig
> make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> drivers/ide/ide-gd.c:364:10: sparse: also defined here
> --
> >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> drivers/scsi/sr.c:691:10: sparse: also defined here
> --
> >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> drivers/block/paride/pcd.c:279:10: sparse: also defined here
> --
> >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> --
> >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
>
> vim +362 drivers/ide/ide-gd.c
>
> c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
>
> :::::: The code at line 362 was first introduced by commit
> :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
Why is 0-day starting to spit out sparse warnings for things that have
nothing to do with the specific patch, or tree?
False-positive errors are the worst, please fix.
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-18 6:08 ` Greg Kroah-Hartman
0 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:08 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3263 bytes --]
On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> reproduce:
> # apt-get install sparse
> # sparse version:
> git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> make ARCH=x86_64 allmodconfig
> make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> drivers/ide/ide-gd.c:364:10: sparse: also defined here
> --
> >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> drivers/scsi/sr.c:691:10: sparse: also defined here
> --
> >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> drivers/block/paride/pcd.c:279:10: sparse: also defined here
> --
> >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> --
> >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
>
> vim +362 drivers/ide/ide-gd.c
>
> c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
>
> :::::: The code at line 362 was first introduced by commit
> :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
Why is 0-day starting to spit out sparse warnings for things that have
nothing to do with the specific patch, or tree?
False-positive errors are the worst, please fix.
greg k-h
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-18 6:08 ` Greg Kroah-Hartman
@ 2020-03-18 6:12 ` Philip Li
-1 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-18 6:12 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > reproduce:
> > # apt-get install sparse
> > # sparse version:
> > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > make ARCH=x86_64 allmodconfig
> > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> >
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > --
> > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > drivers/scsi/sr.c:691:10: sparse: also defined here
> > --
> > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > --
> > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > --
> > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> >
> > vim +362 drivers/ide/ide-gd.c
> >
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> >
> > :::::: The code at line 362 was first introduced by commit
> > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
>
> Why is 0-day starting to spit out sparse warnings for things that have
> nothing to do with the specific patch, or tree?
thanks for the input, Greg, we will check this to avoid
false positive.
>
> False-positive errors are the worst, please fix.
>
> greg k-h
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@lists.01.org
> To unsubscribe send an email to kbuild-all-leave@lists.01.org
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-18 6:12 ` Philip Li
0 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-18 6:12 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3696 bytes --]
On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > reproduce:
> > # apt-get install sparse
> > # sparse version:
> > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > make ARCH=x86_64 allmodconfig
> > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> >
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > --
> > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > drivers/scsi/sr.c:691:10: sparse: also defined here
> > --
> > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > --
> > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > --
> > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> >
> > vim +362 drivers/ide/ide-gd.c
> >
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> >
> > :::::: The code at line 362 was first introduced by commit
> > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
>
> Why is 0-day starting to spit out sparse warnings for things that have
> nothing to do with the specific patch, or tree?
thanks for the input, Greg, we will check this to avoid
false positive.
>
> False-positive errors are the worst, please fix.
>
> greg k-h
> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-18 6:12 ` Philip Li
@ 2020-03-18 6:17 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:17 UTC (permalink / raw)
To: Philip Li; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > reproduce:
> > > # apt-get install sparse
> > > # sparse version:
> > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > make ARCH=x86_64 allmodconfig
> > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > >
> > > If you fix the issue, kindly add following tag
> > > Reported-by: kbuild test robot <lkp@intel.com>
> > >
> > > All warnings (new ones prefixed by >>):
> > >
> > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > --
> > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > --
> > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > --
> > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > --
> > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > >
> > > vim +362 drivers/ide/ide-gd.c
> > >
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > >
> > > :::::: The code at line 362 was first introduced by commit
> > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> >
> > Why is 0-day starting to spit out sparse warnings for things that have
> > nothing to do with the specific patch, or tree?
> thanks for the input, Greg, we will check this to avoid
> false positive.
This was not the only false-positive report I got this morning, so
something went wrong on your end that started spitting out new ones :(
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-18 6:17 ` Greg Kroah-Hartman
0 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:17 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3762 bytes --]
On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > reproduce:
> > > # apt-get install sparse
> > > # sparse version:
> > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > make ARCH=x86_64 allmodconfig
> > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > >
> > > If you fix the issue, kindly add following tag
> > > Reported-by: kbuild test robot <lkp@intel.com>
> > >
> > > All warnings (new ones prefixed by >>):
> > >
> > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > --
> > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > --
> > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > --
> > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > --
> > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > >
> > > vim +362 drivers/ide/ide-gd.c
> > >
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > >
> > > :::::: The code at line 362 was first introduced by commit
> > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> >
> > Why is 0-day starting to spit out sparse warnings for things that have
> > nothing to do with the specific patch, or tree?
> thanks for the input, Greg, we will check this to avoid
> false positive.
This was not the only false-positive report I got this morning, so
something went wrong on your end that started spitting out new ones :(
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-18 6:17 ` Greg Kroah-Hartman
@ 2020-03-18 6:42 ` Philip Li
-1 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-18 6:42 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > reproduce:
> > > > # apt-get install sparse
> > > > # sparse version:
> > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > make ARCH=x86_64 allmodconfig
> > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > >
> > > > If you fix the issue, kindly add following tag
> > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > >
> > > > All warnings (new ones prefixed by >>):
> > > >
> > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > --
> > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > --
> > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > --
> > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > --
> > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > >
> > > > vim +362 drivers/ide/ide-gd.c
> > > >
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > >
> > > > :::::: The code at line 362 was first introduced by commit
> > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > >
> > > Why is 0-day starting to spit out sparse warnings for things that have
> > > nothing to do with the specific patch, or tree?
> > thanks for the input, Greg, we will check this to avoid
> > false positive.
>
> This was not the only false-positive report I got this morning, so
> something went wrong on your end that started spitting out new ones :(
sorry for the noise, we will go through the reports sent to you in recent
days to check. If it is handy for you to fwd me the "wrong" mail, it
can also help me address the problem.
One issue relates to "ERROR: modpost: "sysrq_mask" [drivers/tty/serial/serial_core.ko] undefined"
is caused by modpost logging interface change, we had disabled this
build false positive yestesday, and we will look for a solution for this soon.
Thanks
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-18 6:42 ` Philip Li
0 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-18 6:42 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 4395 bytes --]
On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > reproduce:
> > > > # apt-get install sparse
> > > > # sparse version:
> > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > make ARCH=x86_64 allmodconfig
> > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > >
> > > > If you fix the issue, kindly add following tag
> > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > >
> > > > All warnings (new ones prefixed by >>):
> > > >
> > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > --
> > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > --
> > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > --
> > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > --
> > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > >
> > > > vim +362 drivers/ide/ide-gd.c
> > > >
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > >
> > > > :::::: The code at line 362 was first introduced by commit
> > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > >
> > > Why is 0-day starting to spit out sparse warnings for things that have
> > > nothing to do with the specific patch, or tree?
> > thanks for the input, Greg, we will check this to avoid
> > false positive.
>
> This was not the only false-positive report I got this morning, so
> something went wrong on your end that started spitting out new ones :(
sorry for the noise, we will go through the reports sent to you in recent
days to check. If it is handy for you to fwd me the "wrong" mail, it
can also help me address the problem.
One issue relates to "ERROR: modpost: "sysrq_mask" [drivers/tty/serial/serial_core.ko] undefined"
is caused by modpost logging interface change, we had disabled this
build false positive yestesday, and we will look for a solution for this soon.
Thanks
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-18 6:42 ` Philip Li
@ 2020-03-18 6:53 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:53 UTC (permalink / raw)
To: Philip Li; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 02:42:31PM +0800, Philip Li wrote:
> On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> > On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > > reproduce:
> > > > > # apt-get install sparse
> > > > > # sparse version:
> > > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > make ARCH=x86_64 allmodconfig
> > > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > > >
> > > > > If you fix the issue, kindly add following tag
> > > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > > >
> > > > > All warnings (new ones prefixed by >>):
> > > > >
> > > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > > --
> > > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > > --
> > > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > > --
> > > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > > --
> > > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > > >
> > > > > vim +362 drivers/ide/ide-gd.c
> > > > >
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > > >
> > > > > :::::: The code at line 362 was first introduced by commit
> > > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > > >
> > > > Why is 0-day starting to spit out sparse warnings for things that have
> > > > nothing to do with the specific patch, or tree?
> > > thanks for the input, Greg, we will check this to avoid
> > > false positive.
> >
> > This was not the only false-positive report I got this morning, so
> > something went wrong on your end that started spitting out new ones :(
> sorry for the noise, we will go through the reports sent to you in recent
> days to check. If it is handy for you to fwd me the "wrong" mail, it
> can also help me address the problem.
Subject: [staging:staging-testing 222/222] drivers/mtd/nand/raw/nand_legacy.c:42:17: sparse: warning: cast from restricted __le16
Message-ID: <202003180728.te9ZFdLb%lkp@intel.com>
is also incorrect
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-18 6:53 ` Greg Kroah-Hartman
0 siblings, 0 replies; 14+ messages in thread
From: Greg Kroah-Hartman @ 2020-03-18 6:53 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 4551 bytes --]
On Wed, Mar 18, 2020 at 02:42:31PM +0800, Philip Li wrote:
> On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> > On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > > reproduce:
> > > > > # apt-get install sparse
> > > > > # sparse version:
> > > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > make ARCH=x86_64 allmodconfig
> > > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > > >
> > > > > If you fix the issue, kindly add following tag
> > > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > > >
> > > > > All warnings (new ones prefixed by >>):
> > > > >
> > > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > > --
> > > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > > --
> > > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > > --
> > > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > > --
> > > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > > >
> > > > > vim +362 drivers/ide/ide-gd.c
> > > > >
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > > >
> > > > > :::::: The code at line 362 was first introduced by commit
> > > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > > >
> > > > Why is 0-day starting to spit out sparse warnings for things that have
> > > > nothing to do with the specific patch, or tree?
> > > thanks for the input, Greg, we will check this to avoid
> > > false positive.
> >
> > This was not the only false-positive report I got this morning, so
> > something went wrong on your end that started spitting out new ones :(
> sorry for the noise, we will go through the reports sent to you in recent
> days to check. If it is handy for you to fwd me the "wrong" mail, it
> can also help me address the problem.
Subject: [staging:staging-testing 222/222] drivers/mtd/nand/raw/nand_legacy.c:42:17: sparse: warning: cast from restricted __le16
Message-ID: <202003180728.te9ZFdLb%lkp@intel.com>
is also incorrect
thanks,
greg k-h
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
2020-03-18 6:53 ` Greg Kroah-Hartman
@ 2020-03-19 15:49 ` Philip Li
-1 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-19 15:49 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, Stefano Brivio, Kaaira Gupta, kbuild-all
On Wed, Mar 18, 2020 at 07:53:07AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 02:42:31PM +0800, Philip Li wrote:
> > On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> > > On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > > > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > > > reproduce:
> > > > > > # apt-get install sparse
> > > > > > # sparse version:
> > > > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > > make ARCH=x86_64 allmodconfig
> > > > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > > > >
> > > > > > If you fix the issue, kindly add following tag
> > > > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > > > >
> > > > > > All warnings (new ones prefixed by >>):
> > > > > >
> > > > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > > > --
> > > > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > > > --
> > > > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > > > --
> > > > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > > > --
> > > > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > > > >
> > > > > > vim +362 drivers/ide/ide-gd.c
> > > > > >
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > > > >
> > > > > > :::::: The code at line 362 was first introduced by commit
> > > > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > > > >
> > > > > Why is 0-day starting to spit out sparse warnings for things that have
> > > > > nothing to do with the specific patch, or tree?
> > > > thanks for the input, Greg, we will check this to avoid
> > > > false positive.
> > >
> > > This was not the only false-positive report I got this morning, so
> > > something went wrong on your end that started spitting out new ones :(
> > sorry for the noise, we will go through the reports sent to you in recent
> > days to check. If it is handy for you to fwd me the "wrong" mail, it
> > can also help me address the problem.
>
> Subject: [staging:staging-testing 222/222] drivers/mtd/nand/raw/nand_legacy.c:42:17: sparse: warning: cast from restricted __le16
> Message-ID: <202003180728.te9ZFdLb%lkp@intel.com>
>
> is also incorrect
Thanks a lot Greg, we have analyzed these false positives and provide
a solution to this issue. Because we do partial make sometimes, and it appears
we need do a make clean for the partial make targets to have accurate sparse
result now. So this impacts recent sparse related reports to have some false
positive. We will keep monitor the behavior for the new reports to catch
issue ealier.
>
> thanks,
>
> greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
@ 2020-03-19 15:49 ` Philip Li
0 siblings, 0 replies; 14+ messages in thread
From: Philip Li @ 2020-03-19 15:49 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 5166 bytes --]
On Wed, Mar 18, 2020 at 07:53:07AM +0100, Greg Kroah-Hartman wrote:
> On Wed, Mar 18, 2020 at 02:42:31PM +0800, Philip Li wrote:
> > On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote:
> > > On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote:
> > > > On Wed, Mar 18, 2020 at 07:08:31AM +0100, Greg Kroah-Hartman wrote:
> > > > > On Wed, Mar 18, 2020 at 07:24:29AM +0800, kbuild test robot wrote:
> > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > > > > head: ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > > commit: ba839b32d6f76a68919ed838e9375c47ca05a91a [222/222] staging: media: hantro: remove parentheses
> > > > > > reproduce:
> > > > > > # apt-get install sparse
> > > > > > # sparse version:
> > > > > > git checkout ba839b32d6f76a68919ed838e9375c47ca05a91a
> > > > > > make ARCH=x86_64 allmodconfig
> > > > > > make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> > > > > >
> > > > > > If you fix the issue, kindly add following tag
> > > > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > > > >
> > > > > > All warnings (new ones prefixed by >>):
> > > > > >
> > > > > > >> drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/ide/ide-gd.c:364:10: sparse: also defined here
> > > > > > --
> > > > > > >> drivers/scsi/sr.c:689:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/scsi/sr.c:691:10: sparse: also defined here
> > > > > > --
> > > > > > >> drivers/block/paride/pcd.c:277:10: sparse: warning: Initializer entry defined twice
> > > > > > drivers/block/paride/pcd.c:279:10: sparse: also defined here
> > > > > > --
> > > > > > >> sound/soc/codecs/rt1015.c:844:24: sparse: warning: symbol 'rt1015_aif_dai_ops' was not declared. Should it be static?
> > > > > > >> sound/soc/codecs/rt1015.c:849:27: sparse: warning: symbol 'rt1015_dai' was not declared. Should it be static?
> > > > > > --
> > > > > > >> net/netfilter/nft_set_pipapo.c:739:6: sparse: warning: symbol 'nft_pipapo_get' was not declared. Should it be static?
> > > > > >
> > > > > > vim +362 drivers/ide/ide-gd.c
> > > > > >
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 357
> > > > > > 83d5cde47dedf0 Alexey Dobriyan 2009-09-21 358 static const struct block_device_operations ide_gd_ops = {
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 359 .owner = THIS_MODULE,
> > > > > > 6e9624b8caec29 Arnd Bergmann 2010-08-07 360 .open = ide_gd_unlocked_open,
> > > > > > b2f21e057dfbaa Al Viro 2008-10-16 361 .release = ide_gd_release,
> > > > > > 8a6cfeb6deca3a Arnd Bergmann 2010-07-08 @362 .ioctl = ide_gd_ioctl,
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 363 #ifdef CONFIG_COMPAT
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 364 .ioctl = ide_gd_compat_ioctl,
> > > > > > c103d6ee69f93e Arnd Bergmann 2019-03-15 365 #endif
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 366 .getgeo = ide_gd_getgeo,
> > > > > > 5b03a1b140e13a Tejun Heo 2011-03-09 367 .check_events = ide_gd_check_events,
> > > > > > c3e33e043f5e9c Tejun Heo 2010-05-15 368 .unlock_native_capacity = ide_gd_unlock_native_capacity,
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 369 .revalidate_disk = ide_gd_revalidate_disk
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 370 };
> > > > > > 5fef0e5c028394 Bartlomiej Zolnierkiewicz 2008-10-17 371
> > > > > >
> > > > > > :::::: The code at line 362 was first introduced by commit
> > > > > > :::::: 8a6cfeb6deca3a8fefd639d898b0d163c0b5d368 block: push down BKL into .locked_ioctl
> > > > >
> > > > > Why is 0-day starting to spit out sparse warnings for things that have
> > > > > nothing to do with the specific patch, or tree?
> > > > thanks for the input, Greg, we will check this to avoid
> > > > false positive.
> > >
> > > This was not the only false-positive report I got this morning, so
> > > something went wrong on your end that started spitting out new ones :(
> > sorry for the noise, we will go through the reports sent to you in recent
> > days to check. If it is handy for you to fwd me the "wrong" mail, it
> > can also help me address the problem.
>
> Subject: [staging:staging-testing 222/222] drivers/mtd/nand/raw/nand_legacy.c:42:17: sparse: warning: cast from restricted __le16
> Message-ID: <202003180728.te9ZFdLb%lkp@intel.com>
>
> is also incorrect
Thanks a lot Greg, we have analyzed these false positives and provide
a solution to this issue. Because we do partial make sometimes, and it appears
we need do a make clean for the partial make targets to have accurate sparse
result now. So this impacts recent sparse related reports to have some false
positive. We will keep monitor the behavior for the new reports to catch
issue ealier.
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2020-03-19 15:51 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-17 23:24 [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice kbuild test robot
2020-03-17 23:24 ` kbuild test robot
2020-03-18 6:08 ` Greg Kroah-Hartman
2020-03-18 6:08 ` Greg Kroah-Hartman
2020-03-18 6:12 ` [kbuild-all] " Philip Li
2020-03-18 6:12 ` Philip Li
2020-03-18 6:17 ` [kbuild-all] " Greg Kroah-Hartman
2020-03-18 6:17 ` Greg Kroah-Hartman
2020-03-18 6:42 ` [kbuild-all] " Philip Li
2020-03-18 6:42 ` Philip Li
2020-03-18 6:53 ` [kbuild-all] " Greg Kroah-Hartman
2020-03-18 6:53 ` Greg Kroah-Hartman
2020-03-19 15:49 ` [kbuild-all] " Philip Li
2020-03-19 15:49 ` Philip Li
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.