Improve harness code

Support timestamping with harness -  using VERBOSE=2
Fix also logging in several situation
(i.e. continue logging multiple test in VERBOSE mode,
do not coredump with empty output).

Switch to normal log_verbose message

Here it's not an error case - so do not push this message to stderr.

Fix error path for create_toolcontext

Never return unfinished toolcontext - since error path is hit on
various stages of initialization we cannot leave it partially uninitialized,
since we would need to spread many more test across the code for config_valid.
Instead return NULL and properly release udev library resources as well.

Detect the lvm binary path in lvmetad udev rules.

We can't use 'DM_SBIN_PATH'. This one is set only for DM devices but not
for all block devices - the pvscan is run on all relevant block devices!

This LVM_SBIN_PATH (as well as DM_SBIN_PATH) detection should be removed
eventually but for upstream solution, we still have to do that as there are
known cases where the binaries are put either in /sbin or /usr/sbin
(some installation systems, initrd systems etc.).

Remove some whitespaces.
(Test commit.)

Remove some whitespaces.
(test commit)

Do not unlink the socket on daemon shutdown if activation is done via systemd.

Add 'Restart: on-abort' for lvmetad systemd service.

Restart lvmetad automatically if it exits due to reception of uncaught signal,
leaving a track in systemd log/syslog.

Fix warn message and update man page

Fix regression in man page. The chunk size is in kilobyte units on command line
input though in the source code we work with sector size unit
so make it clear in the man page.

Update chunksize for thin pool in man page - it's max value is 1024M == 1G.

Fix warning range message to show proper max value.

post-release

update reworded string

close ]

Change pvscan --cache syntax

pre-release

Adjust wording

re-word warnings to be clearer to user

Switch pvscan --cache major:minor to --major --minor.

Some more missing supposedly 64bit operations.

Avoid use 32bit math for extent_size.

Use already acquired variable...

Fit thin pool metadata into 128MB

If the lvcreate may decide some automagical values for a user,
try to keep the pool metadata size into 128MB range for optimal
perfomance (as suggested by Joe).

So if the pool metadata size and chunk_size were not specified,
try to select such values they would fit into 128MB size.

Improve warning

Use thin_dump --repair suggestion in log error message
and use just warning on  deactivation path without repair info
(since node has been deactivated).

Also check whether there is not 16 args for thin_check configured.

Use 64bit math

Prevent 32bit overflow and resulting weird error reports when working
with TB sizes..

Add test for name mangling functionality.

Check for multiple mangled names in auto mangling mode.

Auto mode can't deal with multiple mangled names. We can do that while working
in hex mode, but in auto mode, this would lead to device name ambiguity.

Fix dm_task_get_name_unmangled to not unmangle already unmangled name.

In 'auto' and 'hex' mode, these names are already unmangled on ioctl return.
There's no point on trying to do that once again (actually it's a bug!).

Check whether device names are properly mangled on ioctl return.

Be more strict when unmangling names on ioctl return - require the name to be
properly mangled in 'auto' and 'hex' mode. There really should not be any
blacklisted character since the names should be renamed already (by means of
renaming it directly or running 'dmsetup mangle' for automatic rename).

Clean up internal mangling interface.

Validate udev structures

Avoid using NULL pointers from udev. It seems like some older versions of udev
were improperly returning NULL in some case, so do not silently break here,
and give at least a warning to the user.

Return success for deactivation of thin pool

if the thin_check fail on thin pool - still return successful deactivation,
since lvremove would currently fail.

TODO: find some way to not run check with lvremove.

Remove part of FIXME

(and reindent a code below)

Update thin test for thin_check

Test if thin_check is present in system and disable its use, when its missing.

Add testing for poolmetadatasize.

FIXME: Allocation policy for metadata pool might need some relaxing.
(For now it needs to put all block on one PV.)

Speedup test run by a few minutes

Reduce disc excercise for some test and focus on LVM testing by
using smaller extent size.

Reduce number of teardown_devs calls and use vg/lvremove instead.

Don't sleep for seconds on pvmove.

FIXME: shell/lvconvert-mirror-basic.sh seems to need more checking.
Test fails for smalled extent size then 512k.

Add FIXMEs

Processing loop needs some thinking

Just make error message more clear

Make more obvious, the origin LV for snapshot must be active.

Scan all devices for lvmetad if 'pvscan --cache' used without device list.

Obtain VG list from lvmetad before relying on get_vgs, otherwise PVs in VGs
don't appear.

Reinstate accidentally-deleted line.

post-release

.

pre-release

Code refactoring

Properly test for dm_asprintf result.
Keep unlocking of mutex in the same function and do not spread lock and
unlock over functions.

Just space moving

Don't leave space on EOL.

List _thread_registry missed mutex

Operation on _thread_registry needs to be covered by mutex.
Cosmetic move a die code after free for valgind short leak list.

a fixme

Support 16GB for thin pool metadata

Add some hack math to allow 16GB devices to be passed as thinpool metadata.
Since kernel has put in limit to not allow which are just bigger then
some predefined constant in kernel but not matching 16GB so any device bigger
is rejected.

FIXME: Current code still might need more tweaks to be more generic.

fix non-lvmetad pvscan macro

Add support for thin check

Use libdm callback to execute thin_check before activation
thin pool and after deactivation as well.

Supporting thin_check_executable which may pass in extra options for
the tool.

