xfs: pass struct xfs_bmbt_irec to xfs_bmbt_validate_extent

Source kernel commit: dac9c9b137950421a87c1d9ba29f3a6ee54d0e8a

This removed an unaligned load per extent, as well as the manual poking
into the on-disk extent format.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

diff --git a/libxfs/xfs_bmap.c b/libxfs/xfs_bmap.c
index d8e6e7c4ec5e071bad3c4b83ad5749ade03be4af..195f0fb14d7062d4b397b4a3c8669d72afcb2c15 100644 (file)
--- a/libxfs/xfs_bmap.c
+++ b/libxfs/xfs_bmap.c
@@ -1252,13 +1252,13 @@ xfs_iread_extents(
                 */
                frp = XFS_BMBT_REC_ADDR(mp, block, 1);
                for (j = 0; j < num_recs; j++, frp++, i++) {
-                       if (!xfs_bmbt_validate_extent(mp, whichfork, frp)) {
+                       xfs_bmbt_disk_get_all(frp, &new);
+                       if (!xfs_bmbt_validate_extent(mp, whichfork, &new)) {
                                XFS_ERROR_REPORT("xfs_bmap_read_extents(2)",
                                                 XFS_ERRLEVEL_LOW, mp);
                                error = -EFSCORRUPTED;
                                goto out_brelse;
                        }
-                       xfs_bmbt_disk_get_all(frp, &new);
                        xfs_iext_insert(ip, &icur, &new, state);
                        trace_xfs_read_extent(ip, &icur, state, _THIS_IP_);
                        xfs_iext_next(ifp, &icur);

diff --git a/libxfs/xfs_bmap_btree.h b/libxfs/xfs_bmap_btree.h
index 714bfbaf9b2dca8514fb9c1a8e428e1b9963c6cf..135b8c56d23e83c492812982120a093bf7cc5527 100644 (file)
--- a/libxfs/xfs_bmap_btree.h
+++ b/libxfs/xfs_bmap_btree.h
@@ -122,9 +122,9 @@ extern struct xfs_btree_cur *xfs_bmbt_init_cursor(struct xfs_mount *,
  * Check that the extent does not contain an invalid unwritten extent flag.
  */
 static inline bool xfs_bmbt_validate_extent(struct xfs_mount *mp, int whichfork,
-               struct xfs_bmbt_rec *ep)
+               struct xfs_bmbt_irec *irec)
 {
-       if (get_unaligned_be64(&ep->l0) >> (64 - BMBT_EXNTFLAG_BITLEN) == 0)
+       if (irec->br_state == XFS_EXT_NORM)
                return true;
        if (whichfork == XFS_DATA_FORK &&
            xfs_sb_version_hasextflgbit(&mp->m_sb))

diff --git a/libxfs/xfs_inode_fork.c b/libxfs/xfs_inode_fork.c
index b647f63410bcb58a5aad6d21dfd8fb82ada18d62..79e0398c86aa9e0ee34896934107c68b7dbb6b40 100644 (file)
--- a/libxfs/xfs_inode_fork.c
+++ b/libxfs/xfs_inode_fork.c
@@ -352,13 +352,13 @@ xfs_iformat_extents(
 
                xfs_iext_first(ifp, &icur);
                for (i = 0; i < nex; i++, dp++) {
-                       if (!xfs_bmbt_validate_extent(mp, whichfork, dp)) {
+                       xfs_bmbt_disk_get_all(dp, &new);
+                       if (!xfs_bmbt_validate_extent(mp, whichfork, &new)) {
                                XFS_ERROR_REPORT("xfs_iformat_extents(2)",
                                                 XFS_ERRLEVEL_LOW, mp);
                                return -EFSCORRUPTED;
                        }
 
-                       xfs_bmbt_disk_get_all(dp, &new);
                        xfs_iext_insert(ip, &icur, &new, state);
                        trace_xfs_read_extent(ip, &icur, state, _THIS_IP_);
                        xfs_iext_next(ifp, &icur);
@@ -701,9 +701,9 @@ xfs_iextents_copy(
        for_each_xfs_iext(ifp, &icur, &rec) {
                if (isnullstartblock(rec.br_startblock))
                        continue;
+               ASSERT(xfs_bmbt_validate_extent(ip->i_mount, whichfork, &rec));
                xfs_bmbt_disk_set_all(dp, &rec);
                trace_xfs_write_extent(ip, &icur, state, _RET_IP_);
-               ASSERT(xfs_bmbt_validate_extent(ip->i_mount, whichfork, dp));
                copied += sizeof(struct xfs_bmbt_rec);
                dp++;
        }