All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org
Subject: [linux-next:master 2227/3715] drivers/staging/wfx/sta.c:155 wfx_filter_mcast() warn: ignoring unreachable code.
Date: Tue, 21 Apr 2020 20:33:17 +0300	[thread overview]
Message-ID: <20200421173317.GF2682@kadam> (raw)

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   39a314cd1168cbcb05ebcd973098c8f1440a3064

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
drivers/staging/wfx/sta.c:155 wfx_filter_mcast() warn: ignoring unreachable code.

# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=c801351a9d4587aab131d018330d4b35a0b3cdcf
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git remote update linux-next
git checkout c801351a9d4587aab131d018330d4b35a0b3cdcf
vim +155 drivers/staging/wfx/sta.c

c801351a9d4587 Jérôme Pouiller 2020-04-15  147  static void wfx_filter_mcast(struct wfx_vif *wvif, bool filter_mcast)
2c69b2da378243 Jérôme Pouiller 2020-04-15  148  {
2c69b2da378243 Jérôme Pouiller 2020-04-15  149  	int i;
40115bbc40e2fd Jérôme Pouiller 2019-09-19  150  
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  151  	// Temporary workaround for filters
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  152  	hif_set_data_filtering(wvif, false, true);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  153  	return;
                                                        ^^^^^^^
Temporary code creates a temporary static checker warning from the
zero day kbuild bot.

e86bc725ba5f96 Jérôme Pouiller 2020-04-15  154  
c801351a9d4587 Jérôme Pouiller 2020-04-15 @155  	if (!filter_mcast) {
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  156  		hif_set_data_filtering(wvif, false, true);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  157  		return;
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  158  	}
981132aa039d90 Jérôme Pouiller 2020-04-15  159  	for (i = 0; i < wvif->filter_mcast_count; i++)
981132aa039d90 Jérôme Pouiller 2020-04-15  160  		hif_set_mac_addr_condition(wvif, i, wvif->filter_mcast_addr[i]);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  161  	hif_set_uc_mc_bc_condition(wvif, 0,
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  162  				   HIF_FILTER_UNICAST | HIF_FILTER_BROADCAST);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  163  	hif_set_config_data_filter(wvif, true, 0, BIT(1),
981132aa039d90 Jérôme Pouiller 2020-04-15  164  				   BIT(wvif->filter_mcast_count) - 1);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  165  	hif_set_data_filtering(wvif, true, true);
40115bbc40e2fd Jérôme Pouiller 2019-09-19  166  }

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
_______________________________________________
kbuild mailing list -- kbuild(a)lists.01.org
To unsubscribe send an email to kbuild-leave(a)lists.01.org

WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild-all@lists.01.org
Subject: [kbuild] [linux-next:master 2227/3715] drivers/staging/wfx/sta.c:155 wfx_filter_mcast() warn: ignoring unreachable code.
Date: Tue, 21 Apr 2020 20:33:17 +0300	[thread overview]
Message-ID: <20200421173317.GF2682@kadam> (raw)

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   39a314cd1168cbcb05ebcd973098c8f1440a3064

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
drivers/staging/wfx/sta.c:155 wfx_filter_mcast() warn: ignoring unreachable code.

# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=c801351a9d4587aab131d018330d4b35a0b3cdcf
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git remote update linux-next
git checkout c801351a9d4587aab131d018330d4b35a0b3cdcf
vim +155 drivers/staging/wfx/sta.c

c801351a9d4587 Jérôme Pouiller 2020-04-15  147  static void wfx_filter_mcast(struct wfx_vif *wvif, bool filter_mcast)
2c69b2da378243 Jérôme Pouiller 2020-04-15  148  {
2c69b2da378243 Jérôme Pouiller 2020-04-15  149  	int i;
40115bbc40e2fd Jérôme Pouiller 2019-09-19  150  
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  151  	// Temporary workaround for filters
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  152  	hif_set_data_filtering(wvif, false, true);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  153  	return;
                                                        ^^^^^^^
Temporary code creates a temporary static checker warning from the
zero day kbuild bot.

e86bc725ba5f96 Jérôme Pouiller 2020-04-15  154  
c801351a9d4587 Jérôme Pouiller 2020-04-15 @155  	if (!filter_mcast) {
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  156  		hif_set_data_filtering(wvif, false, true);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  157  		return;
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  158  	}
981132aa039d90 Jérôme Pouiller 2020-04-15  159  	for (i = 0; i < wvif->filter_mcast_count; i++)
981132aa039d90 Jérôme Pouiller 2020-04-15  160  		hif_set_mac_addr_condition(wvif, i, wvif->filter_mcast_addr[i]);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  161  	hif_set_uc_mc_bc_condition(wvif, 0,
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  162  				   HIF_FILTER_UNICAST | HIF_FILTER_BROADCAST);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  163  	hif_set_config_data_filter(wvif, true, 0, BIT(1),
981132aa039d90 Jérôme Pouiller 2020-04-15  164  				   BIT(wvif->filter_mcast_count) - 1);
e86bc725ba5f96 Jérôme Pouiller 2020-04-15  165  	hif_set_data_filtering(wvif, true, true);
40115bbc40e2fd Jérôme Pouiller 2019-09-19  166  }

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
_______________________________________________
kbuild mailing list -- kbuild(a)lists.01.org
To unsubscribe send an email to kbuild-leave(a)lists.01.org

             reply	other threads:[~2020-04-21 17:33 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-21 17:33 Dan Carpenter [this message]
2020-04-21 17:33 ` [kbuild] [linux-next:master 2227/3715] drivers/staging/wfx/sta.c:155 wfx_filter_mcast() warn: ignoring unreachable code Dan Carpenter
  -- strict thread matches above, loose matches on Subject: below --
2020-04-21 16:35 kbuild 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=20200421173317.GF2682@kadam \
    --to=dan.carpenter@oracle.com \
    --cc=kbuild@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.