Commit 2d15d2c0 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Darrick J. Wong

xfs: make use of the l_targ field in struct xlog

Use the slightly shorter way to get at the buftarg for the log device
wherever we can in the log and log recovery code.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDave Chinner <dchinner@redhat.com>
Reviewed-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
parent abca1f33
...@@ -927,7 +927,7 @@ xfs_log_unmount_write(xfs_mount_t *mp) ...@@ -927,7 +927,7 @@ xfs_log_unmount_write(xfs_mount_t *mp)
* Or, if we are doing a forced umount (typically because of IO errors). * Or, if we are doing a forced umount (typically because of IO errors).
*/ */
if (mp->m_flags & XFS_MOUNT_NORECOVERY || if (mp->m_flags & XFS_MOUNT_NORECOVERY ||
xfs_readonly_buftarg(log->l_mp->m_logdev_targp)) { xfs_readonly_buftarg(log->l_targ)) {
ASSERT(mp->m_flags & XFS_MOUNT_RDONLY); ASSERT(mp->m_flags & XFS_MOUNT_RDONLY);
return 0; return 0;
} }
...@@ -1442,7 +1442,7 @@ xlog_alloc_log( ...@@ -1442,7 +1442,7 @@ xlog_alloc_log(
* having set it up properly. * having set it up properly.
*/ */
error = -ENOMEM; error = -ENOMEM;
bp = xfs_buf_alloc(mp->m_logdev_targp, XFS_BUF_DADDR_NULL, bp = xfs_buf_alloc(log->l_targ, XFS_BUF_DADDR_NULL,
BTOBB(log->l_iclog_size), XBF_NO_IOACCT); BTOBB(log->l_iclog_size), XBF_NO_IOACCT);
if (!bp) if (!bp)
goto out_free_log; goto out_free_log;
...@@ -1912,7 +1912,7 @@ xlog_sync( ...@@ -1912,7 +1912,7 @@ xlog_sync(
* synchronously here; for an internal log we can simply use the block * synchronously here; for an internal log we can simply use the block
* layer state machine for preflushes. * layer state machine for preflushes.
*/ */
if (log->l_mp->m_logdev_targp != log->l_mp->m_ddev_targp || split) { if (log->l_targ != log->l_mp->m_ddev_targp || split) {
xfs_blkdev_issue_flush(log->l_mp->m_ddev_targp); xfs_blkdev_issue_flush(log->l_mp->m_ddev_targp);
need_flush = false; need_flush = false;
} }
......
...@@ -134,7 +134,7 @@ xlog_get_bp( ...@@ -134,7 +134,7 @@ xlog_get_bp(
nbblks += log->l_sectBBsize; nbblks += log->l_sectBBsize;
nbblks = round_up(nbblks, log->l_sectBBsize); nbblks = round_up(nbblks, log->l_sectBBsize);
bp = xfs_buf_get_uncached(log->l_mp->m_logdev_targp, nbblks, 0); bp = xfs_buf_get_uncached(log->l_targ, nbblks, 0);
if (bp) if (bp)
xfs_buf_unlock(bp); xfs_buf_unlock(bp);
return bp; return bp;
...@@ -1505,7 +1505,7 @@ xlog_find_tail( ...@@ -1505,7 +1505,7 @@ xlog_find_tail(
* But... if the -device- itself is readonly, just skip this. * But... if the -device- itself is readonly, just skip this.
* We can't recover this device anyway, so it won't matter. * We can't recover this device anyway, so it won't matter.
*/ */
if (!xfs_readonly_buftarg(log->l_mp->m_logdev_targp)) if (!xfs_readonly_buftarg(log->l_targ))
error = xlog_clear_stale_blocks(log, tail_lsn); error = xlog_clear_stale_blocks(log, tail_lsn);
done: done:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment