More fixes for prev commit

Fix 34f832faee9f215dfdb61de52506f2905258b147

Lock ordering issues

Fix prev commit

Hardcoded number of queues was leftover from preliminary
work. Use connection_pool_queues.

Multiple queues per threadpool

Tweak mutex acquisition

Slightly reduce held time.

Merge remote-tracking branch 'origin/mdb.master'

Silence warnings

Tweak ITS#7615 fix: Obey strict-aliasing rules.

Do not access a long long as an array of longs.

More for  907b9d487753631371dd969b551b63c41cef30fd

op.o_extra is only used with threading

Merge remote-tracking branch 'origin/mdb.master'

ITS#7664 better fix

For RDONLY, don't get lockfile until we know datafile exists.
Also, don't try to create a new datafile for me_mfd if someone
deleted it after we got me_fd.

Revert "ITS#7664 delete lockfile if there is no environment"

This reverts commit 8725838409512e999d78ea7b60773ce2f94d588f.

Merge remote-tracking branch 'origin/mdb.master'

ITS#7664 delete lockfile if there is no environment

Fix prev commit

Fixes for threaded indexer

Works now, but still 2x slower. Leave disabled.

Merge remote-tracking branch 'origin/mdb.master'

Fix uninit'd mem ref in mdb_page_split

Don't compare cursors beyond their depth. (detected by valgrind)

Merge remote-tracking branch 'origin/mdb.master'

Tweak NODUPDATA flag check

Should allow for the possibility of future flag combos.

Merge remote-tracking branch 'origin/mdb.master'

Fix obscure MDB_MULTIPLE bug

If a key has a single existing value, and then a put (MDB_MULTIPLE)
is done where the first of the multiple values matches the existing
value, the put would return SUCCESS without writing any of the
values. Fixed to loop to the next value as intended.

Merge remote-tracking branch 'origin/mdb.master'

Point to mdb.c for docs on macros in Makefile

Merge remote-tracking branch 'origin/mdb.master'

ITS#7662 cleanup reader txn in mdb_entry_get()

Avoid compiling debug args with C90 compilers.

Use double DPRINTF() parenthesis, so varargs macros are not needed.
Parenthesize MDB_DEBUG, in case the user defines it as foo || bar.

Refuse ops on bad txns

Replace EINVALs with generalized MDB_INCOMPATIBLE.

Also check xcursor instead of DUPSORT, it's slightly briefer.

ITS#7656 fix install target

ITS#7660 Add mdb_txn_env to API

mdb_txn_commit(): Always commit if MDB_TXN_SPILLS.

Checking dirty_list was insufficient after a spill
with no named databases and no positioned cursors.

ITS#7661 mdb_dbi_flags(): Allow main DBI

ITS#7659 add directoryString compatible syntaxes

countryString and telephoneNumber were missing

Merge remote-tracking branch 'origin/mdb.master'

mdb_del must ignore data if db not opened with MDB_DUPSORT

Merge remote-tracking branch 'origin/mdb.master'

Add mdb_env_get_maxkeysize()

Replace unpredictable EINVAL error returns.

Return EINVAL only for simple programmer errors.

Re-fix reader-pid code

mdb_cursorpages_mark: Mark current txn and no more.

Ignore parent txn cursors since it is the current txn's dirty_list
which will be flushed. But check the current txn also when clearing,
since cursors can have pages which are dirty in a parent.

Check !mc_xcursor instead of !MDB_DUPSORT.  Equivalent for valid
data, but a bit safer if the sub-DB flags are corrupt.

Fix mdb_ovpage_free() vs. spill.

Ensure me_pghead has room before removing from spill/dirty list.
Don't return pages to me_pghead in nested txns, use mt_free_pgs.

Fix page spilling when MDB_WRITEMAP.

mdb_page_spill(): Don't binary-search the unsorted dirty_list.
mdb_page_flush(): Don't overwrite unprocessed dirty_list items.

Set MDB_TXN_ERROR when inconsistent txn state

Factor out MDB_env.

MDB_LOCK_VERSION -> MDB_LOCK_FORMAT.

Pid locking needs a different lockfile-version: MDB_env's with and
without pid locking must not coexist, they can sabotage each other.
Store MDB_LOCK_FORMAT = (version | "use locking" flag) instead.

Fix mdb_reader_pid().

Treat unexpected errors as "don't know".  Invert Pidcheck return
value, so nonzero including error codes = "the process may exist".

On Windows: Catch exited but still existing processes.  Handle
undefined PROCESS_QUERY_LIMITED_INFORMATION.
On Unix: don't trust F_GETLK error to leave the input alone,
the fcntl() doc seems unclear.

Use proper printf format on Windows

ITS#7645 more for TLSProtocolMin

