diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2016-08-02 22:29:32 -0400 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2016-08-02 22:29:32 -0400 |
commit | e127fafd1d3c46532903d530dfa69822d3d35d8a (patch) | |
tree | dc8180cbf11ad64a2f2cb0998faed4882bed6ebd | |
parent | 722e251770306ee325151b28e40b5d7e5497d687 (diff) |
xfs: remove unnecessary parentheses from log redo item recovery functions
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
-rw-r--r-- | fs/xfs/xfs_extfree_item.c | 12 | ||||
-rw-r--r-- | fs/xfs/xfs_rmap_item.c | 12 |
2 files changed, 12 insertions, 12 deletions
diff --git a/fs/xfs/xfs_extfree_item.c b/fs/xfs/xfs_extfree_item.c index a8c461735ecb..d7bc14906af8 100644 --- a/fs/xfs/xfs_extfree_item.c +++ b/fs/xfs/xfs_extfree_item.c | |||
@@ -515,13 +515,13 @@ xfs_efi_recover( | |||
515 | * just toss the EFI. | 515 | * just toss the EFI. |
516 | */ | 516 | */ |
517 | for (i = 0; i < efip->efi_format.efi_nextents; i++) { | 517 | for (i = 0; i < efip->efi_format.efi_nextents; i++) { |
518 | extp = &(efip->efi_format.efi_extents[i]); | 518 | extp = &efip->efi_format.efi_extents[i]; |
519 | startblock_fsb = XFS_BB_TO_FSB(mp, | 519 | startblock_fsb = XFS_BB_TO_FSB(mp, |
520 | XFS_FSB_TO_DADDR(mp, extp->ext_start)); | 520 | XFS_FSB_TO_DADDR(mp, extp->ext_start)); |
521 | if ((startblock_fsb == 0) || | 521 | if (startblock_fsb == 0 || |
522 | (extp->ext_len == 0) || | 522 | extp->ext_len == 0 || |
523 | (startblock_fsb >= mp->m_sb.sb_dblocks) || | 523 | startblock_fsb >= mp->m_sb.sb_dblocks || |
524 | (extp->ext_len >= mp->m_sb.sb_agblocks)) { | 524 | extp->ext_len >= mp->m_sb.sb_agblocks) { |
525 | /* | 525 | /* |
526 | * This will pull the EFI from the AIL and | 526 | * This will pull the EFI from the AIL and |
527 | * free the memory associated with it. | 527 | * free the memory associated with it. |
@@ -539,7 +539,7 @@ xfs_efi_recover( | |||
539 | 539 | ||
540 | xfs_rmap_skip_owner_update(&oinfo); | 540 | xfs_rmap_skip_owner_update(&oinfo); |
541 | for (i = 0; i < efip->efi_format.efi_nextents; i++) { | 541 | for (i = 0; i < efip->efi_format.efi_nextents; i++) { |
542 | extp = &(efip->efi_format.efi_extents[i]); | 542 | extp = &efip->efi_format.efi_extents[i]; |
543 | error = xfs_trans_free_extent(tp, efdp, extp->ext_start, | 543 | error = xfs_trans_free_extent(tp, efdp, extp->ext_start, |
544 | extp->ext_len, &oinfo); | 544 | extp->ext_len, &oinfo); |
545 | if (error) | 545 | if (error) |
diff --git a/fs/xfs/xfs_rmap_item.c b/fs/xfs/xfs_rmap_item.c index 7e4743219430..2500f28689d5 100644 --- a/fs/xfs/xfs_rmap_item.c +++ b/fs/xfs/xfs_rmap_item.c | |||
@@ -454,7 +454,7 @@ xfs_rui_recover( | |||
454 | * just toss the RUI. | 454 | * just toss the RUI. |
455 | */ | 455 | */ |
456 | for (i = 0; i < ruip->rui_format.rui_nextents; i++) { | 456 | for (i = 0; i < ruip->rui_format.rui_nextents; i++) { |
457 | rmap = &(ruip->rui_format.rui_extents[i]); | 457 | rmap = &ruip->rui_format.rui_extents[i]; |
458 | startblock_fsb = XFS_BB_TO_FSB(mp, | 458 | startblock_fsb = XFS_BB_TO_FSB(mp, |
459 | XFS_FSB_TO_DADDR(mp, rmap->me_startblock)); | 459 | XFS_FSB_TO_DADDR(mp, rmap->me_startblock)); |
460 | switch (rmap->me_flags & XFS_RMAP_EXTENT_TYPE_MASK) { | 460 | switch (rmap->me_flags & XFS_RMAP_EXTENT_TYPE_MASK) { |
@@ -469,10 +469,10 @@ xfs_rui_recover( | |||
469 | op_ok = false; | 469 | op_ok = false; |
470 | break; | 470 | break; |
471 | } | 471 | } |
472 | if (!op_ok || (startblock_fsb == 0) || | 472 | if (!op_ok || startblock_fsb == 0 || |
473 | (rmap->me_len == 0) || | 473 | rmap->me_len == 0 || |
474 | (startblock_fsb >= mp->m_sb.sb_dblocks) || | 474 | startblock_fsb >= mp->m_sb.sb_dblocks || |
475 | (rmap->me_len >= mp->m_sb.sb_agblocks) || | 475 | rmap->me_len >= mp->m_sb.sb_agblocks || |
476 | (rmap->me_flags & ~XFS_RMAP_EXTENT_FLAGS)) { | 476 | (rmap->me_flags & ~XFS_RMAP_EXTENT_FLAGS)) { |
477 | /* | 477 | /* |
478 | * This will pull the RUI from the AIL and | 478 | * This will pull the RUI from the AIL and |
@@ -490,7 +490,7 @@ xfs_rui_recover( | |||
490 | rudp = xfs_trans_get_rud(tp, ruip); | 490 | rudp = xfs_trans_get_rud(tp, ruip); |
491 | 491 | ||
492 | for (i = 0; i < ruip->rui_format.rui_nextents; i++) { | 492 | for (i = 0; i < ruip->rui_format.rui_nextents; i++) { |
493 | rmap = &(ruip->rui_format.rui_extents[i]); | 493 | rmap = &ruip->rui_format.rui_extents[i]; |
494 | state = (rmap->me_flags & XFS_RMAP_EXTENT_UNWRITTEN) ? | 494 | state = (rmap->me_flags & XFS_RMAP_EXTENT_UNWRITTEN) ? |
495 | XFS_EXT_UNWRITTEN : XFS_EXT_NORM; | 495 | XFS_EXT_UNWRITTEN : XFS_EXT_NORM; |
496 | whichfork = (rmap->me_flags & XFS_RMAP_EXTENT_ATTR_FORK) ? | 496 | whichfork = (rmap->me_flags & XFS_RMAP_EXTENT_ATTR_FORK) ? |