]> git.sur5r.net Git - openldap/blobdiff - libraries/liblmdb/mdb.c
ITS#7736 fix regression in ITS#7733 patch
[openldap] / libraries / liblmdb / mdb.c
index f895c0fb5e2529e0a940747d5db132f3be38dda0..9ade333ed951e459ce623b1c1ca0b29169494cf9 100644 (file)
@@ -75,6 +75,7 @@
 #ifndef _WIN32
 #include <pthread.h>
 #ifdef MDB_USE_POSIX_SEM
+# define MDB_USE_HASH          1
 #include <semaphore.h>
 #endif
 #endif
  *     @{
  */
 #ifdef _WIN32
+#define MDB_USE_HASH   1
+#define MDB_PIDLOCK    0
 #define pthread_t      DWORD
 #define pthread_mutex_t        HANDLE
 #define pthread_key_t  DWORD
 #define GET_PAGESIZE(x) {SYSTEM_INFO si; GetSystemInfo(&si); (x) = si.dwPageSize;}
 #define        close(fd)       (CloseHandle(fd) ? 0 : -1)
 #define        munmap(ptr,len) UnmapViewOfFile(ptr)
+#ifdef PROCESS_QUERY_LIMITED_INFORMATION
+#define MDB_PROCESS_QUERY_LIMITED_INFORMATION PROCESS_QUERY_LIMITED_INFORMATION
 #else
+#define MDB_PROCESS_QUERY_LIMITED_INFORMATION 0x1000
+#endif
+#define        Z       "I"
+#else
+
+#define        Z       "z"                     /**< printf format modifier for size_t */
+
+       /** For MDB_LOCK_FORMAT: True if readers take a pid lock in the lockfile */
+#define MDB_PIDLOCK                    1
 
 #ifdef MDB_USE_POSIX_SEM
 
@@ -281,33 +295,33 @@ typedef MDB_ID    txnid_t;
  *     @{
  */
 #ifndef MDB_DEBUG
-       /**     Enable debug output.
+       /**     Enable debug output.  Needs variable argument macros (a C99 feature).
         *      Set this to 1 for copious tracing. Set to 2 to add dumps of all IDLs
         *      read from and written to the database (used for free space management).
         */
 #define MDB_DEBUG 0
 #endif
 
-#if !(__STDC_VERSION__ >= 199901L || defined(__GNUC__))
-# undef  MDB_DEBUG
-# define MDB_DEBUG     0
-# define DPRINTF       (void)  /* Vararg macros may be unsupported */
-#elif MDB_DEBUG
+#if MDB_DEBUG
 static int mdb_debug;
 static txnid_t mdb_debug_start;
 
-       /**     Print a debug message with printf formatting. */
-# define DPRINTF(fmt, ...)     /**< Requires 2 or more args */ \
-       ((void) ((mdb_debug) && \
-        fprintf(stderr, "%s:%d " fmt "\n", __func__, __LINE__, __VA_ARGS__)))
+       /**     Print a debug message with printf formatting.
+        *      Requires double parenthesis around 2 or more args.
+        */
+# define DPRINTF(args) ((void) ((mdb_debug) && DPRINTF0 args))
+# define DPRINTF0(fmt, ...) \
+       fprintf(stderr, "%s:%d " fmt "\n", __func__, __LINE__, __VA_ARGS__)
 #else
-# define DPRINTF(fmt, ...)     ((void) 0)
-# define MDB_DEBUG_SKIP
+# define DPRINTF(args) ((void) 0)
 #endif
        /**     Print a debug string.
         *      The string is printed literally, with no format processing.
         */
-#define DPUTS(arg)     DPRINTF("%s", arg)
+#define DPUTS(arg)     DPRINTF(("%s", arg))
+       /** Debuging output value of a cursor DBI: Negative in a sub-cursor. */
+#define DDBI(mc) \
+       (((mc)->mc_flags & C_SUB) ? -(int)(mc)->mc_dbi : (int)(mc)->mc_dbi)
 /** @} */
 
        /** A default memory page size.
@@ -344,8 +358,10 @@ static txnid_t mdb_debug_start;
         */
 #define MDB_MAGIC       0xBEEFC0DE
 
-       /**     The version number for a database's file format. */
-#define MDB_VERSION     1
+       /**     The version number for a database's datafile format. */
+#define MDB_DATA_VERSION        1
+       /**     The version number for a database's lockfile format. */
+#define MDB_LOCK_VERSION        1
 
        /**     @brief The maximum size of a key in the database.
         *
@@ -381,7 +397,7 @@ static txnid_t mdb_debug_start;
         */
 #define        DKEY(x) mdb_dkey(x, kbuf)
 #else
-#define        DKBUF   typedef int dummy_kbuf  /* so we can put ';' after */
+#define        DKBUF
 #define DKEY(x)        0
 #endif
 
@@ -414,7 +430,8 @@ typedef uint16_t     indx_t;
  *
  *     If #MDB_NOTLS is set, the slot address is not saved in thread-specific data.
  *
- *     No reader table is used if the database is on a read-only filesystem.
+ *     No reader table is used if the database is on a read-only filesystem, or
+ *     if #MDB_NOLOCK is set.
  *
  *     Since the database uses multi-version concurrency control, readers don't
  *     actually need any locking. This table is used to keep track of which
@@ -513,8 +530,8 @@ typedef struct MDB_txbody {
                /** Stamp identifying this as an MDB file. It must be set
                 *      to #MDB_MAGIC. */
        uint32_t        mtb_magic;
-               /** Version number of this lock file. Must be set to #MDB_VERSION. */
-       uint32_t        mtb_version;
+               /** Format of this lock file. Must be set to #MDB_LOCK_FORMAT. */
+       uint32_t        mtb_format;
 #if defined(_WIN32) || defined(MDB_USE_POSIX_SEM)
        char    mtb_rmname[MNAME_LEN];
 #else
@@ -540,7 +557,7 @@ typedef struct MDB_txninfo {
        union {
                MDB_txbody mtb;
 #define mti_magic      mt1.mtb.mtb_magic
-#define mti_version    mt1.mtb.mtb_version
+#define mti_format     mt1.mtb.mtb_format
 #define mti_mutex      mt1.mtb.mtb_mutex
 #define mti_rmname     mt1.mtb.mtb_rmname
 #define mti_txnid      mt1.mtb.mtb_txnid
@@ -559,6 +576,13 @@ typedef struct MDB_txninfo {
        } mt2;
        MDB_reader      mti_readers[1];
 } MDB_txninfo;
+
+       /** Lockfile format signature: version, features and field layout */
+#define MDB_LOCK_FORMAT \
+       ((uint32_t) \
+        ((MDB_LOCK_VERSION) \
+         /* Flags which describe functionality */ \
+         + (((MDB_PIDLOCK) != 0) << 16)))
 /** @} */
 
 /** Common header for all page types.
@@ -582,7 +606,7 @@ typedef struct MDB_page {
 #define        P_LEAF           0x02           /**< leaf page */
 #define        P_OVERFLOW       0x04           /**< overflow page */
 #define        P_META           0x08           /**< meta page */
-#define        P_DIRTY          0x10           /**< dirty page */
+#define        P_DIRTY          0x10           /**< dirty page, also set for #P_SUBP pages */
 #define        P_LEAF2          0x20           /**< for #MDB_DUPFIXED records */
 #define        P_SUBP           0x40           /**< for #MDB_DUPSORT sub-pages */
 #define        P_KEEP           0x8000         /**< leave this page alone during spill */
@@ -738,9 +762,12 @@ typedef struct MDB_node {
         */
 #define LEAF2KEY(p, i, ks)     ((char *)(p) + PAGEHDRSZ + ((i)*(ks)))
 
-       /** Set the \b node's key into \b key, if requested. */
-#define MDB_GET_KEY(node, key) { if ((key) != NULL) { \
-       (key)->mv_size = NODEKSZ(node); (key)->mv_data = NODEKEY(node); } }
+       /** Set the \b node's key into \b keyptr, if requested. */
+#define MDB_GET_KEY(node, keyptr)      { if ((keyptr) != NULL) { \
+       (keyptr)->mv_size = NODEKSZ(node); (keyptr)->mv_data = NODEKEY(node); } }
+
+       /** Set the \b node's key into \b key. */
+#define MDB_GET_KEY2(node, key)        { key.mv_size = NODEKSZ(node); key.mv_data = NODEKEY(node); }
 
        /** Information about a single database in the environment. */
 typedef struct MDB_db {
@@ -765,12 +792,15 @@ typedef struct MDB_db {
        /** Handle for the default DB. */
 #define        MAIN_DBI        1
 
-       /** Meta page content. */
+       /** Meta page content.
+        *      A meta page is the start point for accessing a database snapshot.
+        *      Pages 0-1 are meta pages. Transaction N writes meta page #(N % 2).
+        */
 typedef struct MDB_meta {
                /** Stamp identifying this as an MDB file. It must be set
                 *      to #MDB_MAGIC. */
        uint32_t        mm_magic;
-               /** Version number of this lock file. Must be set to #MDB_VERSION. */
+               /** Version number of this lock file. Must be set to #MDB_DATA_VERSION. */
        uint32_t        mm_version;
        void            *mm_address;            /**< address for fixed mapping */
        size_t          mm_mapsize;                     /**< size of mmap region */
@@ -825,13 +855,16 @@ struct MDB_txn {
        /** The list of pages that became unused during this transaction.
         */
        MDB_IDL         mt_free_pgs;
-       /** The list of dirty pages we temporarily wrote to disk
-        *      because the dirty list was full.
+       /** The sorted list of dirty pages we temporarily wrote to disk
+        *      because the dirty list was full. page numbers in here are
+        *      shifted left by 1, deleted slots have the LSB set.
         */
        MDB_IDL         mt_spill_pgs;
        union {
-               MDB_ID2L        dirty_list;     /**< for write txns: modified pages */
-               MDB_reader      *reader;        /**< this thread's reader table slot or NULL */
+               /** For write txns: Modified pages. Sorted when not MDB_WRITEMAP. */
+               MDB_ID2L        dirty_list;
+               /** For read txns: This thread/txn's reader table slot, or NULL. */
+               MDB_reader      *reader;
        } mt_u;
        /** Array of records for each DB known in the environment. */
        MDB_dbx         *mt_dbxs;
@@ -841,9 +874,9 @@ struct MDB_txn {
  *     @ingroup internal
  * @{
  */
-#define DB_DIRTY       0x01            /**< DB was written in this txn */
-#define DB_STALE       0x02            /**< DB record is older than txnID */
-#define DB_NEW         0x04            /**< DB handle opened in this txn */
+#define DB_DIRTY       0x01            /**< DB was modified or is DUPSORT data */
+#define DB_STALE       0x02            /**< Named-DB record is older than txnID */
+#define DB_NEW         0x04            /**< Named-DB handle opened in this txn */
 #define DB_VALID       0x08            /**< DB handle is valid, see also #MDB_VALID */
 /** @} */
        /** In write txns, array of cursors for each DB */
@@ -865,12 +898,12 @@ struct MDB_txn {
 #define MDB_TXN_SPILLS         0x08            /**< txn or a parent has spilled pages */
 /** @} */
        unsigned int    mt_flags;               /**< @ref mdb_txn */
-       /** dirty_list maxsize - # of allocated pages allowed, including in parent txns */
-       unsigned int    mt_dirty_room;
-       /** Tracks which of the two meta pages was used at the start
-        *      of this transaction.
+       /** dirty_list room: Array size - #dirty pages visible to this txn.
+        *      Includes ancestor txns' dirty pages not hidden by other txns'
+        *      dirty/spilled pages. Thus commit(nested txn) has room to merge
+        *      dirty_list into mt_parent after freeing hidden mt_parent pages.
         */
-       unsigned int    mt_toggle;
+       unsigned int    mt_dirty_room;
 };
 
 /** Enough space for 2^32 nodes with minimum of 2 keys per node. I.e., plenty.
@@ -881,7 +914,14 @@ struct MDB_txn {
 
 struct MDB_xcursor;
 
-       /** Cursors are used for all DB operations */
+       /** Cursors are used for all DB operations.
+        *      A cursor holds a path of (page pointer, key index) from the DB
+        *      root to a position in the DB, plus other state. #MDB_DUPSORT
+        *      cursors include an xcursor to the current data item. Write txns
+        *      track their cursors and keep them up to date when data moves.
+        *      Exception: An xcursor's pointer to a #P_SUBP page can be stale.
+        *      (A node with #F_DUPDATA but no #F_SUBDATA contains a subpage).
+        */
 struct MDB_cursor {
        /** Next cursor on this DB in this txn */
        MDB_cursor      *mc_next;
@@ -909,6 +949,7 @@ struct MDB_cursor {
 #define C_INITIALIZED  0x01    /**< cursor has been initialized and is valid */
 #define C_EOF  0x02                    /**< No more data */
 #define C_SUB  0x04                    /**< Cursor is a sub-cursor */
+#define C_DEL  0x08                    /**< last op was a cursor_del */
 #define C_SPLITTING    0x20            /**< Cursor is in page_split */
 #define C_UNTRACK      0x40            /**< Un-track cursor when closing */
 /** @} */
@@ -950,6 +991,8 @@ struct MDB_env {
 #define        MDB_ENV_ACTIVE  0x20000000U
        /** me_txkey is set */
 #define        MDB_ENV_TXKEY   0x10000000U
+       /** Have liveness lock in reader table */
+#define        MDB_LIVE_READER 0x08000000U
        uint32_t        me_flags;               /**< @ref mdb_env */
        unsigned int    me_psize;       /**< size of a page, from #GET_PAGESIZE */
        unsigned int    me_maxreaders;  /**< size of the reader table */
@@ -981,6 +1024,7 @@ struct MDB_env {
        /** Max size of a node on a page */
        unsigned int    me_nodemax;
 #ifdef _WIN32
+       int             me_pidquery;            /**< Used in OpenProcess */
        HANDLE          me_rmutex;              /* Windows mutexes don't reside in shared mem */
        HANDLE          me_wmutex;
 #elif defined(MDB_USE_POSIX_SEM)
@@ -991,8 +1035,8 @@ struct MDB_env {
 
        /** Nested transaction */
 typedef struct MDB_ntxn {
-       MDB_txn         mnt_txn;                /* the transaction */
-       MDB_pgstate     mnt_pgstate;    /* parent transaction's saved freestate */
+       MDB_txn         mnt_txn;                /**< the transaction */
+       MDB_pgstate     mnt_pgstate;    /**< parent transaction's saved freestate */
 } MDB_ntxn;
 
        /** max number of pages to commit in one writev() call */
@@ -1014,6 +1058,8 @@ static int  mdb_page_search_root(MDB_cursor *mc,
                            MDB_val *key, int modify);
 #define MDB_PS_MODIFY  1
 #define MDB_PS_ROOTONLY        2
+#define MDB_PS_FIRST   4
+#define MDB_PS_LAST            8
 static int  mdb_page_search(MDB_cursor *mc,
                            MDB_val *key, int flags);
 static int     mdb_page_merge(MDB_cursor *csrc, MDB_cursor *cdst);
@@ -1099,8 +1145,10 @@ static char *const mdb_errstr[] = {
        "MDB_CURSOR_FULL: Internal error - cursor stack limit reached",
        "MDB_PAGE_FULL: Internal error - page has no more space",
        "MDB_MAP_RESIZED: Database contents grew beyond environment mapsize",
-       "MDB_INCOMPATIBLE: Database flags changed or would change",
+       "MDB_INCOMPATIBLE: Operation and DB incompatible, or DB flags changed",
        "MDB_BAD_RSLOT: Invalid reuse of reader locktable slot",
+       "MDB_BAD_TXN: Transaction cannot recover - it must be aborted",
+       "MDB_BAD_VALSIZE: Too big key/data, key is empty, or wrong DUPFIXED size",
 };
 
 char *
@@ -1159,14 +1207,14 @@ mdb_page_list(MDB_page *mp)
        DKBUF;
 
        nkeys = NUMKEYS(mp);
-       fprintf(stderr, "Page %zu numkeys %d\n", mp->mp_pgno, nkeys);
+       fprintf(stderr, "Page %"Z"u numkeys %d\n", mp->mp_pgno, nkeys);
        for (i=0; i<nkeys; i++) {
                node = NODEPTR(mp, i);
                key.mv_size = node->mn_ksize;
                key.mv_data = node->mn_data;
                nsize = NODESIZE + NODEKSZ(node) + sizeof(indx_t);
                if (IS_BRANCH(mp)) {
-                       fprintf(stderr, "key %d: page %zu, %s\n", i, NODEPGNO(node),
+                       fprintf(stderr, "key %d: page %"Z"u, %s\n", i, NODEPGNO(node),
                                DKEY(&key));
                } else {
                        if (F_ISSET(node->mn_flags, F_BIGDATA))
@@ -1197,7 +1245,7 @@ mdb_cursor_chk(MDB_cursor *mc)
 }
 #endif
 
-#if MDB_DEBUG > 2
+#if (MDB_DEBUG) > 2
 /** Count all the pages in each DB and in the freelist
  *  and make sure it matches the actual number of pages
  *  being used.
@@ -1225,7 +1273,7 @@ static void mdb_audit(MDB_txn *txn)
                        txn->mt_dbs[i].md_leaf_pages +
                        txn->mt_dbs[i].md_overflow_pages;
                if (txn->mt_dbs[i].md_flags & MDB_DUPSORT) {
-                       mdb_page_search(&mc, NULL, 0);
+                       mdb_page_search(&mc, NULL, MDB_PS_FIRST);
                        do {
                                unsigned j;
                                MDB_page *mp;
@@ -1262,7 +1310,7 @@ mdb_dcmp(MDB_txn *txn, MDB_dbi dbi, const MDB_val *a, const MDB_val *b)
        return txn->mt_dbxs[dbi].md_dcmp(a, b);
 }
 
-/** Allocate a page.
+/** Allocate memory for a page.
  * Re-use old malloc'd pages first for singletons, otherwise just malloc.
  */
 static MDB_page *
@@ -1299,7 +1347,7 @@ mdb_page_free(MDB_env *env, MDB_page *mp)
        env->me_dpages = mp;
 }
 
-/* Free a dirty page */
+/** Free a dirty page */
 static void
 mdb_dpage_free(MDB_env *env, MDB_page *dp)
 {
@@ -1326,58 +1374,83 @@ mdb_dlist_free(MDB_txn *txn)
        dl[0].mid = 0;
 }
 
-/* Set or clear P_KEEP in non-overflow, non-sub pages in known cursors.
- * When clearing, only consider backup cursors (from parent txns) since
- * other P_KEEP flags have already been cleared.
+/** Set or clear P_KEEP in dirty, non-overflow, non-sub pages watched by txn.
  * @param[in] mc A cursor handle for the current operation.
  * @param[in] pflags Flags of the pages to update:
  * P_DIRTY to set P_KEEP, P_DIRTY|P_KEEP to clear it.
+ * @param[in] all No shortcuts. Needed except after a full #mdb_page_flush().
+ * @return 0 on success, non-zero on failure.
  */
-static void
-mdb_cursorpages_mark(MDB_cursor *mc, unsigned pflags)
+static int
+mdb_pages_xkeep(MDB_cursor *mc, unsigned pflags, int all)
 {
+       enum { Mask = P_SUBP|P_DIRTY|P_KEEP };
        MDB_txn *txn = mc->mc_txn;
-       MDB_cursor *m2, *m3;
+       MDB_cursor *m3;
        MDB_xcursor *mx;
+       MDB_page *dp, *mp;
+       MDB_node *leaf;
        unsigned i, j;
+       int rc = MDB_SUCCESS, level;
 
+       /* Mark pages seen by cursors */
        if (mc->mc_flags & C_UNTRACK)
                mc = NULL;                              /* will find mc in mt_cursors */
        for (i = txn->mt_numdbs;; mc = txn->mt_cursors[--i]) {
                for (; mc; mc=mc->mc_next) {
-                       m2 = pflags == P_DIRTY ? mc : mc->mc_backup;
-                       for (; m2; m2 = m2->mc_backup) {
-                               for (m3=m2; m3->mc_flags & C_INITIALIZED; m3=&mx->mx_cursor) {
-                                       for (j=0; j<m3->mc_snum; j++)
-                                               if ((m3->mc_pg[j]->mp_flags & (P_SUBP|P_DIRTY|P_KEEP))
-                                                               == pflags)
-                                                       m3->mc_pg[j]->mp_flags ^= P_KEEP;
-                                       if (!(m3->mc_db->md_flags & MDB_DUPSORT))
-                                               break;
-                                       /* Cursor backups have mx malloced at the end of m2 */
-                                       mx = (m3 == mc ? m3->mc_xcursor : (MDB_xcursor *)(m3+1));
+                       if (!(mc->mc_flags & C_INITIALIZED))
+                               continue;
+                       for (m3 = mc;; m3 = &mx->mx_cursor) {
+                               mp = NULL;
+                               for (j=0; j<m3->mc_snum; j++) {
+                                       mp = m3->mc_pg[j];
+                                       if ((mp->mp_flags & Mask) == pflags)
+                                               mp->mp_flags ^= P_KEEP;
                                }
+                               mx = m3->mc_xcursor;
+                               /* Proceed to mx if it is at a sub-database */
+                               if (! (mx && (mx->mx_cursor.mc_flags & C_INITIALIZED)))
+                                       break;
+                               if (! (mp && (mp->mp_flags & P_LEAF)))
+                                       break;
+                               leaf = NODEPTR(mp, m3->mc_ki[j-1]);
+                               if (!(leaf->mn_flags & F_SUBDATA))
+                                       break;
                        }
                }
                if (i == 0)
                        break;
        }
+
+       if (all) {
+               /* Mark dirty root pages */
+               for (i=0; i<txn->mt_numdbs; i++) {
+                       if (txn->mt_dbflags[i] & DB_DIRTY) {
+                               pgno_t pgno = txn->mt_dbs[i].md_root;
+                               if (pgno == P_INVALID)
+                                       continue;
+                               if ((rc = mdb_page_get(txn, pgno, &dp, &level)) != MDB_SUCCESS)
+                                       break;
+                               if ((dp->mp_flags & Mask) == pflags && level <= 1)
+                                       dp->mp_flags ^= P_KEEP;
+                       }
+               }
+       }
+
+       return rc;
 }
 
-static int mdb_page_flush(MDB_txn *txn);
+static int mdb_page_flush(MDB_txn *txn, int keep);
 
 /**    Spill pages from the dirty list back to disk.
  * This is intended to prevent running into #MDB_TXN_FULL situations,
  * but note that they may still occur in a few cases:
- *     1) pages in #MDB_DUPSORT sub-DBs are never spilled, so if there
- *      are too many of these dirtied in one txn, the txn may still get
- *      too full.
+ *     1) our estimate of the txn size could be too small. Currently this
+ *      seems unlikely, except with a large number of #MDB_MULTIPLE items.
  *     2) child txns may run out of space if their parents dirtied a
  *      lot of pages and never spilled them. TODO: we probably should do
  *      a preemptive spill during #mdb_txn_begin() of a child txn, if
  *      the parent's dirty_room is below a given threshold.
- *     3) our estimate of the txn size could be too small. At the
- *      moment this seems unlikely.
  *
  * Otherwise, if not using nested txns, it is expected that apps will
  * not run into #MDB_TXN_FULL any more. The pages are flushed to disk
@@ -1407,7 +1480,7 @@ mdb_page_spill(MDB_cursor *m0, MDB_val *key, MDB_val *data)
        MDB_txn *txn = m0->mc_txn;
        MDB_page *dp;
        MDB_ID2L dl = txn->mt_u.dirty_list;
-       unsigned int i, j;
+       unsigned int i, j, need;
        int rc;
 
        if (m0->mc_flags & C_SUB)
@@ -1422,6 +1495,7 @@ mdb_page_spill(MDB_cursor *m0, MDB_val *key, MDB_val *data)
        if (key)
                i += (LEAFSIZE(key, data) + txn->mt_env->me_psize) / txn->mt_env->me_psize;
        i += i; /* double it for good measure */
+       need = i;
 
        if (txn->mt_dirty_room > i)
                return MDB_SUCCESS;
@@ -1430,24 +1504,36 @@ mdb_page_spill(MDB_cursor *m0, MDB_val *key, MDB_val *data)
                txn->mt_spill_pgs = mdb_midl_alloc(MDB_IDL_UM_MAX);
                if (!txn->mt_spill_pgs)
                        return ENOMEM;
-       }
-
-       /* Mark all the dirty root pages we want to preserve */
-       for (i=0; i<txn->mt_numdbs; i++) {
-               if (txn->mt_dbflags[i] & DB_DIRTY) {
-                       j = mdb_mid2l_search(dl, txn->mt_dbs[i].md_root);
-                       if (j <= dl[0].mid) {
-                               dp = dl[j].mptr;
-                               dp->mp_flags |= P_KEEP;
-                       }
+       } else {
+               /* purge deleted slots */
+               MDB_IDL sl = txn->mt_spill_pgs;
+               unsigned int num = sl[0];
+               j=0;
+               for (i=1; i<=num; i++) {
+                       if (!(sl[i] & 1))
+                               sl[++j] = sl[i];
                }
+               sl[0] = j;
        }
 
-       /* Preserve pages used by cursors */
-       mdb_cursorpages_mark(m0, P_DIRTY);
+       /* Preserve pages which may soon be dirtied again */
+       if ((rc = mdb_pages_xkeep(m0, P_DIRTY, 1)) != MDB_SUCCESS)
+               goto done;
+
+       /* Less aggressive spill - we originally spilled the entire dirty list,
+        * with a few exceptions for cursor pages and DB root pages. But this
+        * turns out to be a lot of wasted effort because in a large txn many
+        * of those pages will need to be used again. So now we spill only 1/8th
+        * of the dirty pages. Testing revealed this to be a good tradeoff,
+        * better than 1/2, 1/4, or 1/10.
+        */
+       if (need < MDB_IDL_UM_MAX / 8)
+               need = MDB_IDL_UM_MAX / 8;
 
        /* Save the page IDs of all the pages we're flushing */
-       for (i=1; i<=dl[0].mid; i++) {
+       /* flush from the tail forward, this saves a lot of shifting later on. */
+       for (i=dl[0].mid; i && need; i--) {
+               MDB_ID pn = dl[i].mid << 1;
                dp = dl[i].mptr;
                if (dp->mp_flags & P_KEEP)
                        continue;
@@ -1458,8 +1544,8 @@ mdb_page_spill(MDB_cursor *m0, MDB_val *key, MDB_val *data)
                        MDB_txn *tx2;
                        for (tx2 = txn->mt_parent; tx2; tx2 = tx2->mt_parent) {
                                if (tx2->mt_spill_pgs) {
-                                       j = mdb_midl_search(tx2->mt_spill_pgs, dl[i].mid);
-                                       if (j <= tx2->mt_spill_pgs[0] && tx2->mt_spill_pgs[j] == dl[i].mid) {
+                                       j = mdb_midl_search(tx2->mt_spill_pgs, pn);
+                                       if (j <= tx2->mt_spill_pgs[0] && tx2->mt_spill_pgs[j] == pn) {
                                                dp->mp_flags |= P_KEEP;
                                                break;
                                        }
@@ -1468,38 +1554,21 @@ mdb_page_spill(MDB_cursor *m0, MDB_val *key, MDB_val *data)
                        if (tx2)
                                continue;
                }
-               if ((rc = mdb_midl_append(&txn->mt_spill_pgs, dl[i].mid)))
-                       return rc;
+               if ((rc = mdb_midl_append(&txn->mt_spill_pgs, pn)))
+                       goto done;
+               need--;
        }
        mdb_midl_sort(txn->mt_spill_pgs);
 
-       rc = mdb_page_flush(txn);
+       /* Flush the spilled part of dirty list */
+       if ((rc = mdb_page_flush(txn, i)) != MDB_SUCCESS)
+               goto done;
 
-       mdb_cursorpages_mark(m0, P_DIRTY|P_KEEP);
+       /* Reset any dirty pages we kept that page_flush didn't see */
+       rc = mdb_pages_xkeep(m0, P_DIRTY|P_KEEP, i);
 
-       if (rc == 0) {
-               if (txn->mt_parent) {
-                       MDB_txn *tx2;
-                       pgno_t pgno = dl[i].mid;
-                       txn->mt_dirty_room = txn->mt_parent->mt_dirty_room - dl[0].mid;
-                       /* dirty pages that are dirty in an ancestor don't
-                        * count against this txn's dirty_room.
-                        */
-                       for (i=1; i<=dl[0].mid; i++) {
-                               for (tx2 = txn->mt_parent; tx2; tx2 = tx2->mt_parent) {
-                                       j = mdb_mid2l_search(tx2->mt_u.dirty_list, pgno);
-                                       if (j <= tx2->mt_u.dirty_list[0].mid &&
-                                               tx2->mt_u.dirty_list[j].mid == pgno) {
-                                               txn->mt_dirty_room++;
-                                               break;
-                                       }
-                               }
-                       }
-               } else {
-                       txn->mt_dirty_room = MDB_IDL_UM_MAX - dl[0].mid;
-               }
-               txn->mt_flags |= MDB_TXN_SPILLS;
-       }
+done:
+       txn->mt_flags |= rc ? MDB_TXN_ERROR : MDB_TXN_SPILLS;
        return rc;
 }
 
@@ -1509,12 +1578,14 @@ mdb_find_oldest(MDB_txn *txn)
 {
        int i;
        txnid_t mr, oldest = txn->mt_txnid - 1;
-       MDB_reader *r = txn->mt_env->me_txns->mti_readers;
-       for (i = txn->mt_env->me_txns->mti_numreaders; --i >= 0; ) {
-               if (r[i].mr_pid) {
-                       mr = r[i].mr_txnid;
-                       if (oldest > mr)
-                               oldest = mr;
+       if (txn->mt_env->me_txns) {
+               MDB_reader *r = txn->mt_env->me_txns->mti_readers;
+               for (i = txn->mt_env->me_txns->mti_numreaders; --i >= 0; ) {
+                       if (r[i].mr_pid) {
+                               mr = r[i].mr_txnid;
+                               if (oldest > mr)
+                                       oldest = mr;
+                       }
                }
        }
        return oldest;
@@ -1538,9 +1609,14 @@ mdb_page_dirty(MDB_txn *txn, MDB_page *mp)
        txn->mt_dirty_room--;
 }
 
-/** Allocate pages for writing.
+/** Allocate page numbers and memory for writing.  Maintain me_pglast,
+ * me_pghead and mt_next_pgno.
+ *
  * If there are free pages available from older transactions, they
- * will be re-used first. Otherwise a new page will be allocated.
+ * are re-used first. Otherwise allocate a new page at mt_next_pgno.
+ * Do not modify the freedB, just merge freeDB records into me_pghead[]
+ * and move me_pglast to say which records were consumed.  Only this
+ * function can create me_pghead and move me_pglast/mt_next_pgno.
  * @param[in] mc cursor A cursor handle identifying the transaction and
  *     database for which we are allocating.
  * @param[in] num the number of pages to allocate.
@@ -1604,7 +1680,7 @@ mdb_page_alloc(MDB_cursor *mc, int num, MDB_page **mp)
                        mdb_cursor_init(&m2, txn, FREE_DBI, NULL);
                        if (last) {
                                op = MDB_SET_RANGE;
-                               key.mv_data = &last; /* will loop up last+1 */
+                               key.mv_data = &last; /* will look up last+1 */
                                key.mv_size = sizeof(last);
                        }
                        if (Paranoid && mc->mc_dbi == FREE_DBI)
@@ -1642,11 +1718,11 @@ mdb_page_alloc(MDB_cursor *mc, int num, MDB_page **mp)
                        mop = env->me_pghead;
                }
                env->me_pglast = last;
-#if MDB_DEBUG > 1
-               DPRINTF("IDL read txn %zu root %zu num %u",
-                               last, txn->mt_dbs[FREE_DBI].md_root, i);
+#if (MDB_DEBUG) > 1
+               DPRINTF(("IDL read txn %"Z"u root %"Z"u num %u",
+                       last, txn->mt_dbs[FREE_DBI].md_root, i));
                for (k = i; k; k--)
-                       DPRINTF("IDL %zu", idl[k]);
+                       DPRINTF(("IDL %"Z"u", idl[k]));
 #endif
                /* Merge in descending sorted order */
                j = mop_len;
@@ -1719,26 +1795,28 @@ mdb_page_copy(MDB_page *dst, MDB_page *src, unsigned int psize)
 /** Pull a page off the txn's spill list, if present.
  * If a page being referenced was spilled to disk in this txn, bring
  * it back and make it dirty/writable again.
- * @param[in] tx0 the transaction handle.
+ * @param[in] txn the transaction handle.
  * @param[in] mp the page being referenced.
  * @param[out] ret the writable page, if any. ret is unchanged if
  * mp wasn't spilled.
  */
 static int
-mdb_page_unspill(MDB_txn *tx0, MDB_page *mp, MDB_page **ret)
+mdb_page_unspill(MDB_txn *txn, MDB_page *mp, MDB_page **ret)
 {
-       MDB_env *env = tx0->mt_env;
-       MDB_txn *txn;
+       MDB_env *env = txn->mt_env;
+       const MDB_txn *tx2;
        unsigned x;
-       pgno_t pgno = mp->mp_pgno;
+       pgno_t pgno = mp->mp_pgno, pn = pgno << 1;
 
-       for (txn = tx0; txn; txn=txn->mt_parent) {
-               if (!txn->mt_spill_pgs)
+       for (tx2 = txn; tx2; tx2=tx2->mt_parent) {
+               if (!tx2->mt_spill_pgs)
                        continue;
-               x = mdb_midl_search(txn->mt_spill_pgs, pgno);
-               if (x <= txn->mt_spill_pgs[0] && txn->mt_spill_pgs[x] == pgno) {
+               x = mdb_midl_search(tx2->mt_spill_pgs, pn);
+               if (x <= tx2->mt_spill_pgs[0] && tx2->mt_spill_pgs[x] == pn) {
                        MDB_page *np;
                        int num;
+                       if (txn->mt_dirty_room == 0)
+                               return MDB_TXN_FULL;
                        if (IS_OVERFLOW(mp))
                                num = mp->mp_pages;
                        else
@@ -1754,31 +1832,20 @@ mdb_page_unspill(MDB_txn *tx0, MDB_page *mp, MDB_page **ret)
                                else
                                        mdb_page_copy(np, mp, env->me_psize);
                        }
-                       if (txn == tx0) {
-                               /* If in current txn, this page is no longer spilled */
-                               for (; x < txn->mt_spill_pgs[0]; x++)
-                                       txn->mt_spill_pgs[x] = txn->mt_spill_pgs[x+1];
-                               txn->mt_spill_pgs[0]--;
+                       if (tx2 == txn) {
+                               /* If in current txn, this page is no longer spilled.
+                                * If it happens to be the last page, truncate the spill list.
+                                * Otherwise mark it as deleted by setting the LSB.
+                                */
+                               if (x == txn->mt_spill_pgs[0])
+                                       txn->mt_spill_pgs[0]--;
+                               else
+                                       txn->mt_spill_pgs[x] |= 1;
                        }       /* otherwise, if belonging to a parent txn, the
                                 * page remains spilled until child commits
                                 */
 
-                       if (txn->mt_parent) {
-                               MDB_txn *tx2;
-                               /* If this page is also in a parent's dirty list, then
-                                * it's already accounted in dirty_room, and we need to
-                                * cancel out the decrement that mdb_page_dirty does.
-                                */
-                               for (tx2 = txn->mt_parent; tx2; tx2 = tx2->mt_parent) {
-                                       x = mdb_mid2l_search(tx2->mt_u.dirty_list, pgno);
-                                       if (x <= tx2->mt_u.dirty_list[0].mid &&
-                                               tx2->mt_u.dirty_list[x].mid == pgno) {
-                                               txn->mt_dirty_room++;
-                                               break;
-                                       }
-                               }
-                       }
-                       mdb_page_dirty(tx0, np);
+                       mdb_page_dirty(txn, np);
                        np->mp_flags |= P_DIRTY;
                        *ret = np;
                        break;
@@ -1797,7 +1864,6 @@ mdb_page_touch(MDB_cursor *mc)
        MDB_page *mp = mc->mc_pg[mc->mc_top], *np;
        MDB_txn *txn = mc->mc_txn;
        MDB_cursor *m2, *m3;
-       MDB_dbi dbi;
        pgno_t  pgno;
        int rc;
 
@@ -1814,7 +1880,8 @@ mdb_page_touch(MDB_cursor *mc)
                        (rc = mdb_page_alloc(mc, 1, &np)))
                        return rc;
                pgno = np->mp_pgno;
-               DPRINTF("touched db %u page %zu -> %zu", mc->mc_dbi,mp->mp_pgno,pgno);
+               DPRINTF(("touched db %d page %"Z"u -> %"Z"u", DDBI(mc),
+                       mp->mp_pgno, pgno));
                assert(mp->mp_pgno != pgno);
                mdb_midl_xappend(txn->mt_free_pgs, mp->mp_pgno);
                /* Update the parent page, if any, to point to the new page */
@@ -1860,17 +1927,16 @@ mdb_page_touch(MDB_cursor *mc)
 done:
        /* Adjust cursors pointing to mp */
        mc->mc_pg[mc->mc_top] = np;
-       dbi = mc->mc_dbi;
+       m2 = txn->mt_cursors[mc->mc_dbi];
        if (mc->mc_flags & C_SUB) {
-               dbi--;
-               for (m2 = txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
+               for (; m2; m2=m2->mc_next) {
                        m3 = &m2->mc_xcursor->mx_cursor;
                        if (m3->mc_snum < mc->mc_snum) continue;
                        if (m3->mc_pg[mc->mc_top] == mp)
                                m3->mc_pg[mc->mc_top] = np;
                }
        } else {
-               for (m2 = txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
+               for (; m2; m2=m2->mc_next) {
                        if (m2->mc_snum < mc->mc_snum) continue;
                        if (m2->mc_pg[mc->mc_top] == mp) {
                                m2->mc_pg[mc->mc_top] = np;
@@ -1980,18 +2046,73 @@ mdb_cursors_close(MDB_txn *txn, unsigned merge)
                                }
                                mc = bk;
                        }
+                       /* Only malloced cursors are permanently tracked. */
                        free(mc);
                }
                cursors[i] = NULL;
        }
 }
 
-#ifdef MDB_DEBUG_SKIP
+#if !(MDB_DEBUG)
 #define mdb_txn_reset0(txn, act) mdb_txn_reset0(txn)
 #endif
 static void
 mdb_txn_reset0(MDB_txn *txn, const char *act);
 
+#if !(MDB_PIDLOCK)             /* Currently the same as defined(_WIN32) */
+enum Pidlock_op {
+       Pidset, Pidcheck
+};
+#else
+enum Pidlock_op {
+       Pidset = F_SETLK, Pidcheck = F_GETLK
+};
+#endif
+
+/** Set or check a pid lock. Set returns 0 on success.
+ * Check returns 0 if the process is certainly dead, nonzero if it may
+ * be alive (the lock exists or an error happened so we do not know).
+ *
+ * On Windows Pidset is a no-op, we merely check for the existence
+ * of the process with the given pid. On POSIX we use a single byte
+ * lock on the lockfile, set at an offset equal to the pid.
+ */
+static int
+mdb_reader_pid(MDB_env *env, enum Pidlock_op op, pid_t pid)
+{
+#if !(MDB_PIDLOCK)             /* Currently the same as defined(_WIN32) */
+       int ret = 0;
+       HANDLE h;
+       if (op == Pidcheck) {
+               h = OpenProcess(env->me_pidquery, FALSE, pid);
+               /* No documented "no such process" code, but other program use this: */
+               if (!h)
+                       return ErrCode() != ERROR_INVALID_PARAMETER;
+               /* A process exists until all handles to it close. Has it exited? */
+               ret = WaitForSingleObject(h, 0) != 0;
+               CloseHandle(h);
+       }
+       return ret;
+#else
+       for (;;) {
+               int rc;
+               struct flock lock_info;
+               memset(&lock_info, 0, sizeof(lock_info));
+               lock_info.l_type = F_WRLCK;
+               lock_info.l_whence = SEEK_SET;
+               lock_info.l_start = pid;
+               lock_info.l_len = 1;
+               if ((rc = fcntl(env->me_lfd, op, &lock_info)) == 0) {
+                       if (op == F_GETLK && lock_info.l_type != F_UNLCK)
+                               rc = -1;
+               } else if ((rc = ErrCode()) == EINTR) {
+                       continue;
+               }
+               return rc;
+       }
+#endif
+}
+
 /** Common code for #mdb_txn_begin() and #mdb_txn_renew().
  * @param[in] txn the transaction handle to initialize
  * @return 0 on success, non-zero on failure.
@@ -2000,7 +2121,9 @@ static int
 mdb_txn_renew0(MDB_txn *txn)
 {
        MDB_env *env = txn->mt_env;
-       unsigned int i;
+       MDB_txninfo *ti = env->me_txns;
+       MDB_meta *meta;
+       unsigned int i, nr;
        uint16_t x;
        int rc, new_notls = 0;
 
@@ -2009,9 +2132,9 @@ mdb_txn_renew0(MDB_txn *txn)
        txn->mt_dbxs = env->me_dbxs;    /* mostly static anyway */
 
        if (txn->mt_flags & MDB_TXN_RDONLY) {
-               if (!env->me_txns) {
-                       i = mdb_env_pick_meta(env);
-                       txn->mt_txnid = env->me_metas[i]->mm_txnid;
+               if (!ti) {
+                       meta = env->me_metas[ mdb_env_pick_meta(env) ];
+                       txn->mt_txnid = meta->mm_txnid;
                        txn->mt_u.reader = NULL;
                } else {
                        MDB_reader *r = (env->me_flags & MDB_NOTLS) ? txn->mt_u.reader :
@@ -2023,37 +2146,53 @@ mdb_txn_renew0(MDB_txn *txn)
                                pid_t pid = env->me_pid;
                                pthread_t tid = pthread_self();
 
+                               if (!(env->me_flags & MDB_LIVE_READER)) {
+                                       rc = mdb_reader_pid(env, Pidset, pid);
+                                       if (rc) {
+                                               UNLOCK_MUTEX_R(env);
+                                               return rc;
+                                       }
+                                       env->me_flags |= MDB_LIVE_READER;
+                               }
+
                                LOCK_MUTEX_R(env);
-                               for (i=0; i<env->me_txns->mti_numreaders; i++)
-                                       if (env->me_txns->mti_readers[i].mr_pid == 0)
+                               nr = ti->mti_numreaders;
+                               for (i=0; i<nr; i++)
+                                       if (ti->mti_readers[i].mr_pid == 0)
                                                break;
                                if (i == env->me_maxreaders) {
                                        UNLOCK_MUTEX_R(env);
                                        return MDB_READERS_FULL;
                                }
-                               env->me_txns->mti_readers[i].mr_pid = pid;
-                               env->me_txns->mti_readers[i].mr_tid = tid;
-                               if (i >= env->me_txns->mti_numreaders)
-                                       env->me_txns->mti_numreaders = i+1;
+                               ti->mti_readers[i].mr_pid = pid;
+                               ti->mti_readers[i].mr_tid = tid;
+                               if (i == nr)
+                                       ti->mti_numreaders = ++nr;
                                /* Save numreaders for un-mutexed mdb_env_close() */
-                               env->me_numreaders = env->me_txns->mti_numreaders;
+                               env->me_numreaders = nr;
                                UNLOCK_MUTEX_R(env);
-                               r = &env->me_txns->mti_readers[i];
+
+                               r = &ti->mti_readers[i];
                                new_notls = (env->me_flags & MDB_NOTLS);
                                if (!new_notls && (rc=pthread_setspecific(env->me_txkey, r))) {
                                        r->mr_pid = 0;
                                        return rc;
                                }
                        }
-                       txn->mt_txnid = r->mr_txnid = env->me_txns->mti_txnid;
+                       txn->mt_txnid = r->mr_txnid = ti->mti_txnid;
                        txn->mt_u.reader = r;
+                       meta = env->me_metas[txn->mt_txnid & 1];
                }
-               txn->mt_toggle = txn->mt_txnid & 1;
        } else {
-               LOCK_MUTEX_W(env);
+               if (ti) {
+                       LOCK_MUTEX_W(env);
 
-               txn->mt_txnid = env->me_txns->mti_txnid;
-               txn->mt_toggle = txn->mt_txnid & 1;
+                       txn->mt_txnid = ti->mti_txnid;
+                       meta = env->me_metas[txn->mt_txnid & 1];
+               } else {
+                       meta = env->me_metas[ mdb_env_pick_meta(env) ];
+                       txn->mt_txnid = meta->mm_txnid;
+               }
                txn->mt_txnid++;
 #if MDB_DEBUG
                if (txn->mt_txnid == mdb_debug_start)
@@ -2069,10 +2208,10 @@ mdb_txn_renew0(MDB_txn *txn)
        }
 
        /* Copy the DB info and flags */
-       memcpy(txn->mt_dbs, env->me_metas[txn->mt_toggle]->mm_dbs, 2 * sizeof(MDB_db));
+       memcpy(txn->mt_dbs, meta->mm_dbs, 2 * sizeof(MDB_db));
 
        /* Moved to here to avoid a data race in read TXNs */
-       txn->mt_next_pgno = env->me_metas[txn->mt_toggle]->mm_last_pg+1;
+       txn->mt_next_pgno = meta->mm_last_pg+1;
 
        for (i=2; i<txn->mt_numdbs; i++) {
                x = env->me_dbflags[i];
@@ -2108,9 +2247,9 @@ mdb_txn_renew(MDB_txn *txn)
 
        rc = mdb_txn_renew0(txn);
        if (rc == MDB_SUCCESS) {
-               DPRINTF("renew txn %zu%c %p on mdbenv %p, root page %zu",
+               DPRINTF(("renew txn %"Z"u%c %p on mdbenv %p, root page %"Z"u",
                        txn->mt_txnid, (txn->mt_flags & MDB_TXN_RDONLY) ? 'r' : 'w',
-                       (void *)txn, (void *)txn->mt_env, txn->mt_dbs[MAIN_DBI].md_root);
+                       (void *)txn, (void *)txn->mt_env, txn->mt_dbs[MAIN_DBI].md_root));
        }
        return rc;
 }
@@ -2131,10 +2270,11 @@ mdb_txn_begin(MDB_env *env, MDB_txn *parent, unsigned int flags, MDB_txn **ret)
        if (parent) {
                /* Nested transactions: Max 1 child, write txns only, no writemap */
                if (parent->mt_child ||
-                       (flags & MDB_RDONLY) || (parent->mt_flags & MDB_TXN_RDONLY) ||
+                       (flags & MDB_RDONLY) ||
+                       (parent->mt_flags & (MDB_TXN_RDONLY|MDB_TXN_ERROR)) ||
                        (env->me_flags & MDB_WRITEMAP))
                {
-                       return EINVAL;
+                       return (parent->mt_flags & MDB_TXN_RDONLY) ? EINVAL : MDB_BAD_TXN;
                }
                tsize = sizeof(MDB_ntxn);
        }
@@ -2143,7 +2283,7 @@ mdb_txn_begin(MDB_env *env, MDB_txn *parent, unsigned int flags, MDB_txn **ret)
                size += env->me_maxdbs * sizeof(MDB_cursor *);
 
        if ((txn = calloc(1, size)) == NULL) {
-               DPRINTF("calloc: %s", strerror(ErrCode()));
+               DPRINTF(("calloc: %s", strerror(ErrCode())));
                return ENOMEM;
        }
        txn->mt_dbs = (MDB_db *) ((char *)txn + tsize);
@@ -2167,7 +2307,6 @@ mdb_txn_begin(MDB_env *env, MDB_txn *parent, unsigned int flags, MDB_txn **ret)
                        return ENOMEM;
                }
                txn->mt_txnid = parent->mt_txnid;
-               txn->mt_toggle = parent->mt_toggle;
                txn->mt_dirty_room = parent->mt_dirty_room;
                txn->mt_u.dirty_list[0].mid = 0;
                txn->mt_spill_pgs = NULL;
@@ -2203,14 +2342,21 @@ mdb_txn_begin(MDB_env *env, MDB_txn *parent, unsigned int flags, MDB_txn **ret)
                free(txn);
        else {
                *ret = txn;
-               DPRINTF("begin txn %zu%c %p on mdbenv %p, root page %zu",
+               DPRINTF(("begin txn %"Z"u%c %p on mdbenv %p, root page %"Z"u",
                        txn->mt_txnid, (txn->mt_flags & MDB_TXN_RDONLY) ? 'r' : 'w',
-                       (void *) txn, (void *) env, txn->mt_dbs[MAIN_DBI].md_root);
+                       (void *) txn, (void *) env, txn->mt_dbs[MAIN_DBI].md_root));
        }
 
        return rc;
 }
 
+MDB_env *
+mdb_txn_env(MDB_txn *txn)
+{
+       if(!txn) return NULL;
+       return txn->mt_env;
+}
+
 /** Export or close DBI handles opened in this txn. */
 static void
 mdb_dbis_update(MDB_txn *txn, int keep)
@@ -2240,6 +2386,7 @@ mdb_dbis_update(MDB_txn *txn, int keep)
 /** Common code for #mdb_txn_reset() and #mdb_txn_abort().
  * May be called twice for readonly txns: First reset it, then abort.
  * @param[in] txn the transaction handle to reset
+ * @param[in] act why the transaction is being reset
  */
 static void
 mdb_txn_reset0(MDB_txn *txn, const char *act)
@@ -2249,9 +2396,9 @@ mdb_txn_reset0(MDB_txn *txn, const char *act)
        /* Close any DBI handles opened in this txn */
        mdb_dbis_update(txn, 0);
 
-       DPRINTF("%s txn %zu%c %p on mdbenv %p, root page %zu",
+       DPRINTF(("%s txn %"Z"u%c %p on mdbenv %p, root page %"Z"u",
                act, txn->mt_txnid, (txn->mt_flags & MDB_TXN_RDONLY) ? 'r' : 'w',
-               (void *) txn, (void *)env, txn->mt_dbs[MAIN_DBI].md_root);
+               (void *) txn, (void *)env, txn->mt_dbs[MAIN_DBI].md_root));
 
        if (F_ISSET(txn->mt_flags, MDB_TXN_RDONLY)) {
                if (txn->mt_u.reader) {
@@ -2285,7 +2432,8 @@ mdb_txn_reset0(MDB_txn *txn, const char *act)
 
                env->me_txn = NULL;
                /* The writer mutex was locked in mdb_txn_begin. */
-               UNLOCK_MUTEX_W(env);
+               if (env->me_txns)
+                       UNLOCK_MUTEX_W(env);
        }
 }
 
@@ -2340,7 +2488,7 @@ mdb_freelist_save(MDB_txn *txn)
 
        if (env->me_pghead) {
                /* Make sure first page of freeDB is touched and on freelist */
-               rc = mdb_page_search(&mc, NULL, MDB_PS_MODIFY);
+               rc = mdb_page_search(&mc, NULL, MDB_PS_FIRST|MDB_PS_MODIFY);
                if (rc && rc != MDB_NOTFOUND)
                        return rc;
        }
@@ -2368,9 +2516,7 @@ mdb_freelist_save(MDB_txn *txn)
                if (freecnt < txn->mt_free_pgs[0]) {
                        if (!freecnt) {
                                /* Make sure last page of freeDB is touched and on freelist */
-                               key.mv_size = MDB_MAXKEYSIZE+1;
-                               key.mv_data = NULL;
-                               rc = mdb_page_search(&mc, &key, MDB_PS_MODIFY);
+                               rc = mdb_page_search(&mc, NULL, MDB_PS_LAST|MDB_PS_MODIFY);
                                if (rc && rc != MDB_NOTFOUND)
                                        return rc;
                        }
@@ -2389,13 +2535,13 @@ mdb_freelist_save(MDB_txn *txn)
                        } while (freecnt < free_pgs[0]);
                        mdb_midl_sort(free_pgs);
                        memcpy(data.mv_data, free_pgs, data.mv_size);
-#if MDB_DEBUG > 1
+#if (MDB_DEBUG) > 1
                        {
                                unsigned int i = free_pgs[0];
-                               DPRINTF("IDL write txn %zu root %zu num %u",
-                                       txn->mt_txnid, txn->mt_dbs[FREE_DBI].md_root, i);
+                               DPRINTF(("IDL write txn %"Z"u root %"Z"u num %u",
+                                       txn->mt_txnid, txn->mt_dbs[FREE_DBI].md_root, i));
                                for (; i; i--)
-                                       DPRINTF("IDL %zu", free_pgs[i]);
+                                       DPRINTF(("IDL %"Z"u", free_pgs[i]));
                        }
 #endif
                        continue;
@@ -2437,7 +2583,7 @@ mdb_freelist_save(MDB_txn *txn)
                total_room += head_room;
        }
 
-       /* Fill in the reserved, touched me_pghead records */
+       /* Fill in the reserved me_pghead records */
        rc = MDB_SUCCESS;
        if (mop_len) {
                MDB_val key, data;
@@ -2469,10 +2615,13 @@ mdb_freelist_save(MDB_txn *txn)
        return rc;
 }
 
-/** Flush dirty pages to the map, after clearing their dirty flag.
+/** Flush (some) dirty pages to the map, after clearing their dirty flag.
+ * @param[in] txn the transaction that's being committed
+ * @param[in] keep number of initial pages in dirty_list to keep dirty.
+ * @return 0 on success, non-zero on failure.
  */
 static int
-mdb_page_flush(MDB_txn *txn)
+mdb_page_flush(MDB_txn *txn, int keep)
 {
        MDB_env         *env = txn->mt_env;
        MDB_ID2L        dl = txn->mt_u.dirty_list;
@@ -2490,10 +2639,11 @@ mdb_page_flush(MDB_txn *txn)
        int                     n = 0;
 #endif
 
-       j = 0;
+       j = i = keep;
+
        if (env->me_flags & MDB_WRITEMAP) {
                /* Clear dirty flags */
-               for (i = pagecount; i; i--) {
+               while (++i <= pagecount) {
                        dp = dl[i].mptr;
                        /* Don't flush this page yet */
                        if (dp->mp_flags & P_KEEP) {
@@ -2503,13 +2653,12 @@ mdb_page_flush(MDB_txn *txn)
                        }
                        dp->mp_flags &= ~P_DIRTY;
                }
-               dl[0].mid = j;
-               return MDB_SUCCESS;
+               goto done;
        }
 
        /* Write the pages */
-       for (i = 1;; i++) {
-               if (i <= pagecount) {
+       for (;;) {
+               if (++i <= pagecount) {
                        dp = dl[i].mptr;
                        /* Don't flush this page yet */
                        if (dp->mp_flags & P_KEEP) {
@@ -2534,13 +2683,13 @@ mdb_page_flush(MDB_txn *txn)
                 * the write offset, to at least save the overhead of a Seek
                 * system call.
                 */
-               DPRINTF("committing page %zu", pgno);
+               DPRINTF(("committing page %"Z"u", pgno));
                memset(&ov, 0, sizeof(ov));
                ov.Offset = pos & 0xffffffff;
                ov.OffsetHigh = pos >> 16 >> 16;
                if (!WriteFile(env->me_fd, dp, size, NULL, &ov)) {
                        rc = ErrCode();
-                       DPRINTF("WriteFile: %d", rc);
+                       DPRINTF(("WriteFile: %d", rc));
                        return rc;
                }
 #else
@@ -2556,7 +2705,7 @@ mdb_page_flush(MDB_txn *txn)
                                } else {
                                        if (lseek(env->me_fd, wpos, SEEK_SET) == -1) {
                                                rc = ErrCode();
-                                               DPRINTF("lseek: %s", strerror(rc));
+                                               DPRINTF(("lseek: %s", strerror(rc)));
                                                return rc;
                                        }
                                        wres = writev(env->me_fd, iov, n);
@@ -2565,7 +2714,7 @@ mdb_page_flush(MDB_txn *txn)
                                if (wres != wsize) {
                                        if (wres < 0) {
                                                rc = ErrCode();
-                                               DPRINTF("Write error: %s", strerror(rc));
+                                               DPRINTF(("Write error: %s", strerror(rc)));
                                        } else {
                                                rc = EIO; /* TODO: Use which error code? */
                                                DPUTS("short write, filesystem full?");
@@ -2579,7 +2728,7 @@ mdb_page_flush(MDB_txn *txn)
                        wpos = pos;
                        wsize = 0;
                }
-               DPRINTF("committing page %zu", pgno);
+               DPRINTF(("committing page %"Z"u", pgno));
                next_pos = pos + size;
                iov[n].iov_len = size;
                iov[n].iov_base = (char *)dp;
@@ -2588,8 +2737,7 @@ mdb_page_flush(MDB_txn *txn)
 #endif /* _WIN32 */
        }
 
-       j = 0;
-       for (i=1; i<=pagecount; i++) {
+       for (i = keep; ++i <= pagecount; ) {
                dp = dl[i].mptr;
                /* This is a page we skipped above */
                if (!dl[i].mid) {
@@ -2599,8 +2747,11 @@ mdb_page_flush(MDB_txn *txn)
                }
                mdb_dpage_free(env, dp);
        }
-       dl[0].mid = j;
 
+done:
+       i--;
+       txn->mt_dirty_room += i - j;
+       dl[0].mid = j;
        return MDB_SUCCESS;
 }
 
@@ -2634,20 +2785,24 @@ mdb_txn_commit(MDB_txn *txn)
                DPUTS("error flag is set, can't commit");
                if (txn->mt_parent)
                        txn->mt_parent->mt_flags |= MDB_TXN_ERROR;
-               rc = EINVAL;
+               rc = MDB_BAD_TXN;
                goto fail;
        }
 
        if (txn->mt_parent) {
                MDB_txn *parent = txn->mt_parent;
-               unsigned x, y, len;
                MDB_ID2L dst, src;
+               MDB_IDL pspill;
+               unsigned x, y, len, ps_len;
 
                /* Append our free list to parent's */
                rc = mdb_midl_append_list(&parent->mt_free_pgs, txn->mt_free_pgs);
                if (rc)
                        goto fail;
                mdb_midl_free(txn->mt_free_pgs);
+               /* Failures after this must either undo the changes
+                * to the parent or set MDB_TXN_ERROR in the parent.
+                */
 
                parent->mt_next_pgno = txn->mt_next_pgno;
                parent->mt_flags = txn->mt_flags;
@@ -2669,36 +2824,26 @@ mdb_txn_commit(MDB_txn *txn)
                dst = parent->mt_u.dirty_list;
                src = txn->mt_u.dirty_list;
                /* Remove anything in our dirty list from parent's spill list */
-               if (parent->mt_spill_pgs) {
-                       x = parent->mt_spill_pgs[0];
-                       len = x;
-                       /* zero out our dirty pages in parent spill list */
-                       for (i=1; i<=src[0].mid; i++) {
-                               if (src[i].mid < parent->mt_spill_pgs[x])
-                                       continue;
-                               if (src[i].mid > parent->mt_spill_pgs[x]) {
-                                       if (x <= 1)
-                                               break;
+               if ((pspill = parent->mt_spill_pgs) && (ps_len = pspill[0])) {
+                       x = y = ps_len;
+                       pspill[0] = (pgno_t)-1;
+                       /* Mark our dirty pages as deleted in parent spill list */
+                       for (i=0, len=src[0].mid; ++i <= len; ) {
+                               MDB_ID pn = src[i].mid << 1;
+                               while (pn > pspill[x])
                                        x--;
-                                       continue;
+                               if (pn == pspill[x]) {
+                                       pspill[x] = 1;
+                                       y = --x;
                                }
-                               parent->mt_spill_pgs[x] = 0;
-                               len--;
-                       }
-                       /* OK, we had a few hits, squash zeros from the spill list */
-                       if (len < parent->mt_spill_pgs[0]) {
-                               x=1;
-                               for (y=1; y<=parent->mt_spill_pgs[0]; y++) {
-                                       if (parent->mt_spill_pgs[y]) {
-                                               if (y != x) {
-                                                       parent->mt_spill_pgs[x] = parent->mt_spill_pgs[y];
-                                               }
-                                               x++;
-                                       }
-                               }
-                               parent->mt_spill_pgs[0] = len;
                        }
+                       /* Squash deleted pagenums if we deleted any */
+                       for (x=y; ++x <= ps_len; )
+                               if (!(pspill[x] & 1))
+                                       pspill[++y] = pspill[x];
+                       pspill[0] = y;
                }
+
                /* Find len = length of merging our dirty list with parent's */
                x = dst[0].mid;
                dst[0].mid = 0;         /* simplify loops */
@@ -2732,7 +2877,10 @@ mdb_txn_commit(MDB_txn *txn)
                parent->mt_dirty_room = txn->mt_dirty_room;
                if (txn->mt_spill_pgs) {
                        if (parent->mt_spill_pgs) {
-                               mdb_midl_append_list(&parent->mt_spill_pgs, txn->mt_spill_pgs);
+                               /* TODO: Prevent failure here, so parent does not fail */
+                               rc = mdb_midl_append_list(&parent->mt_spill_pgs, txn->mt_spill_pgs);
+                               if (rc)
+                                       parent->mt_flags |= MDB_TXN_ERROR;
                                mdb_midl_free(txn->mt_spill_pgs);
                                mdb_midl_sort(parent->mt_spill_pgs);
                        } else {
@@ -2743,7 +2891,7 @@ mdb_txn_commit(MDB_txn *txn)
                parent->mt_child = NULL;
                mdb_midl_free(((MDB_ntxn *)txn)->mnt_pgstate.mf_pghead);
                free(txn);
-               return MDB_SUCCESS;
+               return rc;
        }
 
        if (txn != env->me_txn) {
@@ -2754,11 +2902,12 @@ mdb_txn_commit(MDB_txn *txn)
 
        mdb_cursors_close(txn, 0);
 
-       if (!txn->mt_u.dirty_list[0].mid && !(txn->mt_flags & MDB_TXN_DIRTY))
+       if (!txn->mt_u.dirty_list[0].mid &&
+               !(txn->mt_flags & (MDB_TXN_DIRTY|MDB_TXN_SPILLS)))
                goto done;
 
-       DPRINTF("committing txn %zu %p on mdbenv %p, root page %zu",
-           txn->mt_txnid, (void *)txn, (void *)env, txn->mt_dbs[MAIN_DBI].md_root);
+       DPRINTF(("committing txn %"Z"u %p on mdbenv %p, root page %"Z"u",
+           txn->mt_txnid, (void*)txn, (void*)env, txn->mt_dbs[MAIN_DBI].md_root));
 
        /* Update DB root pointers */
        if (txn->mt_numdbs > 2) {
@@ -2787,11 +2936,11 @@ mdb_txn_commit(MDB_txn *txn)
        if (mdb_midl_shrink(&txn->mt_free_pgs))
                env->me_free_pgs = txn->mt_free_pgs;
 
-#if MDB_DEBUG > 2
+#if (MDB_DEBUG) > 2
        mdb_audit(txn);
 #endif
 
-       if ((rc = mdb_page_flush(txn)) ||
+       if ((rc = mdb_page_flush(txn, 0)) ||
                (rc = mdb_env_sync(env, 0)) ||
                (rc = mdb_env_write_meta(txn)))
                goto fail;
@@ -2801,7 +2950,8 @@ done:
        env->me_txn = NULL;
        mdb_dbis_update(txn, 1);
 
-       UNLOCK_MUTEX_W(env);
+       if (env->me_txns)
+               UNLOCK_MUTEX_W(env);
        free(txn);
 
        return MDB_SUCCESS;
@@ -2845,14 +2995,14 @@ mdb_env_read_header(MDB_env *env, MDB_meta *meta)
                        if (rc == 0 && off == 0)
                                return ENOENT;
                        rc = rc < 0 ? (int) ErrCode() : MDB_INVALID;
-                       DPRINTF("read: %s", mdb_strerror(rc));
+                       DPRINTF(("read: %s", mdb_strerror(rc)));
                        return rc;
                }
 
                p = (MDB_page *)&pbuf;
 
                if (!F_ISSET(p->mp_flags, P_META)) {
-                       DPRINTF("page %zu not a meta page", p->mp_pgno);
+                       DPRINTF(("page %"Z"u not a meta page", p->mp_pgno));
                        return MDB_INVALID;
                }
 
@@ -2862,9 +3012,9 @@ mdb_env_read_header(MDB_env *env, MDB_meta *meta)
                        return MDB_INVALID;
                }
 
-               if (m->mm_version != MDB_VERSION) {
-                       DPRINTF("database is version %u, expected version %u",
-                               m->mm_version, MDB_VERSION);
+               if (m->mm_version != MDB_DATA_VERSION) {
+                       DPRINTF(("database is version %u, expected version %u",
+                               m->mm_version, MDB_DATA_VERSION));
                        return MDB_VERSION_MISMATCH;
                }
 
@@ -2885,13 +3035,26 @@ mdb_env_init_meta(MDB_env *env, MDB_meta *meta)
        MDB_page *p, *q;
        int rc;
        unsigned int     psize;
+#ifdef _WIN32
+       DWORD len;
+       OVERLAPPED ov;
+       memset(&ov, 0, sizeof(ov));
+#define DO_PWRITE(rc, fd, ptr, size, len, pos) do { \
+       ov.Offset = pos;        \
+       rc = WriteFile(fd, ptr, size, &len, &ov);       } while(0)
+#else
+       int len;
+#define DO_PWRITE(rc, fd, ptr, size, len, pos) do { \
+       len = pwrite(fd, ptr, size, pos);       \
+       rc = (len >= 0); } while(0)
+#endif
 
        DPUTS("writing new meta page");
 
-       GET_PAGESIZE(psize);
+       psize = env->me_psize;
 
        meta->mm_magic = MDB_MAGIC;
-       meta->mm_version = MDB_VERSION;
+       meta->mm_version = MDB_DATA_VERSION;
        meta->mm_mapsize = env->me_mapsize;
        meta->mm_psize = psize;
        meta->mm_last_pg = 1;
@@ -2910,18 +3073,13 @@ mdb_env_init_meta(MDB_env *env, MDB_meta *meta)
        q->mp_flags = P_META;
        *(MDB_meta *)METADATA(q) = *meta;
 
-#ifdef _WIN32
-       {
-               DWORD len;
-               OVERLAPPED ov;
-               memset(&ov, 0, sizeof(ov));
-               rc = WriteFile(env->me_fd, p, psize * 2, &len, &ov);
-               rc = rc ? (len == psize * 2 ? MDB_SUCCESS : EIO) : ErrCode();
-       }
-#else
-       rc = pwrite(env->me_fd, p, psize * 2, 0);
-       rc = (rc == (int)psize * 2) ? MDB_SUCCESS : rc < 0 ? ErrCode() : EIO;
-#endif
+       DO_PWRITE(rc, env->me_fd, p, psize * 2, len, 0);
+       if (!rc)
+               rc = ErrCode();
+       else if ((unsigned) len == psize * 2)
+               rc = MDB_SUCCESS;
+       else
+               rc = ENOSPC;
        free(p);
        return rc;
 }
@@ -2948,9 +3106,9 @@ mdb_env_write_meta(MDB_txn *txn)
        assert(txn != NULL);
        assert(txn->mt_env != NULL);
 
-       toggle = !txn->mt_toggle;
-       DPRINTF("writing meta page %d for root page %zu",
-               toggle, txn->mt_dbs[MAIN_DBI].md_root);
+       toggle = txn->mt_txnid & 1;
+       DPRINTF(("writing meta page %d for root page %"Z"u",
+               toggle, txn->mt_dbs[MAIN_DBI].md_root));
 
        env = txn->mt_env;
        mp = env->me_metas[toggle];
@@ -3026,6 +3184,7 @@ mdb_env_write_meta(MDB_txn *txn)
                WriteFile(env->me_fd, ptr, len, NULL, &ov);
 #else
                r2 = pwrite(env->me_fd, ptr, len, off);
+               (void)r2;       /* Silence warnings. We don't care about pwrite's return value */
 #endif
 fail:
                env->me_flags |= MDB_FATAL_ERROR;
@@ -3038,7 +3197,8 @@ done:
         * readers will get consistent data regardless of how fresh or
         * how stale their view of these values is.
         */
-       env->me_txns->mti_txnid = txn->mt_txnid;
+       if (env->me_txns)
+               env->me_txns->mti_txnid = txn->mt_txnid;
 
        return MDB_SUCCESS;
 }
@@ -3077,11 +3237,108 @@ mdb_env_create(MDB_env **env)
        return MDB_SUCCESS;
 }
 
+static int
+mdb_env_map(MDB_env *env, void *addr, int newsize)
+{
+       MDB_page *p;
+       unsigned int flags = env->me_flags;
+#ifdef _WIN32
+       int rc;
+       HANDLE mh;
+       LONG sizelo, sizehi;
+       sizelo = env->me_mapsize & 0xffffffff;
+       sizehi = env->me_mapsize >> 16 >> 16; /* only needed on Win64 */
+
+       /* Windows won't create mappings for zero length files.
+        * Just allocate the maxsize right now.
+        */
+       if (newsize) {
+               if (SetFilePointer(env->me_fd, sizelo, &sizehi, 0) != (DWORD)sizelo
+                       || !SetEndOfFile(env->me_fd)
+                       || SetFilePointer(env->me_fd, 0, NULL, 0) != 0)
+                       return ErrCode();
+       }
+       mh = CreateFileMapping(env->me_fd, NULL, flags & MDB_WRITEMAP ?
+               PAGE_READWRITE : PAGE_READONLY,
+               sizehi, sizelo, NULL);
+       if (!mh)
+               return ErrCode();
+       env->me_map = MapViewOfFileEx(mh, flags & MDB_WRITEMAP ?
+               FILE_MAP_WRITE : FILE_MAP_READ,
+               0, 0, env->me_mapsize, addr);
+       rc = env->me_map ? 0 : ErrCode();
+       CloseHandle(mh);
+       if (rc)
+               return rc;
+#else
+       int prot = PROT_READ;
+       if (flags & MDB_WRITEMAP) {
+               prot |= PROT_WRITE;
+               if (ftruncate(env->me_fd, env->me_mapsize) < 0)
+                       return ErrCode();
+       }
+       env->me_map = mmap(addr, env->me_mapsize, prot, MAP_SHARED,
+               env->me_fd, 0);
+       if (env->me_map == MAP_FAILED) {
+               env->me_map = NULL;
+               return ErrCode();
+       }
+
+       if (flags & MDB_NORDAHEAD) {
+               /* Turn off readahead. It's harmful when the DB is larger than RAM. */
+#ifdef MADV_RANDOM
+               madvise(env->me_map, env->me_mapsize, MADV_RANDOM);
+#else
+#ifdef POSIX_MADV_RANDOM
+               posix_madvise(env->me_map, env->me_mapsize, POSIX_MADV_RANDOM);
+#endif /* POSIX_MADV_RANDOM */
+#endif /* MADV_RANDOM */
+       }
+#endif /* _WIN32 */
+
+       /* Can happen because the address argument to mmap() is just a
+        * hint.  mmap() can pick another, e.g. if the range is in use.
+        * The MAP_FIXED flag would prevent that, but then mmap could
+        * instead unmap existing pages to make room for the new map.
+        */
+       if (addr && env->me_map != addr)
+               return EBUSY;   /* TODO: Make a new MDB_* error code? */
+
+       p = (MDB_page *)env->me_map;
+       env->me_metas[0] = METADATA(p);
+       env->me_metas[1] = (MDB_meta *)((char *)env->me_metas[0] + env->me_psize);
+
+       return MDB_SUCCESS;
+}
+
 int
 mdb_env_set_mapsize(MDB_env *env, size_t size)
 {
-       if (env->me_map)
-               return EINVAL;
+       /* If env is already open, caller is responsible for making
+        * sure there are no active txns.
+        */
+       if (env->me_map) {
+               int rc;
+               void *old;
+               if (env->me_txn)
+                       return EINVAL;
+               if (!size)
+                       size = env->me_metas[mdb_env_pick_meta(env)]->mm_mapsize;
+               else if (size < env->me_mapsize) {
+                       /* If the configured size is smaller, make sure it's
+                        * still big enough. Silently round up to minimum if not.
+                        */
+                       size_t minsize = (env->me_metas[mdb_env_pick_meta(env)]->mm_last_pg + 1) * env->me_psize;
+                       if (size < minsize)
+                               size = minsize;
+               }
+               munmap(env->me_map, env->me_mapsize);
+               env->me_mapsize = size;
+               old = (env->me_flags & MDB_FIXEDMAP) ? env->me_map : NULL;
+               rc = mdb_env_map(env, old, 1);
+               if (rc)
+                       return rc;
+       }
        env->me_mapsize = size;
        if (env->me_psize)
                env->me_maxpg = env->me_mapsize / env->me_psize;
@@ -3121,12 +3378,17 @@ static int
 mdb_env_open2(MDB_env *env)
 {
        unsigned int flags = env->me_flags;
-       int i, newenv = 0;
+       int i, newenv = 0, rc;
        MDB_meta meta;
-       MDB_page *p;
-#ifndef _WIN32
-       int prot;
-#endif
+
+#ifdef _WIN32
+       /* See if we should use QueryLimited */
+       rc = GetVersion();
+       if ((rc & 0xff) > 5)
+               env->me_pidquery = MDB_PROCESS_QUERY_LIMITED_INFORMATION;
+       else
+               env->me_pidquery = PROCESS_QUERY_INFORMATION;
+#endif /* _WIN32 */
 
        memset(&meta, 0, sizeof(meta));
 
@@ -3135,6 +3397,9 @@ mdb_env_open2(MDB_env *env)
                        return i;
                DPUTS("new mdbenv");
                newenv = 1;
+               GET_PAGESIZE(env->me_psize);
+       } else {
+               env->me_psize = meta.mm_psize;
        }
 
        /* Was a mapsize configured? */
@@ -3152,58 +3417,9 @@ mdb_env_open2(MDB_env *env)
                        env->me_mapsize = minsize;
        }
 
-#ifdef _WIN32
-       {
-               int rc;
-               HANDLE mh;
-               LONG sizelo, sizehi;
-               sizelo = env->me_mapsize & 0xffffffff;
-               sizehi = env->me_mapsize >> 16 >> 16; /* only needed on Win64 */
-               /* Windows won't create mappings for zero length files.
-                * Just allocate the maxsize right now.
-                */
-               if (newenv) {
-                       if (SetFilePointer(env->me_fd, sizelo, &sizehi, 0) != (DWORD)sizelo
-                               || !SetEndOfFile(env->me_fd)
-                               || SetFilePointer(env->me_fd, 0, NULL, 0) != 0)
-                               return ErrCode();
-               }
-               mh = CreateFileMapping(env->me_fd, NULL, flags & MDB_WRITEMAP ?
-                       PAGE_READWRITE : PAGE_READONLY,
-                       sizehi, sizelo, NULL);
-               if (!mh)
-                       return ErrCode();
-               env->me_map = MapViewOfFileEx(mh, flags & MDB_WRITEMAP ?
-                       FILE_MAP_WRITE : FILE_MAP_READ,
-                       0, 0, env->me_mapsize, meta.mm_address);
-               rc = env->me_map ? 0 : ErrCode();
-               CloseHandle(mh);
-               if (rc)
-                       return rc;
-       }
-#else
-       i = MAP_SHARED;
-       prot = PROT_READ;
-       if (flags & MDB_WRITEMAP) {
-               prot |= PROT_WRITE;
-               if (ftruncate(env->me_fd, env->me_mapsize) < 0)
-                       return ErrCode();
-       }
-       env->me_map = mmap(meta.mm_address, env->me_mapsize, prot, i,
-               env->me_fd, 0);
-       if (env->me_map == MAP_FAILED) {
-               env->me_map = NULL;
-               return ErrCode();
-       }
-       /* Turn off readahead. It's harmful when the DB is larger than RAM. */
-#ifdef MADV_RANDOM
-       madvise(env->me_map, env->me_mapsize, MADV_RANDOM);
-#else
-#ifdef POSIX_MADV_RANDOM
-       posix_madvise(env->me_map, env->me_mapsize, POSIX_MADV_RANDOM);
-#endif /* POSIX_MADV_RANDOM */
-#endif /* MADV_RANDOM */
-#endif /* _WIN32 */
+       rc = mdb_env_map(env, meta.mm_address, newenv);
+       if (rc)
+               return rc;
 
        if (newenv) {
                if (flags & MDB_FIXEDMAP)
@@ -3212,38 +3428,25 @@ mdb_env_open2(MDB_env *env)
                if (i != MDB_SUCCESS) {
                        return i;
                }
-       } else if (meta.mm_address && env->me_map != meta.mm_address) {
-               /* Can happen because the address argument to mmap() is just a
-                * hint.  mmap() can pick another, e.g. if the range is in use.
-                * The MAP_FIXED flag would prevent that, but then mmap could
-                * instead unmap existing pages to make room for the new map.
-                */
-               return EBUSY;   /* TODO: Make a new MDB_* error code? */
        }
-       env->me_psize = meta.mm_psize;
        env->me_maxfree_1pg = (env->me_psize - PAGEHDRSZ) / sizeof(pgno_t) - 1;
        env->me_nodemax = (env->me_psize - PAGEHDRSZ) / MDB_MINKEYS;
 
        env->me_maxpg = env->me_mapsize / env->me_psize;
-
-       p = (MDB_page *)env->me_map;
-       env->me_metas[0] = METADATA(p);
-       env->me_metas[1] = (MDB_meta *)((char *)env->me_metas[0] + meta.mm_psize);
-
 #if MDB_DEBUG
        {
                int toggle = mdb_env_pick_meta(env);
                MDB_db *db = &env->me_metas[toggle]->mm_dbs[MAIN_DBI];
 
-               DPRINTF("opened database version %u, pagesize %u",
-                       env->me_metas[0]->mm_version, env->me_psize);
-               DPRINTF("using meta page %d",  toggle);
-               DPRINTF("depth: %u",           db->md_depth);
-               DPRINTF("entries: %zu",        db->md_entries);
-               DPRINTF("branch pages: %zu",   db->md_branch_pages);
-               DPRINTF("leaf pages: %zu",     db->md_leaf_pages);
-               DPRINTF("overflow pages: %zu", db->md_overflow_pages);
-               DPRINTF("root: %zu",           db->md_root);
+               DPRINTF(("opened database version %u, pagesize %u",
+                       env->me_metas[0]->mm_version, env->me_psize));
+               DPRINTF(("using meta page %d",    toggle));
+               DPRINTF(("depth: %u",             db->md_depth));
+               DPRINTF(("entries: %"Z"u",        db->md_entries));
+               DPRINTF(("branch pages: %"Z"u",   db->md_branch_pages));
+               DPRINTF(("leaf pages: %"Z"u",     db->md_leaf_pages));
+               DPRINTF(("overflow pages: %"Z"u", db->md_overflow_pages));
+               DPRINTF(("root: %"Z"u",           db->md_root));
        }
 #endif
 
@@ -3403,7 +3606,7 @@ mdb_env_excl_lock(MDB_env *env, int *excl)
        return rc;
 }
 
-#if defined(_WIN32) || defined(MDB_USE_POSIX_SEM)
+#ifdef MDB_USE_HASH
 /*
  * hash_64 - 64 bit Fowler/Noll/Vo-0 FNV-1a hash code
  *
@@ -3436,7 +3639,7 @@ typedef unsigned long long        mdb_hash_t;
 #define MDB_HASH_INIT ((mdb_hash_t)0xcbf29ce484222325ULL)
 
 /** perform a 64 bit Fowler/Noll/Vo FNV-1a hash on a buffer
- * @param[in] str string to hash
+ * @param[in] val      value to hash
  * @param[in] hval     initial value for hash
  * @return 64 bit hash
  *
@@ -3463,20 +3666,36 @@ mdb_hash_val(MDB_val *val, mdb_hash_t hval)
        return hval;
 }
 
-/** Hash the string and output the hash in hex.
+/** Hash the string and output the encoded hash.
+ * This uses modified RFC1924 Ascii85 encoding to accommodate systems with
+ * very short name limits. We don't care about the encoding being reversible,
+ * we just want to preserve as many bits of the input as possible in a
+ * small printable string.
  * @param[in] str string to hash
- * @param[out] hexbuf an array of 17 chars to hold the hash
+ * @param[out] encbuf an array of 11 chars to hold the hash
  */
+static const char mdb_a85[]= "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz!#$%&()*+-;<=>?@^_`{|}~";
+
 static void
-mdb_hash_hex(MDB_val *val, char *hexbuf)
+mdb_pack85(unsigned long l, char *out)
 {
        int i;
-       mdb_hash_t h = mdb_hash_val(val, MDB_HASH_INIT);
-       for (i=0; i<8; i++) {
-               hexbuf += sprintf(hexbuf, "%02x", (unsigned int)h & 0xff);
-               h >>= 8;
+
+       for (i=0; i<5; i++) {
+               *out++ = mdb_a85[l % 85];
+               l /= 85;
        }
 }
+
+static void
+mdb_hash_enc(MDB_val *val, char *encbuf)
+{
+       mdb_hash_t h = mdb_hash_val(val, MDB_HASH_INIT);
+
+       mdb_pack85(h, encbuf);
+       mdb_pack85(h>>32, encbuf+5);
+       encbuf[10] = '\0';
+}
 #endif
 
 /** Open and/or initialize the lock region for the environment.
@@ -3589,7 +3808,7 @@ mdb_env_setup_locks(MDB_env *env, char *lpath, int mode, int *excl)
                        DWORD nlow;
                } idbuf;
                MDB_val val;
-               char hexbuf[17];
+               char encbuf[11];
 
                if (!mdb_sec_inited) {
                        InitializeSecurityDescriptor(&mdb_null_sd,
@@ -3606,9 +3825,9 @@ mdb_env_setup_locks(MDB_env *env, char *lpath, int mode, int *excl)
                idbuf.nlow   = stbuf.nFileIndexLow;
                val.mv_data = &idbuf;
                val.mv_size = sizeof(idbuf);
-               mdb_hash_hex(&val, hexbuf);
-               sprintf(env->me_txns->mti_rmname, "Global\\MDBr%s", hexbuf);
-               sprintf(env->me_txns->mti_wmname, "Global\\MDBw%s", hexbuf);
+               mdb_hash_enc(&val, encbuf);
+               sprintf(env->me_txns->mti_rmname, "Global\\MDBr%s", encbuf);
+               sprintf(env->me_txns->mti_wmname, "Global\\MDBw%s", encbuf);
                env->me_rmutex = CreateMutex(&mdb_all_sa, FALSE, env->me_txns->mti_rmname);
                if (!env->me_rmutex) goto fail_errno;
                env->me_wmutex = CreateMutex(&mdb_all_sa, FALSE, env->me_txns->mti_wmname);
@@ -3620,16 +3839,22 @@ mdb_env_setup_locks(MDB_env *env, char *lpath, int mode, int *excl)
                        ino_t ino;
                } idbuf;
                MDB_val val;
-               char hexbuf[17];
+               char encbuf[11];
 
+#if defined(__NetBSD__)
+#define        MDB_SHORT_SEMNAMES      1       /* limited to 14 chars */
+#endif
                if (fstat(env->me_lfd, &stbuf)) goto fail_errno;
                idbuf.dev = stbuf.st_dev;
                idbuf.ino = stbuf.st_ino;
                val.mv_data = &idbuf;
                val.mv_size = sizeof(idbuf);
-               mdb_hash_hex(&val, hexbuf);
-               sprintf(env->me_txns->mti_rmname, "/MDBr%s", hexbuf);
-               sprintf(env->me_txns->mti_wmname, "/MDBw%s", hexbuf);
+               mdb_hash_enc(&val, encbuf);
+#ifdef MDB_SHORT_SEMNAMES
+               encbuf[9] = '\0';       /* drop name from 15 chars to 14 chars */
+#endif
+               sprintf(env->me_txns->mti_rmname, "/MDBr%s", encbuf);
+               sprintf(env->me_txns->mti_wmname, "/MDBw%s", encbuf);
                /* Clean up after a previous run, if needed:  Try to
                 * remove both semaphores before doing anything else.
                 */
@@ -3652,8 +3877,8 @@ mdb_env_setup_locks(MDB_env *env, char *lpath, int mode, int *excl)
                pthread_mutexattr_destroy(&mattr);
 #endif /* _WIN32 || MDB_USE_POSIX_SEM */
 
-               env->me_txns->mti_version = MDB_VERSION;
                env->me_txns->mti_magic = MDB_MAGIC;
+               env->me_txns->mti_format = MDB_LOCK_FORMAT;
                env->me_txns->mti_txnid = 0;
                env->me_txns->mti_numreaders = 0;
 
@@ -3663,9 +3888,9 @@ mdb_env_setup_locks(MDB_env *env, char *lpath, int mode, int *excl)
                        rc = MDB_INVALID;
                        goto fail;
                }
-               if (env->me_txns->mti_version != MDB_VERSION) {
-                       DPRINTF("lock region is version %u, expected version %u",
-                               env->me_txns->mti_version, MDB_VERSION);
+               if (env->me_txns->mti_format != MDB_LOCK_FORMAT) {
+                       DPRINTF(("lock region has format+version 0x%x, expected 0x%x",
+                               env->me_txns->mti_format, MDB_LOCK_FORMAT));
                        rc = MDB_VERSION_MISMATCH;
                        goto fail;
                }
@@ -3704,7 +3929,7 @@ fail:
         *      environment and re-opening it with the new flags.
         */
 #define        CHANGEABLE      (MDB_NOSYNC|MDB_NOMETASYNC|MDB_MAPASYNC)
-#define        CHANGELESS      (MDB_FIXEDMAP|MDB_NOSUBDIR|MDB_RDONLY|MDB_WRITEMAP|MDB_NOTLS)
+#define        CHANGELESS      (MDB_FIXEDMAP|MDB_NOSUBDIR|MDB_RDONLY|MDB_WRITEMAP|MDB_NOTLS|MDB_NOLOCK|MDB_NORDAHEAD)
 
 int
 mdb_env_open(MDB_env *env, const char *path, unsigned int flags, mdb_mode_t mode)
@@ -3756,9 +3981,12 @@ mdb_env_open(MDB_env *env, const char *path, unsigned int flags, mdb_mode_t mode
                goto leave;
        }
 
-       rc = mdb_env_setup_locks(env, lpath, mode, &excl);
-       if (rc)
-               goto leave;
+       /* For RDONLY, get lockfile after we know datafile exists */
+       if (!(flags & (MDB_RDONLY|MDB_NOLOCK))) {
+               rc = mdb_env_setup_locks(env, lpath, mode, &excl);
+               if (rc)
+                       goto leave;
+       }
 
 #ifdef _WIN32
        if (F_ISSET(flags, MDB_RDONLY)) {
@@ -3784,6 +4012,12 @@ mdb_env_open(MDB_env *env, const char *path, unsigned int flags, mdb_mode_t mode
                goto leave;
        }
 
+       if ((flags & (MDB_RDONLY|MDB_NOLOCK)) == MDB_RDONLY) {
+               rc = mdb_env_setup_locks(env, lpath, mode, &excl);
+               if (rc)
+                       goto leave;
+       }
+
        if ((rc = mdb_env_open2(env)) == MDB_SUCCESS) {
                if (flags & (MDB_RDONLY|MDB_WRITEMAP)) {
                        env->me_mfd = env->me_fd;
@@ -3792,10 +4026,12 @@ mdb_env_open(MDB_env *env, const char *path, unsigned int flags, mdb_mode_t mode
                         * MDB_NOSYNC/MDB_NOMETASYNC, in case these get reset.
                         */
 #ifdef _WIN32
+                       len = OPEN_EXISTING;
                        env->me_mfd = CreateFile(dpath, oflags,
                                FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, len,
                                mode | FILE_FLAG_WRITE_THROUGH, NULL);
 #else
+                       oflags &= ~O_CREAT;
                        env->me_mfd = open(dpath, oflags | MDB_DSYNC, mode);
 #endif
                        if (env->me_mfd == INVALID_HANDLE_VALUE) {
@@ -3803,7 +4039,7 @@ mdb_env_open(MDB_env *env, const char *path, unsigned int flags, mdb_mode_t mode
                                goto leave;
                        }
                }
-               DPRINTF("opened dbenv %p", (void *) env);
+               DPRINTF(("opened dbenv %p", (void *) env));
                if (excl > 0) {
                        rc = mdb_env_share_locks(env, &excl);
                }
@@ -3912,6 +4148,14 @@ mdb_env_copyfd(MDB_env *env, HANDLE fd)
        int rc;
        size_t wsize;
        char *ptr;
+#ifdef _WIN32
+       DWORD len, w2;
+#define DO_WRITE(rc, fd, ptr, w2, len) rc = WriteFile(fd, ptr, w2, &len, NULL)
+#else
+       ssize_t len;
+       size_t w2;
+#define DO_WRITE(rc, fd, ptr, w2, len) len = write(fd, ptr, w2); rc = (len >= 0)
+#endif
 
        /* Do the lock/unlock of the reader mutex before starting the
         * write txn.  Otherwise other read txns could block writers.
@@ -3935,52 +4179,50 @@ mdb_env_copyfd(MDB_env *env, HANDLE fd)
        }
 
        wsize = env->me_psize * 2;
-#ifdef _WIN32
-       {
-               DWORD len;
-               rc = WriteFile(fd, env->me_map, wsize, &len, NULL);
-               rc = rc ? (len == wsize ? MDB_SUCCESS : EIO) : ErrCode();
+       ptr = env->me_map;
+       w2 = wsize;
+       while (w2 > 0) {
+               DO_WRITE(rc, fd, ptr, w2, len);
+               if (!rc) {
+                       rc = ErrCode();
+                       break;
+               } else if (len > 0) {
+                       rc = MDB_SUCCESS;
+                       ptr += len;
+                       w2 -= len;
+                       continue;
+               } else {
+                       /* Non-blocking or async handles are not supported */
+                       rc = EIO;
+                       break;
+               }
        }
-#else
-       rc = write(fd, env->me_map, wsize);
-       rc = rc == (int)wsize ? MDB_SUCCESS : rc < 0 ? ErrCode() : EIO;
-#endif
        if (env->me_txns)
                UNLOCK_MUTEX_W(env);
 
        if (rc)
                goto leave;
 
-       ptr = env->me_map + wsize;
        wsize = txn->mt_next_pgno * env->me_psize - wsize;
-#ifdef _WIN32
-       while (wsize > 0) {
-               DWORD len, w2;
-               if (wsize > MAX_WRITE)
-                       w2 = MAX_WRITE;
-               else
-                       w2 = wsize;
-               rc = WriteFile(fd, ptr, w2, &len, NULL);
-               rc = rc ? (len == w2 ? MDB_SUCCESS : EIO) : ErrCode();
-               if (rc) break;
-               wsize -= w2;
-               ptr += w2;
-       }
-#else
        while (wsize > 0) {
-               size_t w2;
-               ssize_t wres;
                if (wsize > MAX_WRITE)
                        w2 = MAX_WRITE;
                else
                        w2 = wsize;
-               wres = write(fd, ptr, w2);
-               rc = wres == (ssize_t)w2 ? MDB_SUCCESS : wres < 0 ? ErrCode() : EIO;
-               if (rc) break;
-               wsize -= wres;
-               ptr += wres;
+               DO_WRITE(rc, fd, ptr, w2, len);
+               if (!rc) {
+                       rc = ErrCode();
+                       break;
+               } else if (len > 0) {
+                       rc = MDB_SUCCESS;
+                       ptr += len;
+                       wsize -= len;
+                       continue;
+               } else {
+                       rc = EIO;
+                       break;
+               }
        }
-#endif
 
 leave:
        mdb_txn_abort(txn);
@@ -4013,17 +4255,18 @@ mdb_env_copy(MDB_env *env, const char *path)
        newfd = CreateFile(lpath, GENERIC_WRITE, 0, NULL, CREATE_NEW,
                                FILE_FLAG_NO_BUFFERING|FILE_FLAG_WRITE_THROUGH, NULL);
 #else
-       newfd = open(lpath, O_WRONLY|O_CREAT|O_EXCL
-#ifdef O_DIRECT
-               |O_DIRECT
-#endif
-               , 0666);
+       newfd = open(lpath, O_WRONLY|O_CREAT|O_EXCL, 0666);
 #endif
        if (newfd == INVALID_HANDLE_VALUE) {
                rc = ErrCode();
                goto leave;
        }
 
+#ifdef O_DIRECT
+       /* Set O_DIRECT if the file system supports it */
+       if ((rc = fcntl(newfd, F_GETFL)) != -1)
+               (void) fcntl(newfd, F_SETFL, rc | O_DIRECT);
+#endif
 #ifdef F_NOCACHE       /* __APPLE__ */
        rc = fcntl(newfd, F_NOCACHE, 1);
        if (rc) {
@@ -4071,7 +4314,7 @@ mdb_cmp_long(const MDB_val *a, const MDB_val *b)
                *(size_t *)a->mv_data > *(size_t *)b->mv_data;
 }
 
-/** Compare two items pointing at aligned int's */
+/** Compare two items pointing at aligned unsigned int's */
 static int
 mdb_cmp_int(const MDB_val *a, const MDB_val *b)
 {
@@ -4079,7 +4322,7 @@ mdb_cmp_int(const MDB_val *a, const MDB_val *b)
                *(unsigned int *)a->mv_data > *(unsigned int *)b->mv_data;
 }
 
-/** Compare two items pointing at ints of unknown alignment.
+/** Compare two items pointing at unsigned ints of unknown alignment.
  *     Nodes and keys are guaranteed to be 2-byte aligned.
  */
 static int
@@ -4170,9 +4413,9 @@ mdb_node_search(MDB_cursor *mc, MDB_val *key, int *exactp)
        {
        pgno_t pgno;
        COPY_PGNO(pgno, mp->mp_pgno);
-       DPRINTF("searching %u keys in %s %spage %zu",
+       DPRINTF(("searching %u keys in %s %spage %"Z"u",
            nkeys, IS_LEAF(mp) ? "leaf" : "branch", IS_SUBP(mp) ? "sub-" : "",
-           pgno);
+           pgno));
        }
 #endif
 
@@ -4199,8 +4442,8 @@ mdb_node_search(MDB_cursor *mc, MDB_val *key, int *exactp)
                        i = (low + high) >> 1;
                        nodekey.mv_data = LEAF2KEY(mp, i, nodekey.mv_size);
                        rc = cmp(key, &nodekey);
-                       DPRINTF("found leaf index %u [%s], rc = %i",
-                           i, DKEY(&nodekey), rc);
+                       DPRINTF(("found leaf index %u [%s], rc = %i",
+                           i, DKEY(&nodekey), rc));
                        if (rc == 0)
                                break;
                        if (rc > 0)
@@ -4219,11 +4462,11 @@ mdb_node_search(MDB_cursor *mc, MDB_val *key, int *exactp)
                        rc = cmp(key, &nodekey);
 #if MDB_DEBUG
                        if (IS_LEAF(mp))
-                               DPRINTF("found leaf index %u [%s], rc = %i",
-                                   i, DKEY(&nodekey), rc);
+                               DPRINTF(("found leaf index %u [%s], rc = %i",
+                                   i, DKEY(&nodekey), rc));
                        else
-                               DPRINTF("found branch index %u [%s -> %zu], rc = %i",
-                                   i, DKEY(&nodekey), NODEPGNO(node), rc);
+                               DPRINTF(("found branch index %u [%s -> %"Z"u], rc = %i",
+                                   i, DKEY(&nodekey), NODEPGNO(node), rc));
 #endif
                        if (rc == 0)
                                break;
@@ -4270,15 +4513,15 @@ static void
 mdb_cursor_pop(MDB_cursor *mc)
 {
        if (mc->mc_snum) {
-#ifndef MDB_DEBUG_SKIP
+#if MDB_DEBUG
                MDB_page        *top = mc->mc_pg[mc->mc_top];
 #endif
                mc->mc_snum--;
                if (mc->mc_snum)
                        mc->mc_top--;
 
-               DPRINTF("popped page %zu off db %u cursor %p", top->mp_pgno,
-                       mc->mc_dbi, (void *) mc);
+               DPRINTF(("popped page %"Z"u off db %d cursor %p", top->mp_pgno,
+                       DDBI(mc), (void *) mc));
        }
 }
 
@@ -4286,8 +4529,8 @@ mdb_cursor_pop(MDB_cursor *mc)
 static int
 mdb_cursor_push(MDB_cursor *mc, MDB_page *mp)
 {
-       DPRINTF("pushing page %zu on db %u cursor %p", mp->mp_pgno,
-               mc->mc_dbi, (void *) mc);
+       DPRINTF(("pushing page %"Z"u on db %d cursor %p", mp->mp_pgno,
+               DDBI(mc), (void *) mc));
 
        if (mc->mc_snum >= CURSOR_STACK) {
                assert(mc->mc_snum < CURSOR_STACK);
@@ -4311,12 +4554,11 @@ mdb_cursor_push(MDB_cursor *mc, MDB_page *mp)
 static int
 mdb_page_get(MDB_txn *txn, pgno_t pgno, MDB_page **ret, int *lvl)
 {
+       MDB_env *env = txn->mt_env;
        MDB_page *p = NULL;
        int level;
 
-       if (!((txn->mt_flags & MDB_TXN_RDONLY) |
-                 (txn->mt_env->me_flags & MDB_WRITEMAP)))
-       {
+       if (!((txn->mt_flags & MDB_TXN_RDONLY) | (env->me_flags & MDB_WRITEMAP))) {
                MDB_txn *tx2 = txn;
                level = 1;
                do {
@@ -4328,9 +4570,10 @@ mdb_page_get(MDB_txn *txn, pgno_t pgno, MDB_page **ret, int *lvl)
                         * leave that unless page_touch happens again).
                         */
                        if (tx2->mt_spill_pgs) {
-                               x = mdb_midl_search(tx2->mt_spill_pgs, pgno);
-                               if (x <= tx2->mt_spill_pgs[0] && tx2->mt_spill_pgs[x] == pgno) {
-                                       p = (MDB_page *)(txn->mt_env->me_map + txn->mt_env->me_psize * pgno);
+                               MDB_ID pn = pgno << 1;
+                               x = mdb_midl_search(tx2->mt_spill_pgs, pn);
+                               if (x <= tx2->mt_spill_pgs[0] && tx2->mt_spill_pgs[x] == pn) {
+                                       p = (MDB_page *)(env->me_map + env->me_psize * pgno);
                                        goto done;
                                }
                        }
@@ -4347,9 +4590,9 @@ mdb_page_get(MDB_txn *txn, pgno_t pgno, MDB_page **ret, int *lvl)
 
        if (pgno < txn->mt_next_pgno) {
                level = 0;
-               p = (MDB_page *)(txn->mt_env->me_map + txn->mt_env->me_psize * pgno);
+               p = (MDB_page *)(env->me_map + env->me_psize * pgno);
        } else {
-               DPRINTF("page %zu not found", pgno);
+               DPRINTF(("page %"Z"u not found", pgno));
                assert(p != NULL);
                return MDB_PAGE_NOTFOUND;
        }
@@ -4361,37 +4604,28 @@ done:
        return MDB_SUCCESS;
 }
 
-/** Search for the page a given key should be in.
- * Pushes parent pages on the cursor stack. This function continues a
- * search on a cursor that has already been initialized. (Usually by
- * #mdb_page_search() but also by #mdb_node_move().)
- * @param[in,out] mc the cursor for this operation.
- * @param[in] key the key to search for. If NULL, search for the lowest
- * page. (This is used by #mdb_cursor_first().)
- * @param[in] modify If true, visited pages are updated with new page numbers.
- * @return 0 on success, non-zero on failure.
+/** Finish #mdb_page_search() / #mdb_page_search_lowest().
+ *     The cursor is at the root page, set up the rest of it.
  */
 static int
-mdb_page_search_root(MDB_cursor *mc, MDB_val *key, int modify)
+mdb_page_search_root(MDB_cursor *mc, MDB_val *key, int flags)
 {
        MDB_page        *mp = mc->mc_pg[mc->mc_top];
-       DKBUF;
        int rc;
-
+       DKBUF;
 
        while (IS_BRANCH(mp)) {
                MDB_node        *node;
                indx_t          i;
 
-               DPRINTF("branch page %zu has %u keys", mp->mp_pgno, NUMKEYS(mp));
+               DPRINTF(("branch page %"Z"u has %u keys", mp->mp_pgno, NUMKEYS(mp)));
                assert(NUMKEYS(mp) > 1);
-               DPRINTF("found index 0 to page %zu", NODEPGNO(NODEPTR(mp, 0)));
+               DPRINTF(("found index 0 to page %"Z"u", NODEPGNO(NODEPTR(mp, 0))));
 
-               if (key == NULL)        /* Initialize cursor to first page. */
+               if (flags & (MDB_PS_FIRST|MDB_PS_LAST)) {
                        i = 0;
-               else if (key->mv_size > MDB_MAXKEYSIZE && key->mv_data == NULL) {
-                                                       /* cursor to last page */
-                       i = NUMKEYS(mp)-1;
+                       if (flags & MDB_PS_LAST)
+                               i = NUMKEYS(mp) - 1;
                } else {
                        int      exact;
                        node = mdb_node_search(mc, key, &exact);
@@ -4404,11 +4638,9 @@ mdb_page_search_root(MDB_cursor *mc, MDB_val *key, int modify)
                                        i--;
                                }
                        }
+                       DPRINTF(("following index %u for key [%s]", i, DKEY(key)));
                }
 
-               if (key)
-                       DPRINTF("following index %u for key [%s]",
-                           i, DKEY(key));
                assert(i < NUMKEYS(mp));
                node = NODEPTR(mp, i);
 
@@ -4419,7 +4651,7 @@ mdb_page_search_root(MDB_cursor *mc, MDB_val *key, int modify)
                if ((rc = mdb_cursor_push(mc, mp)))
                        return rc;
 
-               if (modify) {
+               if (flags & MDB_PS_MODIFY) {
                        if ((rc = mdb_page_touch(mc)) != 0)
                                return rc;
                        mp = mc->mc_pg[mc->mc_top];
@@ -4427,13 +4659,13 @@ mdb_page_search_root(MDB_cursor *mc, MDB_val *key, int modify)
        }
 
        if (!IS_LEAF(mp)) {
-               DPRINTF("internal error, index points to a %02X page!?",
-                   mp->mp_flags);
+               DPRINTF(("internal error, index points to a %02X page!?",
+                   mp->mp_flags));
                return MDB_CORRUPTED;
        }
 
-       DPRINTF("found leaf page %zu for key [%s]", mp->mp_pgno,
-           key ? DKEY(key) : NULL);
+       DPRINTF(("found leaf page %"Z"u for key [%s]", mp->mp_pgno,
+           key ? DKEY(key) : "null"));
        mc->mc_flags |= C_INITIALIZED;
        mc->mc_flags &= ~C_EOF;
 
@@ -4459,18 +4691,17 @@ mdb_page_search_lowest(MDB_cursor *mc)
        mc->mc_ki[mc->mc_top] = 0;
        if ((rc = mdb_cursor_push(mc, mp)))
                return rc;
-       return mdb_page_search_root(mc, NULL, 0);
+       return mdb_page_search_root(mc, NULL, MDB_PS_FIRST);
 }
 
 /** Search for the page a given key should be in.
- * Pushes parent pages on the cursor stack. This function just sets up
- * the search; it finds the root page for \b mc's database and sets this
- * as the root of the cursor's stack. Then #mdb_page_search_root() is
- * called to complete the search.
+ * Push it and its parent pages on the cursor stack.
  * @param[in,out] mc the cursor for this operation.
- * @param[in] key the key to search for. If NULL, search for the lowest
- * page. (This is used by #mdb_cursor_first().)
- * @param[in] flags If MDB_PS_MODIFY set, visited pages are updated with new page numbers.
+ * @param[in] key the key to search for, or NULL for first/last page.
+ * @param[in] flags If MDB_PS_MODIFY is set, visited pages in the DB
+ *   are touched (updated with new page numbers).
+ *   If MDB_PS_FIRST or MDB_PS_LAST is set, find first or last leaf.
+ *   This is used by #mdb_cursor_first() and #mdb_cursor_last().
  *   If MDB_PS_ROOTONLY set, just fetch root node, no further lookups.
  * @return 0 on success, non-zero on failure.
  */
@@ -4481,23 +4712,20 @@ mdb_page_search(MDB_cursor *mc, MDB_val *key, int flags)
        pgno_t           root;
 
        /* Make sure the txn is still viable, then find the root from
-        * the txn's db table.
+        * the txn's db table and set it as the root of the cursor's stack.
         */
        if (F_ISSET(mc->mc_txn->mt_flags, MDB_TXN_ERROR)) {
                DPUTS("transaction has failed, must abort");
-               return EINVAL;
+               return MDB_BAD_TXN;
        } else {
                /* Make sure we're using an up-to-date root */
-               if (mc->mc_dbi > MAIN_DBI) {
-                       if ((*mc->mc_dbflag & DB_STALE) ||
-                       ((flags & MDB_PS_MODIFY) && !(*mc->mc_dbflag & DB_DIRTY))) {
+               if (*mc->mc_dbflag & DB_STALE) {
                                MDB_cursor mc2;
-                               unsigned char dbflag = 0;
                                mdb_cursor_init(&mc2, mc->mc_txn, MAIN_DBI, NULL);
-                               rc = mdb_page_search(&mc2, &mc->mc_dbx->md_name, flags & MDB_PS_MODIFY);
+                               rc = mdb_page_search(&mc2, &mc->mc_dbx->md_name, 0);
                                if (rc)
                                        return rc;
-                               if (*mc->mc_dbflag & DB_STALE) {
+                               {
                                        MDB_val data;
                                        int exact = 0;
                                        uint16_t flags;
@@ -4517,11 +4745,7 @@ mdb_page_search(MDB_cursor *mc, MDB_val *key, int flags)
                                                return MDB_INCOMPATIBLE;
                                        memcpy(mc->mc_db, data.mv_data, sizeof(MDB_db));
                                }
-                               if (flags & MDB_PS_MODIFY)
-                                       dbflag = DB_DIRTY;
                                *mc->mc_dbflag &= ~DB_STALE;
-                               *mc->mc_dbflag |= dbflag;
-                       }
                }
                root = mc->mc_db->md_root;
 
@@ -4539,8 +4763,8 @@ mdb_page_search(MDB_cursor *mc, MDB_val *key, int flags)
        mc->mc_snum = 1;
        mc->mc_top = 0;
 
-       DPRINTF("db %u root page %zu has flags 0x%X",
-               mc->mc_dbi, root, mc->mc_pg[0]->mp_flags);
+       DPRINTF(("db %d root page %"Z"u has flags 0x%X",
+               DDBI(mc), root, mc->mc_pg[0]->mp_flags));
 
        if (flags & MDB_PS_MODIFY) {
                if ((rc = mdb_page_touch(mc)))
@@ -4558,33 +4782,40 @@ mdb_ovpage_free(MDB_cursor *mc, MDB_page *mp)
 {
        MDB_txn *txn = mc->mc_txn;
        pgno_t pg = mp->mp_pgno;
-       unsigned i, ovpages = mp->mp_pages;
+       unsigned x = 0, ovpages = mp->mp_pages;
        MDB_env *env = txn->mt_env;
+       MDB_IDL sl = txn->mt_spill_pgs;
+       MDB_ID pn = pg << 1;
        int rc;
 
-       DPRINTF("free ov page %zu (%d)", pg, ovpages);
+       DPRINTF(("free ov page %"Z"u (%d)", pg, ovpages));
        /* If the page is dirty or on the spill list we just acquired it,
         * so we should give it back to our current free list, if any.
-        * Not currently supported in nested txns.
         * Otherwise put it onto the list of pages we freed in this txn.
+        *
+        * Won't create me_pghead: me_pglast must be inited along with it.
+        * Unsupported in nested txns: They would need to hide the page
+        * range in ancestor txns' dirty and spilled lists.
         */
-       if (!(mp->mp_flags & P_DIRTY) && txn->mt_spill_pgs) {
-               unsigned x = mdb_midl_search(txn->mt_spill_pgs, pg);
-               if (x <= txn->mt_spill_pgs[0] && txn->mt_spill_pgs[x] == pg) {
-                       /* This page is no longer spilled */
-                       for (; x < txn->mt_spill_pgs[0]; x++)
-                               txn->mt_spill_pgs[x] = txn->mt_spill_pgs[x+1];
-                       txn->mt_spill_pgs[0]--;
-                       goto release;
-               }
-       }
-       if ((mp->mp_flags & P_DIRTY) && !txn->mt_parent && env->me_pghead) {
-               unsigned j, x;
+       if (env->me_pghead &&
+               !txn->mt_parent &&
+               ((mp->mp_flags & P_DIRTY) ||
+                (sl && (x = mdb_midl_search(sl, pn)) <= sl[0] && sl[x] == pn)))
+       {
+               unsigned i, j;
                pgno_t *mop;
                MDB_ID2 *dl, ix, iy;
                rc = mdb_midl_need(&env->me_pghead, ovpages);
                if (rc)
                        return rc;
+               if (!(mp->mp_flags & P_DIRTY)) {
+                       /* This page is no longer spilled */
+                       if (x == sl[0])
+                               sl[0]--;
+                       else
+                               sl[x] |= 1;
+                       goto release;
+               }
                /* Remove from dirty list */
                dl = txn->mt_u.dirty_list;
                x = dl[0].mid--;
@@ -4645,7 +4876,7 @@ mdb_node_read(MDB_txn *txn, MDB_node *leaf, MDB_val *data)
        data->mv_size = NODEDSZ(leaf);
        memcpy(&pgno, NODEDATA(leaf), sizeof(pgno));
        if ((rc = mdb_page_get(txn, pgno, &omp, NULL)) != 0) {
-               DPRINTF("read overflow page %zu failed", pgno);
+               DPRINTF(("read overflow page %"Z"u failed", pgno));
                return rc;
        }
        data->mv_data = METADATA(omp);
@@ -4664,13 +4895,16 @@ mdb_get(MDB_txn *txn, MDB_dbi dbi,
 
        assert(key);
        assert(data);
-       DPRINTF("===> get db %u key [%s]", dbi, DKEY(key));
+       DPRINTF(("===> get db %u key [%s]", dbi, DKEY(key)));
 
        if (txn == NULL || !dbi || dbi >= txn->mt_numdbs || !(txn->mt_dbflags[dbi] & DB_VALID))
                return EINVAL;
 
-       if (key->mv_size == 0 || key->mv_size > MDB_MAXKEYSIZE) {
-               return EINVAL;
+       if (txn->mt_flags & MDB_TXN_ERROR)
+               return MDB_BAD_TXN;
+
+       if (key->mv_size > MDB_MAXKEYSIZE) {
+               return MDB_BAD_VALSIZE;
        }
 
        mdb_cursor_init(&mc, txn, dbi, &mx);
@@ -4697,13 +4931,13 @@ mdb_cursor_sibling(MDB_cursor *mc, int move_right)
        }
 
        mdb_cursor_pop(mc);
-       DPRINTF("parent page is page %zu, index %u",
-               mc->mc_pg[mc->mc_top]->mp_pgno, mc->mc_ki[mc->mc_top]);
+       DPRINTF(("parent page is page %"Z"u, index %u",
+               mc->mc_pg[mc->mc_top]->mp_pgno, mc->mc_ki[mc->mc_top]));
 
        if (move_right ? (mc->mc_ki[mc->mc_top] + 1u >= NUMKEYS(mc->mc_pg[mc->mc_top]))
                       : (mc->mc_ki[mc->mc_top] == 0)) {
-               DPRINTF("no more keys left, moving to %s sibling",
-                   move_right ? "right" : "left");
+               DPRINTF(("no more keys left, moving to %s sibling",
+                   move_right ? "right" : "left"));
                if ((rc = mdb_cursor_sibling(mc, move_right)) != MDB_SUCCESS) {
                        /* undo cursor_pop before returning */
                        mc->mc_top++;
@@ -4715,14 +4949,17 @@ mdb_cursor_sibling(MDB_cursor *mc, int move_right)
                        mc->mc_ki[mc->mc_top]++;
                else
                        mc->mc_ki[mc->mc_top]--;
-               DPRINTF("just moving to %s index key %u",
-                   move_right ? "right" : "left", mc->mc_ki[mc->mc_top]);
+               DPRINTF(("just moving to %s index key %u",
+                   move_right ? "right" : "left", mc->mc_ki[mc->mc_top]));
        }
        assert(IS_BRANCH(mc->mc_pg[mc->mc_top]));
 
        indx = NODEPTR(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
-       if ((rc = mdb_page_get(mc->mc_txn, NODEPGNO(indx), &mp, NULL) != 0))
+       if ((rc = mdb_page_get(mc->mc_txn, NODEPGNO(indx), &mp, NULL)) != 0) {
+               /* mc will be inconsistent if caller does mc_snum++ as above */
+               mc->mc_flags &= ~(C_INITIALIZED|C_EOF);
                return rc;
+       }
 
        mdb_cursor_push(mc, mp);
        if (!move_right)
@@ -4752,8 +4989,11 @@ mdb_cursor_next(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
                if (F_ISSET(leaf->mn_flags, F_DUPDATA)) {
                        if (op == MDB_NEXT || op == MDB_NEXT_DUP) {
                                rc = mdb_cursor_next(&mc->mc_xcursor->mx_cursor, data, NULL, MDB_NEXT);
-                               if (op != MDB_NEXT || rc != MDB_NOTFOUND)
+                               if (op != MDB_NEXT || rc != MDB_NOTFOUND) {
+                                       if (rc == MDB_SUCCESS)
+                                               MDB_GET_KEY(leaf, key);
                                        return rc;
+                               }
                        }
                } else {
                        mc->mc_xcursor->mx_cursor.mc_flags &= ~(C_INITIALIZED|C_EOF);
@@ -4762,7 +5002,9 @@ mdb_cursor_next(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
                }
        }
 
-       DPRINTF("cursor_next: top page is %zu in cursor %p", mp->mp_pgno, (void *) mc);
+       DPRINTF(("cursor_next: top page is %"Z"u in cursor %p", mp->mp_pgno, (void *) mc));
+       if (mc->mc_flags & C_DEL)
+               goto skip;
 
        if (mc->mc_ki[mc->mc_top] + 1u >= NUMKEYS(mp)) {
                DPUTS("=====> move to next sibling page");
@@ -4771,12 +5013,13 @@ mdb_cursor_next(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
                        return rc;
                }
                mp = mc->mc_pg[mc->mc_top];
-               DPRINTF("next page is %zu, key index %u", mp->mp_pgno, mc->mc_ki[mc->mc_top]);
+               DPRINTF(("next page is %"Z"u, key index %u", mp->mp_pgno, mc->mc_ki[mc->mc_top]));
        } else
                mc->mc_ki[mc->mc_top]++;
 
-       DPRINTF("==> cursor points to page %zu with %u keys, key index %u",
-           mp->mp_pgno, NUMKEYS(mp), mc->mc_ki[mc->mc_top]);
+skip:
+       DPRINTF(("==> cursor points to page %"Z"u with %u keys, key index %u",
+           mp->mp_pgno, NUMKEYS(mp), mc->mc_ki[mc->mc_top]));
 
        if (IS_LEAF2(mp)) {
                key->mv_size = mc->mc_db->md_pad;
@@ -4819,11 +5062,14 @@ mdb_cursor_prev(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
 
        if (mc->mc_db->md_flags & MDB_DUPSORT) {
                leaf = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
-               if (op == MDB_PREV || op == MDB_PREV_DUP) {
-                       if (F_ISSET(leaf->mn_flags, F_DUPDATA)) {
+               if (F_ISSET(leaf->mn_flags, F_DUPDATA)) {
+                       if (op == MDB_PREV || op == MDB_PREV_DUP) {
                                rc = mdb_cursor_prev(&mc->mc_xcursor->mx_cursor, data, NULL, MDB_PREV);
-                               if (op != MDB_PREV || rc != MDB_NOTFOUND)
+                               if (op != MDB_PREV || rc != MDB_NOTFOUND) {
+                                       if (rc == MDB_SUCCESS)
+                                               MDB_GET_KEY(leaf, key);
                                        return rc;
+                               }
                        } else {
                                mc->mc_xcursor->mx_cursor.mc_flags &= ~(C_INITIALIZED|C_EOF);
                                if (op == MDB_PREV_DUP)
@@ -4832,7 +5078,7 @@ mdb_cursor_prev(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
                }
        }
 
-       DPRINTF("cursor_prev: top page is %zu in cursor %p", mp->mp_pgno, (void *) mc);
+       DPRINTF(("cursor_prev: top page is %"Z"u in cursor %p", mp->mp_pgno, (void *) mc));
 
        if (mc->mc_ki[mc->mc_top] == 0)  {
                DPUTS("=====> move to prev sibling page");
@@ -4841,14 +5087,14 @@ mdb_cursor_prev(MDB_cursor *mc, MDB_val *key, MDB_val *data, MDB_cursor_op op)
                }
                mp = mc->mc_pg[mc->mc_top];
                mc->mc_ki[mc->mc_top] = NUMKEYS(mp) - 1;
-               DPRINTF("prev page is %zu, key index %u", mp->mp_pgno, mc->mc_ki[mc->mc_top]);
+               DPRINTF(("prev page is %"Z"u, key index %u", mp->mp_pgno, mc->mc_ki[mc->mc_top]));
        } else
                mc->mc_ki[mc->mc_top]--;
 
        mc->mc_flags &= ~C_EOF;
 
-       DPRINTF("==> cursor points to page %zu with %u keys, key index %u",
-           mp->mp_pgno, NUMKEYS(mp), mc->mc_ki[mc->mc_top]);
+       DPRINTF(("==> cursor points to page %"Z"u with %u keys, key index %u",
+           mp->mp_pgno, NUMKEYS(mp), mc->mc_ki[mc->mc_top]));
 
        if (IS_LEAF2(mp)) {
                key->mv_size = mc->mc_db->md_pad;
@@ -4889,7 +5135,8 @@ mdb_cursor_set(MDB_cursor *mc, MDB_val *key, MDB_val *data,
 
        assert(mc);
        assert(key);
-       assert(key->mv_size > 0);
+       if (key->mv_size == 0)
+               return MDB_BAD_VALSIZE;
 
        if (mc->mc_xcursor)
                mc->mc_xcursor->mx_cursor.mc_flags &= ~(C_INITIALIZED|C_EOF);
@@ -4908,7 +5155,7 @@ mdb_cursor_set(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                        nodekey.mv_data = LEAF2KEY(mp, 0, nodekey.mv_size);
                } else {
                        leaf = NODEPTR(mp, 0);
-                       MDB_GET_KEY(leaf, &nodekey);
+                       MDB_GET_KEY2(leaf, nodekey);
                }
                rc = mc->mc_dbx->md_cmp(key, &nodekey);
                if (rc == 0) {
@@ -4929,7 +5176,7 @@ mdb_cursor_set(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                                                 nkeys-1, nodekey.mv_size);
                                } else {
                                        leaf = NODEPTR(mp, nkeys-1);
-                                       MDB_GET_KEY(leaf, &nodekey);
+                                       MDB_GET_KEY2(leaf, nodekey);
                                }
                                rc = mc->mc_dbx->md_cmp(key, &nodekey);
                                if (rc == 0) {
@@ -4947,7 +5194,7 @@ mdb_cursor_set(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                                                                 mc->mc_ki[mc->mc_top], nodekey.mv_size);
                                                } else {
                                                        leaf = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
-                                                       MDB_GET_KEY(leaf, &nodekey);
+                                                       MDB_GET_KEY2(leaf, nodekey);
                                                }
                                                rc = mc->mc_dbx->md_cmp(key, &nodekey);
                                                if (rc == 0) {
@@ -4977,7 +5224,11 @@ mdb_cursor_set(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                if (!mc->mc_top) {
                        /* There are no other pages */
                        mc->mc_ki[mc->mc_top] = 0;
-                       return MDB_NOTFOUND;
+                       if (op == MDB_SET_RANGE) {
+                               rc = 0;
+                               goto set1;
+                       } else
+                               return MDB_NOTFOUND;
                }
        }
 
@@ -5041,6 +5292,7 @@ set1:
                        if (rc) {
                                if (op == MDB_GET_BOTH || rc > 0)
                                        return MDB_NOTFOUND;
+                               rc = 0;
                        }
 
                } else {
@@ -5054,7 +5306,7 @@ set1:
        /* The key already matches in all other cases */
        if (op == MDB_SET_RANGE || op == MDB_SET_KEY)
                MDB_GET_KEY(leaf, key);
-       DPRINTF("==> cursor placed on key [%s]", DKEY(key));
+       DPRINTF(("==> cursor placed on key [%s]", DKEY(key)));
 
        return rc;
 }
@@ -5070,7 +5322,7 @@ mdb_cursor_first(MDB_cursor *mc, MDB_val *key, MDB_val *data)
                mc->mc_xcursor->mx_cursor.mc_flags &= ~(C_INITIALIZED|C_EOF);
 
        if (!(mc->mc_flags & C_INITIALIZED) || mc->mc_top) {
-               rc = mdb_page_search(mc, NULL, 0);
+               rc = mdb_page_search(mc, NULL, MDB_PS_FIRST);
                if (rc != MDB_SUCCESS)
                        return rc;
        }
@@ -5116,11 +5368,7 @@ mdb_cursor_last(MDB_cursor *mc, MDB_val *key, MDB_val *data)
        if (!(mc->mc_flags & C_EOF)) {
 
                if (!(mc->mc_flags & C_INITIALIZED) || mc->mc_top) {
-                       MDB_val lkey;
-
-                       lkey.mv_size = MDB_MAXKEYSIZE+1;
-                       lkey.mv_data = NULL;
-                       rc = mdb_page_search(mc, &lkey, 0);
+                       rc = mdb_page_search(mc, NULL, MDB_PS_LAST);
                        if (rc != MDB_SUCCESS)
                                return rc;
                }
@@ -5159,17 +5407,22 @@ mdb_cursor_get(MDB_cursor *mc, MDB_val *key, MDB_val *data,
 {
        int              rc;
        int              exact = 0;
+       int              (*mfunc)(MDB_cursor *mc, MDB_val *key, MDB_val *data);
 
        assert(mc);
 
+       if (mc->mc_txn->mt_flags & MDB_TXN_ERROR)
+               return MDB_BAD_TXN;
+
        switch (op) {
        case MDB_GET_CURRENT:
                if (!(mc->mc_flags & C_INITIALIZED)) {
                        rc = EINVAL;
                } else {
                        MDB_page *mp = mc->mc_pg[mc->mc_top];
-                       if (!NUMKEYS(mp)) {
-                               mc->mc_ki[mc->mc_top] = 0;
+                       int nkeys = NUMKEYS(mp);
+                       if (!nkeys || mc->mc_ki[mc->mc_top] >= nkeys) {
+                               mc->mc_ki[mc->mc_top] = nkeys;
                                rc = MDB_NOTFOUND;
                                break;
                        }
@@ -5182,6 +5435,8 @@ mdb_cursor_get(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                                MDB_GET_KEY(leaf, key);
                                if (data) {
                                        if (F_ISSET(leaf->mn_flags, F_DUPDATA)) {
+                                               if (mc->mc_flags & C_DEL)
+                                                       mdb_xcursor_init1(mc, leaf);
                                                rc = mdb_cursor_get(&mc->mc_xcursor->mx_cursor, data, NULL, MDB_GET_CURRENT);
                                        } else {
                                                rc = mdb_node_read(mc->mc_txn, leaf, data);
@@ -5192,39 +5447,50 @@ mdb_cursor_get(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                break;
        case MDB_GET_BOTH:
        case MDB_GET_BOTH_RANGE:
-               if (data == NULL || mc->mc_xcursor == NULL) {
+               if (data == NULL) {
                        rc = EINVAL;
                        break;
                }
+               if (mc->mc_xcursor == NULL) {
+                       rc = MDB_INCOMPATIBLE;
+                       break;
+               }
                /* FALLTHRU */
        case MDB_SET:
        case MDB_SET_KEY:
        case MDB_SET_RANGE:
-               if (key == NULL || key->mv_size == 0 || key->mv_size > MDB_MAXKEYSIZE) {
+               if (key == NULL) {
                        rc = EINVAL;
+               } else if (key->mv_size > MDB_MAXKEYSIZE) {
+                       rc = MDB_BAD_VALSIZE;
                } else if (op == MDB_SET_RANGE)
                        rc = mdb_cursor_set(mc, key, data, op, NULL);
                else
                        rc = mdb_cursor_set(mc, key, data, op, &exact);
                break;
        case MDB_GET_MULTIPLE:
-               if (data == NULL ||
-                       !(mc->mc_db->md_flags & MDB_DUPFIXED) ||
-                       !(mc->mc_flags & C_INITIALIZED)) {
+               if (data == NULL || !(mc->mc_flags & C_INITIALIZED)) {
                        rc = EINVAL;
                        break;
                }
+               if (!(mc->mc_db->md_flags & MDB_DUPFIXED)) {
+                       rc = MDB_INCOMPATIBLE;
+                       break;
+               }
                rc = MDB_SUCCESS;
                if (!(mc->mc_xcursor->mx_cursor.mc_flags & C_INITIALIZED) ||
                        (mc->mc_xcursor->mx_cursor.mc_flags & C_EOF))
                        break;
                goto fetchm;
        case MDB_NEXT_MULTIPLE:
-               if (data == NULL ||
-                       !(mc->mc_db->md_flags & MDB_DUPFIXED)) {
+               if (data == NULL) {
                        rc = EINVAL;
                        break;
                }
+               if (!(mc->mc_db->md_flags & MDB_DUPFIXED)) {
+                       rc = MDB_INCOMPATIBLE;
+                       break;
+               }
                if (!(mc->mc_flags & C_INITIALIZED))
                        rc = mdb_cursor_first(mc, key, data);
                else
@@ -5267,45 +5533,48 @@ fetchm:
                rc = mdb_cursor_first(mc, key, data);
                break;
        case MDB_FIRST_DUP:
-               if (data == NULL ||
-                       !(mc->mc_db->md_flags & MDB_DUPSORT) ||
-                       !(mc->mc_flags & C_INITIALIZED) ||
-                       !(mc->mc_xcursor->mx_cursor.mc_flags & C_INITIALIZED)) {
+               mfunc = mdb_cursor_first;
+       mmove:
+               if (data == NULL || !(mc->mc_flags & C_INITIALIZED)) {
+                       rc = EINVAL;
+                       break;
+               }
+               if (mc->mc_xcursor == NULL) {
+                       rc = MDB_INCOMPATIBLE;
+                       break;
+               }
+               if (!(mc->mc_xcursor->mx_cursor.mc_flags & C_INITIALIZED)) {
                        rc = EINVAL;
                        break;
                }
-               rc = mdb_cursor_first(&mc->mc_xcursor->mx_cursor, data, NULL);
+               rc = mfunc(&mc->mc_xcursor->mx_cursor, data, NULL);
                break;
        case MDB_LAST:
                rc = mdb_cursor_last(mc, key, data);
                break;
        case MDB_LAST_DUP:
-               if (data == NULL ||
-                       !(mc->mc_db->md_flags & MDB_DUPSORT) ||
-                       !(mc->mc_flags & C_INITIALIZED) ||
-                       !(mc->mc_xcursor->mx_cursor.mc_flags & C_INITIALIZED)) {
-                       rc = EINVAL;
-                       break;
-               }
-               rc = mdb_cursor_last(&mc->mc_xcursor->mx_cursor, data, NULL);
-               break;
+               mfunc = mdb_cursor_last;
+               goto mmove;
        default:
-               DPRINTF("unhandled/unimplemented cursor operation %u", op);
+               DPRINTF(("unhandled/unimplemented cursor operation %u", op));
                rc = EINVAL;
                break;
        }
 
+       if (mc->mc_flags & C_DEL)
+               mc->mc_flags ^= C_DEL;
+
        return rc;
 }
 
-/** Touch all the pages in the cursor stack.
+/** Touch all the pages in the cursor stack. Set mc_top.
  *     Makes sure all the pages are writable, before attempting a write operation.
  * @param[in] mc The cursor to operate on.
  */
 static int
 mdb_cursor_touch(MDB_cursor *mc)
 {
-       int rc;
+       int rc = MDB_SUCCESS;
 
        if (mc->mc_dbi > MAIN_DBI && !(*mc->mc_dbflag & DB_DIRTY)) {
                MDB_cursor mc2;
@@ -5316,13 +5585,14 @@ mdb_cursor_touch(MDB_cursor *mc)
                         return rc;
                *mc->mc_dbflag |= DB_DIRTY;
        }
-       for (mc->mc_top = 0; mc->mc_top < mc->mc_snum; mc->mc_top++) {
-               rc = mdb_page_touch(mc);
-               if (rc)
-                       return rc;
+       mc->mc_top = 0;
+       if (mc->mc_snum) {
+               do {
+                       rc = mdb_page_touch(mc);
+               } while (!rc && ++(mc->mc_top) < mc->mc_snum);
+               mc->mc_top = mc->mc_snum-1;
        }
-       mc->mc_top = mc->mc_snum-1;
-       return MDB_SUCCESS;
+       return rc;
 }
 
 /** Do not spill pages to disk if txn is getting full, may fail instead */
@@ -5353,28 +5623,28 @@ mdb_cursor_put(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                dcount = data[1].mv_size;
                data[1].mv_size = 0;
                if (!F_ISSET(mc->mc_db->md_flags, MDB_DUPFIXED))
-                       return EINVAL;
+                       return MDB_INCOMPATIBLE;
        }
 
        nospill = flags & MDB_NOSPILL;
        flags &= ~MDB_NOSPILL;
 
-       if (F_ISSET(mc->mc_txn->mt_flags, MDB_TXN_RDONLY))
-               return EACCES;
+       if (mc->mc_txn->mt_flags & (MDB_TXN_RDONLY|MDB_TXN_ERROR))
+               return (mc->mc_txn->mt_flags & MDB_TXN_RDONLY) ? EACCES : MDB_BAD_TXN;
 
        if (flags != MDB_CURRENT && (key->mv_size == 0 || key->mv_size > MDB_MAXKEYSIZE))
-               return EINVAL;
+               return MDB_BAD_VALSIZE;
 
        if (F_ISSET(mc->mc_db->md_flags, MDB_DUPSORT) && data->mv_size > MDB_MAXKEYSIZE)
-               return EINVAL;
+               return MDB_BAD_VALSIZE;
 
 #if SIZE_MAX > MAXDATASIZE
        if (data->mv_size > MAXDATASIZE)
-               return EINVAL;
+               return MDB_BAD_VALSIZE;
 #endif
 
-       DPRINTF("==> put db %u key [%s], size %zu, data size %zu",
-               mc->mc_dbi, DKEY(key), key ? key->mv_size:0, data->mv_size);
+       DPRINTF(("==> put db %d key [%s], size %"Z"u, data size %"Z"u",
+               DDBI(mc), DKEY(key), key ? key->mv_size : 0, data->mv_size));
 
        dkey.mv_size = 0;
 
@@ -5385,6 +5655,7 @@ mdb_cursor_put(MDB_cursor *mc, MDB_val *key, MDB_val *data,
        } else if (mc->mc_db->md_root == P_INVALID) {
                /* new database, cursor has nothing to point to */
                mc->mc_snum = 0;
+               mc->mc_top = 0;
                mc->mc_flags &= ~C_INITIALIZED;
                rc = MDB_NO_ROOT;
        } else {
@@ -5407,7 +5678,7 @@ mdb_cursor_put(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                        rc = mdb_cursor_set(mc, key, &d2, MDB_SET, &exact);
                }
                if ((flags & MDB_NOOVERWRITE) && rc == 0) {
-                       DPRINTF("duplicate key [%s]", DKEY(key));
+                       DPRINTF(("duplicate key [%s]", DKEY(key)));
                        *data = d2;
                        return MDB_KEYEXIST;
                }
@@ -5415,6 +5686,9 @@ mdb_cursor_put(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                        return rc;
        }
 
+       if (mc->mc_flags & C_DEL)
+               mc->mc_flags ^= C_DEL;
+
        /* Cursor is positioned, check for room in the dirty list */
        if (!nospill) {
                if (flags & MDB_MULTIPLE) {
@@ -5455,7 +5729,7 @@ mdb_cursor_put(MDB_cursor *mc, MDB_val *key, MDB_val *data,
                if (IS_LEAF2(mc->mc_pg[mc->mc_top])) {
                        unsigned int ksize = mc->mc_db->md_pad;
                        if (key->mv_size != ksize)
-                               return EINVAL;
+                               return MDB_BAD_VALSIZE;
                        if (flags == MDB_CURRENT) {
                                char *ptr = LEAF2KEY(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top], ksize);
                                memcpy(ptr, key->mv_data, ksize);
@@ -5484,9 +5758,16 @@ more:
                                        mc->mc_dbx->md_dcmp = mdb_cmp_cint;
 #endif
 #endif
-                               /* if data matches, ignore it */
-                               if (!mc->mc_dbx->md_dcmp(data, &dkey))
-                                       return (flags == MDB_NODUPDATA) ? MDB_KEYEXIST : MDB_SUCCESS;
+                               /* if data matches, skip it */
+                               if (!mc->mc_dbx->md_dcmp(data, &dkey)) {
+                                       if (flags & MDB_NODUPDATA)
+                                               rc = MDB_KEYEXIST;
+                                       else if (flags & MDB_MULTIPLE)
+                                               goto next_mult;
+                                       else
+                                               rc = MDB_SUCCESS;
+                                       return rc;
+                               }
 
                                /* create a fake page for the dup items */
                                memcpy(dbuf, dkey.mv_data, dkey.mv_size);
@@ -5668,7 +5949,7 @@ current:
                mdb_node_del(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top], 0);
                mc->mc_db->md_entries--;
        } else {
-               DPRINTF("inserting key at index %i", mc->mc_ki[mc->mc_top]);
+               DPRINTF(("inserting key at index %i", mc->mc_ki[mc->mc_top]));
                insert = 1;
        }
 
@@ -5693,9 +5974,6 @@ new_sub:
                        unsigned i = mc->mc_top;
                        MDB_page *mp = mc->mc_pg[i];
 
-                       if (mc->mc_flags & C_SUB)
-                               dbi--;
-
                        for (m2 = mc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                                if (mc->mc_flags & C_SUB)
                                        m3 = &m2->mc_xcursor->mx_cursor;
@@ -5767,15 +6045,16 @@ put_sub:
                        mc->mc_db->md_entries++;
                if (flags & MDB_MULTIPLE) {
                        if (!rc) {
+next_mult:
                                mcount++;
+                               /* let caller know how many succeeded, if any */
+                               data[1].mv_size = mcount;
                                if (mcount < dcount) {
                                        data[0].mv_data = (char *)data[0].mv_data + data[0].mv_size;
                                        leaf = NODEPTR(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
                                        goto more;
                                }
                        }
-                       /* let caller know how many succeeded, if any */
-                       data[1].mv_size = mcount;
                }
        }
 done:
@@ -5791,26 +6070,30 @@ int
 mdb_cursor_del(MDB_cursor *mc, unsigned int flags)
 {
        MDB_node        *leaf;
+       MDB_page        *mp;
        int rc;
 
-       if (F_ISSET(mc->mc_txn->mt_flags, MDB_TXN_RDONLY))
-               return EACCES;
+       if (mc->mc_txn->mt_flags & (MDB_TXN_RDONLY|MDB_TXN_ERROR))
+               return (mc->mc_txn->mt_flags & MDB_TXN_RDONLY) ? EACCES : MDB_BAD_TXN;
 
        if (!(mc->mc_flags & C_INITIALIZED))
                return EINVAL;
 
+       if (mc->mc_ki[mc->mc_top] >= NUMKEYS(mc->mc_pg[mc->mc_top]))
+               return MDB_NOTFOUND;
+
        if (!(flags & MDB_NOSPILL) && (rc = mdb_page_spill(mc, NULL, NULL)))
                return rc;
-       flags &= ~MDB_NOSPILL; /* TODO: Or change (flags != MDB_NODUPDATA) to ~(flags & MDB_NODUPDATA), not looking at the logic of that code just now */
 
        rc = mdb_cursor_touch(mc);
        if (rc)
                return rc;
 
-       leaf = NODEPTR(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
+       mp = mc->mc_pg[mc->mc_top];
+       leaf = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
 
-       if (!IS_LEAF2(mc->mc_pg[mc->mc_top]) && F_ISSET(leaf->mn_flags, F_DUPDATA)) {
-               if (flags != MDB_NODUPDATA) {
+       if (!IS_LEAF2(mp) && F_ISSET(leaf->mn_flags, F_DUPDATA)) {
+               if (!(flags & MDB_NODUPDATA)) {
                        if (!F_ISSET(leaf->mn_flags, F_SUBDATA)) {
                                mc->mc_xcursor->mx_cursor.mc_pg[0] = NODEDATA(leaf);
                        }
@@ -5824,18 +6107,19 @@ mdb_cursor_del(MDB_cursor *mc, unsigned int flags)
                                } else {
                                        MDB_cursor *m2;
                                        /* shrink fake page */
-                                       mdb_node_shrink(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
-                                       leaf = NODEPTR(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
+                                       mdb_node_shrink(mp, mc->mc_ki[mc->mc_top]);
+                                       leaf = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
                                        mc->mc_xcursor->mx_cursor.mc_pg[0] = NODEDATA(leaf);
                                        /* fix other sub-DB cursors pointed at this fake page */
                                        for (m2 = mc->mc_txn->mt_cursors[mc->mc_dbi]; m2; m2=m2->mc_next) {
                                                if (m2 == mc || m2->mc_snum < mc->mc_snum) continue;
-                                               if (m2->mc_pg[mc->mc_top] == mc->mc_pg[mc->mc_top] &&
+                                               if (m2->mc_pg[mc->mc_top] == mp &&
                                                        m2->mc_ki[mc->mc_top] == mc->mc_ki[mc->mc_top])
                                                        m2->mc_xcursor->mx_cursor.mc_pg[0] = NODEDATA(leaf);
                                        }
                                }
                                mc->mc_db->md_entries--;
+                               mc->mc_flags |= C_DEL;
                                return rc;
                        }
                        /* otherwise fall thru and delete the sub-DB */
@@ -5870,8 +6154,8 @@ mdb_page_new(MDB_cursor *mc, uint32_t flags, int num, MDB_page **mp)
 
        if ((rc = mdb_page_alloc(mc, num, &np)))
                return rc;
-       DPRINTF("allocated new mpage %zu, page size %u",
-           np->mp_pgno, mc->mc_txn->mt_env->me_psize);
+       DPRINTF(("allocated new mpage %"Z"u, page size %u",
+           np->mp_pgno, mc->mc_txn->mt_env->me_psize));
        np->mp_flags = flags | P_DIRTY;
        np->mp_lower = PAGEHDRSZ;
        np->mp_upper = mc->mc_txn->mt_env->me_psize;
@@ -5961,6 +6245,7 @@ mdb_node_add(MDB_cursor *mc, indx_t indx,
 {
        unsigned int     i;
        size_t           node_size = NODESIZE;
+       ssize_t          room;
        indx_t           ofs;
        MDB_node        *node;
        MDB_page        *mp = mc->mc_pg[mc->mc_top];
@@ -5969,11 +6254,11 @@ mdb_node_add(MDB_cursor *mc, indx_t indx,
 
        assert(mp->mp_upper >= mp->mp_lower);
 
-       DPRINTF("add to %s %spage %zu index %i, data size %zu key size %zu [%s]",
+       DPRINTF(("add to %s %spage %"Z"u index %i, data size %"Z"u key size %"Z"u [%s]",
            IS_LEAF(mp) ? "leaf" : "branch",
                IS_SUBP(mp) ? "sub-" : "",
            mp->mp_pgno, indx, data ? data->mv_size : 0,
-               key ? key->mv_size : 0, key ? DKEY(key) : NULL);
+               key ? key->mv_size : 0, key ? DKEY(key) : "null"));
 
        if (IS_LEAF2(mp)) {
                /* Move higher keys up one slot. */
@@ -5991,9 +6276,9 @@ mdb_node_add(MDB_cursor *mc, indx_t indx,
                return MDB_SUCCESS;
        }
 
+       room = (ssize_t)SIZELEFT(mp) - (ssize_t)sizeof(indx_t);
        if (key != NULL)
                node_size += key->mv_size;
-
        if (IS_LEAF(mp)) {
                assert(data);
                if (F_ISSET(flags, F_BIGDATA)) {
@@ -6003,28 +6288,25 @@ mdb_node_add(MDB_cursor *mc, indx_t indx,
                        int ovpages = OVPAGES(data->mv_size, mc->mc_txn->mt_env->me_psize);
                        int rc;
                        /* Put data on overflow page. */
-                       DPRINTF("data size is %zu, node would be %zu, put data on overflow page",
-                           data->mv_size, node_size+data->mv_size);
-                       node_size += sizeof(pgno_t);
+                       DPRINTF(("data size is %"Z"u, node would be %"Z"u, put data on overflow page",
+                           data->mv_size, node_size+data->mv_size));
+                       node_size += sizeof(pgno_t) + (node_size & 1);
+                       if ((ssize_t)node_size > room)
+                               goto full;
                        if ((rc = mdb_page_new(mc, P_OVERFLOW, ovpages, &ofp)))
                                return rc;
-                       DPRINTF("allocated overflow page %zu", ofp->mp_pgno);
+                       DPRINTF(("allocated overflow page %"Z"u", ofp->mp_pgno));
                        flags |= F_BIGDATA;
+                       goto update;
                } else {
                        node_size += data->mv_size;
                }
        }
        node_size += node_size & 1;
+       if ((ssize_t)node_size > room)
+               goto full;
 
-       if (node_size + sizeof(indx_t) > SIZELEFT(mp)) {
-               DPRINTF("not enough room in page %zu, got %u ptrs",
-                   mp->mp_pgno, NUMKEYS(mp));
-               DPRINTF("upper - lower = %u - %u = %u", mp->mp_upper, mp->mp_lower,
-                   mp->mp_upper - mp->mp_lower);
-               DPRINTF("node size = %zu", node_size);
-               return MDB_PAGE_FULL;
-       }
-
+update:
        /* Move higher pointers up one slot. */
        for (i = NUMKEYS(mp); i > indx; i--)
                mp->mp_ptrs[i] = mp->mp_ptrs[i - 1];
@@ -6070,6 +6352,13 @@ mdb_node_add(MDB_cursor *mc, indx_t indx,
        }
 
        return MDB_SUCCESS;
+
+full:
+       DPRINTF(("not enough room in page %"Z"u, got %u ptrs",
+               mp->mp_pgno, NUMKEYS(mp)));
+       DPRINTF(("upper-lower = %u - %u = %"Z"d", mp->mp_upper,mp->mp_lower,room));
+       DPRINTF(("node size = %"Z"u", node_size));
+       return MDB_PAGE_FULL;
 }
 
 /** Delete the specified node from a page.
@@ -6090,8 +6379,8 @@ mdb_node_del(MDB_page *mp, indx_t indx, int ksize)
        {
        pgno_t pgno;
        COPY_PGNO(pgno, mp->mp_pgno);
-       DPRINTF("delete node %u on %s page %zu", indx,
-           IS_LEAF(mp) ? "leaf" : "branch", pgno);
+       DPRINTF(("delete node %u on %s page %"Z"u", indx,
+           IS_LEAF(mp) ? "leaf" : "branch", pgno));
        }
 #endif
        assert(indx < NUMKEYS(mp));
@@ -6204,11 +6493,13 @@ mdb_xcursor_init0(MDB_cursor *mc)
        mx->mx_cursor.mc_txn = mc->mc_txn;
        mx->mx_cursor.mc_db = &mx->mx_db;
        mx->mx_cursor.mc_dbx = &mx->mx_dbx;
-       mx->mx_cursor.mc_dbi = mc->mc_dbi+1;
+       mx->mx_cursor.mc_dbi = mc->mc_dbi;
        mx->mx_cursor.mc_dbflag = &mx->mx_dbflag;
        mx->mx_cursor.mc_snum = 0;
        mx->mx_cursor.mc_top = 0;
        mx->mx_cursor.mc_flags = C_SUB;
+       mx->mx_dbx.md_name.mv_size = 0;
+       mx->mx_dbx.md_name.mv_data = NULL;
        mx->mx_dbx.md_cmp = mc->mc_dbx->md_dcmp;
        mx->mx_dbx.md_dcmp = NULL;
        mx->mx_dbx.md_rel = mc->mc_dbx->md_rel;
@@ -6229,6 +6520,7 @@ mdb_xcursor_init1(MDB_cursor *mc, MDB_node *node)
                memcpy(&mx->mx_db, NODEDATA(node), sizeof(MDB_db));
                mx->mx_cursor.mc_pg[0] = 0;
                mx->mx_cursor.mc_snum = 0;
+               mx->mx_cursor.mc_top = 0;
                mx->mx_cursor.mc_flags = C_SUB;
        } else {
                MDB_page *fp = NODEDATA(node);
@@ -6241,8 +6533,8 @@ mdb_xcursor_init1(MDB_cursor *mc, MDB_node *node)
                mx->mx_db.md_entries = NUMKEYS(fp);
                COPY_PGNO(mx->mx_db.md_root, fp->mp_pgno);
                mx->mx_cursor.mc_snum = 1;
-               mx->mx_cursor.mc_flags = C_INITIALIZED|C_SUB;
                mx->mx_cursor.mc_top = 0;
+               mx->mx_cursor.mc_flags = C_INITIALIZED|C_SUB;
                mx->mx_cursor.mc_pg[0] = fp;
                mx->mx_cursor.mc_ki[0] = 0;
                if (mc->mc_db->md_flags & MDB_DUPFIXED) {
@@ -6252,12 +6544,9 @@ mdb_xcursor_init1(MDB_cursor *mc, MDB_node *node)
                                mx->mx_db.md_flags |= MDB_INTEGERKEY;
                }
        }
-       DPRINTF("Sub-db %u for db %u root page %zu", mx->mx_cursor.mc_dbi, mc->mc_dbi,
-               mx->mx_db.md_root);
-       mx->mx_dbflag = DB_VALID | (F_ISSET(mc->mc_pg[mc->mc_top]->mp_flags, P_DIRTY) ?
-               DB_DIRTY : 0);
-       mx->mx_dbx.md_name.mv_data = NODEKEY(node);
-       mx->mx_dbx.md_name.mv_size = node->mn_ksize;
+       DPRINTF(("Sub-db -%u root page %"Z"u", mx->mx_cursor.mc_dbi,
+               mx->mx_db.md_root));
+       mx->mx_dbflag = DB_VALID|DB_DIRTY; /* DB_DIRTY guides mdb_cursor_touch */
 #if UINT_MAX < SIZE_MAX
        if (mx->mx_dbx.md_cmp == mdb_cmp_int && mx->mx_db.md_pad == sizeof(size_t))
 #ifdef MISALIGNED_OK
@@ -6304,6 +6593,9 @@ mdb_cursor_open(MDB_txn *txn, MDB_dbi dbi, MDB_cursor **ret)
        if (txn == NULL || ret == NULL || dbi >= txn->mt_numdbs || !(txn->mt_dbflags[dbi] & DB_VALID))
                return EINVAL;
 
+       if (txn->mt_flags & MDB_TXN_ERROR)
+               return MDB_BAD_TXN;
+
        /* Allow read access to the freelist */
        if (!dbi && !F_ISSET(txn->mt_flags, MDB_TXN_RDONLY))
                return EINVAL;
@@ -6349,8 +6641,8 @@ mdb_cursor_count(MDB_cursor *mc, size_t *countp)
        if (mc == NULL || countp == NULL)
                return EINVAL;
 
-       if (!(mc->mc_db->md_flags & MDB_DUPSORT))
-               return EINVAL;
+       if (mc->mc_xcursor == NULL)
+               return MDB_INCOMPATIBLE;
 
        leaf = NODEPTR(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top]);
        if (!F_ISSET(leaf->mn_flags, F_DUPDATA)) {
@@ -6419,11 +6711,11 @@ mdb_update_key(MDB_cursor *mc, MDB_val *key)
                char kbuf2[(MDB_MAXKEYSIZE*2+1)];
                k2.mv_data = NODEKEY(node);
                k2.mv_size = node->mn_ksize;
-               DPRINTF("update key %u (ofs %u) [%s] to [%s] on page %zu",
+               DPRINTF(("update key %u (ofs %u) [%s] to [%s] on page %"Z"u",
                        indx, ptr,
                        mdb_dkey(&k2, kbuf2),
                        DKEY(key),
-                       mp->mp_pgno);
+                       mp->mp_pgno));
        }
 #endif
 
@@ -6437,7 +6729,7 @@ mdb_update_key(MDB_cursor *mc, MDB_val *key)
                if (delta > 0 && SIZELEFT(mp) < delta) {
                        pgno_t pgno;
                        /* not enough space left, do a delete and split */
-                       DPRINTF("Not enough room, delta = %d, splitting...", delta);
+                       DPRINTF(("Not enough room, delta = %d, splitting...", delta));
                        pgno = NODEPGNO(node);
                        mdb_node_del(mc->mc_pg[mc->mc_top], mc->mc_ki[mc->mc_top], 0);
                        return mdb_page_split(mc, key, NULL, pgno, MDB_SPLIT_REPLACE);
@@ -6547,12 +6839,12 @@ mdb_node_move(MDB_cursor *csrc, MDB_cursor *cdst)
                        return rc;
        }
 
-       DPRINTF("moving %s node %u [%s] on page %zu to node %u on page %zu",
+       DPRINTF(("moving %s node %u [%s] on page %"Z"u to node %u on page %"Z"u",
            IS_LEAF(csrc->mc_pg[csrc->mc_top]) ? "leaf" : "branch",
            csrc->mc_ki[csrc->mc_top],
                DKEY(&key),
            csrc->mc_pg[csrc->mc_top]->mp_pgno,
-           cdst->mc_ki[cdst->mc_top], cdst->mc_pg[cdst->mc_top]->mp_pgno);
+           cdst->mc_ki[cdst->mc_top], cdst->mc_pg[cdst->mc_top]->mp_pgno));
 
        /* Add the node to the destination page.
         */
@@ -6570,9 +6862,6 @@ mdb_node_move(MDB_cursor *csrc, MDB_cursor *cdst)
                MDB_dbi dbi = csrc->mc_dbi;
                MDB_page *mp = csrc->mc_pg[csrc->mc_top];
 
-               if (csrc->mc_flags & C_SUB)
-                       dbi--;
-
                for (m2 = csrc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                        if (csrc->mc_flags & C_SUB)
                                m3 = &m2->mc_xcursor->mx_cursor;
@@ -6598,8 +6887,8 @@ mdb_node_move(MDB_cursor *csrc, MDB_cursor *cdst)
                                key.mv_size = NODEKSZ(srcnode);
                                key.mv_data = NODEKEY(srcnode);
                        }
-                       DPRINTF("update separator for source page %zu to [%s]",
-                               csrc->mc_pg[csrc->mc_top]->mp_pgno, DKEY(&key));
+                       DPRINTF(("update separator for source page %"Z"u to [%s]",
+                               csrc->mc_pg[csrc->mc_top]->mp_pgno, DKEY(&key)));
                        mdb_cursor_copy(csrc, &mn);
                        mn.mc_snum--;
                        mn.mc_top--;
@@ -6626,8 +6915,8 @@ mdb_node_move(MDB_cursor *csrc, MDB_cursor *cdst)
                                key.mv_size = NODEKSZ(srcnode);
                                key.mv_data = NODEKEY(srcnode);
                        }
-                       DPRINTF("update separator for destination page %zu to [%s]",
-                               cdst->mc_pg[cdst->mc_top]->mp_pgno, DKEY(&key));
+                       DPRINTF(("update separator for destination page %"Z"u to [%s]",
+                               cdst->mc_pg[cdst->mc_top]->mp_pgno, DKEY(&key)));
                        mdb_cursor_copy(cdst, &mn);
                        mn.mc_snum--;
                        mn.mc_top--;
@@ -6664,8 +6953,8 @@ mdb_page_merge(MDB_cursor *csrc, MDB_cursor *cdst)
        MDB_val          key, data;
        unsigned        nkeys;
 
-       DPRINTF("merging page %zu into %zu", csrc->mc_pg[csrc->mc_top]->mp_pgno,
-               cdst->mc_pg[cdst->mc_top]->mp_pgno);
+       DPRINTF(("merging page %"Z"u into %"Z"u", csrc->mc_pg[csrc->mc_top]->mp_pgno,
+               cdst->mc_pg[cdst->mc_top]->mp_pgno));
 
        assert(csrc->mc_snum > 1);      /* can't merge root page */
        assert(cdst->mc_snum > 1);
@@ -6717,8 +7006,9 @@ mdb_page_merge(MDB_cursor *csrc, MDB_cursor *cdst)
                }
        }
 
-       DPRINTF("dst page %zu now has %u keys (%.1f%% filled)",
-           cdst->mc_pg[cdst->mc_top]->mp_pgno, NUMKEYS(cdst->mc_pg[cdst->mc_top]), (float)PAGEFILL(cdst->mc_txn->mt_env, cdst->mc_pg[cdst->mc_top]) / 10);
+       DPRINTF(("dst page %"Z"u now has %u keys (%.1f%% filled)",
+           cdst->mc_pg[cdst->mc_top]->mp_pgno, NUMKEYS(cdst->mc_pg[cdst->mc_top]),
+               (float)PAGEFILL(cdst->mc_txn->mt_env, cdst->mc_pg[cdst->mc_top]) / 10));
 
        /* Unlink the src page from parent and add to free list.
         */
@@ -6746,9 +7036,6 @@ mdb_page_merge(MDB_cursor *csrc, MDB_cursor *cdst)
                MDB_dbi dbi = csrc->mc_dbi;
                MDB_page *mp = cdst->mc_pg[cdst->mc_top];
 
-               if (csrc->mc_flags & C_SUB)
-                       dbi--;
-
                for (m2 = csrc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                        if (csrc->mc_flags & C_SUB)
                                m3 = &m2->mc_xcursor->mx_cursor;
@@ -6808,9 +7095,10 @@ mdb_rebalance(MDB_cursor *mc)
        {
        pgno_t pgno;
        COPY_PGNO(pgno, mc->mc_pg[mc->mc_top]->mp_pgno);
-       DPRINTF("rebalancing %s page %zu (has %u keys, %.1f%% full)",
+       DPRINTF(("rebalancing %s page %"Z"u (has %u keys, %.1f%% full)",
            IS_LEAF(mc->mc_pg[mc->mc_top]) ? "leaf" : "branch",
-           pgno, NUMKEYS(mc->mc_pg[mc->mc_top]), (float)PAGEFILL(mc->mc_txn->mt_env, mc->mc_pg[mc->mc_top]) / 10);
+           pgno, NUMKEYS(mc->mc_pg[mc->mc_top]),
+               (float)PAGEFILL(mc->mc_txn->mt_env, mc->mc_pg[mc->mc_top]) / 10));
        }
 #endif
 
@@ -6819,8 +7107,8 @@ mdb_rebalance(MDB_cursor *mc)
 #if MDB_DEBUG
                pgno_t pgno;
                COPY_PGNO(pgno, mc->mc_pg[mc->mc_top]->mp_pgno);
-               DPRINTF("no need to rebalance page %zu, above fill threshold",
-                   pgno);
+               DPRINTF(("no need to rebalance page %"Z"u, above fill threshold",
+                   pgno));
 #endif
                return MDB_SUCCESS;
        }
@@ -6842,13 +7130,11 @@ mdb_rebalance(MDB_cursor *mc)
                        /* Adjust cursors pointing to mp */
                        mc->mc_snum = 0;
                        mc->mc_top = 0;
+                       mc->mc_flags &= ~C_INITIALIZED;
                        {
                                MDB_cursor *m2, *m3;
                                MDB_dbi dbi = mc->mc_dbi;
 
-                               if (mc->mc_flags & C_SUB)
-                                       dbi--;
-
                                for (m2 = mc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                                        if (mc->mc_flags & C_SUB)
                                                m3 = &m2->mc_xcursor->mx_cursor;
@@ -6858,6 +7144,7 @@ mdb_rebalance(MDB_cursor *mc)
                                        if (m3->mc_pg[0] == mp) {
                                                m3->mc_snum = 0;
                                                m3->mc_top = 0;
+                                               m3->mc_flags &= ~C_INITIALIZED;
                                        }
                                }
                        }
@@ -6878,9 +7165,6 @@ mdb_rebalance(MDB_cursor *mc)
                                MDB_cursor *m2, *m3;
                                MDB_dbi dbi = mc->mc_dbi;
 
-                               if (mc->mc_flags & C_SUB)
-                                       dbi--;
-
                                for (m2 = mc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                                        if (mc->mc_flags & C_SUB)
                                                m3 = &m2->mc_xcursor->mx_cursor;
@@ -6888,10 +7172,13 @@ mdb_rebalance(MDB_cursor *mc)
                                                m3 = m2;
                                        if (m3 == mc || m3->mc_snum < mc->mc_snum) continue;
                                        if (m3->mc_pg[0] == mp) {
-                                               m3->mc_pg[0] = mc->mc_pg[0];
-                                               m3->mc_snum = 1;
-                                               m3->mc_top = 0;
-                                               m3->mc_ki[0] = m3->mc_ki[1];
+                                               int i;
+                                               m3->mc_snum--;
+                                               m3->mc_top--;
+                                               for (i=0; i<m3->mc_snum; i++) {
+                                                       m3->mc_pg[i] = m3->mc_pg[i+1];
+                                                       m3->mc_ki[i] = m3->mc_ki[i+1];
+                                               }
                                        }
                                }
                        }
@@ -6940,8 +7227,9 @@ mdb_rebalance(MDB_cursor *mc)
                mc->mc_ki[mc->mc_top] = 0;
        }
 
-       DPRINTF("found neighbor page %zu (%u keys, %.1f%% full)",
-           mn.mc_pg[mn.mc_top]->mp_pgno, NUMKEYS(mn.mc_pg[mn.mc_top]), (float)PAGEFILL(mc->mc_txn->mt_env, mn.mc_pg[mn.mc_top]) / 10);
+       DPRINTF(("found neighbor page %"Z"u (%u keys, %.1f%% full)",
+           mn.mc_pg[mn.mc_top]->mp_pgno, NUMKEYS(mn.mc_pg[mn.mc_top]),
+               (float)PAGEFILL(mc->mc_txn->mt_env, mn.mc_pg[mn.mc_top]) / 10));
 
        /* If the neighbor page is above threshold and has enough keys,
         * move one key from it. Otherwise we should try to merge them.
@@ -6953,8 +7241,11 @@ mdb_rebalance(MDB_cursor *mc)
        else {
                if (mc->mc_ki[ptop] == 0)
                        rc = mdb_page_merge(&mn, mc);
-               else
+               else {
+                       mn.mc_ki[mn.mc_top] += mc->mc_ki[mn.mc_top] + 1;
                        rc = mdb_page_merge(mc, &mn);
+                       mdb_cursor_copy(&mn, mc);
+               }
                mc->mc_flags &= ~(C_INITIALIZED|C_EOF);
        }
        return rc;
@@ -6967,6 +7258,7 @@ mdb_cursor_del0(MDB_cursor *mc, MDB_node *leaf)
        int rc;
        MDB_page *mp;
        indx_t ki;
+       unsigned int nkeys;
 
        mp = mc->mc_pg[mc->mc_top];
        ki = mc->mc_ki[mc->mc_top];
@@ -6986,30 +7278,34 @@ mdb_cursor_del0(MDB_cursor *mc, MDB_node *leaf)
        rc = mdb_rebalance(mc);
        if (rc != MDB_SUCCESS)
                mc->mc_txn->mt_flags |= MDB_TXN_ERROR;
-       /* if mc points past last node in page, invalidate */
-       else if (mc->mc_ki[mc->mc_top] >= NUMKEYS(mc->mc_pg[mc->mc_top]))
-               mc->mc_flags &= ~(C_INITIALIZED|C_EOF);
-
-       {
-               /* Adjust other cursors pointing to mp */
+       else {
                MDB_cursor *m2;
-               unsigned int nkeys;
                MDB_dbi dbi = mc->mc_dbi;
 
                mp = mc->mc_pg[mc->mc_top];
                nkeys = NUMKEYS(mp);
+
+               /* if mc points past last node in page, find next sibling */
+               if (mc->mc_ki[mc->mc_top] >= nkeys)
+                       mdb_cursor_sibling(mc, 1);
+
+               /* Adjust other cursors pointing to mp */
                for (m2 = mc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                        if (m2 == mc)
                                continue;
                        if (!(m2->mc_flags & C_INITIALIZED))
                                continue;
                        if (m2->mc_pg[mc->mc_top] == mp) {
-                               if (m2->mc_ki[mc->mc_top] > ki)
-                                       m2->mc_ki[mc->mc_top]--;
+                               if (m2->mc_ki[mc->mc_top] >= ki) {
+                                       m2->mc_flags |= C_DEL;
+                                       if (m2->mc_ki[mc->mc_top] > ki)
+                                               m2->mc_ki[mc->mc_top]--;
+                               }
                                if (m2->mc_ki[mc->mc_top] >= nkeys)
-                                       m2->mc_flags &= ~(C_INITIALIZED|C_EOF);
+                                       mdb_cursor_sibling(m2, 1);
                        }
                }
+               mc->mc_flags |= C_DEL;
        }
 
        return rc;
@@ -7028,22 +7324,25 @@ mdb_del(MDB_txn *txn, MDB_dbi dbi,
 
        assert(key != NULL);
 
-       DPRINTF("====> delete db %u key [%s]", dbi, DKEY(key));
+       DPRINTF(("====> delete db %u key [%s]", dbi, DKEY(key)));
 
        if (txn == NULL || !dbi || dbi >= txn->mt_numdbs || !(txn->mt_dbflags[dbi] & DB_VALID))
                return EINVAL;
 
-       if (F_ISSET(txn->mt_flags, MDB_TXN_RDONLY)) {
-               return EACCES;
-       }
+       if (txn->mt_flags & (MDB_TXN_RDONLY|MDB_TXN_ERROR))
+               return (txn->mt_flags & MDB_TXN_RDONLY) ? EACCES : MDB_BAD_TXN;
 
-       if (key->mv_size == 0 || key->mv_size > MDB_MAXKEYSIZE) {
-               return EINVAL;
+       if (key->mv_size > MDB_MAXKEYSIZE) {
+               return MDB_BAD_VALSIZE;
        }
 
        mdb_cursor_init(&mc, txn, dbi, &mx);
 
        exact = 0;
+       if (!F_ISSET(txn->mt_dbs[dbi].md_flags, MDB_DUPSORT)) {
+               /* must ignore any data */
+               data = NULL;
+       }
        if (data) {
                op = MDB_GET_BOTH;
                rdata = *data;
@@ -7086,29 +7385,31 @@ mdb_page_split(MDB_cursor *mc, MDB_val *newkey, MDB_val *newdata, pgno_t newpgno
        unsigned int nflags)
 {
        unsigned int flags;
-       int              rc = MDB_SUCCESS, ins_new = 0, new_root = 0, newpos = 1, did_split = 0;
+       int              rc = MDB_SUCCESS, new_root = 0, did_split = 0;
        indx_t           newindx;
        pgno_t           pgno = 0;
-       unsigned int     i, j, split_indx, nkeys, pmax;
+       int      i, j, split_indx, nkeys, pmax;
+       MDB_env         *env = mc->mc_txn->mt_env;
        MDB_node        *node;
        MDB_val  sepkey, rkey, xdata, *rdata = &xdata;
-       MDB_page        *copy;
+       MDB_page        *copy = NULL;
        MDB_page        *mp, *rp, *pp;
-       unsigned int ptop;
+       int ptop;
        MDB_cursor      mn;
        DKBUF;
 
        mp = mc->mc_pg[mc->mc_top];
        newindx = mc->mc_ki[mc->mc_top];
+       nkeys = NUMKEYS(mp);
 
-       DPRINTF("-----> splitting %s page %zu and adding [%s] at index %i",
+       DPRINTF(("-----> splitting %s page %"Z"u and adding [%s] at index %i/%i",
            IS_LEAF(mp) ? "leaf" : "branch", mp->mp_pgno,
-           DKEY(newkey), mc->mc_ki[mc->mc_top]);
+           DKEY(newkey), mc->mc_ki[mc->mc_top], nkeys));
 
        /* Create a right sibling. */
        if ((rc = mdb_page_new(mc, mp->mp_flags, 1, &rp)))
                return rc;
-       DPRINTF("new right sibling: page %zu", rp->mp_pgno);
+       DPRINTF(("new right sibling: page %"Z"u", rp->mp_pgno));
 
        if (mc->mc_snum < 2) {
                if ((rc = mdb_page_new(mc, P_BRANCH, 1, &pp)))
@@ -7119,7 +7420,7 @@ mdb_page_split(MDB_cursor *mc, MDB_val *newkey, MDB_val *newdata, pgno_t newpgno
                mc->mc_pg[0] = pp;
                mc->mc_ki[0] = 0;
                mc->mc_db->md_root = pp->mp_pgno;
-               DPRINTF("root split! new root = %zu", pp->mp_pgno);
+               DPRINTF(("root split! new root = %"Z"u", pp->mp_pgno));
                mc->mc_db->md_depth++;
                new_root = 1;
 
@@ -7137,7 +7438,7 @@ mdb_page_split(MDB_cursor *mc, MDB_val *newkey, MDB_val *newdata, pgno_t newpgno
                ptop = 0;
        } else {
                ptop = mc->mc_top-1;
-               DPRINTF("parent branch page is %zu", mc->mc_pg[ptop]->mp_pgno);
+               DPRINTF(("parent branch page is %"Z"u", mc->mc_pg[ptop]->mp_pgno));
        }
 
        mc->mc_flags |= C_SPLITTING;
@@ -7150,141 +7451,146 @@ mdb_page_split(MDB_cursor *mc, MDB_val *newkey, MDB_val *newdata, pgno_t newpgno
                sepkey = *newkey;
                split_indx = newindx;
                nkeys = 0;
-               goto newsep;
-       }
+       } else {
 
-       nkeys = NUMKEYS(mp);
-       split_indx = nkeys / 2;
-       if (newindx < split_indx)
-               newpos = 0;
-
-       if (IS_LEAF2(rp)) {
-               char *split, *ins;
-               int x;
-               unsigned int lsize, rsize, ksize;
-               /* Move half of the keys to the right sibling */
-               copy = NULL;
-               x = mc->mc_ki[mc->mc_top] - split_indx;
-               ksize = mc->mc_db->md_pad;
-               split = LEAF2KEY(mp, split_indx, ksize);
-               rsize = (nkeys - split_indx) * ksize;
-               lsize = (nkeys - split_indx) * sizeof(indx_t);
-               mp->mp_lower -= lsize;
-               rp->mp_lower += lsize;
-               mp->mp_upper += rsize - lsize;
-               rp->mp_upper -= rsize - lsize;
-               sepkey.mv_size = ksize;
-               if (newindx == split_indx) {
-                       sepkey.mv_data = newkey->mv_data;
-               } else {
-                       sepkey.mv_data = split;
-               }
-               if (x<0) {
-                       ins = LEAF2KEY(mp, mc->mc_ki[mc->mc_top], ksize);
-                       memcpy(rp->mp_ptrs, split, rsize);
-                       sepkey.mv_data = rp->mp_ptrs;
-                       memmove(ins+ksize, ins, (split_indx - mc->mc_ki[mc->mc_top]) * ksize);
-                       memcpy(ins, newkey->mv_data, ksize);
-                       mp->mp_lower += sizeof(indx_t);
-                       mp->mp_upper -= ksize - sizeof(indx_t);
+               split_indx = (nkeys+1) / 2;
+
+               if (IS_LEAF2(rp)) {
+                       char *split, *ins;
+                       int x;
+                       unsigned int lsize, rsize, ksize;
+                       /* Move half of the keys to the right sibling */
+                       copy = NULL;
+                       x = mc->mc_ki[mc->mc_top] - split_indx;
+                       ksize = mc->mc_db->md_pad;
+                       split = LEAF2KEY(mp, split_indx, ksize);
+                       rsize = (nkeys - split_indx) * ksize;
+                       lsize = (nkeys - split_indx) * sizeof(indx_t);
+                       mp->mp_lower -= lsize;
+                       rp->mp_lower += lsize;
+                       mp->mp_upper += rsize - lsize;
+                       rp->mp_upper -= rsize - lsize;
+                       sepkey.mv_size = ksize;
+                       if (newindx == split_indx) {
+                               sepkey.mv_data = newkey->mv_data;
+                       } else {
+                               sepkey.mv_data = split;
+                       }
+                       if (x<0) {
+                               ins = LEAF2KEY(mp, mc->mc_ki[mc->mc_top], ksize);
+                               memcpy(rp->mp_ptrs, split, rsize);
+                               sepkey.mv_data = rp->mp_ptrs;
+                               memmove(ins+ksize, ins, (split_indx - mc->mc_ki[mc->mc_top]) * ksize);
+                               memcpy(ins, newkey->mv_data, ksize);
+                               mp->mp_lower += sizeof(indx_t);
+                               mp->mp_upper -= ksize - sizeof(indx_t);
+                       } else {
+                               if (x)
+                                       memcpy(rp->mp_ptrs, split, x * ksize);
+                               ins = LEAF2KEY(rp, x, ksize);
+                               memcpy(ins, newkey->mv_data, ksize);
+                               memcpy(ins+ksize, split + x * ksize, rsize - x * ksize);
+                               rp->mp_lower += sizeof(indx_t);
+                               rp->mp_upper -= ksize - sizeof(indx_t);
+                               mc->mc_ki[mc->mc_top] = x;
+                               mc->mc_pg[mc->mc_top] = rp;
+                       }
                } else {
-                       if (x)
-                               memcpy(rp->mp_ptrs, split, x * ksize);
-                       ins = LEAF2KEY(rp, x, ksize);
-                       memcpy(ins, newkey->mv_data, ksize);
-                       memcpy(ins+ksize, split + x * ksize, rsize - x * ksize);
-                       rp->mp_lower += sizeof(indx_t);
-                       rp->mp_upper -= ksize - sizeof(indx_t);
-                       mc->mc_ki[mc->mc_top] = x;
-                       mc->mc_pg[mc->mc_top] = rp;
-               }
-               goto newsep;
-       }
+                       int psize, nsize, k;
+                       /* Maximum free space in an empty page */
+                       pmax = env->me_psize - PAGEHDRSZ;
+                       if (IS_LEAF(mp))
+                               nsize = mdb_leaf_size(env, newkey, newdata);
+                       else
+                               nsize = mdb_branch_size(env, newkey);
+                       nsize += nsize & 1;
 
-       /* For leaf pages, check the split point based on what
-        * fits where, since otherwise mdb_node_add can fail.
-        *
-        * This check is only needed when the data items are
-        * relatively large, such that being off by one will
-        * make the difference between success or failure.
-        *
-        * It's also relevant if a page happens to be laid out
-        * such that one half of its nodes are all "small" and
-        * the other half of its nodes are "large." If the new
-        * item is also "large" and falls on the half with
-        * "large" nodes, it also may not fit.
-        */
-       if (IS_LEAF(mp)) {
-               unsigned int psize, nsize;
-               /* Maximum free space in an empty page */
-               pmax = mc->mc_txn->mt_env->me_psize - PAGEHDRSZ;
-               nsize = mdb_leaf_size(mc->mc_txn->mt_env, newkey, newdata);
-               if ((nkeys < 20) || (nsize > pmax/16)) {
-                       if (newindx <= split_indx) {
-                               psize = nsize;
-                               newpos = 0;
-                               for (i=0; i<split_indx; i++) {
-                                       node = NODEPTR(mp, i);
-                                       psize += NODESIZE + NODEKSZ(node) + sizeof(indx_t);
-                                       if (F_ISSET(node->mn_flags, F_BIGDATA))
-                                               psize += sizeof(pgno_t);
-                                       else
-                                               psize += NODEDSZ(node);
-                                       psize += psize & 1;
-                                       if (psize > pmax) {
-                                               if (i <= newindx) {
-                                                       split_indx = newindx;
-                                                       if (i < newindx)
-                                                               newpos = 1;
+                       /* grab a page to hold a temporary copy */
+                       copy = mdb_page_malloc(mc->mc_txn, 1);
+                       if (copy == NULL)
+                               return ENOMEM;
+                       copy->mp_pgno  = mp->mp_pgno;
+                       copy->mp_flags = mp->mp_flags;
+                       copy->mp_lower = PAGEHDRSZ;
+                       copy->mp_upper = env->me_psize;
+
+                       /* prepare to insert */
+                       for (i=0, j=0; i<nkeys; i++) {
+                               if (i == newindx) {
+                                       copy->mp_ptrs[j++] = 0;
+                               }
+                               copy->mp_ptrs[j++] = mp->mp_ptrs[i];
+                       }
+
+                       /* When items are relatively large the split point needs
+                        * to be checked, because being off-by-one will make the
+                        * difference between success or failure in mdb_node_add.
+                        *
+                        * It's also relevant if a page happens to be laid out
+                        * such that one half of its nodes are all "small" and
+                        * the other half of its nodes are "large." If the new
+                        * item is also "large" and falls on the half with
+                        * "large" nodes, it also may not fit.
+                        *
+                        * As a final tweak, if the new item goes on the last
+                        * spot on the page (and thus, onto the new page), bias
+                        * the split so the new page is emptier than the old page.
+                        * This yields better packing during sequential inserts.
+                        */
+                       if (nkeys < 20 || nsize > pmax/16 || newindx >= nkeys) {
+                               /* Find split point */
+                               psize = 0;
+                               if (newindx <= split_indx || newindx >= nkeys) {
+                                       i = 0; j = 1;
+                                       k = newindx >= nkeys ? nkeys : split_indx+1;
+                               } else {
+                                       i = nkeys; j = -1;
+                                       k = split_indx-1;
+                               }
+                               for (; i!=k; i+=j) {
+                                       if (i == newindx) {
+                                               psize += nsize;
+                                               node = NULL;
+                                       } else {
+                                               node = (MDB_node *)((char *)mp + copy->mp_ptrs[i]);
+                                               psize += NODESIZE + NODEKSZ(node) + sizeof(indx_t);
+                                               if (IS_LEAF(mp)) {
+                                                       if (F_ISSET(node->mn_flags, F_BIGDATA))
+                                                               psize += sizeof(pgno_t);
+                                                       else
+                                                               psize += NODEDSZ(node);
                                                }
-                                               else
-                                                       split_indx = i;
-                                               break;
+                                               psize += psize & 1;
                                        }
-                               }
-                       } else {
-                               psize = nsize;
-                               for (i=nkeys-1; i>=split_indx; i--) {
-                                       node = NODEPTR(mp, i);
-                                       psize += NODESIZE + NODEKSZ(node) + sizeof(indx_t);
-                                       if (F_ISSET(node->mn_flags, F_BIGDATA))
-                                               psize += sizeof(pgno_t);
-                                       else
-                                               psize += NODEDSZ(node);
-                                       psize += psize & 1;
                                        if (psize > pmax) {
-                                               if (i >= newindx) {
-                                                       split_indx = newindx;
-                                                       newpos = 0;
-                                               } else
-                                                       split_indx = i+1;
+                                               split_indx = i + (j<0);
                                                break;
                                        }
                                }
+                               /* special case: when the new node was on the last
+                                * slot we may not have tripped the break inside the loop.
+                                * In all other cases we either hit the break condition,
+                                * or the original split_indx was already safe.
+                                */
+                               if (newindx >= nkeys && i == k)
+                                       split_indx = nkeys-1;
+                       }
+                       if (split_indx == newindx) {
+                               sepkey.mv_size = newkey->mv_size;
+                               sepkey.mv_data = newkey->mv_data;
+                       } else {
+                               node = (MDB_node *)((char *)mp + copy->mp_ptrs[split_indx]);
+                               sepkey.mv_size = node->mn_ksize;
+                               sepkey.mv_data = NODEKEY(node);
                        }
                }
        }
 
-       /* First find the separating key between the split pages.
-        * The case where newindx == split_indx is ambiguous; the
-        * new item could go to the new page or stay on the original
-        * page. If newpos == 1 it goes to the new page.
-        */
-       if (newindx == split_indx && newpos) {
-               sepkey.mv_size = newkey->mv_size;
-               sepkey.mv_data = newkey->mv_data;
-       } else {
-               node = NODEPTR(mp, split_indx);
-               sepkey.mv_size = node->mn_ksize;
-               sepkey.mv_data = NODEKEY(node);
-       }
-
-newsep:
-       DPRINTF("separator is [%s]", DKEY(&sepkey));
+       DPRINTF(("separator is %d [%s]", split_indx, DKEY(&sepkey)));
 
        /* Copy separator key to the parent.
         */
-       if (SIZELEFT(mn.mc_pg[ptop]) < mdb_branch_size(mc->mc_txn->mt_env, &sepkey)) {
+       if (SIZELEFT(mn.mc_pg[ptop]) < mdb_branch_size(env, &sepkey)) {
                mn.mc_snum--;
                mn.mc_top--;
                did_split = 1;
@@ -7329,117 +7635,97 @@ newsep:
                        return rc;
                for (i=0; i<mc->mc_top; i++)
                        mc->mc_ki[i] = mn.mc_ki[i];
-               goto done;
-       }
-       if (IS_LEAF2(rp)) {
-               goto done;
-       }
-
-       /* Move half of the keys to the right sibling. */
+       } else if (!IS_LEAF2(mp)) {
+               /* Move nodes */
+               mc->mc_pg[mc->mc_top] = rp;
+               i = split_indx;
+               j = 0;
+               do {
+                       if (i == newindx) {
+                               rkey.mv_data = newkey->mv_data;
+                               rkey.mv_size = newkey->mv_size;
+                               if (IS_LEAF(mp)) {
+                                       rdata = newdata;
+                               } else
+                                       pgno = newpgno;
+                               flags = nflags;
+                               /* Update index for the new key. */
+                               mc->mc_ki[mc->mc_top] = j;
+                       } else {
+                               node = (MDB_node *)((char *)mp + copy->mp_ptrs[i]);
+                               rkey.mv_data = NODEKEY(node);
+                               rkey.mv_size = node->mn_ksize;
+                               if (IS_LEAF(mp)) {
+                                       xdata.mv_data = NODEDATA(node);
+                                       xdata.mv_size = NODEDSZ(node);
+                                       rdata = &xdata;
+                               } else
+                                       pgno = NODEPGNO(node);
+                               flags = node->mn_flags;
+                       }
 
-       /* grab a page to hold a temporary copy */
-       copy = mdb_page_malloc(mc->mc_txn, 1);
-       if (copy == NULL)
-               return ENOMEM;
+                       if (!IS_LEAF(mp) && j == 0) {
+                               /* First branch index doesn't need key data. */
+                               rkey.mv_size = 0;
+                       }
 
-       copy->mp_pgno  = mp->mp_pgno;
-       copy->mp_flags = mp->mp_flags;
-       copy->mp_lower = PAGEHDRSZ;
-       copy->mp_upper = mc->mc_txn->mt_env->me_psize;
-       mc->mc_pg[mc->mc_top] = copy;
-       for (i = j = 0; i <= nkeys; j++) {
-               if (i == split_indx) {
-               /* Insert in right sibling. */
-               /* Reset insert index for right sibling. */
-                       if (i != newindx || (newpos ^ ins_new)) {
+                       rc = mdb_node_add(mc, j, &rkey, rdata, pgno, flags);
+                       if (rc) {
+                               /* return tmp page to freelist */
+                               mdb_page_free(env, copy);
+                               return rc;
+                       }
+                       if (i == nkeys) {
+                               i = 0;
                                j = 0;
-                               mc->mc_pg[mc->mc_top] = rp;
+                               mc->mc_pg[mc->mc_top] = copy;
+                       } else {
+                               i++;
+                               j++;
+                       }
+               } while (i != split_indx);
+
+               nkeys = NUMKEYS(copy);
+               for (i=0; i<nkeys; i++)
+                       mp->mp_ptrs[i] = copy->mp_ptrs[i];
+               mp->mp_lower = copy->mp_lower;
+               mp->mp_upper = copy->mp_upper;
+               memcpy(NODEPTR(mp, nkeys-1), NODEPTR(copy, nkeys-1),
+                       env->me_psize - copy->mp_upper);
+
+               /* reset back to original page */
+               if (newindx < split_indx) {
+                       mc->mc_pg[mc->mc_top] = mp;
+                       if (nflags & MDB_RESERVE) {
+                               node = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
+                               if (!(node->mn_flags & F_BIGDATA))
+                                       newdata->mv_data = NODEDATA(node);
                        }
-               }
-
-               if (i == newindx && !ins_new) {
-                       /* Insert the original entry that caused the split. */
-                       rkey.mv_data = newkey->mv_data;
-                       rkey.mv_size = newkey->mv_size;
-                       if (IS_LEAF(mp)) {
-                               rdata = newdata;
-                       } else
-                               pgno = newpgno;
-                       flags = nflags;
-
-                       ins_new = 1;
-
-                       /* Update index for the new key. */
-                       mc->mc_ki[mc->mc_top] = j;
-               } else if (i == nkeys) {
-                       break;
                } else {
-                       node = NODEPTR(mp, i);
-                       rkey.mv_data = NODEKEY(node);
-                       rkey.mv_size = node->mn_ksize;
-                       if (IS_LEAF(mp)) {
-                               xdata.mv_data = NODEDATA(node);
-                               xdata.mv_size = NODEDSZ(node);
-                               rdata = &xdata;
-                       } else
-                               pgno = NODEPGNO(node);
-                       flags = node->mn_flags;
-
-                       i++;
-               }
-
-               if (!IS_LEAF(mp) && j == 0) {
-                       /* First branch index doesn't need key data. */
-                       rkey.mv_size = 0;
-               }
-
-               rc = mdb_node_add(mc, j, &rkey, rdata, pgno, flags);
-               if (rc) break;
-       }
-
-       nkeys = NUMKEYS(copy);
-       for (i=0; i<nkeys; i++)
-               mp->mp_ptrs[i] = copy->mp_ptrs[i];
-       mp->mp_lower = copy->mp_lower;
-       mp->mp_upper = copy->mp_upper;
-       memcpy(NODEPTR(mp, nkeys-1), NODEPTR(copy, nkeys-1),
-               mc->mc_txn->mt_env->me_psize - copy->mp_upper);
-
-       /* reset back to original page */
-       if (newindx < split_indx || (!newpos && newindx == split_indx)) {
-               mc->mc_pg[mc->mc_top] = mp;
-               if (nflags & MDB_RESERVE) {
-                       node = NODEPTR(mp, mc->mc_ki[mc->mc_top]);
-                       if (!(node->mn_flags & F_BIGDATA))
-                               newdata->mv_data = NODEDATA(node);
-               }
-       } else {
-               mc->mc_ki[ptop]++;
-               /* Make sure mc_ki is still valid.
-                */
-               if (mn.mc_pg[ptop] != mc->mc_pg[ptop] &&
-                   mc->mc_ki[ptop] >= NUMKEYS(mc->mc_pg[ptop])) {
-                       for (i=0; i<ptop; i++) {
-                               mc->mc_pg[i] = mn.mc_pg[i];
-                               mc->mc_ki[i] = mn.mc_ki[i];
+                       mc->mc_pg[mc->mc_top] = rp;
+                       mc->mc_ki[ptop]++;
+                       /* Make sure mc_ki is still valid.
+                        */
+                       if (mn.mc_pg[ptop] != mc->mc_pg[ptop] &&
+                               mc->mc_ki[ptop] >= NUMKEYS(mc->mc_pg[ptop])) {
+                               for (i=0; i<ptop; i++) {
+                                       mc->mc_pg[i] = mn.mc_pg[i];
+                                       mc->mc_ki[i] = mn.mc_ki[i];
+                               }
+                               mc->mc_pg[ptop] = mn.mc_pg[ptop];
+                               mc->mc_ki[ptop] = mn.mc_ki[ptop] - 1;
                        }
-                       mc->mc_pg[ptop] = mn.mc_pg[ptop];
-                       mc->mc_ki[ptop] = mn.mc_ki[ptop] - 1;
                }
+               /* return tmp page to freelist */
+               mdb_page_free(env, copy);
        }
 
-       /* return tmp page to freelist */
-       mdb_page_free(mc->mc_txn->mt_env, copy);
-done:
        {
                /* Adjust other cursors pointing to mp */
                MDB_cursor *m2, *m3;
                MDB_dbi dbi = mc->mc_dbi;
                int fixup = NUMKEYS(mp);
 
-               if (mc->mc_flags & C_SUB)
-                       dbi--;
-
                for (m2 = mc->mc_txn->mt_cursors[dbi]; m2; m2=m2->mc_next) {
                        if (mc->mc_flags & C_SUB)
                                m3 = &m2->mc_xcursor->mx_cursor;
@@ -7467,7 +7753,7 @@ done:
                                m3->mc_snum++;
                                m3->mc_top++;
                        }
-                       if (m3->mc_pg[mc->mc_top] == mp) {
+                       if (m3->mc_top >= mc->mc_top && m3->mc_pg[mc->mc_top] == mp) {
                                if (m3->mc_ki[mc->mc_top] >= newindx && !(nflags & MDB_SPLIT_REPLACE))
                                        m3->mc_ki[mc->mc_top]++;
                                if (m3->mc_ki[mc->mc_top] >= fixup) {
@@ -7475,12 +7761,13 @@ done:
                                        m3->mc_ki[mc->mc_top] -= fixup;
                                        m3->mc_ki[ptop] = mn.mc_ki[ptop];
                                }
-                       } else if (!did_split && m3->mc_pg[ptop] == mc->mc_pg[ptop] &&
+                       } else if (!did_split && m3->mc_top >= ptop && m3->mc_pg[ptop] == mc->mc_pg[ptop] &&
                                m3->mc_ki[ptop] >= mc->mc_ki[ptop]) {
                                m3->mc_ki[ptop]++;
                        }
                }
        }
+       DPRINTF(("mp left: %d, rp left: %d", SIZELEFT(mp), SIZELEFT(rp)));
        return rc;
 }
 
@@ -7497,14 +7784,6 @@ mdb_put(MDB_txn *txn, MDB_dbi dbi,
        if (txn == NULL || !dbi || dbi >= txn->mt_numdbs || !(txn->mt_dbflags[dbi] & DB_VALID))
                return EINVAL;
 
-       if (F_ISSET(txn->mt_flags, MDB_TXN_RDONLY)) {
-               return EACCES;
-       }
-
-       if (key->mv_size == 0 || key->mv_size > MDB_MAXKEYSIZE) {
-               return EINVAL;
-       }
-
        if ((flags & (MDB_NOOVERWRITE|MDB_NODUPDATA|MDB_RESERVE|MDB_APPEND|MDB_APPENDDUP)) != flags)
                return EINVAL;
 
@@ -7544,6 +7823,16 @@ mdb_env_get_path(MDB_env *env, const char **arg)
        return MDB_SUCCESS;
 }
 
+int
+mdb_env_get_fd(MDB_env *env, mdb_filehandle_t *arg)
+{
+       if (!env || !arg)
+               return EINVAL;
+
+       *arg = env->me_fd;
+       return MDB_SUCCESS;
+}
+
 /** Common code for #mdb_stat() and #mdb_env_stat().
  * @param[in] env the environment to operate in.
  * @param[in] db the #MDB_db record containing the stats to return.
@@ -7587,7 +7876,12 @@ mdb_env_info(MDB_env *env, MDB_envinfo *arg)
        arg->me_mapaddr = (env->me_flags & MDB_FIXEDMAP) ? env->me_map : 0;
        arg->me_mapsize = env->me_mapsize;
        arg->me_maxreaders = env->me_maxreaders;
-       arg->me_numreaders = env->me_numreaders;
+
+       /* me_numreaders may be zero if this process never used any readers. Use
+        * the shared numreader count if it exists.
+        */
+       arg->me_numreaders = env->me_txns ? env->me_txns->mti_numreaders : env->me_numreaders;
+
        arg->me_last_pgno = env->me_metas[toggle]->mm_last_pg;
        arg->me_last_txnid = env->me_metas[toggle]->mm_txnid;
        return MDB_SUCCESS;
@@ -7631,6 +7925,8 @@ int mdb_dbi_open(MDB_txn *txn, const char *name, unsigned int flags, MDB_dbi *db
 
        if ((flags & VALID_FLAGS) != flags)
                return EINVAL;
+       if (txn->mt_flags & MDB_TXN_ERROR)
+               return MDB_BAD_TXN;
 
        /* main DB? */
        if (!name) {
@@ -7685,7 +7981,7 @@ int mdb_dbi_open(MDB_txn *txn, const char *name, unsigned int flags, MDB_dbi *db
                /* make sure this is actually a DB */
                MDB_node *node = NODEPTR(mc.mc_pg[mc.mc_top], mc.mc_ki[mc.mc_top]);
                if (!(node->mn_flags & F_SUBDATA))
-                       return EINVAL;
+                       return MDB_INCOMPATIBLE;
        } else if (rc == MDB_NOTFOUND && (flags & MDB_CREATE)) {
                /* Create if requested */
                MDB_db dummy;
@@ -7707,7 +8003,6 @@ int mdb_dbi_open(MDB_txn *txn, const char *name, unsigned int flags, MDB_dbi *db
                txn->mt_dbflags[slot] = dbflag;
                memcpy(&txn->mt_dbs[slot], data.mv_data, sizeof(MDB_db));
                *dbi = slot;
-               txn->mt_env->me_dbflags[slot] = txn->mt_dbs[slot].md_flags;
                mdb_default_cmp(txn, slot);
                if (!unused) {
                        txn->mt_numdbs++;
@@ -7743,12 +8038,12 @@ void mdb_dbi_close(MDB_env *env, MDB_dbi dbi)
        free(ptr);
 }
 
-int mdb_dbi_flags(MDB_env *env, MDB_dbi dbi, unsigned int *flags)
+int mdb_dbi_flags(MDB_txn *txn, MDB_dbi dbi, unsigned int *flags)
 {
        /* We could return the flags for the FREE_DBI too but what's the point? */
-       if (dbi <= MAIN_DBI || dbi >= env->me_numdbs)
+       if (txn == NULL || dbi < MAIN_DBI || dbi >= txn->mt_numdbs)
                return EINVAL;
-       *flags = env->me_dbflags[dbi];
+       *flags = txn->mt_dbs[dbi].md_flags & PERSISTENT_FLAGS;
        return MDB_SUCCESS;
 }
 
@@ -7762,7 +8057,7 @@ mdb_drop0(MDB_cursor *mc, int subs)
 {
        int rc;
 
-       rc = mdb_page_search(mc, NULL, 0);
+       rc = mdb_page_search(mc, NULL, MDB_PS_FIRST);
        if (rc == MDB_SUCCESS) {
                MDB_txn *txn = mc->mc_txn;
                MDB_node *ni;
@@ -7916,4 +8211,133 @@ int mdb_set_relctx(MDB_txn *txn, MDB_dbi dbi, void *ctx)
        return MDB_SUCCESS;
 }
 
+int mdb_env_get_maxkeysize(MDB_env *env)
+{
+       return MDB_MAXKEYSIZE;
+}
+
+int mdb_reader_list(MDB_env *env, MDB_msg_func *func, void *ctx)
+{
+       unsigned int i, rdrs;
+       MDB_reader *mr;
+       char buf[64];
+       int first = 1;
+
+       if (!env || !func)
+               return -1;
+       if (!env->me_txns) {
+               return func("(no reader locks)\n", ctx);
+       }
+       rdrs = env->me_txns->mti_numreaders;
+       mr = env->me_txns->mti_readers;
+       for (i=0; i<rdrs; i++) {
+               if (mr[i].mr_pid) {
+                       size_t tid;
+                       int rc;
+                       tid = mr[i].mr_tid;
+                       if (mr[i].mr_txnid == (txnid_t)-1) {
+                               sprintf(buf, "%10d %"Z"x -\n", mr[i].mr_pid, tid);
+                       } else {
+                               sprintf(buf, "%10d %"Z"x %"Z"u\n", mr[i].mr_pid, tid, mr[i].mr_txnid);
+                       }
+                       if (first) {
+                               first = 0;
+                               func("    pid     thread     txnid\n", ctx);
+                       }
+                       rc = func(buf, ctx);
+                       if (rc < 0)
+                               return rc;
+               }
+       }
+       if (first) {
+               func("(no active readers)\n", ctx);
+       }
+       return 0;
+}
+
+/** Insert pid into list if not already present.
+ * return -1 if already present.
+ */
+static int mdb_pid_insert(pid_t *ids, pid_t pid)
+{
+       /* binary search of pid in list */
+       unsigned base = 0;
+       unsigned cursor = 1;
+       int val = 0;
+       unsigned n = ids[0];
+
+       while( 0 < n ) {
+               unsigned pivot = n >> 1;
+               cursor = base + pivot + 1;
+               val = pid - ids[cursor];
+
+               if( val < 0 ) {
+                       n = pivot;
+
+               } else if ( val > 0 ) {
+                       base = cursor;
+                       n -= pivot + 1;
+
+               } else {
+                       /* found, so it's a duplicate */
+                       return -1;
+               }
+       }
+
+       if( val > 0 ) {
+               ++cursor;
+       }
+       ids[0]++;
+       for (n = ids[0]; n > cursor; n--)
+               ids[n] = ids[n-1];
+       ids[n] = pid;
+       return 0;
+}
+
+int mdb_reader_check(MDB_env *env, int *dead)
+{
+       unsigned int i, j, rdrs;
+       MDB_reader *mr;
+       pid_t *pids, pid;
+       int count = 0;
+
+       if (!env)
+               return EINVAL;
+       if (dead)
+               *dead = 0;
+       if (!env->me_txns)
+               return MDB_SUCCESS;
+       rdrs = env->me_txns->mti_numreaders;
+       pids = malloc((rdrs+1) * sizeof(pid_t));
+       if (!pids)
+               return ENOMEM;
+       pids[0] = 0;
+       mr = env->me_txns->mti_readers;
+       j = 0;
+       for (i=0; i<rdrs; i++) {
+               if (mr[i].mr_pid && mr[i].mr_pid != env->me_pid) {
+                       pid = mr[i].mr_pid;
+                       if (mdb_pid_insert(pids, pid) == 0) {
+                               if (!mdb_reader_pid(env, Pidcheck, pid)) {
+                                       LOCK_MUTEX_R(env);
+                                       /* Recheck, a new process may have reused pid */
+                                       if (!mdb_reader_pid(env, Pidcheck, pid)) {
+                                               for (j=i; j<rdrs; j++)
+                                                       if (mr[j].mr_pid == pid) {
+                                                               DPRINTF(("clear stale reader pid %u txn %"Z"d",
+                                                                       (unsigned) pid, mr[j].mr_txnid));
+                                                               mr[j].mr_pid = 0;
+                                                               count++;
+                                                       }
+                                       }
+                                       UNLOCK_MUTEX_R(env);
+                               }
+                       }
+               }
+       }
+       free(pids);
+       if (dead)
+               *dead = count;
+       return MDB_SUCCESS;
+}
 /** @} */