From: Justin Iurman <justin.iurman@uliege.be> To: kernel test robot <rong.a.chen@intel.com> Cc: netdev@vger.kernel.org, kbuild-all@lists.01.org, davem@davemloft.net, kuba@kernel.org, tom@herbertland.com Subject: Re: [PATCH net-next 4/5] ipv6: ioam: Support for IOAM injection with lwtunnels Date: Mon, 22 Mar 2021 13:56:26 +0100 (CET) [thread overview] Message-ID: <1448994361.68241255.1616417786080.JavaMail.zimbra@uliege.be> (raw) In-Reply-To: <20210311005631.GJ219708@shao2-debian> > Hi Justin, > > 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/Justin-Iurman/Support-for-the-IOAM-Pre-allocated-Trace-with-IPv6/20210311-005727 > base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git > d310ec03a34e92a77302edb804f7d68ee4f01ba0 > :::::: branch date: 2 hours ago > :::::: commit date: 2 hours ago > config: sparc64-randconfig-s032-20210310 (attached as .config) > compiler: sparc64-linux-gcc (GCC) 9.3.0 > reproduce: > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O > ~/bin/make.cross > chmod +x ~/bin/make.cross > # apt-get install sparse > # sparse version: v0.6.3-262-g5e674421-dirty > # > https://github.com/0day-ci/linux/commit/f20771586508a195a44298f251d68446c10830ce > git remote add linux-review https://github.com/0day-ci/linux > git fetch --no-tags linux-review > Justin-Iurman/Support-for-the-IOAM-Pre-allocated-Trace-with-IPv6/20210311-005727 > git checkout f20771586508a195a44298f251d68446c10830ce > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 > CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=sparc64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <lkp@intel.com> > > > "sparse warnings: (new ones prefixed by >>)" >>> net/ipv6/ioam6.c:856:1: sparse: sparse: unused label 'out_unregister_genl' Apologies for this small oversight, will update it and post a v3 (forgot the "v2" tag for this patchset by the way). > vim +/out_unregister_genl +856 net/ipv6/ioam6.c > > f20771586508a1 Justin Iurman 2021-03-10 851 > e1a4cb7c14f537 Justin Iurman 2021-03-10 852 pr_info("In-situ OAM (IOAM) with > IPv6\n"); > ffece959dfe59a Justin Iurman 2021-03-10 853 > ffece959dfe59a Justin Iurman 2021-03-10 854 out: > ffece959dfe59a Justin Iurman 2021-03-10 855 return err; > f20771586508a1 Justin Iurman 2021-03-10 @856 out_unregister_genl: > f20771586508a1 Justin Iurman 2021-03-10 857 > genl_unregister_family(&ioam6_genl_family); > ffece959dfe59a Justin Iurman 2021-03-10 858 out_unregister_pernet_subsys: > ffece959dfe59a Justin Iurman 2021-03-10 859 > unregister_pernet_subsys(&ioam6_net_ops); > ffece959dfe59a Justin Iurman 2021-03-10 860 goto out; > e1a4cb7c14f537 Justin Iurman 2021-03-10 861 } > e1a4cb7c14f537 Justin Iurman 2021-03-10 862
WARNING: multiple messages have this Message-ID (diff)
From: Justin Iurman <justin.iurman@uliege.be> To: kbuild-all@lists.01.org Subject: Re: [PATCH net-next 4/5] ipv6: ioam: Support for IOAM injection with lwtunnels Date: Mon, 22 Mar 2021 13:56:26 +0100 [thread overview] Message-ID: <1448994361.68241255.1616417786080.JavaMail.zimbra@uliege.be> (raw) In-Reply-To: <20210311005631.GJ219708@shao2-debian> [-- Attachment #1: Type: text/plain, Size: 2636 bytes --] > Hi Justin, > > 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/Justin-Iurman/Support-for-the-IOAM-Pre-allocated-Trace-with-IPv6/20210311-005727 > base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git > d310ec03a34e92a77302edb804f7d68ee4f01ba0 > :::::: branch date: 2 hours ago > :::::: commit date: 2 hours ago > config: sparc64-randconfig-s032-20210310 (attached as .config) > compiler: sparc64-linux-gcc (GCC) 9.3.0 > reproduce: > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O > ~/bin/make.cross > chmod +x ~/bin/make.cross > # apt-get install sparse > # sparse version: v0.6.3-262-g5e674421-dirty > # > https://github.com/0day-ci/linux/commit/f20771586508a195a44298f251d68446c10830ce > git remote add linux-review https://github.com/0day-ci/linux > git fetch --no-tags linux-review > Justin-Iurman/Support-for-the-IOAM-Pre-allocated-Trace-with-IPv6/20210311-005727 > git checkout f20771586508a195a44298f251d68446c10830ce > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 > CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=sparc64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot <lkp@intel.com> > > > "sparse warnings: (new ones prefixed by >>)" >>> net/ipv6/ioam6.c:856:1: sparse: sparse: unused label 'out_unregister_genl' Apologies for this small oversight, will update it and post a v3 (forgot the "v2" tag for this patchset by the way). > vim +/out_unregister_genl +856 net/ipv6/ioam6.c > > f20771586508a1 Justin Iurman 2021-03-10 851 > e1a4cb7c14f537 Justin Iurman 2021-03-10 852 pr_info("In-situ OAM (IOAM) with > IPv6\n"); > ffece959dfe59a Justin Iurman 2021-03-10 853 > ffece959dfe59a Justin Iurman 2021-03-10 854 out: > ffece959dfe59a Justin Iurman 2021-03-10 855 return err; > f20771586508a1 Justin Iurman 2021-03-10 @856 out_unregister_genl: > f20771586508a1 Justin Iurman 2021-03-10 857 > genl_unregister_family(&ioam6_genl_family); > ffece959dfe59a Justin Iurman 2021-03-10 858 out_unregister_pernet_subsys: > ffece959dfe59a Justin Iurman 2021-03-10 859 > unregister_pernet_subsys(&ioam6_net_ops); > ffece959dfe59a Justin Iurman 2021-03-10 860 goto out; > e1a4cb7c14f537 Justin Iurman 2021-03-10 861 } > e1a4cb7c14f537 Justin Iurman 2021-03-10 862
next prev parent reply other threads:[~2021-03-22 12:59 UTC|newest] Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-03-10 16:44 [PATCH net-next 0/5] Support for the IOAM Pre-allocated Trace with IPv6 Justin Iurman 2021-03-10 16:44 ` [PATCH net-next 1/5] uapi: IPv6 IOAM headers definition Justin Iurman 2021-03-10 16:44 ` [PATCH net-next 2/5] ipv6: ioam: Data plane support for Pre-allocated Trace Justin Iurman 2021-03-10 16:44 ` [PATCH net-next 3/5] ipv6: ioam: IOAM Generic Netlink API Justin Iurman 2021-03-10 16:44 ` [PATCH net-next 4/5] ipv6: ioam: Support for IOAM injection with lwtunnels Justin Iurman 2021-03-10 20:15 ` kernel test robot 2021-03-10 20:15 ` kernel test robot 2021-03-11 0:56 ` kernel test robot 2021-03-11 0:56 ` kernel test robot 2021-03-22 12:56 ` Justin Iurman [this message] 2021-03-22 12:56 ` Justin Iurman 2021-03-10 16:44 ` [PATCH net-next 5/5] ipv6: ioam: Documentation for new IOAM sysctls Justin Iurman 2021-03-10 18:43 [PATCH net-next 4/5] ipv6: ioam: Support for IOAM injection with lwtunnels kernel test robot
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=1448994361.68241255.1616417786080.JavaMail.zimbra@uliege.be \ --to=justin.iurman@uliege.be \ --cc=davem@davemloft.net \ --cc=kbuild-all@lists.01.org \ --cc=kuba@kernel.org \ --cc=netdev@vger.kernel.org \ --cc=rong.a.chen@intel.com \ --cc=tom@herbertland.com \ /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: linkBe 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.