Replacei sleep wait with udev settle

Use new udev_wait command instead of unpredictable sleep waiting.
As with more devices in the system, udev processing is slower.

Query only test related vg name

Add command to wait for udevadm settle

Do not break pipes

As the option 'set -e -o pipefail' is very sensite on pipe breaking
stop using '-q' for grep commands.

Otherwise this command (with large enough table) would fail:

 dmsetup table | egrep -q

with exit code 141 (128 + SIGPIPE)

As Peter suggested, he prefers to keep '-o pipefail' - so make sure all
piped commands will read the whole output and will not exit too early.

Accelerate mirror tests

Instead of recreation of whole device set for each test round,
just cleanup created LVs to empty VG.
Lower the size of PV devices to 16MB

Display duration of tests when it passed.

Enhance output with info about the test duration.
Cleanup few declarations in the code.

Updating man pages

Compile code for memory debuging only with DEBUG_MEM

When it's not in use - do not compile this code.
Improves lcov code coverage results for this code a lot :)

Set DM_UDEV_DISABLE_OTHER_RULES_FLAG for suspended DM devices in udev rules.

This is to avoid any scanning and processing of DM devices while they are in
suspended state (e.g. a rename while the device is suspended - a CHANGE event
is generated!). Otherwise, any scanning in the rules could end up with locking
the calling process until the device is resumed and so we don't receive a
notification about udev rules completion until then (and that effectively
locks out the process awaiting the notification!).

However, we still keep 'disk' and any 'subsystem' related udev rules running.
We trust these and these should check themselves whether a device is suspended
or not, not trying to run any scanning if it is.

Use memcpy and add error message

strncpy (which check each byte for \0) is not need as we always copy
the length size - so using memcpy is a bit cheaper.

Add missing log_error message for failed allocation.

Skip NULL check before dm_free

dm_free checks for NULL itself.

Avoid rebuilding of uuid validation table

Small CPU relax...

Improve lvcreate "insufficient extents" errors to "insufficient free space".

Use O_DIRECT when reading block devices.

Add missing lvm_quit in test

Fix occasional confusing memory leak report in testing.

Initialize pool object for each row

Fix assert abort of dmsetup (when compiled with pool debug)
dmsetup splitname --nameprefixes --noheadings --rows gvg-a2

Move pool begin in the inner loop - otherwise it would using
already 'ended' pool object.

post-release

pre-release

Bring lvscan man page up-to-date.

Fix lvchange --test to exit cleanly.

Add change_tag to toollib.
Allow multiple pvchange command line options to be specified together.

If other process finishes (or aborts) pvmove operation and
polling function cannot find any lv with PVMOVE flag, return
success and do not print  "aborting" message.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=602389

In some versions (RHEL6) dlm_create_lockspace() always
return lockspace reference (even if lockspace already exists)
and thus increases DLM lockspace count. It means that after
clvmd restart the lockspace is still in use.

(The only way to clean environment to enable clean cluster
shutdown is call "dlm_tool leave clvmd" several times.)

Because only one clvmd can run in time, we can use simpler logic,
try to open lockspace with dlm_open_lockspace() and only if it fails
try to create new one. This way the lockspace reference doesn not
increase.

Very easily reproducible with  "clvmd -S" command.

Patch also fixes return code when clvmd_restart fails and fixes
double free if debug option was specified during restart.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=612862

Remove duplicate statement (pasted twice by patch at some point?)

Add test to make sure that a log device is retained when a
mirror image fails and the policies are set to:
 mirror_image_fault_policy = "replace"
 mirror_log_fault_policy = "remove"

Remove DEBUGLOG from dmeventd.

Add -f (don't fork) option to clvmd and fix clvmd -d<num> description.

remove unused definitions

post-release

pre-release

Remove -f from list of targets for cleanup

rm -f is already in used for this cleanup.

Do not scan devices unnecessarily for reseting error counter

For reseting error counter use directly btree cached elements and do not
create whole dev_iterator.

Set default value to counter_last

Missed in previous commit and could be using some random stack value.

Do not log backtrace for correctly processed command

Include default make.tmpl

Use default clean targets for this Makefile
Use also same build flags for compiling testing tools like harness.

Enable monitoring for cluster tests

Skip unnecessary lock_vol() call after volume deactivation

Improve condition within lock_vol so we are not calling extra unlock
if the volume just has been deactivated.

Patch uses lck_type and replaces negative 'and' condition to more
readable 'or' condition.
Few missing strace traces added.

Add exec_cmd paramater  sync_needed

As sync_local_dev_names() cannot be called within activation context,
add new parametr which allows to select if the sync call is needed
before executing new command.

Check for file with clvmd pid

Instead of checking  $LOCAL_CLVMD set during some 'aux' execution which
doesn't seem to be propagated to this shell - check for existance of pid
file of clvmd process - so this test is skipped in singlenode cluster test.

Hotfix to stop harness busylooping

Harness seems to be able to busyloop in while cycle and not moving forward
for certain buffer - so check whethere there was some progress.
This fix allows to continue after failed cluster test.

Fix gcc warning for hiding global variable 's' -> sig.

Replace fs_unlock by sync_local_dev_names to notify local clvmd. (2.02.80)
Introduce sync_local_dev_names and CLVMD_CMD_SYNC_NAMES to issue fs_unlock.

Kill clvmd properly after locking_type 3 tests (using clvmd/singlenode).

add fio

s/log_verbose/log_error/ - Increase log level on error message.

Add disk to mirrored log type conversion.

post-release

reissue 2.02.80

Fix missing declaration for fs_unlock

Avoid cookie sharing between forked processes

Before fork, ensure cookie is reset so it's not shared between processes.

post-release

pre-release

Speedup consequent activation calls

Stop calling fs_unlock() from lv_de/activate().
Start using internal lvm fs cookie for dm_tree.
Stop directly calling dm_udev_wait() and
dm_tree_set/get_cookie() from activate code -
it's now called through fs_unlock() function.

Add lvm_do_fs_unlock()

Call fs_unlock() when unlocking vg where implicit unlock solves the
problem also for cluster - thus no extra command for clustering
environment is required - only lvm_do_fs_unlock() function is added
to call lvm's fs_unlock() while holding lvm_lock mutex in clvmd.

Add fs_unlock() also to set_lv() so the command waits until devices
are ready for regular open (i.e. wiping its begining).

Move fs_unlock() prototype to activation.h to keep fs.h private
in lib/activate dir and not expose other functions from this header.

Add internal fs cookie

Add functions for handling internal lvm cookie used for
all dm_tree operations until fs_unlock is called.

Define DM_COOKIE_AUTO_CREATE

TODO: Use it also for already written code.

Use strndup as GNU extension

strndup needs _GNU_SOURCE defined as it's GNU extension.
Remove duplicate string.h header added in previous harness commit.

Cache config_tree

Start to use config_tree for cache just like vgmetadata.
When vgmetadata are erased destroy its cached config tree.

Change import_vg_from_buffer to use config_tree

Change function import_vg_from_buffer() to import_vg_from_config_tree().
Instead of creating config tree inside the function allow config tree to
be passed as parameter - usable later for caching.

Add default error path for get_property

Set invalid property value for error path when NULL handler is passed.
Fixes use of uninitialized prop structure as we return 'v' by value.
---

Improve OCF dir support

Add --with-ocfdir configurable option.
Fix superfluous /usr in ocf_scriptdir instalation path.

Add aclocal.m4 (for pkgconfig).

Skip the api/percent test on kernels <= 2.6.32.

Add a test for the interaction of snapshots of mirrors and lvconvert --repair.

Add a test for the interaction of transient errors and dmeventd mirror repair.

Fix a typo in the new relaxed mirror check (test/lib/check.sh).

Using Fedora 14's autoreconf.

Remove *.t in test/api's make clean target.

Disable activation/monitoring in testing lvm.conf by default.

Relax the mirror test in test/lib/check.sh.

In aux, exit 200 no longer works as skip -- replace with real skip call.

Fix up the make clean target in test/.

Fix memory leak in filter creation error path

If some allocation for peristent filter fails its memory reference
was lost, fix it by calling filter's destructor.

Fix log_error messages for failing allocation.

Intentionaly ignore result from get_config_uint32

Remove dead assignment of 'ret' value

Variable 'ret' assigned from _do_event() was actually not used and replaced with next
assignment without any read of the returned value.

Code is reformated - so the error path is put in the if() branch and normal
code is put after the 'if' together with FIXME comment.

FIXME lowprio: logging needs to be fixed in this code,
 - multiple log_errors are printed, stacks are missing...

Remove unneeded stack prints after log_error

Reverting recent commit to disallow adding/removing mirror log while
removing/adding mirror images... There was already code in there to
do the job - I just didn't find it in WHATS_NEW (or in the code right
away).

