From mboxrd@z Thu Jan 1 00:00:00 1970 From: kbuild test robot Subject: Re: [PATCH net-next v2 3/5] bpf: Split lwt inout verifier structures Date: Thu, 26 Apr 2018 23:29:42 +0800 Message-ID: <201804262333.IFS7v1gT%fengguang.wu@intel.com> References: <1a00b0c169c46a6444dea2b9f76e9c83430faa87.1524591163.git.m.xhonneux@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kbuild-all@01.org, netdev@vger.kernel.org, dlebrun@google.com, alexei.starovoitov@gmail.com To: Mathieu Xhonneux Return-path: Received: from mga18.intel.com ([134.134.136.126]:59968 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756749AbeDZPaM (ORCPT ); Thu, 26 Apr 2018 11:30:12 -0400 Content-Disposition: inline In-Reply-To: <1a00b0c169c46a6444dea2b9f76e9c83430faa87.1524591163.git.m.xhonneux@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: Hi Mathieu, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Mathieu-Xhonneux/ipv6-sr-introduce-seg6local-End-BPF-action/20180426-082209 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) net/core/filter.c:110:48: sparse: expression using sizeof(void) net/core/filter.c:110:48: sparse: expression using sizeof(void) net/core/filter.c:323:33: sparse: subtraction of functions? Share your drugs net/core/filter.c:326:33: sparse: subtraction of functions? Share your drugs net/core/filter.c:329:33: sparse: subtraction of functions? Share your drugs net/core/filter.c:332:33: sparse: subtraction of functions? Share your drugs net/core/filter.c:335:33: sparse: subtraction of functions? Share your drugs include/linux/filter.h:612:16: sparse: expression using sizeof(void) include/linux/filter.h:612:16: sparse: expression using sizeof(void) include/linux/filter.h:612:16: sparse: expression using sizeof(void) include/linux/filter.h:612:16: sparse: expression using sizeof(void) net/core/filter.c:1189:39: sparse: incorrect type in argument 1 (different address spaces) @@ expected struct sock_filter const *filter @@ got struct sockstruct sock_filter const *filter @@ net/core/filter.c:1189:39: expected struct sock_filter const *filter net/core/filter.c:1189:39: got struct sock_filter [noderef] *filter include/linux/filter.h:612:16: sparse: expression using sizeof(void) include/linux/filter.h:612:16: sparse: expression using sizeof(void) net/core/filter.c:1291:39: sparse: incorrect type in argument 1 (different address spaces) @@ expected struct sock_filter const *filter @@ got struct sockstruct sock_filter const *filter @@ net/core/filter.c:1291:39: expected struct sock_filter const *filter net/core/filter.c:1291:39: got struct sock_filter [noderef] *filter include/linux/filter.h:612:16: sparse: expression using sizeof(void) net/core/filter.c:1552:43: sparse: incorrect type in argument 2 (different base types) @@ expected restricted __wsum [usertype] diff @@ got unsigned lonrestricted __wsum [usertype] diff @@ net/core/filter.c:1552:43: expected restricted __wsum [usertype] diff net/core/filter.c:1552:43: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1555:36: sparse: incorrect type in argument 2 (different base types) @@ expected restricted __be16 [usertype] old @@ got unsigned lonrestricted __be16 [usertype] old @@ net/core/filter.c:1555:36: expected restricted __be16 [usertype] old net/core/filter.c:1555:36: got unsigned long long [unsigned] [usertype] from net/core/filter.c:1555:42: sparse: incorrect type in argument 3 (different base types) @@ expected restricted __be16 [usertype] new @@ got unsigned lonrestricted __be16 [usertype] new @@ net/core/filter.c:1555:42: expected restricted __be16 [usertype] new net/core/filter.c:1555:42: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1558:36: sparse: incorrect type in argument 2 (different base types) @@ expected restricted __be32 [usertype] from @@ got unsigned lonrestricted __be32 [usertype] from @@ net/core/filter.c:1558:36: expected restricted __be32 [usertype] from net/core/filter.c:1558:36: got unsigned long long [unsigned] [usertype] from net/core/filter.c:1558:42: sparse: incorrect type in argument 3 (different base types) @@ expected restricted __be32 [usertype] to @@ got unsigned lonrestricted __be32 [usertype] to @@ net/core/filter.c:1558:42: expected restricted __be32 [usertype] to net/core/filter.c:1558:42: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1603:59: sparse: incorrect type in argument 3 (different base types) @@ expected restricted __wsum [usertype] diff @@ got unsigned lonrestricted __wsum [usertype] diff @@ net/core/filter.c:1603:59: expected restricted __wsum [usertype] diff net/core/filter.c:1603:59: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1606:52: sparse: incorrect type in argument 3 (different base types) @@ expected restricted __be16 [usertype] from @@ got unsigned lonrestricted __be16 [usertype] from @@ net/core/filter.c:1606:52: expected restricted __be16 [usertype] from net/core/filter.c:1606:52: got unsigned long long [unsigned] [usertype] from net/core/filter.c:1606:58: sparse: incorrect type in argument 4 (different base types) @@ expected restricted __be16 [usertype] to @@ got unsigned lonrestricted __be16 [usertype] to @@ net/core/filter.c:1606:58: expected restricted __be16 [usertype] to net/core/filter.c:1606:58: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1609:52: sparse: incorrect type in argument 3 (different base types) @@ expected restricted __be32 [usertype] from @@ got unsigned lonrestricted __be32 [usertype] from @@ net/core/filter.c:1609:52: expected restricted __be32 [usertype] from net/core/filter.c:1609:52: got unsigned long long [unsigned] [usertype] from net/core/filter.c:1609:58: sparse: incorrect type in argument 4 (different base types) @@ expected restricted __be32 [usertype] to @@ got unsigned lonrestricted __be32 [usertype] to @@ net/core/filter.c:1609:58: expected restricted __be32 [usertype] to net/core/filter.c:1609:58: got unsigned long long [unsigned] [usertype] to net/core/filter.c:1655:28: sparse: incorrect type in return expression (different base types) @@ expected unsigned long long @@ got nsigned long long @@ net/core/filter.c:1655:28: expected unsigned long long net/core/filter.c:1655:28: got restricted __wsum net/core/filter.c:1677:35: sparse: incorrect type in return expression (different base types) @@ expected unsigned long long @@ got restricted unsigned long long @@ net/core/filter.c:1677:35: expected unsigned long long net/core/filter.c:1677:35: got restricted __wsum [usertype] csum net/core/filter.c:3462:41: sparse: expression using sizeof(void) net/core/filter.c:3466:41: sparse: expression using sizeof(void) net/core/filter.c:3470:46: sparse: expression using sizeof(void) net/core/filter.c:3470:46: sparse: expression using sizeof(void) net/core/filter.c:3538:47: sparse: expression using sizeof(void) net/core/filter.c:3728:5: sparse: symbol 'bpf_push_seg6_encap' was not declared. Should it be static? >> net/core/filter.c:5806:31: sparse: symbol 'lwt_out_verifier_ops' was not declared. Should it be static? >> net/core/filter.c:5812:27: sparse: symbol 'lwt_out_prog_ops' was not declared. Should it be static? Please review and possibly fold the followup patch. --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation