All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* drivers/net/ethernet/chelsio/cxgb/espi.c:329:9: sparse: sparse: context imbalance in 't1_espi_get_mon' - different lock contexts for basic block
@ 2021-06-08 23:46 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2021-06-08 23:46 UTC (permalink / raw
  To: kbuild

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

CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Greg Ungerer <gerg@linux-m68k.org>
CC: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   368094df48e680fa51cedb68537408cfa64b788e
commit: d4aa8affa1e9e51c237a1ec47a97e96dce76c98c m68knommu: fix use of cpu_to_le() on IO access
date:   11 months ago
:::::: branch date: 6 hours ago
:::::: commit date: 11 months ago
config: m68k-randconfig-s031-20210608 (attached as .config)
compiler: m68k-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-341-g8af24329-dirty
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d4aa8affa1e9e51c237a1ec47a97e96dce76c98c
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout d4aa8affa1e9e51c237a1ec47a97e96dce76c98c
        # 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__' W=1 ARCH=m68k 

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 >>)
>> drivers/net/ethernet/chelsio/cxgb/espi.c:329:9: sparse: sparse: context imbalance in 't1_espi_get_mon' - different lock contexts for basic block
>> drivers/net/ethernet/chelsio/cxgb/espi.c:356:9: sparse: sparse: context imbalance in 't1_espi_get_mon_t204' - different lock contexts for basic block

vim +/t1_espi_get_mon +329 drivers/net/ethernet/chelsio/cxgb/espi.c

8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  313  
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  314  u32 t1_espi_get_mon(adapter_t *adapter, u32 addr, u8 wait)
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  315  {
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  316  	struct peespi *espi = adapter->espi;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  317  	u32 sel;
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  318  
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  319  	if (!is_T2(adapter))
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  320  		return 0;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  321  
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  322  	sel = V_MONITORED_PORT_NUM((addr & 0x3c) >> 2);
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  323  	if (!wait) {
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  324  		if (!spin_trylock(&espi->lock))
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  325  			return 0;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  326  	} else
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  327  		spin_lock(&espi->lock);
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  328  
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30 @329  	if ((sel != (espi->misc_ctrl & MON_MASK))) {
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  330  		writel(((espi->misc_ctrl & ~MON_MASK) | sel),
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  331  		       adapter->regs + A_ESPI_MISC_CONTROL);
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  332  		sel = readl(adapter->regs + A_ESPI_SCH_TOKEN3);
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  333  		writel(espi->misc_ctrl, adapter->regs + A_ESPI_MISC_CONTROL);
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  334  	} else
559fb51ba7e66f drivers/net/chelsio/espi.c Scott Bardone     2005-06-23  335  		sel = readl(adapter->regs + A_ESPI_SCH_TOKEN3);
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  336  	spin_unlock(&espi->lock);
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  337  	return sel;
8199d3a79c224b drivers/net/chelsio/espi.c Christoph Lameter 2005-03-30  338  }
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  339  
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  340  /*
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  341   * This function is for T204 only.
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  342   * compare with t1_espi_get_mon(), it reads espiInTxSop[0 ~ 3] in
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  343   * one shot, since there is no per port counter on the out side.
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  344   */
356bd1460d1e1c drivers/net/chelsio/espi.c Francois Romieu   2006-12-11  345  int t1_espi_get_mon_t204(adapter_t *adapter, u32 *valp, u8 wait)
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  346  {
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  347  	struct peespi *espi = adapter->espi;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  348  	u8 i, nport = (u8)adapter->params.nports;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  349  
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  350  	if (!wait) {
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  351  		if (!spin_trylock(&espi->lock))
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  352  			return -1;
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  353  	} else
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  354  		spin_lock(&espi->lock);
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01  355  
f1d3d38af75789 drivers/net/chelsio/espi.c Stephen Hemminger 2006-12-01 @356  	if ((espi->misc_ctrl & MON_MASK) != F_MONITORED_DIRECTION) {

:::::: The code at line 329 was first introduced by commit
:::::: 8199d3a79c224bbe5943fa08684e1f93a17881b0 [PATCH] A new 10GB Ethernet Driver by Chelsio Communications

:::::: TO: Christoph Lameter <christoph@graphe.net>
:::::: CC: Jeff Garzik <jgarzik@pobox.com>

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-06-08 23:46 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-06-08 23:46 drivers/net/ethernet/chelsio/cxgb/espi.c:329:9: sparse: sparse: context imbalance in 't1_espi_get_mon' - different lock contexts for basic block kernel test robot

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.