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

Hide unused code into if 0

Make it obvious for lcov coverage and static analyzis we
are not interested in this piece of code.

Detect errors from dm_task_set calls

Check for errors in dm_task_set calls.
Use  goto_bad macro with stack trace.
Replace  label  failed: with bad:.

Fix last pthread_join fix commit

Call for pthread_join() does not set errno value even though return values
looks like that. For now assign errno from return value and still use
strerror() to print some error message as this seems to be commonly used.

Add also log_sys_error() message for error close of local pipe.

Add backtraces for archive and backup_locally

If archive or back_locally fails - add stack trace.

Fix memory leak in debug mode of restart_clvmd() error path

Log error state from pthread_join operation

Value jstat is unused - so replace it with logging via log_sys_error().

Remove dead assignment of lv_total and lv_capasity_total

Variables 'lv_total' and 'lv_capasity_total' are unused.

post-release

Copyright notices for new files.

pre-release

Add missing test for reallocation error.

Remove dead assignment

Variable 'r' is never read so remove it and just cast result from
_error_device function to (void).

Verbose log old_umask value

Use old_umask value and print its content through verbose log.

Add internal error if pointer is uninitialized

Add simple check for existance of 'pl' and printer internal error message
if device is missing instead of plain crash.

Add check for unlink errors

Remove unused variable label

Variable 'label' is unused in _format1_pv_write().

Remove unused variable dev_dir_provided

Remove dead assignment of segh

Variable 'segh' is never read again after this assignment.

Remove dead store in lvm_run_command

Variable 'ret' is not read before its next assignment.

Removed unused pointer

Pointer 'duplicate' is unused.

Use dm_free for dm_malloc-ed areas in _clog_ctr/_clog_dtr (cmirrord).

Use dm_zalloc to obtain zeroed memory block.
Use dm_free for dm_ allocated memory blocks.
Test close() for error.

Add checks for allocation errors in config node clonning.

Add checks for clonning allocation a fail-out when something is
not allocated correctly.

Also move var declaration to the begining of the function
and fix log_error messages.

Replace multiple fprintf calls with one large string parameter

Fix error path if regex engine cannot be created in _build_matcher().

Fix only 'stack' printing with full function error exit.

Use const char * for name and old_name in vg

Switch to use const char pointers to avoid changes of these structure
members and having better control over, were these members could
be modified.

Use const char* for offset calculation

As 'const' types are also passed to macro dm_list_struct_base -
keep offset calculation with const char pointers.
Fixes several gcc constness warnings.

Switch void* to char* arithmetic