Purge remaining trim bits from code

missing reply frees

Pass struct device around internally rather than dev_t.
Add 3rd daemon return state "unknown" for lookups that are carried out
successfully but don't find the item requested.
Avoid issuing error messages when it's expected that a device that's
being looked up in lvmetad might not be there.

Merge metadata size checking

Move the code for poolmetadatasize operation into one place.
Report override for minimum and maximum size.
Drop _read_thin_params function its error reporting is handled elsewhere.

Change pvscan --lvmetad to pvscan --cache.

Added dm_tree_node_set_callback() for preload and deactivation hooks

Run users hook after preload for the node is finished,
or after the node has been deactivated.

Fix estimation of pool metadata device size

If no size was give the later added minimal size check efectively
disable this code. Also the argument for size now must be kept
in sector_size, so adding division by SECTOR_SIZE (moved into
a const expression)

Allow multiple device names with pvscan --lvmetad.
Hold global lock in pvscan --lvmetad.  (This might need refinement.)
Add PV name to "PV gone" messages.
Adjust some log message severities.  (More changes needed.)

Remove support for TRIM message

It's been unsupporte for now - and it's not going to be
implemented for thin pool kernel driver - so dropping
appearence of TRIM from libdm and lvm.

Reinstate error if e.g. daemon comms fails.

Wait for mirror in sync

1s could be too short

Check for error from dup2

Do not take correct fd as error result.

Add traceback for failpath

Skip zero length messages

In case of zero length message, there would be a memory leak on
return path from _do_process_request.

Check allocated pointers

Test pointers from allocation against NULL.
Error paths should be checked, some of them probably need
some extesions.

Improve logging

Use %m for strerror.
Switch to use 0 for fail return code.

Check pointer before deref in debug message

Explicitely ignore fail from this label_read

Add some more pointer validation

Ensure _display_name() and _add_dep() would not deref NULL names.
Switch to use internal dm_basename().

Test alloc fail

Replace goto with just log_debug

FIXME: temporal quick hack to make things green
There might be a better reaction.

Check for alloc error

Simplify segtype_str usage and check for NULL segtype.

Log sys error for lseek

Debug log for hold_lock failure

Log sys errors

Backtrace for failing drop_cached_metadata

Check for udev_get_dev_dir result

Don't use NULL return value.

Correct enum type

Using debug_t and some forgetten alloc_policy_t, force_t from past commit.

Few more close and dev_close trace

Adding (void) where we cannot really report an error.

Use 64 bit integers for device size & label sector in _pv_populate_lvmcache in
lvmetad client code. Fixes RHBZ 798267.

Add a regression test for the PSize bug in pvs/lvmetad.

Use 64 bit integers whenever extracting numbers from daemon replies.

s/CPG_/CS_: Various CPG constants are going away, even though CPG itself stays

F17 is getting rid of OpenAIS libraries (and checkpointing).  While the
CPG stuff is staying, some if its constants are being removed.  So, we
must adjust and use the remaining constants which the CPG constants were based on.

[~]# egrep 'CPG_DISPATCH_ALL|CPG_OK' /usr/include/*/*
corosync/corotypes.h:#define CPG_DISPATCH_ALL     CS_DISPATCH_ALL
corosync/corotypes.h:#define CPG_OK               CS_OK

Missed ;

More useful debug message

Add _rimage as reserved suffix

Add cast for ptrdiff_t - cleanup gcc warning

Improve error logging

Log errors instead of plain return 0.
Check for f->private strdup result.

Add stack trace for lv_reduce

Use common code call with stack trace.

TODO: maybe the release_lv_segment_area()
should be actually able to return error code to upper level.

Check for version string buffer

Since lvm seems to call driver_version(NULL, 0)  this would lead
to crash. Though the combination of the code is probably very hard to hit.
If the user doesn't supply version buffer, just skip printing to buffer.

Check for allocation error

return ENOMEM when malloc fails.

Log fail of pthread_join

Syslog error outside of mutex section.

Fix leak of FID structure

Wipe initial 4KiB for non-zeroed thin volumes

If the thin pool has disabled zeroing  (created with -Zn), we at least
clear initial 4KiB of such thin volume (provisions 1st block).

If lvcreate is executed with '-an' command will abort (same way like we for
normal LV - however for normal LV option -Zn may skip clearing completely,
for thin volumes this option is not supported (applies only for pools).

Allow cluster mirrors to handle the absence of the checkpoint lib (libSaCkpt).

The OpenAIS checkpoint library is going away; therefore, cmirrord must
operate without it.  The algorithms the handle the timing of when to send
a checkpoint, the determination of what to send, and which ongoing cluster
requests are relevent with respect to the checkpoints are unaffected.  We
need only replace the functions that actually perform the storing/transmitting
and retrieving/receiving of the checkpoint data.  Rather than store the
checkpoint data in an OpenAIS checkpoint file, we simply transmit it along
with the message that notifies the incoming node that the checkpoint is
ready.

Pass 'single_device' parameter down to suppress 'Can't find uuid' messages
when reading VG text metadate and called from pvscan --lvmetad.

(Longer-term, that check needs moving outside of that code.)

Fix a whitespace bug in the last checkin.

Attempt a fix for lvm shell accumulating copies of orphan PVs with each "pvs"
invocation.

expose new include files to build

Reflect new file locations, include file updates etc.

remove old makefile

move daemon files

new makefiles