All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH net-next 8/8] mld: change context of mld module
Date: Tue, 09 Feb 2021 10:33:44 +0800	[thread overview]
Message-ID: <202102091014.g53e3tl1-lkp@intel.com> (raw)
In-Reply-To: <20210208175952.5880-1-ap420073@gmail.com>

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

Hi Taehee,

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/Taehee-Yoo/mld-change-context-from-atomic-to-sleepable/20210209-072339
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 6626a0266566c5aea16178c5e6cd7fc4db3f2f56
config: mips-allyesconfig (attached as .config)
compiler: mips-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/9ac7d23c98251efc265073a0d63930c91739cc02
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Taehee-Yoo/mld-change-context-from-atomic-to-sleepable/20210209-072339
        git checkout 9ac7d23c98251efc265073a0d63930c91739cc02
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=mips 

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

All warnings (new ones prefixed by >>):

>> net/ipv6/mcast.c:333:6: warning: no previous prototype for 'mld_check_leave_group' [-Wmissing-prototypes]
     333 | bool mld_check_leave_group(struct ipv6_mc_socklist *mc_lst, int omode)
         |      ^~~~~~~~~~~~~~~~~~~~~
   net/ipv6/mcast.c: In function 'ip6_mc_del_src':
   net/ipv6/mcast.c:2153:6: warning: variable 'i' set but not used [-Wunused-but-set-variable]
    2153 |  int i, err, rv;
         |      ^


vim +/mld_check_leave_group +333 net/ipv6/mcast.c

   331	
   332	/* special case - (INCLUDE, empty) == LEAVE_GROUP */
 > 333	bool mld_check_leave_group(struct ipv6_mc_socklist *mc_lst, int omode)
   334	{
   335		if (atomic_read(&mc_lst->sl_count) == 1 && omode == MCAST_INCLUDE)
   336			return true;
   337		else
   338			return false;
   339	}
   340	

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

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Taehee Yoo <ap420073@gmail.com>,
	davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org,
	dsahern@kernel.org, xiyou.wangcong@gmail.com, jwi@linux.ibm.com,
	kgraul@linux.ibm.com, hca@linux.ibm.com, gor@linux.ibm.com,
	borntraeger@de.ibm.com
Cc: kbuild-all@lists.01.org
Subject: Re: [PATCH net-next 8/8] mld: change context of mld module
Date: Tue, 9 Feb 2021 10:33:44 +0800	[thread overview]
Message-ID: <202102091014.g53e3tl1-lkp@intel.com> (raw)
In-Reply-To: <20210208175952.5880-1-ap420073@gmail.com>

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

Hi Taehee,

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/Taehee-Yoo/mld-change-context-from-atomic-to-sleepable/20210209-072339
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 6626a0266566c5aea16178c5e6cd7fc4db3f2f56
config: mips-allyesconfig (attached as .config)
compiler: mips-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/9ac7d23c98251efc265073a0d63930c91739cc02
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Taehee-Yoo/mld-change-context-from-atomic-to-sleepable/20210209-072339
        git checkout 9ac7d23c98251efc265073a0d63930c91739cc02
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=mips 

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

All warnings (new ones prefixed by >>):

>> net/ipv6/mcast.c:333:6: warning: no previous prototype for 'mld_check_leave_group' [-Wmissing-prototypes]
     333 | bool mld_check_leave_group(struct ipv6_mc_socklist *mc_lst, int omode)
         |      ^~~~~~~~~~~~~~~~~~~~~
   net/ipv6/mcast.c: In function 'ip6_mc_del_src':
   net/ipv6/mcast.c:2153:6: warning: variable 'i' set but not used [-Wunused-but-set-variable]
    2153 |  int i, err, rv;
         |      ^


vim +/mld_check_leave_group +333 net/ipv6/mcast.c

   331	
   332	/* special case - (INCLUDE, empty) == LEAVE_GROUP */
 > 333	bool mld_check_leave_group(struct ipv6_mc_socklist *mc_lst, int omode)
   334	{
   335		if (atomic_read(&mc_lst->sl_count) == 1 && omode == MCAST_INCLUDE)
   336			return true;
   337		else
   338			return false;
   339	}
   340	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 69939 bytes --]

  reply	other threads:[~2021-02-09  2:33 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-08 17:59 [PATCH net-next 8/8] mld: change context of mld module Taehee Yoo
2021-02-09  2:33 ` kernel test robot [this message]
2021-02-09  2:33   ` kernel test robot
2021-02-09  3:17 ` kernel test robot
2021-02-09  3:17   ` kernel test robot
2021-02-09  3:34 ` kernel test robot
2021-02-09  3:34   ` kernel test robot
2021-02-09  3:34 ` [RFC PATCH] mld: mld_check_leave_group() can be static kernel test robot
2021-02-09  3:34   ` kernel test robot
2021-02-09  8:26 ` [PATCH net-next 8/8] mld: change context of mld module Julian Wiedmann
2021-02-09 10:00   ` Taehee Yoo

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=202102091014.g53e3tl1-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.