Add simple test for read-only table reload suppression

Just a small test for previous libdm commit.

Fix read-only identical table reload supression.

Compare also file size to detect changed config file

Clvmd detects modifed config file before it takes lv_lock.
If the config file is changed rapidly - the change was ignored within
a seocnd ranged.  This patch adds also compare of file size.
So change like some flag for 0 to 1 would pass unnoticed - but
it's quick fix for failing test suite.

FIXME: Implement inotify solution.

lvmetad: Obliterate vg_status by returning the same information from
update_pv_status, saving a dozen lines of code and execution time of one
walkthrough of the PV list.

lvmetad: Fix a possible infinite loop in vg_status.

lvmetad: Robustify update_pv_status and remove an useless lookup.

First stab at making lvmetad-core threadsafe. The current design should allow
very reasonable amount of parallel access, although the hash tables may become
a point of contention under heavy loads. Nevertheless, there should be orders
of magnitude less contention on the hash table locks than we currently have on
block device scanning.

lvmetad: Avoid stale PV -> VG mappings on metadata update.

Optimise PV -> VG lookups by using a UUID (hash) map.

Free up allocated memory before exiting, in lvmetad.

Fix two small (but eventually unbounded) leaks in daemon-server.c.

Can't have a global memory pool in lvmetad (that would constitute an ongoing
memory leak) => remove it (it's been unused anyway).

Make lvmetad also report VGID in reply when adding a PV without MDAs (this
obviously only works for VGs that already had at least some MDA discovered).

Add code to lvmetad's testclient that scans an actual physical device,
effectively emulating a future "pvscan --lvmetad" command.

Make lvmetad report the VG ID and status (complete, partial) in reply to pv_add
requests.

Towards MISSING (PV) flag management in lvmetad.

Make it possible to represent type-correct single-item arrays in config trees.

Add an API to config.h for creating values in the config_tree mempool.

Work out some more details in metadata update in lvmetad.

Remove and unneeded parameter from build_parallel_areas_from_lv()

Fix potential null ptr deref in 'origin_from_cow'

return NULL rather than segfaulting if lv->snapshot is not set

More work on cache maintenance code in lvmetad: keep track of PV status.

Start filling in the core LVMetaD functionality and the corresponding
testclient bits.

Various improvements to the daemon-common code, including automated response
formatting from config trees provided by the daemon implementation.

Improve format_buffer in daemon-shared.c, adding block formatting in addition
to string/integer (this propagates to the *simple* family of request/response
functionality).

Revert the #include changes. Need to fix this at the #include site for now, and
eventually refactor the way we structure #includes in the all of the library.

Slightly refactor the config code to allow better reuse (no functional change).

Add a missing libdevmapper.h #include to lvm-types.h (for dm_list).

Add a couple missing #includes in uuid.h.

Better version of failing 'should' test

where the naming is left completely on lvm.
(Commited code has been different version of test).
So here it should be able to figure out new free name and create a new LV.

Adding test for repair when confusing metadata are found

LVM2 RAID design doc

Fix t-vgreduce-usage to stop relying on the persistent cache not seeing a
device that has been brought back from the dead: this sometimes fails with
clvmd (the cache is updated "too soon"). Instead, force a pvscan and rely on an
up-to-date cache as usual.

post-release

remove unnecessary lvconvert.h

pre-release

Downgrade error message - it isn't strictly an internal error in the
library, and the known cause within lvm2 got fixed.

move doc to doc dir

missing ifdef when new fn unused

 Remove dev name prefix from dmsetup line output if exactly one dev requested.

Ignore activation/verify_udev_operations if dm kernel driver vsn < 4.18.

Fix warning for pvcreate over MD linear.

If MD linear device has set rounding (overload chunk size attribute),
the pvcreate command prints this warning:

  /dev/md0 sysfs attr level not in expected format: linear

Need to snapshot lookup by uuid instead of name in case it's renamed.

Log cookie values when incrementing/decrementing to give clues about races.

Move snapshot removal activation logic into lib/activate.

Move snapshot deactivation logic into lib/activate, fixing the
teardown sequence.  (Previously the snapshot was deactivated while its
origin was active and before its removal was committed to disk, so
restarting after a crash at the point would leave corruption.)

report which dev was not found

Increase timeout for udev settle

For some unknown reason (atm) udev settle takes longer time on recent rawhide.
To pass the test - inrease the settle timeout.

Cope with a PV only discovered missing when creating deptree.

Abort operation if dm_tree_node_add_target_area fails.

Always perform preload logic before suspending - not only in the case when we
have precommitted metadata.  (Necessary to avoid loading tables
while suspend in lvchange --refresh.)

decode cookie flags in debug msgs

Snapshots LVs are never loaded in their own right, only along with their
origin.

Fix conditions using no_merging: only those using lv_is_merging_cow() should
have been converted, not pure lv_is_cow ones.
(Merging has no impact on how the pre-merged cow segment itself is loaded.)

first attempt to fix test

reinstate accidentally-removed lines to fix pvmove again

Report internal error if suspending a device using an already-suspended dev.
This catches the recent pvmove problem trapping I/O between layers.

RAID updates for gdbinit file

Add framework for validation of ioctls.  Doesn't do any checks yet.
dmsetup --checks
libdevmapper: dm_task_enable_checks()
lvm.conf: activation/checks=1

update pv_attr check for new missing attr

When suspending, automatically preload newly-visible existing LVs
Let's find out if this makes things better or worse overall...

Report internal error when parameters are missing on table load

When some target is passing empty parameters to some dm target,
report this as an internal error to better catch some broken
table construction (some mirror conversions seem to be doing
this for now).

use remove -f

teardown stray filesystems too

Use -f to remove device

Since some test may leave devices in suspend mode which would require
carefull order of resume operation - use '-f' to replace them with
error targets

For disable_dev  - when  'error' target is used for open count - treat
return code as ok (|| true) to avoid breaking futher test processing.

Try to force remove dm devices in after test fails.

Try to remove any stray loop devices left behind by earlier aborted tests.

test attempt to remove stray loop devs

Differentiate the request and response format, in daemon/common.

Add age filter to dmsetup udevcomplete_all to minimise concurrency problems.

make udev teardown conditional on using udev

teardown aborts if it hits an error, so better teardown semaphores earlier
as we need them later in the teardown process

print any loop devs that weren't torn down

remove any stray semaphores after tearing down devs

Reinstate correct permissions when creating mirrors.

debug log readonly flag with ioctls

Append 'm' attribute to pv_attr for missing PVs.

Add uuids to dmsetup create cmds.

use dmsetup remove -f as a better way of simulating a device "disappearing"

Remove temporary failures now, but continue to give INTERNAL_ERROR.

Temporary conversion to internal error and failure, to see how many
instances of this problem this flushes out.

Fix error message - the parameter name is '--mirrorlog' not '--log'

Remove unnecesary conditional.

The conditional is not just unnecessary, it would have been wrong.  The code
is suppose to be checking if the 'splitmirrors_ARG' is negative, but it
instead is checking 'mirrors_ARG'.  Rather than changing the argument being
checked, I've pulled the check entirely because 'splitmirrors_ARG' is already
guarenteed to not be negative by virtue of the fact that it is a 'int_arg'.
Negative values will be caught in _process_command_line().

Annotate CLVMD_CMD_SYNC_NAMES in decode_cmd

More readahead tests.

more fixes to readahead etc.

Change to still sync with udev (to test waiting for rules) then to verify udev
operations like it used to (which will perform them on systems with
non-std dev dirs where test udev rules aren't installed).

Remove enforcement of udev verification when using non-standard /dev location.
If you change the dev dir, it's your responsibility to adjust udev rules
or tell lvm not to use udev too.

Selectively enable/disable udev rules.

Move _set_lvm_fallback into toolcontext, fix string comparison (/devtest
matched /dev) and note that function should go anyway as it can be
overriding a valid config.

Extend debug log messages to distinguish between the 3 states:
trust udev; verify udev; perform dev node operations directly.

Move udev_only logic inside stacked node op code.
(We still need to treat add+readhead+del as a no-op.)
Rename udev_fallback to verify_udev_operations.
Rename --udevfallback to --verifyudev

Add int/str lookup routines specific to the reply (in client) and request (in
server) for nicer-looking code (thin wrapping around find_config_{int,str}).

Also parse the config_tree on the client end (daemon-client.c).

Parse the incoming config tree in daemon-server.c, providing the
daemon-specific handler with a struct config_tree pointer to look things up in.

LVMetaD build hack: link to lvm-internal &c. and add a simple test script.

Implement daemon_send_simple and use it in the testclient.

Do not open the socket with SOCK_NONBLOCK in daemon-client, since we have no
use for that behaviour (at least for now).

Call daemon_close before exit in the testclient.

Sketch out the actual on-wire format in daemon-client.h documentation, and add
a simplified send interface.