drivers/scsi/bnx2fc/bnx2fc_hwi.c:638:16: warning: variable 'index' set but not used

Monday, 15 June 2020

Hi zhengbin,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   b3a9e3b9622ae10064826dccb4f7a52bd88c7407
commit: 84769706dde5881b8da0dc3da666f7443be95a8a scsi: bnx2fc: remove set but not used
variables 'task','port','orig_task'
date:   10 months ago
config: ia64-randconfig-r003-20200615 (attached as .config)
compiler: ia64-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
        git checkout 84769706dde5881b8da0dc3da666f7443be95a8a
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com&gt;

All warnings (new ones prefixed by >>, old ones prefixed by <<):

In file included from arch/ia64/include/asm/pgtable.h:154,
from include/linux/mm.h:99,
from arch/ia64/include/asm/uaccess.h:38,
from include/linux/uaccess.h:11,
from include/linux/crypto.h:21,
from include/crypto/hash.h:11,
from include/linux/uio.h:10,
from include/linux/socket.h:8,
from include/linux/skbuff.h:19,
from drivers/scsi/bnx2fc/bnx2fc.h:21,
from drivers/scsi/bnx2fc/bnx2fc_hwi.c:16:
arch/ia64/include/asm/mmu_context.h: In function 'reload_context':
arch/ia64/include/asm/mmu_context.h:137:41: warning: variable 'old_rr4' set but
not used [-Wunused-but-set-variable]
137 |  unsigned long rr0, rr1, rr2, rr3, rr4, old_rr4;
|                                         ^~~~~~~
drivers/scsi/bnx2fc/bnx2fc_hwi.c: In function 'bnx2fc_process_unsol_compl':
<<                  from drivers/scsi/bnx2fc/bnx2fc_hwi.c:16:
...
> drivers/scsi/bnx2fc/bnx2fc_hwi.c:638:16: warning: variable
'index' set but not used [-Wunused-but-set-variable] 638 |  int task_idx,
index;
|                ^~~~~
<<                  from drivers/scsi/bnx2fc/bnx2fc_hwi.c:16:
...
> drivers/scsi/bnx2fc/bnx2fc_hwi.c:638:6: warning: variable
'task_idx' set but not used [-Wunused-but-set-variable] 638 |  int task_idx,
index;
|      ^~~~~~~~
drivers/scsi/bnx2fc/bnx2fc_hwi.c: In function 'bnx2fc_init_seq_cleanup_task':
drivers/scsi/bnx2fc/bnx2fc_hwi.c:1466:21: warning: variable 'index' set but not
used [-Wunused-but-set-variable]
1466 |  int orig_task_idx, index;
|                     ^~~~~
<<                  from drivers/scsi/bnx2fc/bnx2fc_hwi.c:16:
...
> drivers/scsi/bnx2fc/bnx2fc_hwi.c:1466:6: warning: variable
'orig_task_idx' set but not used [-Wunused-but-set-variable] 1466 |  int
orig_task_idx, index;
|      ^~~~~~~~~~~~~
drivers/scsi/bnx2fc/bnx2fc_hwi.c: In function 'bnx2fc_allocate_hash_table':
drivers/scsi/bnx2fc/bnx2fc_hwi.c:2046:7: warning: variable 'hi' set but not used
[-Wunused-but-set-variable]
2046 |   u32 hi;
|       ^~
drivers/scsi/bnx2fc/bnx2fc_hwi.c:2045:7: warning: variable 'lo' set but not used
[-Wunused-but-set-variable]
2045 |   u32 lo;
|       ^~

vim +/index +638 drivers/scsi/bnx2fc/bnx2fc_hwi.c

