xfs: replace xfs_dir3_data_endp with xfs_dir3_data_end_offset

Source kernel commit: 5c072127d31d3a605a3048dc5d3fdfc0cdd47212

All the callers really want an offset into the buffer, so adopt
the helper to return that instead.

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_dir2.h b/libxfs/xfs_dir2.h
index a160f2d4ff37f4b85704c899d62f3bc99a35c219..3a4b98d4973db0ac6d878504992df7673588b7c1 100644 (file)
--- a/libxfs/xfs_dir2.h
+++ b/libxfs/xfs_dir2.h
@@ -266,7 +266,7 @@ xfs_dir2_leaf_tail_p(struct xfs_da_geometry *geo, struct xfs_dir2_leaf *lp)
 #define XFS_READDIR_BUFSIZE    (32768)
 
 unsigned char xfs_dir3_get_dtype(struct xfs_mount *mp, uint8_t filetype);
-void *xfs_dir3_data_endp(struct xfs_da_geometry *geo,
+unsigned int xfs_dir3_data_end_offset(struct xfs_da_geometry *geo,
                struct xfs_dir2_data_hdr *hdr);
 bool xfs_dir2_namecheck(const void *name, size_t length);
 

diff --git a/libxfs/xfs_dir2_data.c b/libxfs/xfs_dir2_data.c
index 0ee4ee341ba35c89eef11b3ce8dbb1c95069c0c4..d6e0c8787f336b560446cf69756ecc67f7d81aeb 100644 (file)
--- a/libxfs/xfs_dir2_data.c
+++ b/libxfs/xfs_dir2_data.c
@@ -52,7 +52,6 @@ __xfs_dir3_data_check(
        int                     count;          /* count of entries found */
        xfs_dir2_data_hdr_t     *hdr;           /* data block header */
        xfs_dir2_data_free_t    *dfp;           /* bestfree entry */
-       void                    *endp;          /* end of useful data */
        int                     freeseen;       /* mask of bestfrees seen */
        xfs_dahash_t            hash;           /* hash of current name */
        int                     i;              /* leaf index */
@@ -99,10 +98,9 @@ __xfs_dir3_data_check(
        default:
                return __this_address;
        }
-       endp = xfs_dir3_data_endp(geo, hdr);
-       if (!endp)
+       end = xfs_dir3_data_end_offset(geo, hdr);
+       if (!end)
                return __this_address;
-       end = endp - bp->b_addr;
 
        /*
         * Account for zero bestfree entries.
@@ -587,7 +585,7 @@ xfs_dir2_data_freescan_int(
        memset(bf, 0, sizeof(*bf) * XFS_DIR2_DATA_FD_COUNT);
        *loghead = 1;
 
-       end = xfs_dir3_data_endp(geo, addr) - addr;
+       end = xfs_dir3_data_end_offset(geo, addr);
        while (offset < end) {
                struct xfs_dir2_data_unused     *dup = addr + offset;
                struct xfs_dir2_data_entry      *dep = addr + offset;
@@ -781,11 +779,11 @@ xfs_dir2_data_make_free(
 {
        xfs_dir2_data_hdr_t     *hdr;           /* data block pointer */
        xfs_dir2_data_free_t    *dfp;           /* bestfree pointer */
-       char                    *endptr;        /* end of data area */
        int                     needscan;       /* need to regen bestfree */
        xfs_dir2_data_unused_t  *newdup;        /* new unused entry */
        xfs_dir2_data_unused_t  *postdup;       /* unused entry after us */
        xfs_dir2_data_unused_t  *prevdup;       /* unused entry before us */
+       unsigned int            end;
        struct xfs_dir2_data_free *bf;
 
        hdr = bp->b_addr;
@@ -793,8 +791,8 @@ xfs_dir2_data_make_free(
        /*
         * Figure out where the end of the data area is.
         */
-       endptr = xfs_dir3_data_endp(args->geo, hdr);
-       ASSERT(endptr != NULL);
+       end = xfs_dir3_data_end_offset(args->geo, hdr);
+       ASSERT(end != 0);
 
        /*
         * If this isn't the start of the block, then back up to
@@ -813,7 +811,7 @@ xfs_dir2_data_make_free(
         * If this isn't the end of the block, see if the entry after
         * us is free.
         */
-       if ((char *)hdr + offset + len < endptr) {
+       if (offset + len < end) {
                postdup =
                        (xfs_dir2_data_unused_t *)((char *)hdr + offset + len);
                if (be16_to_cpu(postdup->freetag) != XFS_DIR2_DATA_FREE_TAG)
@@ -1141,19 +1139,22 @@ corrupt:
 }
 
 /* Find the end of the entry data in a data/block format dir block. */
-void *
-xfs_dir3_data_endp(
+unsigned int
+xfs_dir3_data_end_offset(
        struct xfs_da_geometry          *geo,
        struct xfs_dir2_data_hdr        *hdr)
 {
+       void                            *p;
+
        switch (hdr->magic) {
        case cpu_to_be32(XFS_DIR3_BLOCK_MAGIC):
        case cpu_to_be32(XFS_DIR2_BLOCK_MAGIC):
-               return xfs_dir2_block_leaf_p(xfs_dir2_block_tail_p(geo, hdr));
+               p = xfs_dir2_block_leaf_p(xfs_dir2_block_tail_p(geo, hdr));
+               return p - (void *)hdr;
        case cpu_to_be32(XFS_DIR3_DATA_MAGIC):
        case cpu_to_be32(XFS_DIR2_DATA_MAGIC):
-               return (char *)hdr + geo->blksize;
+               return geo->blksize;
        default:
-               return NULL;
+               return 0;
        }
 }

diff --git a/libxfs/xfs_dir2_sf.c b/libxfs/xfs_dir2_sf.c
index c620357420e257f94ecbc8314233567bd28067a6..a707180adfa4c97a34a1d04006543cd2c9cf3f62 100644 (file)
--- a/libxfs/xfs_dir2_sf.c
+++ b/libxfs/xfs_dir2_sf.c
@@ -283,7 +283,7 @@ xfs_dir2_block_to_sf(
         * Loop over the active and unused entries.  Stop when we reach the
         * leaf/tail portion of the block.
         */
-       end = xfs_dir3_data_endp(args->geo, bp->b_addr) - bp->b_addr;
+       end = xfs_dir3_data_end_offset(args->geo, bp->b_addr);
        sfep = xfs_dir2_sf_firstentry(sfp);
        while (offset < end) {
                struct xfs_dir2_data_unused     *dup = bp->b_addr + offset;