Hi Alain, Thank you for the patch! Yet something to improve: [auto build test ERROR on bluetooth-next/master] [also build test ERROR on next-20200611] [cannot apply to v5.7] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Alain-Michaud/sco-Add-support-for-BT_PKT_STATUS-CMSG-data/20200611-212907 base: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master config: x86_64-randconfig-s022-20200611 (attached as .config) compiler: gcc-9 (Debian 9.3.0-13) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.1-250-g42323db3-dirty # save the attached .config to linux build tree make W=1 C=1 ARCH=x86_64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> All errors (new ones prefixed by >>, old ones prefixed by <<): net/bluetooth/sco.c: In function 'sco_skb_put_cmsg': >> net/bluetooth/sco.c:456:36: error: passing argument 2 of 'test_bit' from incompatible pointer type [-Werror=incompatible-pointer-types] 456 | if (test_bit(SCO_CMSG_PKT_STATUS, &sco_pi(sk)->cmsg_mask)) | ^~~~~~~~~~~~~~~~~~~~~~ | | | __u8 * {aka unsigned char *} In file included from arch/x86/include/asm/bitops.h:392, from include/linux/bitops.h:29, from include/linux/kernel.h:12, from include/linux/list.h:9, from include/linux/module.h:12, from net/bluetooth/sco.c:27: include/asm-generic/bitops/instrumented-non-atomic.h:108:68: note: expected 'const volatile long unsigned int *' but argument is of type '__u8 *' {aka 'unsigned char *'} 108 | static inline bool test_bit(long nr, const volatile unsigned long *addr) | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~ net/bluetooth/sco.c: In function 'sco_sock_setsockopt': >> net/bluetooth/sco.c:868:33: error: passing argument 2 of 'set_bit' from incompatible pointer type [-Werror=incompatible-pointer-types] 868 | set_bit(SCO_CMSG_PKT_STATUS, &sco_pi(sk)->cmsg_mask); | ^~~~~~~~~~~~~~~~~~~~~~ | | | __u8 * {aka unsigned char *} In file included from arch/x86/include/asm/bitops.h:391, from include/linux/bitops.h:29, from include/linux/kernel.h:12, from include/linux/list.h:9, from include/linux/module.h:12, from net/bluetooth/sco.c:27: include/asm-generic/bitops/instrumented-atomic.h:26:61: note: expected 'volatile long unsigned int *' but argument is of type '__u8 *' {aka 'unsigned char *'} 26 | static inline void set_bit(long nr, volatile unsigned long *addr) | ~~~~~~~~~~~~~~~~~~~~~~~~^~~~ >> net/bluetooth/sco.c:870:35: error: passing argument 2 of 'clear_bit' from incompatible pointer type [-Werror=incompatible-pointer-types] 870 | clear_bit(SCO_CMSG_PKT_STATUS, &sco_pi(sk)->cmsg_mask); | ^~~~~~~~~~~~~~~~~~~~~~ | | | __u8 * {aka unsigned char *} In file included from arch/x86/include/asm/bitops.h:391, from include/linux/bitops.h:29, from include/linux/kernel.h:12, from include/linux/list.h:9, from include/linux/module.h:12, from net/bluetooth/sco.c:27: include/asm-generic/bitops/instrumented-atomic.h:39:63: note: expected 'volatile long unsigned int *' but argument is of type '__u8 *' {aka 'unsigned char *'} 39 | static inline void clear_bit(long nr, volatile unsigned long *addr) | ~~~~~~~~~~~~~~~~~~~~~~~~^~~~ net/bluetooth/sco.c: In function 'sco_sock_getsockopt': net/bluetooth/sco.c:999:11: error: passing argument 2 of 'test_bit' from incompatible pointer type [-Werror=incompatible-pointer-types] 999 | &(sco_pi(sk)->cmsg_mask)); | ^~~~~~~~~~~~~~~~~~~~~~~~ | | | __u8 * {aka unsigned char *} In file included from arch/x86/include/asm/bitops.h:392, from include/linux/bitops.h:29, from include/linux/kernel.h:12, from include/linux/list.h:9, from include/linux/module.h:12, from net/bluetooth/sco.c:27: include/asm-generic/bitops/instrumented-non-atomic.h:108:68: note: expected 'const volatile long unsigned int *' but argument is of type '__u8 *' {aka 'unsigned char *'} 108 | static inline bool test_bit(long nr, const volatile unsigned long *addr) | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~ cc1: some warnings being treated as errors sparse warnings: (new ones prefixed by >>) >> net/bluetooth/sco.c:456:44: sparse: sparse: incorrect type in argument 2 (different type sizes) @@ expected unsigned long const volatile *addr @@ got unsigned char * @@ >> net/bluetooth/sco.c:456:44: sparse: expected unsigned long const volatile *addr >> net/bluetooth/sco.c:456:44: sparse: got unsigned char * >> net/bluetooth/sco.c:868:55: sparse: sparse: incorrect type in argument 2 (different type sizes) @@ expected unsigned long volatile *addr @@ got unsigned char * @@ >> net/bluetooth/sco.c:868:55: sparse: expected unsigned long volatile *addr net/bluetooth/sco.c:868:55: sparse: got unsigned char * net/bluetooth/sco.c:870:57: sparse: sparse: incorrect type in argument 2 (different type sizes) @@ expected unsigned long volatile *addr @@ got unsigned char * @@ net/bluetooth/sco.c:870:57: sparse: expected unsigned long volatile *addr net/bluetooth/sco.c:870:57: sparse: got unsigned char * net/bluetooth/sco.c:999:41: sparse: sparse: incorrect type in argument 2 (different type sizes) @@ expected unsigned long const volatile *addr @@ got unsigned char * @@ net/bluetooth/sco.c:999:41: sparse: expected unsigned long const volatile *addr net/bluetooth/sco.c:999:41: sparse: got unsigned char * vim +/test_bit +456 net/bluetooth/sco.c 452 453 static void sco_skb_put_cmsg(struct sk_buff *skb, struct msghdr *msg, 454 struct sock *sk) 455 { > 456 if (test_bit(SCO_CMSG_PKT_STATUS, &sco_pi(sk)->cmsg_mask)) 457 put_cmsg(msg, SOL_BLUETOOTH, BT_SCM_PKT_STATUS, 458 sizeof(bt_cb(skb)->sco.pkt_status), 459 &bt_cb(skb)->sco.pkt_status); 460 } 461 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
Attachment:
.config.gz
Description: application/gzip