853e2bd2103aaa Bhanu Gollapudi         2011-02-04  625  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  626  static void
bnx2fc_process_unsol_compl(struct bnx2fc_rport *tgt, u16 wqe)
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  627  {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  628  	u8 num_rq;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  629  	struct fcoe_err_report_entry
*err_entry;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  630  	unsigned char *rq_data;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  631  	unsigned char *buf = NULL,
*buf1;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  632  	int i;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  633  	u16 xid;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  634  	u32 frame_len, len;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  635  	struct bnx2fc_cmd *io_req =
NULL;
aea71a024914e8 Bhanu Prakash Gollapudi 2011-07-26  636  	struct bnx2fc_interface
*interface = tgt->port->priv;
aea71a024914e8 Bhanu Prakash Gollapudi 2011-07-26  637  	struct bnx2fc_hba *hba =
interface->hba;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04 @638  	int task_idx, index;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  639  	int rc = 0;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  640  	u64 err_warn_bit_map;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  641  	u8 err_warn = 0xff;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  642  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  643  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  644  	BNX2FC_TGT_DBG(tgt, "Entered
UNSOL COMPLETION wqe = 0x%x\n", wqe);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  645  	switch (wqe &
FCOE_UNSOLICITED_CQE_SUBTYPE) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  646  	case
FCOE_UNSOLICITED_FRAME_CQE_TYPE:
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  647  		frame_len = (wqe &
FCOE_UNSOLICITED_CQE_PKT_LEN) >>
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  648  			    
FCOE_UNSOLICITED_CQE_PKT_LEN_SHIFT;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  649  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  650  		num_rq = (frame_len +
BNX2FC_RQ_BUF_SZ - 1) / BNX2FC_RQ_BUF_SZ;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  651  
686959736a8543 Nithin Sujir            2011-03-17  652 
		spin_lock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  653  		rq_data = (unsigned char
*)bnx2fc_get_next_rqe(tgt, num_rq);
686959736a8543 Nithin Sujir            2011-03-17  654 
		spin_unlock_bh(&tgt->tgt_lock);
686959736a8543 Nithin Sujir            2011-03-17  655  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  656  		if (rq_data) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  657  			buf = rq_data;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  658  		} else {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  659  			buf1 = buf = kmalloc((num_rq *
BNX2FC_RQ_BUF_SZ),
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  660  					      GFP_ATOMIC);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  661  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  662  			if (!buf1) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  663  				BNX2FC_TGT_DBG(tgt,
"Memory alloc failure\n");
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  664  				break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  665  			}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  666  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  667  			for (i = 0; i < num_rq; i++)
{
686959736a8543 Nithin Sujir            2011-03-17  668 
				spin_lock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  669  				rq_data = (unsigned char *)
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  670  					   bnx2fc_get_next_rqe(tgt,
1);
686959736a8543 Nithin Sujir            2011-03-17  671 
				spin_unlock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  672  				len = BNX2FC_RQ_BUF_SZ;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  673  				memcpy(buf1, rq_data, len);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  674  				buf1 += len;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  675  			}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  676  		}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  677 
		bnx2fc_process_l2_frame_compl(tgt, buf, frame_len,
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  678  					      FC_XID_UNKNOWN);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  679  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  680  		if (buf != rq_data)
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  681  			kfree(buf);
686959736a8543 Nithin Sujir            2011-03-17  682 
		spin_lock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  683  		bnx2fc_return_rqe(tgt, num_rq);
686959736a8543 Nithin Sujir            2011-03-17  684 
		spin_unlock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  685  		break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  686  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  687  	case
FCOE_ERROR_DETECTION_CQE_TYPE:
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  688  		/*
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  689  		 * In case of error reporting
CQE a single RQ entry
686959736a8543 Nithin Sujir            2011-03-17  690  		 * is consumed.
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  691  		 */
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  692 
		spin_lock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  693  		num_rq = 1;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  694  		err_entry = (struct
fcoe_err_report_entry *)
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  695  			     bnx2fc_get_next_rqe(tgt,
1);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  696  		xid =
err_entry->fc_hdr.ox_id;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  697  		BNX2FC_TGT_DBG(tgt, "Unsol
Error Frame OX_ID = 0x%x\n", xid);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  698  		BNX2FC_TGT_DBG(tgt,
"err_warn_bitmap = %08x:%08x\n",
619c5cb6885b93 Vlad Zolotarov          2011-06-14  699 
			err_entry->data.err_warn_bitmap_hi,
619c5cb6885b93 Vlad Zolotarov          2011-06-14  700 
			err_entry->data.err_warn_bitmap_lo);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  701  		BNX2FC_TGT_DBG(tgt,
"buf_offsets - tx = 0x%x, rx = 0x%x\n",
619c5cb6885b93 Vlad Zolotarov          2011-06-14  702  			err_entry->data.tx_buf_off,
err_entry->data.rx_buf_off);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  703  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  704  
0eb43b4bb081a1 Bhanu Prakash Gollapudi 2013-04-22  705  		if (xid > hba->max_xid) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  706  			BNX2FC_TGT_DBG(tgt,
"xid(0x%x) out of FW range\n",
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  707  				   xid);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  708  			goto ret_err_rqe;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  709  		}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  710  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  711  		task_idx = xid /
BNX2FC_TASKS_PER_PAGE;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  712  		index = xid %
BNX2FC_TASKS_PER_PAGE;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  713  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  714  		io_req = (struct bnx2fc_cmd
*)hba->cmd_mgr->cmds[xid];
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  715  		if (!io_req)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  716  			goto ret_err_rqe;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  717  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  718  		if (io_req->cmd_type !=
BNX2FC_SCSI_CMD) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  719  			printk(KERN_ERR PFX
"err_warn: Not a SCSI cmd\n");
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  720  			goto ret_err_rqe;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  721  		}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  722  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  723  		if
(test_and_clear_bit(BNX2FC_FLAG_IO_CLEANUP,
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  724  				      
&io_req->req_flags)) {
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  725  			BNX2FC_IO_DBG(io_req,
"unsol_err: cleanup in "
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  726  					    "progress.. ignore
unsol err\n");
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  727  			goto ret_err_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  728  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  729  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  730  		err_warn_bit_map = (u64)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  731 
			((u64)err_entry->data.err_warn_bitmap_hi << 32) |
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  732 
			(u64)err_entry->data.err_warn_bitmap_lo;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  733  		for (i = 0; i <
BNX2FC_NUM_ERR_BITS; i++) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  734  			if (err_warn_bit_map &
(u64)((u64)1 << i)) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  735  				err_warn = i;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  736  				break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  737  			}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  738  		}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  739  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  740  		/*
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  741  		 * If ABTS is already in
progress, and FW error is
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  742  		 * received after that, do not
cancel the timeout_work
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  743  		 * and let the error recovery
continue by explicitly
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  744  		 * logging out the target, when
the ABTS eventually
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  745  		 * times out.
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  746  		 */
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  747  		if
(test_bit(BNX2FC_FLAG_ISSUE_ABTS, &io_req->req_flags)) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  748  			printk(KERN_ERR PFX
"err_warn: io_req (0x%x) already "
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  749  					    "in ABTS
processing\n", xid);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  750  			goto ret_err_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  751  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  752  		BNX2FC_TGT_DBG(tgt, "err =
0x%x\n", err_warn);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  753  		if (tgt->dev_type !=
TYPE_TAPE)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  754  			goto skip_rec;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  755  		switch (err_warn) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  756  		case
FCOE_ERROR_CODE_REC_TOV_TIMER_EXPIRATION:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  757  		case
FCOE_ERROR_CODE_DATA_OOO_RO:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  758  		case
FCOE_ERROR_CODE_COMMON_INCORRECT_SEQ_CNT:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  759  		case
FCOE_ERROR_CODE_DATA_SOFI3_SEQ_ACTIVE_SET:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  760  		case
FCOE_ERROR_CODE_FCP_RSP_OPENED_SEQ:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  761  		case
FCOE_ERROR_CODE_DATA_SOFN_SEQ_ACTIVE_RESET:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  762  			BNX2FC_TGT_DBG(tgt, "REC
TOV popped for xid - 0x%x\n",
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  763  				   xid);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  764 
			memcpy(&io_req->err_entry, err_entry,
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  765  			       sizeof(struct
fcoe_err_report_entry));
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  766  			if
(!test_bit(BNX2FC_FLAG_SRR_SENT,
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  767  				     
&io_req->req_flags)) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  768 
				spin_unlock_bh(&tgt->tgt_lock);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  769  				rc = bnx2fc_send_rec(io_req);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  770 
				spin_lock_bh(&tgt->tgt_lock);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  771  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  772  				if (rc)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  773  					goto skip_rec;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  774  			} else
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  775  				printk(KERN_ERR PFX "SRR
in progress\n");
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  776  			goto ret_err_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  777  			break;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  778  		default:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  779  			break;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  780  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  781  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  782  skip_rec:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  783  		set_bit(BNX2FC_FLAG_ISSUE_ABTS,
&io_req->req_flags);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  784  		/*
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  785  		 * Cancel the timeout_work, as
we received IO
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  786  		 * completion with FW error.
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  787  		 */
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  788  		if
(cancel_delayed_work(&io_req->timeout_work))
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  789 
			kref_put(&io_req->refcount, bnx2fc_cmd_release);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  790  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  791  		rc =
bnx2fc_initiate_abts(io_req);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  792  		if (rc != SUCCESS) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  793  			printk(KERN_ERR PFX
"err_warn: initiate_abts "
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  794  				"failed xid = 0x%x. issue
cleanup\n",
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  795  				io_req->xid);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  796 
			bnx2fc_initiate_cleanup(io_req);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  797  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  798  ret_err_rqe:
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  799  		bnx2fc_return_rqe(tgt, 1);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  800 
		spin_unlock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  801  		break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  802  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  803  	case
FCOE_WARNING_DETECTION_CQE_TYPE:
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  804  		/*
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  805  		 *In case of warning reporting
CQE a single RQ entry
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  806  		 * is consumes.
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  807  		 */
686959736a8543 Nithin Sujir            2011-03-17  808 
		spin_lock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  809  		num_rq = 1;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  810  		err_entry = (struct
fcoe_err_report_entry *)
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  811  			     bnx2fc_get_next_rqe(tgt,
1);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  812  		xid =
cpu_to_be16(err_entry->fc_hdr.ox_id);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  813  		BNX2FC_TGT_DBG(tgt, "Unsol
Warning Frame OX_ID = 0x%x\n", xid);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  814  		BNX2FC_TGT_DBG(tgt,
"err_warn_bitmap = %08x:%08x",
619c5cb6885b93 Vlad Zolotarov          2011-06-14  815 
			err_entry->data.err_warn_bitmap_hi,
619c5cb6885b93 Vlad Zolotarov          2011-06-14  816 
			err_entry->data.err_warn_bitmap_lo);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  817  		BNX2FC_TGT_DBG(tgt,
"buf_offsets - tx = 0x%x, rx = 0x%x",
619c5cb6885b93 Vlad Zolotarov          2011-06-14  818  			err_entry->data.tx_buf_off,
err_entry->data.rx_buf_off);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  819  
0eb43b4bb081a1 Bhanu Prakash Gollapudi 2013-04-22  820  		if (xid > hba->max_xid) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  821  			BNX2FC_TGT_DBG(tgt,
"xid(0x%x) out of FW range\n", xid);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  822  			goto ret_warn_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  823  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  824  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  825  		err_warn_bit_map = (u64)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  826 
			((u64)err_entry->data.err_warn_bitmap_hi << 32) |
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  827 
			(u64)err_entry->data.err_warn_bitmap_lo;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  828  		for (i = 0; i <
BNX2FC_NUM_ERR_BITS; i++) {
d0c0d902339249 Colin Ian King          2019-05-04  829  			if (err_warn_bit_map &
((u64)1 << i)) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  830  				err_warn = i;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  831  				break;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  832  			}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  833  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  834  		BNX2FC_TGT_DBG(tgt, "warn =
0x%x\n", err_warn);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  835  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  836  		task_idx = xid /
BNX2FC_TASKS_PER_PAGE;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  837  		index = xid %
BNX2FC_TASKS_PER_PAGE;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  838  		io_req = (struct bnx2fc_cmd
*)hba->cmd_mgr->cmds[xid];
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  839  		if (!io_req)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  840  			goto ret_warn_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  841  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  842  		if (io_req->cmd_type !=
BNX2FC_SCSI_CMD) {
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  843  			printk(KERN_ERR PFX
"err_warn: Not a SCSI cmd\n");
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  844  			goto ret_warn_rqe;
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  845  		}
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  846  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  847 
		memcpy(&io_req->err_entry, err_entry,
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  848  		       sizeof(struct
fcoe_err_report_entry));
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  849  
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  850  		if (err_warn ==
FCOE_ERROR_CODE_REC_TOV_TIMER_EXPIRATION)
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  851  			/* REC_TOV is not a warning
code */
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  852  			BUG_ON(1);
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  853  		else
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  854  			BNX2FC_TGT_DBG(tgt,
"Unsolicited warning\n");
7b594769120b43 Bhanu Prakash Gollapudi 2011-07-27  855  ret_warn_rqe:
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  856  		bnx2fc_return_rqe(tgt, 1);
686959736a8543 Nithin Sujir            2011-03-17  857 
		spin_unlock_bh(&tgt->tgt_lock);
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  858  		break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  859  
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  860  	default:
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  861  		printk(KERN_ERR PFX "Unsol
Compl: Invalid CQE Subtype\n");
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  862  		break;
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  863  	}
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  864  }
853e2bd2103aaa Bhanu Gollapudi         2011-02-04  865  

:::::: The code at line 638 was first introduced by commit
:::::: 853e2bd2103aaa91d1ba1c0b57ba17628d836f03 [SCSI] bnx2fc: Broadcom FCoE offload
driver

:::::: TO: Bhanu Gollapudi <bprakash(a)broadcom.com&gt;
:::::: CC: James Bottomley <James.Bottomley(a)suse.de&gt;

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

2022

2021

2020

2019