Re: [PATCH v4 1/2] geneve: implement support for IPv6-based tunnels

From: kbuild test robot <lkp@intel.com>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: kbuild-all@01.org, netdev@vger.kernel.org,
	Dave Miller <davem@davemloft.net>,
	Pravin B Shelar <pshelar@nicira.com>,
	Jesse Gross <jesse@nicira.com>, Jiri Benc <jbenc@redhat.com>,
	"John W. Linville" <linville@tuxdriver.com>
Subject: Re: [PATCH v4 1/2] geneve: implement support for IPv6-based tunnels
Date: Wed, 21 Oct 2015 06:55:52 +0800	[thread overview]
Message-ID: <201510210631.PB30gY26%fengguang.wu@intel.com> (raw)
In-Reply-To: <1445353866-32710-1-git-send-email-linville@tuxdriver.com>

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

Hi John,

[auto build test WARNING on v4.3-rc6 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url:    https://github.com/0day-ci/linux/commits/John-W-Linville/geneve-implement-support-for-IPv6-based-tunnels/20151020-232128
config: x86_64-randconfig-a0-10210538 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from drivers/net/geneve.c:13:
   drivers/net/geneve.c: In function 'geneve_rx':
   drivers/net/geneve.c:239:6: error: 'ip6h' undeclared (first use in this function)
     if (ip6h)
         ^
   include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
                               ^
>> drivers/net/geneve.c:239:2: note: in expansion of macro 'if'
     if (ip6h)
     ^
   drivers/net/geneve.c:239:6: note: each undeclared identifier is reported only once for each function it appears in
     if (ip6h)
         ^
   include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
                               ^
>> drivers/net/geneve.c:239:2: note: in expansion of macro 'if'
     if (ip6h)
     ^

vim +/if +239 drivers/net/geneve.c

   223		skb_reset_mac_header(skb);
   224		skb_scrub_packet(skb, !net_eq(geneve->net, dev_net(geneve->dev)));
   225		skb->protocol = eth_type_trans(skb, geneve->dev);
   226		skb_postpull_rcsum(skb, eth_hdr(skb), ETH_HLEN);
   227	
   228		if (tun_dst)
   229			skb_dst_set(skb, &tun_dst->dst);
   230	
   231		/* Ignore packet loops (and multicast echo) */
   232		if (ether_addr_equal(eth_hdr(skb)->h_source, geneve->dev->dev_addr))
   233			goto drop;
   234	
   235		skb_reset_network_header(skb);
   236	
   237		if (iph)
   238			err = IP_ECN_decapsulate(iph, skb);
 > 239		if (ip6h)
   240			err = IP6_ECN_decapsulate(ip6h, skb);
   241	
   242		if (unlikely(err)) {
   243			if (log_ecn_error) {
   244				if (iph)
   245					net_info_ratelimited("non-ECT from %pI4 "
   246							     "with TOS=%#x\n",
   247							     &iph->saddr, iph->tos);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 24631 bytes --]