mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* drivers/net/ethernet/marvell/octeontx2/af/cgx.c:622 cgx_lmac_enadis_rx_pause_fwding() error: uninitialized symbol 'rx_pause'.
@ 2022-05-23 13:16 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2022-05-23 13:16 UTC (permalink / raw)
  To: kbuild, Hariprasad Kelam
  Cc: lkp, kbuild-all, linux-kernel, Sunil Kovvuri Goutham

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   eaea45fc0e7b6ae439526b4a41d91230c8517336
commit: e740003874edc13e468d19233f99787bedb4bb8e octeontx2-af: Flow control resource management
config: s390-randconfig-m031-20220522 (https://download.01.org/0day-ci/archive/20220522/202205222253.Mj6ywHyJ-lkp@intel.com/config)
compiler: s390-linux-gcc (GCC) 11.3.0

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

smatch warnings:
drivers/net/ethernet/marvell/octeontx2/af/cgx.c:622 cgx_lmac_enadis_rx_pause_fwding() error: uninitialized symbol 'rx_pause'.

vim +/rx_pause +622 drivers/net/ethernet/marvell/octeontx2/af/cgx.c

5d9b976d4480dc Sunil Goutham    2019-11-14  602  void cgx_lmac_enadis_rx_pause_fwding(void *cgxd, int lmac_id, bool enable)
5d9b976d4480dc Sunil Goutham    2019-11-14  603  {
5d9b976d4480dc Sunil Goutham    2019-11-14  604  	struct cgx *cgx = cgxd;
e740003874edc1 Hariprasad Kelam 2022-02-09  605  	u8 rx_pause, tx_pause;
e740003874edc1 Hariprasad Kelam 2022-02-09  606  	bool is_pfc_enabled;
e740003874edc1 Hariprasad Kelam 2022-02-09  607  	struct lmac *lmac;
5d9b976d4480dc Sunil Goutham    2019-11-14  608  	u64 cfg;
5d9b976d4480dc Sunil Goutham    2019-11-14  609  
5d9b976d4480dc Sunil Goutham    2019-11-14  610  	if (!cgx)
5d9b976d4480dc Sunil Goutham    2019-11-14  611  		return;
5d9b976d4480dc Sunil Goutham    2019-11-14  612  
e740003874edc1 Hariprasad Kelam 2022-02-09  613  	lmac = lmac_pdata(lmac_id, cgx);
e740003874edc1 Hariprasad Kelam 2022-02-09  614  	if (!lmac)
e740003874edc1 Hariprasad Kelam 2022-02-09  615  		return;
e740003874edc1 Hariprasad Kelam 2022-02-09  616  
e740003874edc1 Hariprasad Kelam 2022-02-09  617  	/* Pause frames are not enabled just return */
e740003874edc1 Hariprasad Kelam 2022-02-09  618  	if (!bitmap_weight(lmac->rx_fc_pfvf_bmap.bmap, lmac->rx_fc_pfvf_bmap.max))
e740003874edc1 Hariprasad Kelam 2022-02-09  619  		return;
e740003874edc1 Hariprasad Kelam 2022-02-09  620  
e740003874edc1 Hariprasad Kelam 2022-02-09  621  	cgx_lmac_get_pause_frm_status(cgx, lmac_id, &rx_pause, &tx_pause);

The first two returns in cgx_lmac_get_pause_frm_status() don't initialize
rx or tx pause.

e740003874edc1 Hariprasad Kelam 2022-02-09 @622  	is_pfc_enabled = rx_pause ? false : true;
                                                                         ^^^^^^^^

e740003874edc1 Hariprasad Kelam 2022-02-09  623  
5d9b976d4480dc Sunil Goutham    2019-11-14  624  	if (enable) {
e740003874edc1 Hariprasad Kelam 2022-02-09  625  		if (!is_pfc_enabled) {
5d9b976d4480dc Sunil Goutham    2019-11-14  626  			cfg = cgx_read(cgx, lmac_id, CGXX_GMP_GMI_RXX_FRM_CTL);
5d9b976d4480dc Sunil Goutham    2019-11-14  627  			cfg |= CGX_GMP_GMI_RXX_FRM_CTL_CTL_BCK;
5d9b976d4480dc Sunil Goutham    2019-11-14  628  			cgx_write(cgx, lmac_id, CGXX_GMP_GMI_RXX_FRM_CTL, cfg);
5d9b976d4480dc Sunil Goutham    2019-11-14  629  
5d9b976d4480dc Sunil Goutham    2019-11-14  630  			cfg = cgx_read(cgx, lmac_id, CGXX_SMUX_RX_FRM_CTL);
5d9b976d4480dc Sunil Goutham    2019-11-14  631  			cfg |= CGX_SMUX_RX_FRM_CTL_CTL_BCK;

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp


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

only message in thread, other threads:[~2022-05-23 13:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-23 13:16 drivers/net/ethernet/marvell/octeontx2/af/cgx.c:622 cgx_lmac_enadis_rx_pause_fwding() error: uninitialized symbol 'rx_pause' Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®