]> granicus.if.org Git - zfs/commitdiff
Conserve stack in zfs_mkdir()
authorBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 7 Jan 2011 21:47:47 +0000 (13:47 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 10 Feb 2011 17:21:42 +0000 (09:21 -0800)
Move the sa_attrs array from the stack to the heap to minimize stack
space usage.

module/zfs/zfs_znode.c

index 93f2f026e75f2dfb718dde4b117eb59902d860f6..b7f95f4f0c4a399ea5c0fc35e61985a1d3cd39c0 100644 (file)
@@ -773,7 +773,7 @@ zfs_mknode(znode_t *dzp, vattr_t *vap, dmu_tx_t *tx, cred_t *cr,
        int             bonuslen;
        sa_handle_t     *sa_hdl;
        dmu_object_type_t obj_type;
-       sa_bulk_attr_t  sa_attrs[ZPL_END];
+       sa_bulk_attr_t  *sa_attrs;
        int             cnt = 0;
        zfs_acl_locator_cb_t locate = { 0 };
 
@@ -899,6 +899,7 @@ zfs_mknode(znode_t *dzp, vattr_t *vap, dmu_tx_t *tx, cred_t *cr,
         * order for  DMU_OT_ZNODE is critical since it needs to be constructed
         * in the old znode_phys_t format.  Don't change this ordering
         */
+       sa_attrs = kmem_alloc(sizeof(sa_bulk_attr_t) * ZPL_END, KM_SLEEP);
 
        if (obj_type == DMU_OT_ZNODE) {
                SA_ADD_BULK_ATTR(sa_attrs, cnt, SA_ZPL_ATIME(zfsvfs),
@@ -1002,6 +1003,7 @@ zfs_mknode(znode_t *dzp, vattr_t *vap, dmu_tx_t *tx, cred_t *cr,
                err = zfs_aclset_common(*zpp, acl_ids->z_aclp, cr, tx);
                ASSERT3S(err, ==, 0);
        }
+       kmem_free(sa_attrs, sizeof(sa_bulk_attr_t) * ZPL_END);
        ZFS_OBJ_HOLD_EXIT(zfsvfs, obj);
 }