All of lore.kernel.org
 help / color / mirror / Atom feed
* drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c:1960 rvu_mbox_handler_npc_mcam_shift_entry() error: uninitialized symbol 'rc'.
@ 2021-07-23  5:24 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2021-07-23  5:24 UTC (permalink / raw)
  To: kbuild

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

CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Jiaxun Yang <jiaxun.yang@flygoat.com>
CC: Thomas Bogendoerfer <tsbogend@alpha.franken.de>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   8baef6386baaefb776bdd09b5c7630cf057c51c6
commit: b13812ddea615b6507beef24f76540c0c1143c5c MIPS: Loongson2ef: Disable Loongson MMI instructions
date:   10 months ago
:::::: branch date: 2 hours ago
:::::: commit date: 10 months ago
config: mips-randconfig-m031-20210723 (attached as .config)
compiler: mips64-linux-gcc (GCC) 10.3.0

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

New smatch warnings:
drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c:1960 rvu_mbox_handler_npc_mcam_shift_entry() error: uninitialized symbol 'rc'.
drivers/net/ethernet/dec/tulip/de4x5.c:3173 dc2114x_autoconf() warn: inconsistent indenting
drivers/net/ethernet/dec/tulip/de4x5.c:4328 srom_infoleaf_info() warn: inconsistent indenting
drivers/net/ethernet/dec/tulip/de4x5.c:5228 de4x5_dbg_open() warn: argument 2 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/neterion/s2io.c:3041 tx_intr_handler() error: calling 'spin_unlock_irqrestore()' with bogus flags
drivers/net/ethernet/3com/3c574_cs.c:966 update_stats() warn: inconsistent indenting
drivers/net/ethernet/3com/3c574_cs.c:1067 el3_ioctl() warn: inconsistent indenting
arch/mips/loongson2ef/common/mem.c:20 prom_init_memory() warn: should 'memsize << 20' be a 64 bit type?
arch/mips/loongson2ef/common/mem.c:45 prom_init_memory() warn: should 'highmemsize << 20' be a 64 bit type?
drivers/scsi/pm8001/pm8001_init.c:199 pm8001_tasklet() error: we previously assumed 'pm8001_ha' could be null (see line 197)
drivers/clk/renesas/r7s9210-cpg-mssr.c:141 r7s9210_update_clk_table() error: uninitialized symbol 'index'.
drivers/net/ethernet/smsc/smc91x.c:550 smc_hardware_send_pkt() error: uninitialized symbol 'flags'.
drivers/scsi/bnx2fc/bnx2fc_fcoe.c:480 bnx2fc_rcv() error: double free of 'skb'
drivers/net/ethernet/3com/3c589_cs.c:732 media_check() warn: if statement not indented
drivers/net/wan/c101.c:392 c101_run() warn: 'winbase' not released on lines: 350.
drivers/mtd/devices/bcm47xxsflash.c:236 bcm47xxsflash_write() error: uninitialized symbol 'written'.

Old smatch warnings:
drivers/net/ethernet/dec/tulip/de4x5.c:5228 de4x5_dbg_open() warn: argument 3 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/dec/tulip/de4x5.c:5232 de4x5_dbg_open() warn: argument 2 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/dec/tulip/de4x5.c:5235 de4x5_dbg_open() warn: argument 2 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/dec/tulip/de4x5.c:5239 de4x5_dbg_open() warn: argument 2 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/dec/tulip/de4x5.c:5242 de4x5_dbg_open() warn: argument 2 to %8.8lx specifier is cast from pointer
drivers/net/ethernet/neterion/s2io.c:856 init_shared_mem() error: uninitialized symbol 'tmp_p_addr'.
drivers/net/ethernet/neterion/s2io.c:3061 tx_intr_handler() error: calling 'spin_unlock_irqrestore()' with bogus flags
drivers/net/ethernet/3com/3c574_cs.c:973 update_stats() warn: inconsistent indenting
drivers/net/ethernet/3com/3c574_cs.c:979 update_stats() warn: inconsistent indenting
arch/mips/loongson2ef/common/mem.c:22 prom_init_memory() warn: should 'memsize << 20' be a 64 bit type?
drivers/scsi/pm8001/pm8001_init.c:447 pm8001_ioremap() warn: argument 6 to %llx specifier is cast from pointer
drivers/clk/renesas/r7s9210-cpg-mssr.c:144 r7s9210_update_clk_table() error: uninitialized symbol 'index'.
drivers/clk/renesas/r7s9210-cpg-mssr.c:147 r7s9210_update_clk_table() error: uninitialized symbol 'index'.
drivers/clk/renesas/r7s9210-cpg-mssr.c:151 r7s9210_update_clk_table() error: uninitialized symbol 'index'.
drivers/net/ethernet/smsc/smc91x.c:2128 smc_request_attrib() warn: should '(64 * 1024) << (lp->io_shift)' be a 64 bit type?
drivers/net/ethernet/smsc/smc91x.c:2141 smc_release_attrib() warn: should '(64 * 1024) << (lp->io_shift)' be a 64 bit type?
drivers/net/ethernet/smsc/smc91x.c:2338 smc_drv_probe() warn: should '16 << (lp->io_shift)' be a 64 bit type?
drivers/net/ethernet/smsc/smc91x.c:2397 smc_drv_probe() warn: should '16 << (lp->io_shift)' be a 64 bit type?
drivers/net/ethernet/smsc/smc91x.c:2428 smc_drv_remove() warn: should '16 << (lp->io_shift)' be a 64 bit type?

