STATIC int xfs_attr_refillstate(xfs_da_state_t *state);
 STATIC int xfs_attr_set_iter(struct xfs_delattr_context *dac,
                             struct xfs_buf **leaf_bp);
-STATIC int xfs_attr_node_remove_name(struct xfs_da_args *args,
-                                    struct xfs_da_state *state);
+STATIC int xfs_attr_node_removename(struct xfs_da_args *args,
+                                   struct xfs_da_state *state);
 
 int
 xfs_inode_hasattr(
 }
 
 STATIC int
-xfs_attr_set_fmt(
+xfs_attr_sf_addname(
        struct xfs_delattr_context      *dac,
        struct xfs_buf                  **leaf_bp)
 {
                 * release the hold once we return with a clean transaction.
                 */
                if (xfs_attr_is_shortform(dp))
-                       return xfs_attr_set_fmt(dac, leaf_bp);
+                       return xfs_attr_sf_addname(dac, leaf_bp);
                if (*leaf_bp != NULL) {
                        xfs_trans_bhold_release(args->trans, *leaf_bp);
                        *leaf_bp = NULL;
        if (retval == -EEXIST) {
                if (args->attr_flags & XATTR_CREATE)
                        return retval;
-               retval = xfs_attr_shortform_remove(args);
+               retval = xfs_attr_sf_removename(args);
                if (retval)
                        return retval;
                /*
        if (error)
                goto out;
 
-       error = xfs_attr_node_remove_name(args, state);
+       error = xfs_attr_node_removename(args, state);
 
        /*
         * Check to see if the tree needs to be collapsed.
 }
 
 STATIC int
-xfs_attr_node_remove_name(
+xfs_attr_node_removename(
        struct xfs_da_args      *args,
        struct xfs_da_state     *state)
 {
                 * thus state transitions. Call the right helper and return.
                 */
                if (dp->i_afp->if_format == XFS_DINODE_FMT_LOCAL)
-                       return xfs_attr_shortform_remove(args);
+                       return xfs_attr_sf_removename(args);
 
                if (xfs_attr_is_leaf(dp))
                        return xfs_attr_leaf_removename(args);
                                goto out;
                }
 
-               retval = xfs_attr_node_remove_name(args, state);
+               retval = xfs_attr_node_removename(args, state);
 
                /*
                 * Check to see if the tree needs to be collapsed. If so, roll
 
 int    xfs_attr_shortform_getvalue(struct xfs_da_args *args);
 int    xfs_attr_shortform_to_leaf(struct xfs_da_args *args,
                        struct xfs_buf **leaf_bp);
-int    xfs_attr_shortform_remove(struct xfs_da_args *args);
+int    xfs_attr_sf_removename(struct xfs_da_args *args);
 int    xfs_attr_sf_findname(struct xfs_da_args *args,
                             struct xfs_attr_sf_entry **sfep,
                             unsigned int *basep);