Some fixmes

'len' calculation is unused ?
Unreachable code could be removed or moved upward ?

Switch to return void

List delete cannot fail, so there is no reason to test for error.

Use dm_snprintf and improve error handling

Add standard error reporting with error logging.
Use plain alloc instead of zalloc for string buffer.
Use dm_snprintf with valid test for <0.

Add range test for device number

Check the output of atoi is in valid range.

Remove duplicate test

Tested condition has been already evaluated before
For strlen() code has already excluded <ID_LEN.
For repairing, already tested (!argc && !repairing) before.

Replace snprintf with dm_snprintf

snprintf testing for negative is replaced with dm_snprintf where this
test really works.
Add missing test for result of dm_snprintf().

Remove unneeded assignments

Variables have (or will have) those values set.

Keep page_size as signed number

Since it's return value from sysconf and is checked for <0.

Move done jump lower

Since before 'goto done' is bufused zeroed, it would otherwise write 1
byte in front of buffer.

Add boundary test for number of mirror devs and logs

As atoi may return negative value - test for both limits.
Test log_args for limits before calling alloca().
Code from dmeventd mirror plugin should probably share same code as
we have in mirrored.c.

Set all parameters to 0

Since the function dm_get_next_target() returns NULL as 'next' pointer
so it's not a 'real' error - set 0 to all parameters when NULL is
returned because of missing head.

i.e. one of use case::
do {
next = dm_get_next_target(dmt, next, &start, &length,
  &target_type, &params);
size += length;
} while (next);

Check that whole locking_dir fits _lock_dir buffer

Move close few lines

Since the function dev_close() has code path, which really could close
file (for unlocked vg) and destroy dev handler, stay on safe side and move
the close few lines later, even our current use case shouldn't trigger
such scenario.

Use dm_list_iterate_items_safe

And avoid direct access to list member variables.
Inline _free_li().

Fix fd resource leak in error path

Use 'goto bad' to cleanup fd on error path.

Ensure strncpy() function always ends with '\0'

Since last character needs to be \0 for string,
pass buffer size smaller by 1 byte.

Set status for error path

Do not leave status unitialized, since in some cases, it's tested,
when the function returns error.

Add missing deps for lvm2api

Hmm, wasted some time because of this missing deps....

Fix resource leaks for failing allocation

In case, something would fail during format initialization,
return allocated memory.

Release allocated resources in error path

If composite_filter_create() fails, release filters.

Check result of lstat

If lstat returns errno different from ENOENT, do not use the content of
struct stat 'buf'.

What's new.

Remove a dubious log message ("another thread is handling an event") from LVM
dmeventd plugins. Fixes RHBZ 771419.

post-release

pre-release

Adjust comments

Add synchornization point in mirror log init.

Put extra sync point when mirror log is deactivated and before
it's activated for the second time.

Disable partial activation for thin LVs and LVs with all missing segments

Count number of error and existing areas and if there is no existing area
for the LV avoid its activation.

Always disable partial activatio for thin volumes.

For mirrors currently put in hack to let it pass with a special name
since current mirror code needs to activate such LV during some operations.

Avoid warning for small pv_min_size

Do not print warning for pv_min_size set in range between 512KB and 2MB.

Clean up systemd unit ordering and requirements.

Minor consistency update for debugging messages

Use mlock/munlock for debugging lines.

User correct base dir for lcov reports

Fix problem when srcdir != builddir.

lost line

Track unreserved space for all alloc policies and then permit NORMAL to place
log and data on same single PV.

lp.pool may be (null) too

Automatically detect whether corosync clvmd needs to use confdb or cmap. (fabio)

Fix data% reporting

For reading % of mapped size of thin volume use as origin for
old style snapshot '-real' device needs to be queried.
Fix log_error report given for lvs -a in this case.

TARGETS macro has to be defined before inclusion of make.tmpl

Use @TESTING@ check in this case.

post-release

Build test run target only for $(TESTING)

Add libraries properly in LDLIBS.

Thin  for_each_sub_lv

Adapt to scan thin dependency LVs

If clvmd is not started, die test instead of skipping it

Oops missed braces in previous commit

This has disabled clvmd for being executed.
(FIXME improve testing part to catch this fault)

pre-release

pre-release

Cosmetic change use return macro with implicit stack

Set missing header define

Fix leak of hash table

Minor leak on command initialization.

Instrument code that pointer are already released

Set pointers to NULL since on the function exit they are no longer valid.

Set to a defined value vars used after error path

Static analyzer noticed this vars are used even when error is reported
back thus their state is undefined - set to 0 for this case.

Test for uname result

in fail path initialize to 0.

Fix missing dmt destructor

Also always initialize maj,min,patchlevel when success is returned.

Free allocated segment type in error path

Limit alignment to 32bit values

to get the same behavior on 32/64 machines.

Ensure whole info is initialised

Since _create_dm_tree_node is copying whole structure,
make sure all members are initialized.

Fix resource leak of file handle

Introduces when added dm_device_get_name.
Close file handle in all error paths.

Check for correctness of uint64 value if exists

Add breaks for cases

Ensure reply struct has all fields defined

Reply is returned by value.

Check and print perror for syscalls

& -> &&

Fix compilation with disabled devmapper

During release preparation things has changed, so making sure
we are compilable with --disable-devmapper.

Thin read  percent info from layered device

Since we only need layered device - check for tpool device presence,
so user doesn't need to activate thin pool explicitely for percent info.

