Commit f52edf6c authored by Darrick J. Wong's avatar Darrick J. Wong

Merge tag 'unit-conversion-cleanups-5.14_2021-06-03' of...

Merge tag 'unit-conversion-cleanups-5.14_2021-06-03' of https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux into xfs-5.14-merge2

xfs: various unit conversions

Crafting the realtime file extent size hint fixes revealed various
opportunities to clean up unit conversions, so now that gets its own
series.

* tag 'unit-conversion-cleanups-5.14_2021-06-03' of https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfs-linux:
  xfs: remove unnecessary shifts
  xfs: clean up open-coded fs block unit conversions
parents 9ba0889e 20bd8e63
...@@ -612,7 +612,7 @@ xfs_inode_validate_extsize( ...@@ -612,7 +612,7 @@ xfs_inode_validate_extsize(
*/ */
if (rt_flag) if (rt_flag)
blocksize_bytes = mp->m_sb.sb_rextsize << mp->m_sb.sb_blocklog; blocksize_bytes = XFS_FSB_TO_B(mp, mp->m_sb.sb_rextsize);
else else
blocksize_bytes = mp->m_sb.sb_blocksize; blocksize_bytes = mp->m_sb.sb_blocksize;
......
...@@ -945,7 +945,7 @@ xfs_flush_unmap_range( ...@@ -945,7 +945,7 @@ xfs_flush_unmap_range(
xfs_off_t rounding, start, end; xfs_off_t rounding, start, end;
int error; int error;
rounding = max_t(xfs_off_t, 1 << mp->m_sb.sb_blocklog, PAGE_SIZE); rounding = max_t(xfs_off_t, mp->m_sb.sb_blocksize, PAGE_SIZE);
start = round_down(offset, rounding); start = round_down(offset, rounding);
end = round_up(offset + len, rounding) - 1; end = round_up(offset + len, rounding) - 1;
...@@ -1053,9 +1053,9 @@ xfs_prepare_shift( ...@@ -1053,9 +1053,9 @@ xfs_prepare_shift(
* extent (after split) during the shift and corrupt the file. Start * extent (after split) during the shift and corrupt the file. Start
* with the block just prior to the start to stabilize the boundary. * with the block just prior to the start to stabilize the boundary.
*/ */
offset = round_down(offset, 1 << mp->m_sb.sb_blocklog); offset = round_down(offset, mp->m_sb.sb_blocksize);
if (offset) if (offset)
offset -= (1 << mp->m_sb.sb_blocklog); offset -= mp->m_sb.sb_blocksize;
/* /*
* Writeback and invalidate cache for the remainder of the file as we're * Writeback and invalidate cache for the remainder of the file as we're
......
...@@ -543,7 +543,7 @@ xfs_stat_blksize( ...@@ -543,7 +543,7 @@ xfs_stat_blksize(
* always return the realtime extent size. * always return the realtime extent size.
*/ */
if (XFS_IS_REALTIME_INODE(ip)) if (XFS_IS_REALTIME_INODE(ip))
return xfs_get_extsz_hint(ip) << mp->m_sb.sb_blocklog; return XFS_FSB_TO_B(mp, xfs_get_extsz_hint(ip));
/* /*
* Allow large block sizes to be reported to userspace programs if the * Allow large block sizes to be reported to userspace programs if the
...@@ -560,7 +560,7 @@ xfs_stat_blksize( ...@@ -560,7 +560,7 @@ xfs_stat_blksize(
*/ */
if (mp->m_flags & XFS_MOUNT_LARGEIO) { if (mp->m_flags & XFS_MOUNT_LARGEIO) {
if (mp->m_swidth) if (mp->m_swidth)
return mp->m_swidth << mp->m_sb.sb_blocklog; return XFS_FSB_TO_B(mp, mp->m_swidth);
if (mp->m_flags & XFS_MOUNT_ALLOCSIZE) if (mp->m_flags & XFS_MOUNT_ALLOCSIZE)
return 1U << mp->m_allocsize_log; return 1U << mp->m_allocsize_log;
} }
......
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