All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PACTH net-next 4/5] net: add net available in build_state
Date: Thu, 20 Feb 2020 06:38:22 +0800	[thread overview]
Message-ID: <202002200626.IDbujHc8%lkp@intel.com> (raw)
In-Reply-To: <20200217223541.18862-5-alex.aring@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 16397 bytes --]

Hi Alexander,

I love your patch! Yet something to improve:

[auto build test ERROR on net-next/master]
[also build test ERROR on net/master linus/master v5.6-rc2 next-20200219]
[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/Alexander-Aring/net-ipv6-add-rpl-source-routing/20200220-033031
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 00796b929ce8c9e7567fe7e395763418eb579100
config: x86_64-defconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All error/warnings (new ones prefixed by >>):

   net/ipv4/fib_semantics.c: In function 'fib_nh_common_init':
>> net/ipv4/fib_semantics.c:593:30: warning: passing argument 1 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                 ^~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'u16 {aka short unsigned int}' but argument is of type 'struct net *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
>> net/ipv4/fib_semantics.c:593:35: warning: passing argument 2 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                      ^~~~~~~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct nlattr *' but argument is of type 'u16 {aka short unsigned int}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:593:47: warning: passing argument 3 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                                  ^~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'unsigned int' but argument is of type 'struct nlattr *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:594:9: warning: passing argument 4 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
            nhc->nhc_family, cfg, &lwtstate,
            ^~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'const void *' but argument is of type 'u8 {aka unsigned char}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
>> net/ipv4/fib_semantics.c:594:31: error: passing argument 6 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
            nhc->nhc_family, cfg, &lwtstate,
                                  ^
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct netlink_ext_ack *' but argument is of type 'struct lwtunnel_state **'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
>> net/ipv4/fib_semantics.c:593:9: error: too many arguments to function 'lwtunnel_build_state'
      err = lwtunnel_build_state(net, encap_type, encap,
            ^~~~~~~~~~~~~~~~~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: declared here
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c: In function 'fib_encap_match':
   net/ipv4/fib_semantics.c:831:29: warning: passing argument 1 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                ^~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'u16 {aka short unsigned int}' but argument is of type 'struct net *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:831:34: warning: passing argument 2 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                     ^~~~~~~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct nlattr *' but argument is of type 'u16 {aka short unsigned int}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:831:46: warning: passing argument 3 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                                 ^~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'unsigned int' but argument is of type 'struct nlattr *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   In file included from net/ipv4/fib_semantics.c:19:0:
>> include/linux/socket.h:165:18: warning: passing argument 4 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
    #define AF_INET  2 /* Internet IP Protocol  */
                     ^
>> net/ipv4/fib_semantics.c:831:53: note: in expansion of macro 'AF_INET'
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                                        ^~~~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'const void *' but argument is of type 'int'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:832:8: error: passing argument 5 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
           cfg, &lwtstate, extack);
           ^~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct lwtunnel_state **' but argument is of type 'const struct fib_config *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:832:13: error: passing argument 6 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
           cfg, &lwtstate, extack);
                ^
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct netlink_ext_ack *' but argument is of type 'struct lwtunnel_state **'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net/ipv4/fib_semantics.c:831:8: error: too many arguments to function 'lwtunnel_build_state'
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
           ^~~~~~~~~~~~~~~~~~~~
   In file included from net/ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: declared here
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   net//ipv4/fib_semantics.c: In function 'fib_nh_common_init':
   net//ipv4/fib_semantics.c:593:30: warning: passing argument 1 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                 ^~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'u16 {aka short unsigned int}' but argument is of type 'struct net *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:593:35: warning: passing argument 2 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                      ^~~~~~~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct nlattr *' but argument is of type 'u16 {aka short unsigned int}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:593:47: warning: passing argument 3 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
      err = lwtunnel_build_state(net, encap_type, encap,
                                                  ^~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'unsigned int' but argument is of type 'struct nlattr *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:594:9: warning: passing argument 4 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
            nhc->nhc_family, cfg, &lwtstate,
            ^~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'const void *' but argument is of type 'u8 {aka unsigned char}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:594:31: error: passing argument 6 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
            nhc->nhc_family, cfg, &lwtstate,
                                  ^
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct netlink_ext_ack *' but argument is of type 'struct lwtunnel_state **'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:593:9: error: too many arguments to function 'lwtunnel_build_state'
      err = lwtunnel_build_state(net, encap_type, encap,
            ^~~~~~~~~~~~~~~~~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: declared here
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c: In function 'fib_encap_match':
   net//ipv4/fib_semantics.c:831:29: warning: passing argument 1 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                ^~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'u16 {aka short unsigned int}' but argument is of type 'struct net *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:831:34: warning: passing argument 2 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                     ^~~~~~~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct nlattr *' but argument is of type 'u16 {aka short unsigned int}'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:831:46: warning: passing argument 3 of 'lwtunnel_build_state' makes integer from pointer without a cast [-Wint-conversion]
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                                 ^~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'unsigned int' but argument is of type 'struct nlattr *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   In file included from net//ipv4/fib_semantics.c:19:0:
>> include/linux/socket.h:165:18: warning: passing argument 4 of 'lwtunnel_build_state' makes pointer from integer without a cast [-Wint-conversion]
    #define AF_INET  2 /* Internet IP Protocol  */
                     ^
   net//ipv4/fib_semantics.c:831:53: note: in expansion of macro 'AF_INET'
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
                                                        ^~~~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'const void *' but argument is of type 'int'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:832:8: error: passing argument 5 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
           cfg, &lwtstate, extack);
           ^~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct lwtunnel_state **' but argument is of type 'const struct fib_config *'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:832:13: error: passing argument 6 of 'lwtunnel_build_state' from incompatible pointer type [-Werror=incompatible-pointer-types]
           cfg, &lwtstate, extack);
                ^
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: expected 'struct netlink_ext_ack *' but argument is of type 'struct lwtunnel_state **'
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   net//ipv4/fib_semantics.c:831:8: error: too many arguments to function 'lwtunnel_build_state'
     ret = lwtunnel_build_state(net, encap_type, encap, AF_INET,
           ^~~~~~~~~~~~~~~~~~~~
   In file included from net//ipv4/fib_semantics.c:44:0:
   include/net/lwtunnel.h:212:19: note: declared here
    static inline int lwtunnel_build_state(u16 encap_type,
                      ^~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/lwtunnel_build_state +594 net/ipv4/fib_semantics.c

   572	
   573	int fib_nh_common_init(struct net *net, struct fib_nh_common *nhc,
   574			       struct nlattr *encap, u16 encap_type,
   575			       void *cfg, gfp_t gfp_flags,
   576			       struct netlink_ext_ack *extack)
   577	{
   578		int err;
   579	
   580		nhc->nhc_pcpu_rth_output = alloc_percpu_gfp(struct rtable __rcu *,
   581							    gfp_flags);
   582		if (!nhc->nhc_pcpu_rth_output)
   583			return -ENOMEM;
   584	
   585		if (encap) {
   586			struct lwtunnel_state *lwtstate;
   587	
   588			if (encap_type == LWTUNNEL_ENCAP_NONE) {
   589				NL_SET_ERR_MSG(extack, "LWT encap type not specified");
   590				err = -EINVAL;
   591				goto lwt_failure;
   592			}
 > 593			err = lwtunnel_build_state(net, encap_type, encap,
 > 594						   nhc->nhc_family, cfg, &lwtstate,
   595						   extack);
   596			if (err)
   597				goto lwt_failure;
   598	
   599			nhc->nhc_lwtstate = lwtstate_get(lwtstate);
   600		}
   601	
   602		return 0;
   603	
   604	lwt_failure:
   605		rt_fibinfo_free_cpus(nhc->nhc_pcpu_rth_output);
   606		nhc->nhc_pcpu_rth_output = NULL;
   607		return err;
   608	}
   609	EXPORT_SYMBOL_GPL(fib_nh_common_init);
   610	

---
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: 29012 bytes --]

  reply	other threads:[~2020-02-19 22:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-17 22:35 [PACTH net-next 0/5] net: ipv6: add rpl source routing Alexander Aring
2020-02-17 22:35 ` [PACTH net-next 1/5] include: uapi: linux: add rpl sr header definition Alexander Aring
2020-02-17 22:35 ` [PACTH net-next 2/5] addrconf: add functionality to check on rpl requirements Alexander Aring
2020-02-17 22:35 ` [PACTH net-next 3/5] net: ipv6: add support for rpl sr exthdr Alexander Aring
2020-02-17 22:35 ` [PACTH net-next 4/5] net: add net available in build_state Alexander Aring
2020-02-19 22:38   ` kbuild test robot [this message]
2020-02-17 22:35 ` [PACTH net-next 5/5] net: ipv6: add rpl sr tunnel Alexander Aring
2020-02-18  5:47   ` David Miller
2020-02-19 14:41     ` Alexander Aring
2020-02-19 19:12       ` Jakub Kicinski
2020-02-20 23:02       ` David Miller
2020-02-19 14:45   ` Alexander Aring

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202002200626.IDbujHc8%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.