Rename  origin_only to more generic use_layer flag

Since now we have more layered devices i.e. thin volumes - support
selection of layer via flag.

Clean var declarations to the front of the function

Update verbose lvs  to print metadata_percent info

Update lvs  -o fields in WHATS_NEW.

Thin clear stacked message for thin pool

Before removing thin pool LV always make sure, stacked message
for previous run are cleared - but allow to remove any
device that should have been created
(i.e. creation of snapshot failed - so the message for snapshot creation
may be replaced with delete message within unfinished transaction).

Also commit messages after lv remove - so free space is released in pool.

Make commented out code more obvious

Thin skip activation when there are no thin message

If the list with thin messages is empty, do not touch thin pool device.

Thin correct activation order

When the message is passed only in resume path the order needs
to be corrected.

Thin use  suspend/resume_lv_origin

Use origin_only support for thin volume when thin snapshot is created.

Thin use origin_only for thin pools as well

Extend the usage of origin_only flag to allow resume of thin pool LV
(when it's active) to pass only the messages.

origin_only flag will skip detection of already resumed tree for thin_pool,
so we do not need to suspend the tree and we just send messages.

Thin add support for origin_only suspend of thin volumes

Pass in the origin_only flag also for thin volumes - but curently the flag
is not used to its best.

FIXME: achieve the state where only  thin volume snapshot origin is
suspended without its childrens -  let's explore whether this may
happen automatically inside libdm (might be generic for other targets).
So the code would not need to annotate the node for this.

Thin add messages only for activation tree

Extend lv_activate_opts with bool flag to know for which purpose
dtree is created - and add message only for activation tree
(since that's the only place that may send them).

Extend validation check for thin snapshot creation and test whether
active snapshot origin is suspended before its snapshot is created
(useful in recover scenarios) -  in this case also detect, whether
transaction has been already completed and avoid such suspend check
failure in that case.

Thin check for lv_thin_pool_percent error status

Check has been missing.

Update lv_info whats_new

lv_info using -real layer only for origin_only LV

If the origin_only flag is passed for non lv_is_origin LVs,
the extension is not added.

Thin volumes may also use origin_only flag.

Thin prevent removal of its data and metadata LVs

LVs cannot be removed while there are linked to thin pool.
(Gives better error message, than validation).

Thin fix transaction_id incrementation and code refactoring

Add pool_has_message and use it in attach_pool_message.
Also update header to make more obvious which segment type is
expected as parameter.
Rename  'read_only' to  'no_update' (no auto update transaction_id)
to better fit how it's used.
Fix problem when there was only one stacked message replaced with delete
message that caused unwanted transaction_id increase.

Comment cleanups

Move comment where it applies and remove unused attribe when the var
is actually used.

Thin dependency scan support

Go through pool_lv and metadata_lv LVs when doing recursive scan.

Thin add lv_thin_pool_transaction_id

Easy function to get transaction_id status value.

Thin send messages on activation resume code path

Using PRELOAD part would lead to problems when the problem
would happen before vg_write and vg_commit.
Also this change is necessary for snapshot creation sequence.

Caller is still entitled to reference an LV that's unlinked, so don't
tamper with struct contents.

Use suspend|resume_origin_only when up-converting RAID LVs, as mirrors do.

Failure to do so results in "Performing unsafe table load while X device(s) are
known to be suspended" errors.  While fixing the problem in this way works and
is consistent with the way the mirror segment type does it, it would be nice
to find a solution that uses the generic suspend/resume calls.

Also included in this check-in are additions to the test suite that perform
conversions on RAID LVs under a snapshot.  These tests are disabled for the
time being due to a kernel bug that is yet to be tracked down.

Properly show LV removal message.

(Fix regression in commit 6e181ba96dc1f806afc2d7fe825a8d3dd8093052)

Use chunk_size consistently for thin_pool within LVM.

Fix the way RAID meta LVs are added to the dependency tree.

Similar to the "mirror" segment type's log device, _add_dev_to_dtree should
be called and not _add_lv_to_dtree when adding metadata sub-LVs to the deptree.
Since _add_lv_to_dtree was being called, 'origin_only' could be set if a
snapshot sits on top of the RAID device.  This would cause the actual device
that needed to be added to be skipped in favor of the non-existant device,
"<foo>-real".

Fix metad help.

add const

Reorder fns in libdm-deptree.
Tweak dm_config interface and remove FIXMEs.

Add CLVMD_FLAG_REMOTE to skip processing on local node.

Attempt to improve clustered 'lvchange -aey' behaviour to try local node before
remote nodes and address some existing anomalies.

Prompt if request is made to remove a snapshot whose "Merge failed".

Allow removal of an invalid snapshot that was to be merged on next activation.

Don't allow a user to merge an invalid snapshot.

Use m and M lv_attr to indicate that a snapshot merge failed in lvs.

snapshot (m)erge failed, suspended snapshot (M)erge failed

Differentiate between snapshot status of "Invalid" and "Merge failed".

Lookup snapshot usage percent of origin when a snapshot is merging.

Update lvdisplay to show more info about thin LVs

Reformat name and path how the LV is represented with lvm1 compatible option,
to switch to the old way - which had number of  problem - i.e. many links
do not exist - since for private devices we are not creating them.
Add more info about thin pools and volumes.

Minor cleanup

Simplier and more readable char pointer math.