vim +/rc +1960 drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c

651cd2652339d5 Sunil Goutham 2018-11-19  1893  
651cd2652339d5 Sunil Goutham 2018-11-19  1894  int rvu_mbox_handler_npc_mcam_shift_entry(struct rvu *rvu,
651cd2652339d5 Sunil Goutham 2018-11-19  1895  					  struct npc_mcam_shift_entry_req *req,
651cd2652339d5 Sunil Goutham 2018-11-19  1896  					  struct npc_mcam_shift_entry_rsp *rsp)
651cd2652339d5 Sunil Goutham 2018-11-19  1897  {
651cd2652339d5 Sunil Goutham 2018-11-19  1898  	struct npc_mcam *mcam = &rvu->hw->mcam;
651cd2652339d5 Sunil Goutham 2018-11-19  1899  	u16 pcifunc = req->hdr.pcifunc;
651cd2652339d5 Sunil Goutham 2018-11-19  1900  	u16 old_entry, new_entry;
a958dd59f9cef9 Sunil Goutham 2018-11-19  1901  	u16 index, cntr;
651cd2652339d5 Sunil Goutham 2018-11-19  1902  	int blkaddr, rc;
651cd2652339d5 Sunil Goutham 2018-11-19  1903  
651cd2652339d5 Sunil Goutham 2018-11-19  1904  	blkaddr = rvu_get_blkaddr(rvu, BLKTYPE_NPC, 0);
651cd2652339d5 Sunil Goutham 2018-11-19  1905  	if (blkaddr < 0)
651cd2652339d5 Sunil Goutham 2018-11-19  1906  		return NPC_MCAM_INVALID_REQ;
651cd2652339d5 Sunil Goutham 2018-11-19  1907  
651cd2652339d5 Sunil Goutham 2018-11-19  1908  	if (req->shift_count > NPC_MCAM_MAX_SHIFTS)
651cd2652339d5 Sunil Goutham 2018-11-19  1909  		return NPC_MCAM_INVALID_REQ;
651cd2652339d5 Sunil Goutham 2018-11-19  1910  
651cd2652339d5 Sunil Goutham 2018-11-19  1911  	mutex_lock(&mcam->lock);
651cd2652339d5 Sunil Goutham 2018-11-19  1912  	for (index = 0; index < req->shift_count; index++) {
651cd2652339d5 Sunil Goutham 2018-11-19  1913  		old_entry = req->curr_entry[index];
651cd2652339d5 Sunil Goutham 2018-11-19  1914  		new_entry = req->new_entry[index];
651cd2652339d5 Sunil Goutham 2018-11-19  1915  
651cd2652339d5 Sunil Goutham 2018-11-19  1916  		/* Check if both old and new entries are valid and
651cd2652339d5 Sunil Goutham 2018-11-19  1917  		 * does belong to this PFFUNC or not.
651cd2652339d5 Sunil Goutham 2018-11-19  1918  		 */
651cd2652339d5 Sunil Goutham 2018-11-19  1919  		rc = npc_mcam_verify_entry(mcam, pcifunc, old_entry);
651cd2652339d5 Sunil Goutham 2018-11-19  1920  		if (rc)
651cd2652339d5 Sunil Goutham 2018-11-19  1921  			break;
651cd2652339d5 Sunil Goutham 2018-11-19  1922  
651cd2652339d5 Sunil Goutham 2018-11-19  1923  		rc = npc_mcam_verify_entry(mcam, pcifunc, new_entry);
651cd2652339d5 Sunil Goutham 2018-11-19  1924  		if (rc)
651cd2652339d5 Sunil Goutham 2018-11-19  1925  			break;
651cd2652339d5 Sunil Goutham 2018-11-19  1926  
a958dd59f9cef9 Sunil Goutham 2018-11-19  1927  		/* new_entry should not have a counter mapped */
a958dd59f9cef9 Sunil Goutham 2018-11-19  1928  		if (mcam->entry2cntr_map[new_entry] != NPC_MCAM_INVALID_MAP) {
a958dd59f9cef9 Sunil Goutham 2018-11-19  1929  			rc = NPC_MCAM_PERM_DENIED;
a958dd59f9cef9 Sunil Goutham 2018-11-19  1930  			break;
a958dd59f9cef9 Sunil Goutham 2018-11-19  1931  		}
a958dd59f9cef9 Sunil Goutham 2018-11-19  1932  
651cd2652339d5 Sunil Goutham 2018-11-19  1933  		/* Disable the new_entry */
651cd2652339d5 Sunil Goutham 2018-11-19  1934  		npc_enable_mcam_entry(rvu, mcam, blkaddr, new_entry, false);
651cd2652339d5 Sunil Goutham 2018-11-19  1935  
651cd2652339d5 Sunil Goutham 2018-11-19  1936  		/* Copy rule from old entry to new entry */
651cd2652339d5 Sunil Goutham 2018-11-19  1937  		npc_copy_mcam_entry(rvu, mcam, blkaddr, old_entry, new_entry);
651cd2652339d5 Sunil Goutham 2018-11-19  1938  
a958dd59f9cef9 Sunil Goutham 2018-11-19  1939  		/* Copy counter mapping, if any */
a958dd59f9cef9 Sunil Goutham 2018-11-19  1940  		cntr = mcam->entry2cntr_map[old_entry];
a958dd59f9cef9 Sunil Goutham 2018-11-19  1941  		if (cntr != NPC_MCAM_INVALID_MAP) {
a958dd59f9cef9 Sunil Goutham 2018-11-19  1942  			npc_unmap_mcam_entry_and_cntr(rvu, mcam, blkaddr,
a958dd59f9cef9 Sunil Goutham 2018-11-19  1943  						      old_entry, cntr);
a958dd59f9cef9 Sunil Goutham 2018-11-19  1944  			npc_map_mcam_entry_and_cntr(rvu, mcam, blkaddr,
a958dd59f9cef9 Sunil Goutham 2018-11-19  1945  						    new_entry, cntr);
a958dd59f9cef9 Sunil Goutham 2018-11-19  1946  		}
a958dd59f9cef9 Sunil Goutham 2018-11-19  1947  
651cd2652339d5 Sunil Goutham 2018-11-19  1948  		/* Enable new_entry and disable old_entry */
651cd2652339d5 Sunil Goutham 2018-11-19  1949  		npc_enable_mcam_entry(rvu, mcam, blkaddr, new_entry, true);
651cd2652339d5 Sunil Goutham 2018-11-19  1950  		npc_enable_mcam_entry(rvu, mcam, blkaddr, old_entry, false);
651cd2652339d5 Sunil Goutham 2018-11-19  1951  	}
651cd2652339d5 Sunil Goutham 2018-11-19  1952  
651cd2652339d5 Sunil Goutham 2018-11-19  1953  	/* If shift has failed then report the failed index */
651cd2652339d5 Sunil Goutham 2018-11-19  1954  	if (index != req->shift_count) {
651cd2652339d5 Sunil Goutham 2018-11-19  1955  		rc = NPC_MCAM_PERM_DENIED;
651cd2652339d5 Sunil Goutham 2018-11-19  1956  		rsp->failed_entry_idx = index;
651cd2652339d5 Sunil Goutham 2018-11-19  1957  	}
651cd2652339d5 Sunil Goutham 2018-11-19  1958  
651cd2652339d5 Sunil Goutham 2018-11-19  1959  	mutex_unlock(&mcam->lock);
651cd2652339d5 Sunil Goutham 2018-11-19 @1960  	return rc;
651cd2652339d5 Sunil Goutham 2018-11-19  1961  }
7fbb3f238c09e8 Sunil Goutham 2018-11-19  1962  

:::::: The code at line 1960 was first introduced by commit
:::::: 651cd2652339d5223bab9802faaf0a7f14c870b7 octeontx2-af: MCAM entry installation support

:::::: TO: Sunil Goutham <sgoutham@marvell.com>
:::::: CC: David S. Miller <davem@davemloft.net>

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

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

only message in thread, other threads:[~2021-07-23  5:24 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-07-23  5:24 drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c:1960 rvu_mbox_handler_npc_mcam_shift_entry() error: uninitialized symbol 'rc' 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.