CC: [email protected]
BCC: [email protected]
TO: Dave Chinner <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/dgc/linux-xfs.git 
xfs-5.19-compose
head:   d8fa98c66946d5b1e01531cf7cb50151817f6894
commit: fd920008784ead369e79c2be2f8d9cc736e306ca [15/43] xfs: Set up 
infrastructure for log attribute replay
:::::: branch date: 9 hours ago
:::::: commit date: 3 days ago
config: sparc64-randconfig-c003-20220506 
(https://download.01.org/0day-ci/archive/20220507/[email protected]/config)
compiler: sparc64-linux-gcc (GCC) 11.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Julia Lawall <[email protected]>


cocci warnings: (new ones prefixed by >>)
>> fs/xfs/xfs_attr_item.c:58:6-25: atomic_dec_and_test variation before object 
>> free at line 61.

vim +58 fs/xfs/xfs_attr_item.c

fd920008784ead Allison Henderson 2022-05-04  45  
fd920008784ead Allison Henderson 2022-05-04  46  /*
fd920008784ead Allison Henderson 2022-05-04  47   * Freeing the attrip requires 
that we remove it from the AIL if it has already
fd920008784ead Allison Henderson 2022-05-04  48   * been placed there. However, 
the ATTRI may not yet have been placed in the
fd920008784ead Allison Henderson 2022-05-04  49   * AIL when called by 
xfs_attri_release() from ATTRD processing due to the
fd920008784ead Allison Henderson 2022-05-04  50   * ordering of committed vs 
unpin operations in bulk insert operations. Hence
fd920008784ead Allison Henderson 2022-05-04  51   * the reference count to 
ensure only the last caller frees the ATTRI.
fd920008784ead Allison Henderson 2022-05-04  52   */
fd920008784ead Allison Henderson 2022-05-04  53  STATIC void
fd920008784ead Allison Henderson 2022-05-04  54  xfs_attri_release(
fd920008784ead Allison Henderson 2022-05-04  55         struct 
xfs_attri_log_item       *attrip)
fd920008784ead Allison Henderson 2022-05-04  56  {
fd920008784ead Allison Henderson 2022-05-04  57         
ASSERT(atomic_read(&attrip->attri_refcount) > 0);
fd920008784ead Allison Henderson 2022-05-04 @58         if 
(!atomic_dec_and_test(&attrip->attri_refcount))
fd920008784ead Allison Henderson 2022-05-04  59                 return;
fd920008784ead Allison Henderson 2022-05-04  60  
fd920008784ead Allison Henderson 2022-05-04 @61         
xfs_trans_ail_delete(&attrip->attri_item, 0);
fd920008784ead Allison Henderson 2022-05-04  62         
xfs_attri_item_free(attrip);
fd920008784ead Allison Henderson 2022-05-04  63  }
fd920008784ead Allison Henderson 2022-05-04  64  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to