lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202108010828.KHzoyX7e-lkp@intel.com>
Date:   Sun, 1 Aug 2021 08:35:40 +0800
From:   kernel test robot <lkp@...el.com>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
        Jakub Kicinski <kuba@...nel.org>,
        Matthieu Baerts <matthieu.baerts@...sares.net>
Subject: drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:993:44: warning: variable 'size6'
 set but not used

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   c7d102232649226a69dddd58a4942cf13cff4f7c
commit: 010b430d5df556d5d232e3751ac691ba9e88c041 mptcp: MPTCP_IPV6 should depend on IPV6 instead of selecting it
date:   9 months ago
config: ia64-randconfig-r014-20210801 (attached as .config)
compiler: ia64-linux-gcc (GCC) 10.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
        # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=010b430d5df556d5d232e3751ac691ba9e88c041
        git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
        git fetch --no-tags linus master
        git checkout 010b430d5df556d5d232e3751ac691ba9e88c041
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross ARCH=ia64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   In file included from arch/ia64/include/asm/pgtable.h:154,
                    from include/linux/pgtable.h:6,
                    from arch/ia64/include/asm/uaccess.h:40,
                    from include/linux/uaccess.h:9,
                    from include/linux/sched/task.h:11,
                    from include/linux/sched/signal.h:9,
                    from include/linux/rcuwait.h:6,
                    from include/linux/percpu-rwsem.h:7,
                    from include/linux/fs.h:33,
                    from include/linux/seq_file.h:11,
                    from include/scsi/scsi_host.h:10,
                    from drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:19:
   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/cxgbi/cxgb4i/cxgb4i.c: In function 'csk_act_open_retry_timer':
>> drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:993:44: warning: variable 'size6' set but not used [-Wunused-but-set-variable]
     993 |  int t4 = is_t4(lldi->adapter_type), size, size6;
         |                                            ^~~~~
   drivers/scsi/cxgbi/cxgb4i/cxgb4i.c: In function 'init_act_open':
   drivers/scsi/cxgbi/cxgb4i/cxgb4i.c:1774:21: warning: variable 'size6' set but not used [-Wunused-but-set-variable]
    1774 |  unsigned int size, size6;
         |                     ^~~~~


vim +/size6 +993 drivers/scsi/cxgbi/cxgb4i/cxgb4i.c

7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   985  
cd07f958e88250 Kees Cook        2017-08-28   986  static void csk_act_open_retry_timer(struct timer_list *t)
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   987  {
001586a737ee8c Anish Bhatt      2014-10-15   988  	struct sk_buff *skb = NULL;
cd07f958e88250 Kees Cook        2017-08-28   989  	struct cxgbi_sock *csk = from_timer(csk, t, retry_timer);
3bd3e8bf6250f3 Karen Xie        2013-05-29   990  	struct cxgb4_lld_info *lldi = cxgbi_cdev_priv(csk->cdev);
759a0cc5a3e1bc Anish Bhatt      2014-07-17   991  	void (*send_act_open_func)(struct cxgbi_sock *, struct sk_buff *,
759a0cc5a3e1bc Anish Bhatt      2014-07-17   992  				   struct l2t_entry *);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  @993  	int t4 = is_t4(lldi->adapter_type), size, size6;
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   994  
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   995  	log_debug(1 << CXGBI_DBG_TOE | 1 << CXGBI_DBG_SOCK,
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   996  		"csk 0x%p,%u,0x%lx,%u.\n",
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   997  		csk, csk->state, csk->flags, csk->tid);
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   998  
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16   999  	cxgbi_sock_get(csk);
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1000  	spin_lock_bh(&csk->lock);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1001  
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1002  	if (t4) {
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1003  		size = sizeof(struct cpl_act_open_req);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1004  		size6 = sizeof(struct cpl_act_open_req6);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1005  	} else {
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1006  		size = sizeof(struct cpl_t5_act_open_req);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1007  		size6 = sizeof(struct cpl_t5_act_open_req6);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1008  	}
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1009  
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1010  	if (csk->csk_family == AF_INET) {
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1011  		send_act_open_func = send_act_open_req;
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1012  		skb = alloc_wr(size, 0, GFP_ATOMIC);
f42bb57c61fd21 Anish Bhatt      2014-10-14  1013  #if IS_ENABLED(CONFIG_IPV6)
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1014  	} else {
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1015  		send_act_open_func = send_act_open_req6;
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1016  		skb = alloc_wr(size6, 0, GFP_ATOMIC);
f42bb57c61fd21 Anish Bhatt      2014-10-14  1017  #endif
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1018  	}
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1019  
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1020  	if (!skb)
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1021  		cxgbi_sock_fail_act_open(csk, -ENOMEM);
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1022  	else {
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1023  		skb->sk = (struct sock *)csk;
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1024  		t4_set_arp_err_handler(skb, csk,
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1025  				       cxgbi_sock_act_open_req_arp_failure);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1026  		send_act_open_func(csk, skb, csk->l2t);
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1027  	}
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1028  
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1029  	spin_unlock_bh(&csk->lock);
7b36b6e03b0d6c kxie@...lsio.com 2010-08-16  1030  	cxgbi_sock_put(csk);
759a0cc5a3e1bc Anish Bhatt      2014-07-17  1031  

:::::: The code at line 993 was first introduced by commit
:::::: 759a0cc5a3e1bc2cc48fa3c0b91bdcad8b8f87d6 cxgb4i: Add ipv6 code to driver, call into libcxgbi ipv6 api

:::::: TO: Anish Bhatt <anish@...lsio.com>
:::::: CC: David S. Miller <davem@...emloft.net>

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

Download attachment ".config.gz" of type "application/gzip" (42441 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