aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDarrick J. Wong <darrick.wong@oracle.com>2016-08-02 22:29:32 -0400
committerDave Chinner <david@fromorbit.com>2016-08-02 22:29:32 -0400
commite127fafd1d3c46532903d530dfa69822d3d35d8a (patch)
treedc8180cbf11ad64a2f2cb0998faed4882bed6ebd
parent722e251770306ee325151b28e40b5d7e5497d687 (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.c12
-rw-r--r--fs/xfs/xfs_rmap_item.c12
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) ?