Prevent the user from simultaneously adding a mirror log while removing
a mirror image (or removing a log while adding a mirror).  Advise the
user to use two separate commands instead.

This issue become especially problematic when PVs are specified, as they
tend to mean different things when adding vs removing.  In a command that
mixes adding and removing, it is impossible to decern exactly what the
user wants.

This change prevents bug 603912.

Check result of dm_snprintf for error

Add sys_debug loging for unlink

This unlink intentionally silently ignores any errors.
It's still worth to trace its error status in debug mode.

Fix gcc warnings for missing headers and prototypes

Add missing header  string.h (strcmp, strndup).
Add 'static' to fix missing prototypes gcc warning.
Remove unused variables verbose_off, TESTDIR, PREFIX.

Remove obsolete bits of support code from test/.

Re-enable t-lvconvert-repair-transient, avoiding the deadlock by running mkfs
asynchronously.

Add missing error path tests

Fail deamonization if lvmcache_init fail

FIXME Add proper cleanup

Return PERCENT_INVALID for error case

If the percent value could not be determined return PERCENT_INVALID.
Indent function with tabs.

Remove unused variable mirr_state and its assignment

Remove dead assignment to thisfd

Value of 'thisfd' is not read again after its assigment

Fix another bug in t-pvcreate-operation-md (should pass now).

Add a test for RHBZ 640621 (metadata/dirs + no PV MDAs broken).

Fix typo in t-pvcreate-operation-md.sh.

Move the mkdtemp functionality into test/lib/utils.sh.

Fix a couple of bugs in the new (test/lib/)aux.sh.

Substantial rework of the functional test support code. Some new features:

- somewhat neater, more consistent and more readable output
- possible to set any lvm.conf value: aux lvmconf "section/key = value"
- LVM_TEST_NODEBUG to suppress the (lengthy) "## DEBUG" output
- back-substitution on test output ($TESTDIR/$PREFIX -> @TESTDIR@/@PREFIX@)
- support code moved from test/ to test/lib/ --> less clutter

Fix bug 635949: lvconvert man page clarification

Change to correct cmirrord return code is user visible - update WHATS_NEW
file.

Fix bug 632681: cmirrord shouldn't fail 'start' if already started

Return 0 from the cmirrord init script if the daemon is already
running.

http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generic/iniscrptact.html

Export DM_CONTROL_NODE_UMASK and use it while creating /dev/mapper/control.

Use system assert in test/api/percent.c, for now.

Fix wrongly paired unlocking of global lock in pvchange. (2.02.66)

Remove check for existance of vg pointer

Checking for vg being != NULL in this place is not needed.
Pointer vg is already dereferced in this function above this code line.
Also this internal function _read_pv is always called with valid 'vg' pointer.

Add backtraces for backup and backup_remove fail paths