xfs: split xfs_iformat_fork

Source kernel commit: 9229d18e801bdbdf79d963d8c944980fc77b5d6b

xfs_iformat_fork is a weird catchall.  Split it into one helper for
the data fork and one for the attr fork, and then call both helper
as well as the COW fork initialization from xfs_inode_from_disk.  Order
the COW fork initialization after the attr fork initialization given
that it can't fail to simplify the error handling.

Note that the newly split helpers are moved down the file in
xfs_inode_fork.c to avoid the need for forward declarations.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Brian Foster <bfoster@redhat.com>
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_inode_buf.c b/libxfs/xfs_inode_buf.c
index ed04883beee79ad5c8223d48a3427acee55307c1..4679819bf0223bbe6a6ceff3f682b6d12def7c11 100644 (file)
--- a/libxfs/xfs_inode_buf.c
+++ b/libxfs/xfs_inode_buf.c
@@ -184,6 +184,10 @@ xfs_inode_from_disk(
 {
        struct xfs_icdinode     *to = &ip->i_d;
        struct inode            *inode = VFS_I(ip);
+       int                     error;
+
+       ASSERT(ip->i_cowfp == NULL);
+       ASSERT(ip->i_afp == NULL);
 
        /*
         * Convert v1 inodes immediately to v2 inode format as this is the
@@ -239,7 +243,21 @@ xfs_inode_from_disk(
                to->di_cowextsize = be32_to_cpu(from->di_cowextsize);
        }
 
-       return xfs_iformat_fork(ip, from);
+       error = xfs_iformat_data_fork(ip, from);
+       if (error)
+               return error;
+       if (XFS_DFORK_Q(from)) {
+               error = xfs_iformat_attr_fork(ip, from);
+               if (error)
+                       goto out_destroy_data_fork;
+       }
+       if (xfs_is_reflink_inode(ip))
+               xfs_ifork_init_cow(ip);
+       return 0;
+
+out_destroy_data_fork:
+       xfs_idestroy_fork(ip, XFS_DATA_FORK);
+       return error;
 }
 
 void

diff --git a/libxfs/xfs_inode_fork.c b/libxfs/xfs_inode_fork.c
index 5ecfecaf454c5fa34fbb505abe69d72a9bd9740f..9e8edd45c92b96005956b641b80eebbaa1a7dc1a 100644 (file)
--- a/libxfs/xfs_inode_fork.c
+++ b/libxfs/xfs_inode_fork.c
@@ -24,110 +24,6 @@
 
 kmem_zone_t *xfs_ifork_zone;
 
-STATIC int xfs_iformat_local(xfs_inode_t *, xfs_dinode_t *, int, int);
-STATIC int xfs_iformat_extents(xfs_inode_t *, xfs_dinode_t *, int);
-STATIC int xfs_iformat_btree(xfs_inode_t *, xfs_dinode_t *, int);
-
-/*
- * Copy inode type and data and attr format specific information from the
- * on-disk inode to the in-core inode and fork structures.  For fifos, devices,
- * and sockets this means set i_rdev to the proper value.  For files,
- * directories, and symlinks this means to bring in the in-line data or extent
- * pointers as well as the attribute fork.  For a fork in B-tree format, only
- * the root is immediately brought in-core.  The rest will be read in later when
- * first referenced (see xfs_iread_extents()).
- */
-int
-xfs_iformat_fork(
-       struct xfs_inode        *ip,
-       struct xfs_dinode       *dip)
-{
-       struct inode            *inode = VFS_I(ip);
-       struct xfs_attr_shortform *atp;
-       int                     size;
-       int                     error = 0;
-       xfs_fsize_t             di_size;
-
-       switch (inode->i_mode & S_IFMT) {
-       case S_IFIFO:
-       case S_IFCHR:
-       case S_IFBLK:
-       case S_IFSOCK:
-               ip->i_d.di_size = 0;
-               inode->i_rdev = xfs_to_linux_dev_t(xfs_dinode_get_rdev(dip));
-               break;
-
-       case S_IFREG:
-       case S_IFLNK:
-       case S_IFDIR:
-               switch (dip->di_format) {
-               case XFS_DINODE_FMT_LOCAL:
-                       di_size = be64_to_cpu(dip->di_size);
-                       size = (int)di_size;
-                       error = xfs_iformat_local(ip, dip, XFS_DATA_FORK, size);
-                       break;
-               case XFS_DINODE_FMT_EXTENTS:
-                       error = xfs_iformat_extents(ip, dip, XFS_DATA_FORK);
-                       break;
-               case XFS_DINODE_FMT_BTREE:
-                       error = xfs_iformat_btree(ip, dip, XFS_DATA_FORK);
-                       break;
-               default:
-                       xfs_inode_verifier_error(ip, -EFSCORRUPTED, __func__,
-                                       dip, sizeof(*dip), __this_address);
-                       return -EFSCORRUPTED;
-               }
-               break;
-
-       default:
-               xfs_inode_verifier_error(ip, -EFSCORRUPTED, __func__, dip,
-                               sizeof(*dip), __this_address);
-               return -EFSCORRUPTED;
-       }
-       if (error)
-               return error;
-
-       if (xfs_is_reflink_inode(ip)) {
-               ASSERT(ip->i_cowfp == NULL);
-               xfs_ifork_init_cow(ip);
-       }
-
-       if (!XFS_DFORK_Q(dip))
-               return 0;
-
-       ASSERT(ip->i_afp == NULL);
-       ip->i_afp = kmem_zone_zalloc(xfs_ifork_zone, KM_NOFS);
-
-       switch (dip->di_aformat) {
-       case XFS_DINODE_FMT_LOCAL:
-               atp = (xfs_attr_shortform_t *)XFS_DFORK_APTR(dip);
-               size = be16_to_cpu(atp->hdr.totsize);
-
-               error = xfs_iformat_local(ip, dip, XFS_ATTR_FORK, size);
-               break;
-       case XFS_DINODE_FMT_EXTENTS:
-               error = xfs_iformat_extents(ip, dip, XFS_ATTR_FORK);
-               break;
-       case XFS_DINODE_FMT_BTREE:
-               error = xfs_iformat_btree(ip, dip, XFS_ATTR_FORK);
-               break;
-       default:
-               xfs_inode_verifier_error(ip, error, __func__, dip,
-                               sizeof(*dip), __this_address);
-               error = -EFSCORRUPTED;
-               break;
-       }
-       if (error) {
-               kmem_cache_free(xfs_ifork_zone, ip->i_afp);
-               ip->i_afp = NULL;
-               if (ip->i_cowfp)
-                       kmem_cache_free(xfs_ifork_zone, ip->i_cowfp);
-               ip->i_cowfp = NULL;
-               xfs_idestroy_fork(ip, XFS_DATA_FORK);
-       }
-       return error;
-}
-
 void
 xfs_init_local_fork(
        struct xfs_inode        *ip,
@@ -323,6 +219,88 @@ xfs_iformat_btree(
        return 0;
 }
 
+int
+xfs_iformat_data_fork(
+       struct xfs_inode        *ip,
+       struct xfs_dinode       *dip)
+{
+       struct inode            *inode = VFS_I(ip);
+
+       switch (inode->i_mode & S_IFMT) {
+       case S_IFIFO:
+       case S_IFCHR:
+       case S_IFBLK:
+       case S_IFSOCK:
+               ip->i_d.di_size = 0;
+               inode->i_rdev = xfs_to_linux_dev_t(xfs_dinode_get_rdev(dip));
+               return 0;
+       case S_IFREG:
+       case S_IFLNK:
+       case S_IFDIR:
+               switch (dip->di_format) {
+               case XFS_DINODE_FMT_LOCAL:
+                       return xfs_iformat_local(ip, dip, XFS_DATA_FORK,
+                                       be64_to_cpu(dip->di_size));
+               case XFS_DINODE_FMT_EXTENTS:
+                       return xfs_iformat_extents(ip, dip, XFS_DATA_FORK);
+               case XFS_DINODE_FMT_BTREE:
+                       return xfs_iformat_btree(ip, dip, XFS_DATA_FORK);
+               default:
+                       xfs_inode_verifier_error(ip, -EFSCORRUPTED, __func__,
+                                       dip, sizeof(*dip), __this_address);
+                       return -EFSCORRUPTED;
+               }
+               break;
+       default:
+               xfs_inode_verifier_error(ip, -EFSCORRUPTED, __func__, dip,
+                               sizeof(*dip), __this_address);
+               return -EFSCORRUPTED;
+       }
+}
+
+static uint16_t
+xfs_dfork_attr_shortform_size(
+       struct xfs_dinode               *dip)
+{
+       struct xfs_attr_shortform       *atp =
+               (struct xfs_attr_shortform *)XFS_DFORK_APTR(dip);
+
+       return be16_to_cpu(atp->hdr.totsize);
+}
+
+int
+xfs_iformat_attr_fork(
+       struct xfs_inode        *ip,
+       struct xfs_dinode       *dip)
+{
+       int                     error = 0;
+
+       ip->i_afp = kmem_zone_zalloc(xfs_ifork_zone, KM_NOFS);
+       switch (dip->di_aformat) {
+       case XFS_DINODE_FMT_LOCAL:
+               error = xfs_iformat_local(ip, dip, XFS_ATTR_FORK,
+                               xfs_dfork_attr_shortform_size(dip));
+               break;
+       case XFS_DINODE_FMT_EXTENTS:
+               error = xfs_iformat_extents(ip, dip, XFS_ATTR_FORK);
+               break;
+       case XFS_DINODE_FMT_BTREE:
+               error = xfs_iformat_btree(ip, dip, XFS_ATTR_FORK);
+               break;
+       default:
+               xfs_inode_verifier_error(ip, error, __func__, dip,
+                               sizeof(*dip), __this_address);
+               error = -EFSCORRUPTED;
+               break;
+       }
+
+       if (error) {
+               kmem_cache_free(xfs_ifork_zone, ip->i_afp);
+               ip->i_afp = NULL;
+       }
+       return error;
+}
+
 /*
  * Reallocate the space for if_broot based on the number of records
  * being added or deleted as indicated in rec_diff.  Move the records

diff --git a/libxfs/xfs_inode_fork.h b/libxfs/xfs_inode_fork.h
index 668ee942be224f87c0d76093fbb3dd5d0435b190..8487b0c88a75e04b276c1a7f771bed8b0e87d6a0 100644 (file)
--- a/libxfs/xfs_inode_fork.h
+++ b/libxfs/xfs_inode_fork.h
@@ -88,7 +88,8 @@ struct xfs_ifork {
 
 struct xfs_ifork *xfs_iext_state_to_fork(struct xfs_inode *ip, int state);
 
-int            xfs_iformat_fork(struct xfs_inode *, struct xfs_dinode *);
+int            xfs_iformat_data_fork(struct xfs_inode *, struct xfs_dinode *);
+int            xfs_iformat_attr_fork(struct xfs_inode *, struct xfs_dinode *);
 void           xfs_iflush_fork(struct xfs_inode *, struct xfs_dinode *,
                                struct xfs_inode_log_item *, int);
 void           xfs_idestroy_fork(struct xfs_inode *, int);