From mboxrd@z Thu Jan 1 00:00:00 1970 From: Toshiaki Makita Subject: Re: [PATCH bpf] xdp: Fix handling of devmap in generic XDP Date: Thu, 14 Jun 2018 09:33:26 +0900 Message-ID: References: <1528877178-2521-1-git-send-email-makita.toshiaki@lab.ntt.co.jp> <201806131752.fqIMeuHk%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: kbuild-all@01.org, Alexei Starovoitov , Daniel Borkmann , netdev@vger.kernel.org, Jesper Dangaard Brouer To: kbuild test robot Return-path: Received: from tama500.ecl.ntt.co.jp ([129.60.39.148]:43758 "EHLO tama500.ecl.ntt.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935903AbeFNAek (ORCPT ); Wed, 13 Jun 2018 20:34:40 -0400 In-Reply-To: <201806131752.fqIMeuHk%fengguang.wu@intel.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 2018/06/13 18:27, kbuild test robot wrote: > Hi Toshiaki, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on bpf/master] > > url: https://github.com/0day-ci/linux/commits/Toshiaki-Makita/xdp-Fix-handling-of-devmap-in-generic-XDP/20180613-161204 > base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf.git master > config: i386-randconfig-a1-201823 (attached as .config) > compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4 > reproduce: > # save the attached .config to linux build tree > make ARCH=i386 > > All warnings (new ones prefixed by >>): > > In file included from net//bpf/test_run.c:7:0: >>> include/linux/bpf.h:594:16: warning: 'struct sk_buff' declared inside parameter list > struct bpf_prog *xdp_prog) > ^ >>> include/linux/bpf.h:594:16: warning: its scope is only this definition or declaration, which is probably not what you want > > vim +594 include/linux/bpf.h > > 591 > 592 static inline int dev_map_generic_redirect(struct bpf_dtab_netdev *dst, > 593 struct sk_buff *skb, > > 594 struct bpf_prog *xdp_prog) > 595 { > 596 return 0; > 597 } > 598 Ugh I did build test for the entire tree with CONFIG_BPF_SYSCALL, and net/core and kernel/bpf without CONFIG_BPF_SYSCALL but not net/bpf. I'll make sure to test the entire tree next. will send v2. -- Toshiaki Makita