Update core ZFS code from build 121 to build 141.
[zfs.git] / module / zfs / dmu_tx.c
index c6fbeee..5fc062c 100644 (file)
@@ -19,8 +19,7 @@
  * CDDL HEADER END
  */
 /*
- * Copyright 2009 Sun Microsystems, Inc.  All rights reserved.
- * Use is subject to license terms.
+ * Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved.
  */
 
 #include <sys/dmu.h>
 #include <sys/dsl_pool.h>
 #include <sys/zap_impl.h> /* for fzap_default_block_shift */
 #include <sys/spa.h>
+#include <sys/sa.h>
+#include <sys/sa_impl.h>
 #include <sys/zfs_context.h>
+#include <sys/varargs.h>
 
 typedef void (*dmu_tx_hold_func_t)(dmu_tx_t *tx, struct dnode *dn,
     uint64_t arg1, uint64_t arg2);
@@ -48,6 +50,8 @@ dmu_tx_create_dd(dsl_dir_t *dd)
                tx->tx_pool = dd->dd_pool;
        list_create(&tx->tx_holds, sizeof (dmu_tx_hold_t),
            offsetof(dmu_tx_hold_t, txh_node));
+       list_create(&tx->tx_callbacks, sizeof (dmu_tx_callback_t),
+           offsetof(dmu_tx_callback_t, dcb_node));
 #ifdef ZFS_DEBUG
        refcount_create(&tx->tx_space_written);
        refcount_create(&tx->tx_space_freed);
@@ -58,9 +62,9 @@ dmu_tx_create_dd(dsl_dir_t *dd)
 dmu_tx_t *
 dmu_tx_create(objset_t *os)
 {
-       dmu_tx_t *tx = dmu_tx_create_dd(os->os->os_dsl_dataset->ds_dir);
+       dmu_tx_t *tx = dmu_tx_create_dd(os->os_dsl_dataset->ds_dir);
        tx->tx_objset = os;
-       tx->tx_lastsnap_txg = dsl_dataset_prev_snap_txg(os->os->os_dsl_dataset);
+       tx->tx_lastsnap_txg = dsl_dataset_prev_snap_txg(os->os_dsl_dataset);
        return (tx);
 }
 
@@ -98,7 +102,7 @@ dmu_tx_hold_object_impl(dmu_tx_t *tx, objset_t *os, uint64_t object,
        int err;
 
        if (object != DMU_NEW_OBJECT) {
-               err = dnode_hold(os->os, object, tx, &dn);
+               err = dnode_hold(os, object, tx, &dn);
                if (err) {
                        tx->tx_err = err;
                        return (NULL);
@@ -161,38 +165,47 @@ dmu_tx_check_ioerr(zio_t *zio, dnode_t *dn, int level, uint64_t blkid)
 }
 
 static void
-dmu_tx_count_indirects(dmu_tx_hold_t *txh, dmu_buf_impl_t *db,
-    boolean_t freeable, dmu_buf_impl_t **history)
+dmu_tx_count_twig(dmu_tx_hold_t *txh, dnode_t *dn, dmu_buf_impl_t *db,
+    int level, uint64_t blkid, boolean_t freeable, uint64_t *history)
 {
-       int i = db->db_level + 1;
-       dnode_t *dn = db->db_dnode;
-
-       if (i >= dn->dn_nlevels)
+       objset_t *os = dn->dn_objset;
+       dsl_dataset_t *ds = os->os_dsl_dataset;
+       int epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT;
+       dmu_buf_impl_t *parent = NULL;
+       blkptr_t *bp = NULL;
+       uint64_t space;
+
+       if (level >= dn->dn_nlevels || history[level] == blkid)
                return;
 
-       db = db->db_parent;
-       if (db == NULL) {
-               uint64_t lvls = dn->dn_nlevels - i;
+       history[level] = blkid;
 
-               txh->txh_space_towrite += lvls << dn->dn_indblkshift;
-               return;
+       space = (level == 0) ? dn->dn_datablksz : (1ULL << dn->dn_indblkshift);
+
+       if (db == NULL || db == dn->dn_dbuf) {
+               ASSERT(level != 0);
+               db = NULL;
+       } else {
+               ASSERT(db->db_dnode == dn);
+               ASSERT(db->db_level == level);
+               ASSERT(db->db.db_size == space);
+               ASSERT(db->db_blkid == blkid);
+               bp = db->db_blkptr;
+               parent = db->db_parent;
        }
 
-       if (db != history[i]) {
-               dsl_dataset_t *ds = dn->dn_objset->os_dsl_dataset;
-               uint64_t space = 1ULL << dn->dn_indblkshift;
+       freeable = (bp && (freeable ||
+           dsl_dataset_block_freeable(ds, bp, bp->blk_birth)));
 
-               freeable = (db->db_blkptr && (freeable ||
-                   dsl_dataset_block_freeable(ds, db->db_blkptr->blk_birth)));
-               if (freeable)
-                       txh->txh_space_tooverwrite += space;
-               else
-                       txh->txh_space_towrite += space;
-               if (db->db_blkptr)
-                       txh->txh_space_tounref += space;
-               history[i] = db;
-               dmu_tx_count_indirects(txh, db, freeable, history);
-       }
+       if (freeable)
+               txh->txh_space_tooverwrite += space;
+       else
+               txh->txh_space_towrite += space;
+       if (bp)
+               txh->txh_space_tounref += bp_get_dsize(os->os_spa, bp);
+
+       dmu_tx_count_twig(txh, dn, parent, level + 1,
+           blkid >> epbs, freeable, history);
 }
 
 /* ARGSUSED */
@@ -213,7 +226,7 @@ dmu_tx_count_write(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
        max_ibs = DN_MAX_INDBLKSHIFT;
 
        if (dn) {
-               dmu_buf_impl_t *last[DN_MAX_LEVELS];
+               uint64_t history[DN_MAX_LEVELS];
                int nlvls = dn->dn_nlevels;
                int delta;
 
@@ -289,29 +302,24 @@ dmu_tx_count_write(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
                 * If this write is not off the end of the file
                 * we need to account for overwrites/unref.
                 */
-               if (start <= dn->dn_maxblkid)
-                       bzero(last, sizeof (dmu_buf_impl_t *) * DN_MAX_LEVELS);
+               if (start <= dn->dn_maxblkid) {
+                       for (int l = 0; l < DN_MAX_LEVELS; l++)
+                               history[l] = -1ULL;
+               }
                while (start <= dn->dn_maxblkid) {
-                       spa_t *spa = txh->txh_tx->tx_pool->dp_spa;
-                       dsl_dataset_t *ds = dn->dn_objset->os_dsl_dataset;
                        dmu_buf_impl_t *db;
 
                        rw_enter(&dn->dn_struct_rwlock, RW_READER);
-                       db = dbuf_hold_level(dn, 0, start, FTAG);
+                       err = dbuf_hold_impl(dn, 0, start, FALSE, FTAG, &db);
                        rw_exit(&dn->dn_struct_rwlock);
-                       if (db->db_blkptr && dsl_dataset_block_freeable(ds,
-                           db->db_blkptr->blk_birth)) {
-                               dprintf_bp(db->db_blkptr, "can free old%s", "");
-                               txh->txh_space_tooverwrite += dn->dn_datablksz;
-                               txh->txh_space_tounref += dn->dn_datablksz;
-                               dmu_tx_count_indirects(txh, db, TRUE, last);
-                       } else {
-                               txh->txh_space_towrite += dn->dn_datablksz;
-                               if (db->db_blkptr)
-                                       txh->txh_space_tounref +=
-                                           bp_get_dasize(spa, db->db_blkptr);
-                               dmu_tx_count_indirects(txh, db, FALSE, last);
+
+                       if (err) {
+                               txh->txh_tx->tx_err = err;
+                               return;
                        }
+
+                       dmu_tx_count_twig(txh, dn, db, 0, start, B_FALSE,
+                           history);
                        dbuf_rele(db, FTAG);
                        if (++start > end) {
                                /*
@@ -376,13 +384,13 @@ static void
 dmu_tx_count_dnode(dmu_tx_hold_t *txh)
 {
        dnode_t *dn = txh->txh_dnode;
-       dnode_t *mdn = txh->txh_tx->tx_objset->os->os_meta_dnode;
+       dnode_t *mdn = txh->txh_tx->tx_objset->os_meta_dnode;
        uint64_t space = mdn->dn_datablksz +
            ((mdn->dn_nlevels-1) << mdn->dn_indblkshift);
 
        if (dn && dn->dn_dbuf->db_blkptr &&
            dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset,
-           dn->dn_dbuf->db_blkptr->blk_birth)) {
+           dn->dn_dbuf->db_blkptr, dn->dn_dbuf->db_blkptr->blk_birth)) {
                txh->txh_space_tooverwrite += space;
                txh->txh_space_tounref += space;
        } else {
@@ -427,7 +435,7 @@ dmu_tx_count_free(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
         * The struct_rwlock protects us against dn_nlevels
         * changing, in case (against all odds) we manage to dirty &
         * sync out the changes after we check for being dirty.
-        * Also, dbuf_hold_level() wants us to have the struct_rwlock.
+        * Also, dbuf_hold_impl() wants us to have the struct_rwlock.
         */
        rw_enter(&dn->dn_struct_rwlock, RW_READER);
        epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT;
@@ -457,9 +465,9 @@ dmu_tx_count_free(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
                        blkptr_t *bp = dn->dn_phys->dn_blkptr;
                        ASSERT3U(blkid + i, <, dn->dn_nblkptr);
                        bp += blkid + i;
-                       if (dsl_dataset_block_freeable(ds, bp->blk_birth)) {
+                       if (dsl_dataset_block_freeable(ds, bp, bp->blk_birth)) {
                                dprintf_bp(bp, "can free old%s", "");
-                               space += bp_get_dasize(spa, bp);
+                               space += bp_get_dsize(spa, bp);
                        }
                        unref += BP_GET_ASIZE(bp);
                }
@@ -515,14 +523,22 @@ dmu_tx_count_free(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
                blkoff = P2PHASE(blkid, epb);
                tochk = MIN(epb - blkoff, nblks);
 
-               dbuf = dbuf_hold_level(dn, 1, blkid >> epbs, FTAG);
-
-               txh->txh_memory_tohold += dbuf->db.db_size;
-               if (txh->txh_memory_tohold > DMU_MAX_ACCESS) {
-                       txh->txh_tx->tx_err = E2BIG;
-                       dbuf_rele(dbuf, FTAG);
+               err = dbuf_hold_impl(dn, 1, blkid >> epbs, FALSE, FTAG, &dbuf);
+               if (err) {
+                       txh->txh_tx->tx_err = err;
                        break;
                }
+
+               txh->txh_memory_tohold += dbuf->db.db_size;
+
+               /*
+                * We don't check memory_tohold against DMU_MAX_ACCESS because
+                * memory_tohold is an over-estimation (especially the >L1
+                * indirect blocks), so it could fail.  Callers should have
+                * already verified that they will not be holding too much
+                * memory.
+                */
+
                err = dbuf_read(dbuf, NULL, DB_RF_HAVESTRUCT | DB_RF_CANFAIL);
                if (err != 0) {
                        txh->txh_tx->tx_err = err;
@@ -534,9 +550,10 @@ dmu_tx_count_free(dmu_tx_hold_t *txh, uint64_t off, uint64_t len)
                bp += blkoff;
 
                for (i = 0; i < tochk; i++) {
-                       if (dsl_dataset_block_freeable(ds, bp[i].blk_birth)) {
+                       if (dsl_dataset_block_freeable(ds, &bp[i],
+                           bp[i].blk_birth)) {
                                dprintf_bp(&bp[i], "can free old%s", "");
-                               space += bp_get_dasize(spa, &bp[i]);
+                               space += bp_get_dsize(spa, &bp[i]);
                        }
                        unref += BP_GET_ASIZE(bp);
                }
@@ -581,6 +598,8 @@ dmu_tx_hold_free(dmu_tx_t *tx, uint64_t object, uint64_t off, uint64_t len)
        if (len != DMU_OBJECT_END)
                dmu_tx_count_write(txh, off+len, 1);
 
+       dmu_tx_count_dnode(txh);
+
        if (off >= (dn->dn_maxblkid+1) * dn->dn_datablksz)
                return;
        if (len == DMU_OBJECT_END)
@@ -623,7 +642,6 @@ dmu_tx_hold_free(dmu_tx_t *tx, uint64_t object, uint64_t off, uint64_t len)
                }
        }
 
-       dmu_tx_count_dnode(txh);
        dmu_tx_count_free(txh, off, len);
 }
 
@@ -673,6 +691,7 @@ dmu_tx_hold_zap(dmu_tx_t *tx, uint64_t object, int add, const char *name)
                 * the size will change between now and the dbuf dirty call.
                 */
                if (dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset,
+                   &dn->dn_phys->dn_blkptr[0],
                    dn->dn_phys->dn_blkptr[0].blk_birth)) {
                        txh->txh_space_tooverwrite += SPA_MAXBLOCKSIZE;
                } else {
@@ -688,7 +707,7 @@ dmu_tx_hold_zap(dmu_tx_t *tx, uint64_t object, int add, const char *name)
                 * access the name in this fat-zap so that we'll check
                 * for i/o errors to the leaf blocks, etc.
                 */
-               err = zap_lookup(&dn->dn_objset->os, dn->dn_object, name,
+               err = zap_lookup(dn->dn_objset, dn->dn_object, name,
                    8, 0, NULL);
                if (err == EIO) {
                        tx->tx_err = err;
@@ -696,7 +715,7 @@ dmu_tx_hold_zap(dmu_tx_t *tx, uint64_t object, int add, const char *name)
                }
        }
 
-       err = zap_count_write(&dn->dn_objset->os, dn->dn_object, name, add,
+       err = zap_count_write(dn->dn_objset, dn->dn_object, name, add,
            &txh->txh_space_towrite, &txh->txh_space_tooverwrite);
 
        /*
@@ -771,7 +790,7 @@ dmu_tx_dirty_buf(dmu_tx_t *tx, dmu_buf_impl_t *db)
        dnode_t *dn = db->db_dnode;
 
        ASSERT(tx->tx_txg != 0);
-       ASSERT(tx->tx_objset == NULL || dn->dn_objset == tx->tx_objset->os);
+       ASSERT(tx->tx_objset == NULL || dn->dn_objset == tx->tx_objset);
        ASSERT3U(dn->dn_object, ==, db->db.db_object);
 
        if (tx->tx_anyobj)
@@ -808,10 +827,11 @@ dmu_tx_dirty_buf(dmu_tx_t *tx, dmu_buf_impl_t *db)
                                        match_offset = TRUE;
                                /*
                                 * We will let this hold work for the bonus
-                                * buffer so that we don't need to hold it
-                                * when creating a new object.
+                                * or spill buffer so that we don't need to
+                                * hold it when creating a new object.
                                 */
-                               if (blkid == DB_BONUS_BLKID)
+                               if (blkid == DMU_BONUS_BLKID ||
+                                   blkid == DMU_SPILL_BLKID)
                                        match_offset = TRUE;
                                /*
                                 * They might have to increase nlevels,
@@ -832,8 +852,12 @@ dmu_tx_dirty_buf(dmu_tx_t *tx, dmu_buf_impl_t *db)
                                    txh->txh_arg2 == DMU_OBJECT_END))
                                        match_offset = TRUE;
                                break;
+                       case THT_SPILL:
+                               if (blkid == DMU_SPILL_BLKID)
+                                       match_offset = TRUE;
+                               break;
                        case THT_BONUS:
-                               if (blkid == DB_BONUS_BLKID)
+                               if (blkid == DMU_BONUS_BLKID)
                                        match_offset = TRUE;
                                break;
                        case THT_ZAP:
@@ -931,7 +955,7 @@ dmu_tx_try_assign(dmu_tx_t *tx, uint64_t txg_how)
         * assume that we won't be able to free or overwrite anything.
         */
        if (tx->tx_objset &&
-           dsl_dataset_prev_snap_txg(tx->tx_objset->os->os_dsl_dataset) >
+           dsl_dataset_prev_snap_txg(tx->tx_objset->os_dsl_dataset) >
            tx->tx_lastsnap_txg) {
                towrite += tooverwrite;
                tooverwrite = tofree = 0;
@@ -1112,8 +1136,13 @@ dmu_tx_commit(dmu_tx_t *tx)
        if (tx->tx_tempreserve_cookie)
                dsl_dir_tempreserve_clear(tx->tx_tempreserve_cookie, tx);
 
+       if (!list_is_empty(&tx->tx_callbacks))
+               txg_register_callbacks(&tx->tx_txgh, &tx->tx_callbacks);
+
        if (tx->tx_anyobj == FALSE)
                txg_rele_to_sync(&tx->tx_txgh);
+
+       list_destroy(&tx->tx_callbacks);
        list_destroy(&tx->tx_holds);
 #ifdef ZFS_DEBUG
        dprintf("towrite=%llu written=%llu tofree=%llu freed=%llu\n",
@@ -1142,6 +1171,14 @@ dmu_tx_abort(dmu_tx_t *tx)
                if (dn != NULL)
                        dnode_rele(dn, tx);
        }
+
+       /*
+        * Call any registered callbacks with an error code.
+        */
+       if (!list_is_empty(&tx->tx_callbacks))
+               dmu_tx_do_callbacks(&tx->tx_callbacks, ECANCELED);
+
+       list_destroy(&tx->tx_callbacks);
        list_destroy(&tx->tx_holds);
 #ifdef ZFS_DEBUG
        refcount_destroy_many(&tx->tx_space_written,
@@ -1158,3 +1195,169 @@ dmu_tx_get_txg(dmu_tx_t *tx)
        ASSERT(tx->tx_txg != 0);
        return (tx->tx_txg);
 }
+
+void
+dmu_tx_callback_register(dmu_tx_t *tx, dmu_tx_callback_func_t *func, void *data)
+{
+       dmu_tx_callback_t *dcb;
+
+       dcb = kmem_alloc(sizeof (dmu_tx_callback_t), KM_SLEEP);
+
+       dcb->dcb_func = func;
+       dcb->dcb_data = data;
+
+       list_insert_tail(&tx->tx_callbacks, dcb);
+}
+
+/*
+ * Call all the commit callbacks on a list, with a given error code.
+ */
+void
+dmu_tx_do_callbacks(list_t *cb_list, int error)
+{
+       dmu_tx_callback_t *dcb;
+
+       while (dcb = list_head(cb_list)) {
+               list_remove(cb_list, dcb);
+               dcb->dcb_func(dcb->dcb_data, error);
+               kmem_free(dcb, sizeof (dmu_tx_callback_t));
+       }
+}
+
+/*
+ * Interface to hold a bunch of attributes.
+ * used for creating new files.
+ * attrsize is the total size of all attributes
+ * to be added during object creation
+ *
+ * For updating/adding a single attribute dmu_tx_hold_sa() should be used.
+ */
+
+/*
+ * hold necessary attribute name for attribute registration.
+ * should be a very rare case where this is needed.  If it does
+ * happen it would only happen on the first write to the file system.
+ */
+static void
+dmu_tx_sa_registration_hold(sa_os_t *sa, dmu_tx_t *tx)
+{
+       int i;
+
+       if (!sa->sa_need_attr_registration)
+               return;
+
+       for (i = 0; i != sa->sa_num_attrs; i++) {
+               if (!sa->sa_attr_table[i].sa_registered) {
+                       if (sa->sa_reg_attr_obj)
+                               dmu_tx_hold_zap(tx, sa->sa_reg_attr_obj,
+                                   B_TRUE, sa->sa_attr_table[i].sa_name);
+                       else
+                               dmu_tx_hold_zap(tx, DMU_NEW_OBJECT,
+                                   B_TRUE, sa->sa_attr_table[i].sa_name);
+               }
+       }
+}
+
+
+void
+dmu_tx_hold_spill(dmu_tx_t *tx, uint64_t object)
+{
+       dnode_t *dn;
+       dmu_tx_hold_t *txh;
+       blkptr_t *bp;
+
+       txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, object,
+           THT_SPILL, 0, 0);
+
+       dn = txh->txh_dnode;
+
+       if (dn == NULL)
+               return;
+
+       /* If blkptr doesn't exist then add space to towrite */
+       bp = &dn->dn_phys->dn_spill;
+       if (BP_IS_HOLE(bp)) {
+               txh->txh_space_towrite += SPA_MAXBLOCKSIZE;
+               txh->txh_space_tounref = 0;
+       } else {
+               if (dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset,
+                   bp, bp->blk_birth))
+                       txh->txh_space_tooverwrite += SPA_MAXBLOCKSIZE;
+               else
+                       txh->txh_space_towrite += SPA_MAXBLOCKSIZE;
+               if (bp->blk_birth)
+                       txh->txh_space_tounref += SPA_MAXBLOCKSIZE;
+       }
+}
+
+void
+dmu_tx_hold_sa_create(dmu_tx_t *tx, int attrsize)
+{
+       sa_os_t *sa = tx->tx_objset->os_sa;
+
+       dmu_tx_hold_bonus(tx, DMU_NEW_OBJECT);
+
+       if (tx->tx_objset->os_sa->sa_master_obj == 0)
+               return;
+
+       if (tx->tx_objset->os_sa->sa_layout_attr_obj)
+               dmu_tx_hold_zap(tx, sa->sa_layout_attr_obj, B_TRUE, NULL);
+       else {
+               dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_LAYOUTS);
+               dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_REGISTRY);
+               dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL);
+               dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL);
+       }
+
+       dmu_tx_sa_registration_hold(sa, tx);
+
+       if (attrsize <= DN_MAX_BONUSLEN && !sa->sa_force_spill)
+               return;
+
+       (void) dmu_tx_hold_object_impl(tx, tx->tx_objset, DMU_NEW_OBJECT,
+           THT_SPILL, 0, 0);
+}
+
+/*
+ * Hold SA attribute
+ *
+ * dmu_tx_hold_sa(dmu_tx_t *tx, sa_handle_t *, attribute, add, size)
+ *
+ * variable_size is the total size of all variable sized attributes
+ * passed to this function.  It is not the total size of all
+ * variable size attributes that *may* exist on this object.
+ */
+void
+dmu_tx_hold_sa(dmu_tx_t *tx, sa_handle_t *hdl, boolean_t may_grow)
+{
+       uint64_t object;
+       sa_os_t *sa = tx->tx_objset->os_sa;
+
+       ASSERT(hdl != NULL);
+
+       object = sa_handle_object(hdl);
+
+       dmu_tx_hold_bonus(tx, object);
+
+       if (tx->tx_objset->os_sa->sa_master_obj == 0)
+               return;
+
+       if (tx->tx_objset->os_sa->sa_reg_attr_obj == 0 ||
+           tx->tx_objset->os_sa->sa_layout_attr_obj == 0) {
+               dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_LAYOUTS);
+               dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_REGISTRY);
+               dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL);
+               dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL);
+       }
+
+       dmu_tx_sa_registration_hold(sa, tx);
+
+       if (may_grow && tx->tx_objset->os_sa->sa_layout_attr_obj)
+               dmu_tx_hold_zap(tx, sa->sa_layout_attr_obj, B_TRUE, NULL);
+
+       if (sa->sa_force_spill || may_grow || hdl->sa_spill ||
+           ((dmu_buf_impl_t *)hdl->sa_bonus)->db_dnode->dn_have_spill) {
+               ASSERT(tx->tx_txg == 0);
+               dmu_tx_hold_spill(tx, object);
+       }
+}