CC: [email protected]
CC: "Darrick J. Wong" <[email protected]>
TO: "Darrick J. Wong" <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux.git 
refactor-log-recovery
head:   2dcdc305cfc2b8b1beee483cf8ae5c674d9fac86
commit: 9c766c8019d8de7f1a65645f01b179209b3f7b3c [294/314] xfs: online repair 
of the realtime rmap btree
:::::: branch date: 12 hours ago
:::::: commit date: 12 hours ago

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


cppcheck warnings: (new ones prefixed by >>)

>> fs/xfs/scrub/rtrmap_repair.c:111:16: warning: Argument 'error==0' to 
>> function ASSERT is always 0 [constArgument]
     ASSERT(error == 0);
                  ^
   fs/xfs/scrub/rtrmap_repair.c:110:6: note: Assuming that condition 'error' is 
not redundant
    if (error)
        ^
   fs/xfs/scrub/rtrmap_repair.c:111:16: note: Argument 'error==0' to function 
ASSERT is always 0
     ASSERT(error == 0);
                  ^
   fs/xfs/scrub/rtrmap_repair.c:115:16: warning: Argument 'error==0' to 
function ASSERT is always 0 [constArgument]
     ASSERT(error == 0);
                  ^
   fs/xfs/scrub/rtrmap_repair.c:114:6: note: Assuming that condition 'error' is 
not redundant
    if (error)
        ^
   fs/xfs/scrub/rtrmap_repair.c:115:16: note: Argument 'error==0' to function 
ASSERT is always 0
     ASSERT(error == 0);
                  ^

# 
https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux.git/commit/?id=9c766c8019d8de7f1a65645f01b179209b3f7b3c
git remote add djwong-xfs 
https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux.git
git remote update djwong-xfs
git checkout 9c766c8019d8de7f1a65645f01b179209b3f7b3c
vim +111 fs/xfs/scrub/rtrmap_repair.c

9c766c8019d8de Darrick J. Wong 2020-03-06   88  
9c766c8019d8de Darrick J. Wong 2020-03-06   89  /* Compare two rtrmapbt 
extents. */
9c766c8019d8de Darrick J. Wong 2020-03-06   90  static int
9c766c8019d8de Darrick J. Wong 2020-03-06   91  xrep_rtrmap_extent_cmp(
9c766c8019d8de Darrick J. Wong 2020-03-06   92          const void              
        *a,
9c766c8019d8de Darrick J. Wong 2020-03-06   93          const void              
        *b)
9c766c8019d8de Darrick J. Wong 2020-03-06   94  {
9c766c8019d8de Darrick J. Wong 2020-03-06   95          const struct 
xrep_rtrmap_extent *ap = a;
9c766c8019d8de Darrick J. Wong 2020-03-06   96          const struct 
xrep_rtrmap_extent *bp = b;
9c766c8019d8de Darrick J. Wong 2020-03-06   97          struct xfs_rmap_irec    
        ar = {
9c766c8019d8de Darrick J. Wong 2020-03-06   98                  .rm_startblock  
        = ap->startblock,
9c766c8019d8de Darrick J. Wong 2020-03-06   99                  .rm_blockcount  
        = ap->blockcount,
9c766c8019d8de Darrick J. Wong 2020-03-06  100                  .rm_owner       
        = ap->owner,
9c766c8019d8de Darrick J. Wong 2020-03-06  101          };
9c766c8019d8de Darrick J. Wong 2020-03-06  102          struct xfs_rmap_irec    
        br = {
9c766c8019d8de Darrick J. Wong 2020-03-06  103                  .rm_startblock  
        = bp->startblock,
9c766c8019d8de Darrick J. Wong 2020-03-06  104                  .rm_blockcount  
        = bp->blockcount,
9c766c8019d8de Darrick J. Wong 2020-03-06  105                  .rm_owner       
        = bp->owner,
9c766c8019d8de Darrick J. Wong 2020-03-06  106          };
9c766c8019d8de Darrick J. Wong 2020-03-06  107          int                     
        error;
9c766c8019d8de Darrick J. Wong 2020-03-06  108  
9c766c8019d8de Darrick J. Wong 2020-03-06  109          error = 
xfs_rmap_irec_offset_unpack(ap->offset, &ar);
9c766c8019d8de Darrick J. Wong 2020-03-06  110          if (error)
9c766c8019d8de Darrick J. Wong 2020-03-06 @111                  ASSERT(error == 
0);
9c766c8019d8de Darrick J. Wong 2020-03-06  112  
9c766c8019d8de Darrick J. Wong 2020-03-06  113          error = 
xfs_rmap_irec_offset_unpack(bp->offset, &br);
9c766c8019d8de Darrick J. Wong 2020-03-06  114          if (error)
9c766c8019d8de Darrick J. Wong 2020-03-06  115                  ASSERT(error == 
0);
9c766c8019d8de Darrick J. Wong 2020-03-06  116  
9c766c8019d8de Darrick J. Wong 2020-03-06  117          return 
xfs_rmap_compare(&ar, &br);
9c766c8019d8de Darrick J. Wong 2020-03-06  118  }
9c766c8019d8de Darrick J. Wong 2020-03-06  119  

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

Reply via email to