include/linux/list.h:72:19: warning: storing the address of local variable 'pbufl' in '*&buf_33(D)->list.prev'

[kernel test robot <lkp@intel.com>]

Hi Shiraz,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   897d54018cc9aa97fd1529ca08a53b429d05a566
commit: fa0cf568fd76550c1ddb806c03a65a1a4a1ea909 RDMA/irdma: Add irdma Kconfig/Makefile and remove i40iw
date:   4 years, 11 months ago
config: mips-allyesconfig (https://download.01.org/0day-ci/archive/20260426/202604261334.fKf5nOTO-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260426/202604261334.fKf5nOTO-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Fixes: fa0cf568fd76 ("RDMA/irdma: Add irdma Kconfig/Makefile and remove i40iw")
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202604261334.fKf5nOTO-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/pci.h:32,
                    from drivers/infiniband/hw/irdma/osdep.h:6,
                    from drivers/infiniband/hw/irdma/puda.c:3:
   In function '__list_add',
       inlined from 'list_add' at include/linux/list.h:86:2,
       inlined from 'irdma_ieq_handle_partial' at drivers/infiniband/hw/irdma/puda.c:1384:2:
>> include/linux/list.h:72:19: warning: storing the address of local variable 'pbufl' in '*&buf_33(D)->list.prev' [-Wdangling-pointer=]
      72 |         new->prev = prev;
         |         ~~~~~~~~~~^~~~~~
   drivers/infiniband/hw/irdma/puda.c: In function 'irdma_ieq_handle_partial':
   drivers/infiniband/hw/irdma/puda.c:1377:26: note: 'pbufl' declared here
    1377 |         struct list_head pbufl; /* partial buffer list */
         |                          ^~~~~
   drivers/infiniband/hw/irdma/puda.c:1371:49: note: 'buf' declared here
    1371 |                          struct irdma_puda_buf *buf, u16 fpdu_len)
         |                          ~~~~~~~~~~~~~~~~~~~~~~~^~~


vim +72 include/linux/list.h

d7c816733d501b Kees Cook        2016-08-17  56  
^1da177e4c3f41 Linus Torvalds   2005-04-16  57  /*
^1da177e4c3f41 Linus Torvalds   2005-04-16  58   * Insert a new entry between two known consecutive entries.
^1da177e4c3f41 Linus Torvalds   2005-04-16  59   *
^1da177e4c3f41 Linus Torvalds   2005-04-16  60   * This is only for internal list manipulation where we know
^1da177e4c3f41 Linus Torvalds   2005-04-16  61   * the prev/next entries already!
^1da177e4c3f41 Linus Torvalds   2005-04-16  62   */
^1da177e4c3f41 Linus Torvalds   2005-04-16  63  static inline void __list_add(struct list_head *new,
^1da177e4c3f41 Linus Torvalds   2005-04-16  64  			      struct list_head *prev,
^1da177e4c3f41 Linus Torvalds   2005-04-16  65  			      struct list_head *next)
^1da177e4c3f41 Linus Torvalds   2005-04-16  66  {
d7c816733d501b Kees Cook        2016-08-17  67  	if (!__list_add_valid(new, prev, next))
d7c816733d501b Kees Cook        2016-08-17  68  		return;
d7c816733d501b Kees Cook        2016-08-17  69  
^1da177e4c3f41 Linus Torvalds   2005-04-16  70  	next->prev = new;
^1da177e4c3f41 Linus Torvalds   2005-04-16  71  	new->next = next;
^1da177e4c3f41 Linus Torvalds   2005-04-16 @72  	new->prev = prev;
1c97be677f72b3 Paul E. McKenney 2015-09-20  73  	WRITE_ONCE(prev->next, new);
^1da177e4c3f41 Linus Torvalds   2005-04-16  74  }
^1da177e4c3f41 Linus Torvalds   2005-04-16  75  
^1da177e4c3f41 Linus Torvalds   2005-04-16  76  /**
^1da177e4c3f41 Linus Torvalds   2005-04-16  77   * list_add - add a new entry
^1da177e4c3f41 Linus Torvalds   2005-04-16  78   * @new: new entry to be added
^1da177e4c3f41 Linus Torvalds   2005-04-16  79   * @head: list head to add it after
^1da177e4c3f41 Linus Torvalds   2005-04-16  80   *
^1da177e4c3f41 Linus Torvalds   2005-04-16  81   * Insert a new entry after the specified head.
^1da177e4c3f41 Linus Torvalds   2005-04-16  82   * This is good for implementing stacks.
^1da177e4c3f41 Linus Torvalds   2005-04-16  83   */
^1da177e4c3f41 Linus Torvalds   2005-04-16  84  static inline void list_add(struct list_head *new, struct list_head *head)
^1da177e4c3f41 Linus Torvalds   2005-04-16  85  {
^1da177e4c3f41 Linus Torvalds   2005-04-16 @86  	__list_add(new, head, head->next);
^1da177e4c3f41 Linus Torvalds   2005-04-16  87  }
199a9afc3dbe98 Dave Jones       2006-09-29  88  

:::::: The code at line 72 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2

:::::: TO: Linus Torvalds <torvalds@ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds@ppc970.osdl.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki