CC: [email protected]
CC: [email protected]
TO: Luc Van Oostenryck <[email protected]>
CC: Andrew Morton <[email protected]>
CC: Linux Memory Management List <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   8404c9fbc84b741f66cff7d4934a25dd2c344452
commit: d991bb1c8da842a2a0b9dc83b1005e655783f861 include/linux/compiler-gcc.h: 
sparse can do constant folding of __builtin_bswap*()
date:   5 days ago
:::::: branch date: 2 hours ago
:::::: commit date: 5 days ago
config: powerpc64-randconfig-m031-20210505 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.0

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

smatch warnings:
fs/ext4/fast_commit.c:1809 ext4_fc_replay_del_range() warn: should 
'__builtin_bswap32(((lrange->fc_lblk))) << sb->s_blocksize_bits' be a 64 bit 
type?
fs/ext4/fast_commit.c:1810 ext4_fc_replay_del_range() warn: should 
'__builtin_bswap32(((lrange->fc_len))) << sb->s_blocksize_bits' be a 64 bit 
type?

vim +1809 fs/ext4/fast_commit.c

8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1760  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1761  /* Replay DEL_RANGE tag */
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1762  static int
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1763  
ext4_fc_replay_del_range(struct super_block *sb, struct ext4_fc_tl *tl)
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1764  {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1765      struct inode *inode;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1766      struct 
ext4_fc_del_range *lrange;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1767      struct ext4_map_blocks 
map;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1768      ext4_lblk_t cur, 
remaining;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1769      int ret;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1770  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1771      lrange = (struct 
ext4_fc_del_range *)ext4_fc_tag_val(tl);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1772      cur = 
le32_to_cpu(lrange->fc_lblk);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1773      remaining = 
le32_to_cpu(lrange->fc_len);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1774  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1775      
trace_ext4_fc_replay(sb, EXT4_FC_TAG_DEL_RANGE,
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1776              
le32_to_cpu(lrange->fc_ino), cur, remaining);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1777  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1778      inode = ext4_iget(sb, 
le32_to_cpu(lrange->fc_ino), EXT4_IGET_NORMAL);
23dd561ad9eae0 Yi Li              2020-12-30  1779      if (IS_ERR(inode)) {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1780              jbd_debug(1, 
"Inode %d not found", le32_to_cpu(lrange->fc_ino));
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1781              return 0;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1782      }
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1783  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1784      ret = 
ext4_fc_record_modified_inode(sb, inode->i_ino);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1785  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1786      jbd_debug(1, 
"DEL_RANGE, inode %ld, lblk %d, len %d\n",
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1787                      
inode->i_ino, le32_to_cpu(lrange->fc_lblk),
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1788                      
le32_to_cpu(lrange->fc_len));
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1789      while (remaining > 0) {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1790              map.m_lblk = 
cur;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1791              map.m_len = 
remaining;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1792  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1793              ret = 
ext4_map_blocks(NULL, inode, &map, 0);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1794              if (ret < 0) {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1795                      
iput(inode);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1796                      return 
0;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1797              }
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1798              if (ret > 0) {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1799                      
remaining -= ret;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1800                      cur += 
ret;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1801                      
ext4_mb_mark_bb(inode->i_sb, map.m_pblk, map.m_len, 0);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1802              } else {
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1803                      
remaining -= map.m_len;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1804                      cur += 
map.m_len;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1805              }
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1806      }
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1807  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1808      ret = 
ext4_punch_hole(inode,
8016e29f4362e2 Harshad Shirwadkar 2020-10-15 @1809              
le32_to_cpu(lrange->fc_lblk) << sb->s_blocksize_bits,
8016e29f4362e2 Harshad Shirwadkar 2020-10-15 @1810              
le32_to_cpu(lrange->fc_len) <<  sb->s_blocksize_bits);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1811      if (ret)
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1812              jbd_debug(1, 
"ext4_punch_hole returned %d", ret);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1813      
ext4_ext_replay_shrink_inode(inode,
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1814              
i_size_read(inode) >> sb->s_blocksize_bits);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1815      
ext4_mark_inode_dirty(NULL, inode);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1816      iput(inode);
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1817  
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1818      return 0;
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1819  }
8016e29f4362e2 Harshad Shirwadkar 2020-10-15  1820  

:::::: The code at line 1809 was first introduced by commit
:::::: 8016e29f4362e285f0f7e38fadc61a5b7bdfdfa2 ext4: fast commit recovery path

:::::: TO: Harshad Shirwadkar <[email protected]>
:::::: CC: Theodore Ts'o <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to