Silence warnings

Tweak comments

Clarify doc: mdb_copy, nested txns, mdb_drop().

mdb_copy: Does not copy lockfile. Can trigger file growth.
mdb_txn_begin(): Clarify usage restrictions.
mdb_drop(): State what to do rather than what will be done, since
closing the handle could otherwise be read as happening even at failure.

ITS#7591 fix back-meta missing result msgs

ITS#7654 fix implicit declaration warnings

Merge remote-tracking branch 'origin/mdb.master'

Tweak prev commit again

Make sure errors are propagated from init_meta

Merge remote-tracking branch 'origin/mdb.master'

Fix typo in Win32 branch

Merge remote-tracking branch 'origin/mdb.master'

Cleanup prev commit

Loop on copyfd meta write, since pipes may return after partial write.

ITS#7652 fix I/O error checks

partial revert of d6d2638acc245116b8f091ac425b6700d06c4713 and
26a25df5fcc2fcddae6597a61c1b867fc27c568b

The original code was already tested and working correctly.

ITS#7645 use ldap_pvt_tls_config for TLSProtocolMin

ITS#7645 more OpenSSL TLS versions

ITS#7645, #5655 TLSProtocolMin docs

ITS#5655 cleanup

ITS#5655 TLS_PROTOCOL_MIN setting

Merge remote-tracking branch 'origin/mdb.master'

Tweak mdb_envinfo numreaders

Return the actual shared reader count when it exists, not
just the current process env's reader count.

ITS#7641 don't muck with controls if slapi doesn't use them

ITS#7615 use shorter names for semaphores

NetBSD can only handle up to 14 chars, we were using 21. Now
we encode to 15, and for NetBSD truncate the last char.

Merge remote-tracking branch 'origin/mdb.master'

Tweak reader_pid check

Check again after acquiring rmutex. Avoids potential issue with
a duplicate pid coming in between initial check and rmutex.

Get pid lock outside of rmutex

Avoid holding rmutex for longer than necessary.

Merge remote-tracking branch 'origin/mdb.master'

Tweak reader checks

Use mti_numreaders for loop limit, not me_maxreaders.

Merge remote-tracking branch 'origin/mdb.master'

Add mdb_reader_check()

Split MDB_VERSION to MDB_DATA/MDB_LOCK VERSION

Tweak reader_list

Tweak mdb_stat(1)

Don't obtain reader txn before displaying reader table. Exit
after reader table if no other DB query options were given.

Add mdb_reader_list()

Dump the active slots in the reader table.

Add mdb_dbi_flags()

Retrieve the flags from a DB handle.

Merge remote-tracking branch 'origin/mdb.master'

Fix child txn dirty_room counts in spill/unspill

Don't count pages twice if they're already accounted in an ancestor txn.

More for stale sub-cursor flags

Same fix for cursor_first/last.

Fix stale sub-cursor C_INIT flag

Whenever we enter cursor_set() the sub-cursor's flag must be
cleared. If the new cursor position has valid subdata it will
be initialized again, if not then the sub-cursor has nothing
to point to.

Tweak comments, defaults should be OK already

Bump version to 0.9.7

Merge remote-tracking branch 'origin/mdb.master'

Also set/clear P_KEEP in parent txn's cursors

Spill pages, take 3

Delay touching pages until cursor is positioned.

This avoids unnecessary rewrites of pages that do not change.
(Restructuring for upcoming mdb_page_spill work.)

Simplify: Always set C_UNTRACK for tracked cursors.

TODO: Rename C_UNTRACK to C_TRACKED.  Omitted now for readability.
The current name is because it's lazy: not always set when tracked.

Save freelist using proper mdb_cursor_put().

(Restructuring for upcoming mdb_page_spill work.)
mdb_freelist_save() can't just Get() the destination, since
mdb_page_spill() may have put the destination in the read-only map.

TODO: Can this new put() modify the freelist, which would break it? The
final iteration's put() can shorten the node, the rest uses MDB_CURRENT.

We could set P_KEEP on dirty freeDB leaves and ovpages, since they are
all about to be modified.  But the code in this commit must stay anyway,
if mdb should support dropping a 256G DB. I.e. too big for dirty_list.

Move code out to mdb_page_dirty()

Factor out parent

Fix env_read_header() on Windows

Commit d6d2638acc245116b8f091ac425b6700d06c4713 broke read
on zero-length files.

Do not follow uninited cursors' page pointers.

Nor uninited cursors' subcursors' page pointers.

ITS#7636 slapi: don't free filterstr if filter didn't change

Fix rebalance/cursor adjust

When collapsing root, must also move cursor index down,
not just the page pointer.

Also in mtest, break from NEXT loops on error, otherwise it just
prints the previous key/data again, which looks confusing.