man: add man page for lvm dumpconfig

fix spurios char

Mistyped char left in code.

tests: more pvchange options examined

tests: cover more commands

Visit more code lines.

cleanup: indent change

debug: avoid warning when compiled with valgrind

Declare 'c' only when compiling without valgrind.
This cleans compiler warning about unused var when
compiled with valgrind pool support.

lv_manip: drop cmd pointer from for_each_sub_lv

Drop unused passed cmd pointer from function.

TODO:

We have two similar functions (though not identical)

lv_manip.c: for_each_sub_lv()
metadata.c: _lv_each_dependency()

They seem to not always match - we should probably convert
to use only a single function.

lv_rename: validate renamed sublv

Use proper vgmem memory pool for allocation of LV name in the vg
and check if new renamed LV is a valid name.

TODO: validation should really use also VG name, othewise we are not
able to tell "vgname-lvname" will be valid.

lvrename: fix exit code

Return invalid cmd line (3) when error is detect on cmd line input.
Cmd failed (5) is used when command processing fails.

pvchange: fix exit code regression

Commit 1a832398a7caa96faab2ccbac773cc4047c8f9c1 moved
some code from _pvchange_single() to main pvchange() and
introduced exit code regression as return codes have not
been properly changed, thus pvchange command exited
with '0' exit code, even though it has reported error.
Also there is a missing vg unlock in error path.

Fix it by counting the total number of expected calls before
checking for pvname and also unlock and relase vg when
pv is not found.

cleanup: easier casting

Use 32bit for extent calcs.
Drop uneeded implicit case.

cleanup: simplify pv name size estimation

Reuse buffer with size of 2 * PATH_MAX to handle worst case escape
and avoid extra calculation of espaced len.

cleanup: lvmetad use dm_hash_iterator

Simplify code with the use of dm_hash_iterator

man: colorization fixes

Fix style

man: add repair info for thin pools

Document lvconvert --repair for thin pools.

lvmetad: Fix an endless wait loop introduced in c13f5dbb.

tests: missed exclusive activation

tests: pass list devs to allocate from

tests: cover code from more commands

tests: check forced backup

tests: disable test for broken kernel raid targe

Since using raid5 - validate it's usable on the system

tests: remove also 3.14 fc21 kernels

lvmetad: Wait at least 80s for the initial scan.

lvmetad: add missing dev_close in error path

Fixes missing dev_close() in dev_read error path
introduced in commit
a36869867275b8ef0f6d324bc5a7f3e2b8e445e9
3e5bec37e9cec455f20656858ba510b69010676d

(in-release fix)

lvmdump: add lvm dumpconfig --type diff/missing

For a quick overview of config when debugging and to quickly check
which values are different from defaults and which are not defined
in the config and for which defaults are used.

pvresize: fail exit code for negative size

Pvresize with negative value retuns invalid cmd line exit code.

tests: wipe signature

Check for usable blkid.

tests: update

tests: use exclusive activation

pvmove in cluster needs exlusive activation or cmirrord running.

tests: argless pvremove

tests: more pvresize tests

Maybe move to separate pvresize test,
remove vgck size this is already executed elsewhere.

tests: relocate vgimport test

It seem like a better-fit here.

debug: add proper backtrace debug

When failure happens inside refresh, print proper debug backtrace.

cleanup: drop init of already zalloced mem

cleanup: use dm_free without extra test

It's ok to free(NULL).

cleanup: use trigraph

cleanup: initilization of vars

cleanup: indent

cleanup: use NULL for pointer reset

clvmd: validate open device state

If clvmd does not hold any lock, it should also not keep any opened
device.

The reason for this patch is, that refresh_toolcontext calls
dev_cache_exit() which destroys whole device cache (even those with
opened file) - previous patch added recovery path to avoid memory
corruption, but opened files are still bugs that need to be fixed.

So this patch certainly kills many internal mirror & raid tests,
since they leak opened file descriptors (when tests are executed
with 'abort_on_error').

clvmd: hardening leak on exit

Operate with lvm_thread_exit while holding lvm_thread_mutex.
Don't leave unfinished work in the lvm thread queue
and always finish all queued tasks before exit,
so no cmd struct is left in the list.

(in-release fix)

dev-cache: fix mem corruption on refresh context

When lvm2 command works with clvmd and uses locking in wrong way,
it may 'leak' certain file descriptors in opened (incorrect) state.

dev_cache_exit then destroys memory pool of cached devices, while
_open_devices list in dev-io.c was still referencing them if they
were still opened.

Patch properly calls _close() function to 'self-heal' from this
invalid state, but it will report internal error (so execution
with abort_on_internal_error causes immediate death). On the
normal 'execution', error is only reported, but memory state is
corrected, and linked list is not referencing devices from
released mempool.

For crash see: https://bugzilla.redhat.com/show_bug.cgi?id=1073886

WHATS_NEW: commit f12ee43

libdaemon: fix misleading  "WARNING: Ignoring unsupported value for expected." when communicating with daemon

When we're trying to search for certain tree node
in daemon's reply, we default to a blank string ""
if the node is not found. This happens during lvmetad
initialization.

However, when the default blank string is used, we
can't use dm_config_find_str at the same time - the
dm_config_find_str_allow_empty should be used instead.
Otherwise a a warning message:

  "WARNING: Ignoring unsupported value for ..."

is issued.

config: define default value for global/thin_disabled_features as NULL instead

Before:
  thin_disabled_features = ""
Now:
  thin_disabled_features = []

Which is a more correct and consistent way of specifying void array
though parses can handle both forms.

config: also check empty arrays for difference against default values

dumpconfig: add dumpconfig --type diff to show differences from defaults

config: add support for comparing used config values with default ones

cleanup: move _get_def_array_values fn

So we can use reuse it for the code that will follow...

dev-swap: use SECTOR_SHIFT

dev-swap: detect swap signature on devices smaller then 2MB

Smallest supported size for swap device is 40KB, however current
test skipped devices smaller then 4096 sectors (2MB).

Since page is in bytes, convert it to sectors before comparing
with device size (in sectors).

WHATS_NEW update

tests: wait for clvmd.pid file

Just like with dmeventd and lvmetad.
Do few indent changes around.

tests: simplify

tests: add quotes around device paths

cleanup: clvmd add more info debug message

cleanup: clvmd uses struct initializers

Code easier to read

cleanup: clvmd zombie removal loop

Simplier code to start scan from the next node,
and remove matching pipe client.

cleanup: clvmd reindent read_from_local_sock

Shift indent of else branch to right since
error path returns in the front.
(Simplier to read)

cleanup: clvmd indent change

Plain indent changes.

cleanup: clvmd dump_messages

cleanup: clvmd update log_error

Drop \n from log_error() and add '.'

cleanup: clvmd drop goto

cleanup: clvmd skip reset of null pointer

cleanup: clvmd move destroy_lvhash into main

Keep destruction code path consistent and simple and
destroy lvhash in the place it's been created.
Also issue debug message from a single place.

cleanup: memlock line indent

singlenode: reset pointer when hash is destroyed

Just keep pointer consistent.

clvmd: drop dead code

Since pipe_client has been already reset to NULL,
remove this dead code.

clvmd: drop unused local_sock parameter

clvmd: update add_reply_to_list

Take mutex lock after the allocation just before
the structure is merged into reply list.

clvmd: use dm_malloc

Use libdm malloc wrapper functions.

clvmd: move call of cluster_closedown

We have to close cluster in some predicatable way,
otherwise we may access released memory from different
threads.

So move closedown till the point we know all thread
are closed. New messages from cluster are discarded.

clvmd: move destroy_lvm into lvm thread

Since lvm was initialized in lvm thread call destroy there as well.

clvmd: add special lvm thread exit

When multiple threads act on the same 'quit' variable
the order of exit becomes unpredictable.

So let the main_loop() finish first and then clean up
all queued lvm jobs.

Do not add any new work, when lvm_thread_exit is set.

clvmd: always set cleanup_needed

When thread is created, set the cleanup_needed flag
immediately so it could be used properly any time later
when cleanup_zombie() is needed.

clvmd: fix clean memory on exit

Properly clean 'client' structure only for LOCAL_SOCK type.
(Fixes bug from commit 460c19df621d35260caceec7f598da1566263cd4)
(in release fix)

Also cleanup-up associated pthreads by using cleanup_zombie() function.
Since this function may change the list, restart scanning always from
the list header.

Note: couple following patches are necessary to make this working properly.

memlock: drop locked mem in critical section

When daemon releases memory and it is still in critical
section, issue an error message and drop memory.

We cannot do anything better for now and we at least
release allocated resource.

FIXME:

This code is triggered when i.e. clvmd is killed while
some LVs are suspend - in this case suspended devices leak,
so if this happens during i.e. clvmd upgrade we have
unresolved problem - even locked rootfs...

activate: report release with critical section

This function is typically called for cmd context refresh or destroy.
On the non-clustered case we already unlocked all messages,
however when i.e. 'clvmd' gets break signal it may have
still couple messages queued.

For now just report an error.

lvmcache: fix debug trace

Recent debug tracing commit introduce read of uninitialized memory,
since VGID is not really a proper string which ends with '\0'.
Enforce at most 32 (ID_LEN) chars are read from vgid.
(in release fix)

lvmcache: handle reinit without error

Since commit f12ee43f2eaba5d38b1925a5a34b1746c1d66985 call destroy,
it start to check all VGs are unlocked. However when we become_daemon,
we simply reset locking (since lock is still kept by parent process).
So implement a simple 'reset' flag.

cmirrord: Clean-up stray warning message (attempt #2)

There are two types of CPG communications in a corosync cluster:
messages and state transitions.  Cmirrord processes the state
transitions first.

When a cluster mirror issues a POSTSUSPEND, it signals the end of
cluster communication with the rest of the nodes in the cluster.
The POSTSUSPEND marks the last communication of the 'message'
type that will go around the cluster.  The node then calls
cpg_leave which causes a final 'state transition' communication to
all of the nodes.  Once the out-going node receives its own state
transition notice from the cluster, it finalizes the leave.  At this
point, the state of the log is 'INVALID'; but it is possible that
there remains some cluster trafic that was queued up behind the
state transition that still wants to be processed.  It is harmless
to attempt to dispatch any remaining messages - they won't be
delivered because the node is no longer in the cluster.  However,
there was a warning message that was being printed in this case
that is now removed by this patch.  The failure of the dispatch
created a false positive condition that triggered the message.

polldaemon: Re-initialise lvmcache properly on fork (fixes RHBZ 1073670).

WHATS_NEW: commit 5eef269

cleanup: add cmd_context reference to struct cft_check_handle

So we have all things needed to do a configuration check packaged
in one handle. It makes function calls inside a bit more readable.

cleanup: indent

pvscan: cleanup updates

Simplify code:
 remove unneeded assignments
 use unsigned values for length

pvscan: fix report of long pv names

pvscan --uuid was broken since it was using only 128 char buffers
without checking any write size, so any longer device path leads to
crash.

Also ansure format is properly aligned into columns with this option.

cleanup: simplify pv name print

pv_vg_name() now already hides orphans.

pv_vg_name: do not expose internal orphans to lvm2 users

Check for orphan VG name and return just empty string,
Use internally pv->vg_name if the real orphan name is needed.

pvdisplay: fix man to refer to sectors, not KB

The PV size is displayed in sectors, not kilobytes
for 'pvdisplay -c'

Signed-off-by: Thomas Fehr <fehr@suse.de>
Acked-by: Hannes Reinecke <hare@suse.de>

pvdisplay: use log_print_unless_silent for info messages

metadata: print VG with invalid chars in quotes

We we report invalid chars, put quotes around vg name.

clvmd: avoid resending local sync commands

Instead of sending repeatedly  LOCAL_SYNC commands to clvmds
like 'lvs', rememeber the last sent commmand, and if there was no other
clvmd command, drop this redundant SYNC call message.

The problem has started with commit:
56cab8cc037fc4d31085705987b3a4c04d5e5938

This introduced correct synchronisation of name, when user requests to know
open_count (needs to wait for udev), however it is also executed for
read-only cases like 'lvs' command.

For now implement very simple solution, which is only monitoring
outgoing clvmd command, and when sequence of LOCAL sync names are
recognized, they are skipped automatically.

TODO:
Future solution might move this variable info 'cmd_context' and
use  'needs_sync' flag also i.e. in file locking code.

file_locking: use PATH_MAX for dir name

Using just 128 chars for locking dir may wail if longer
dir entry is used, swich to default linux max path.

archiver: drop unneeded backup check

When the backup is disabled, avoid testing backup presence.
This only leads to errors being logged in debug trace and the missing
backup can't be fixed, since it's disabled.

dumpconfig: fix memleak when using --mergedconfig

Check whether lvm dumpconfig --mergedconfig is used only
with --type current (where we're merging current config and
the config supplied on command line). With other types
the config was merged, but it was thrown away since we're
generating other type of config anyway. This lead to a memleak.

Error out if --mergedconfig is used with anything else than
--type current (or without specifying --type in which case
the --type current is used by default).

tests: use check lv_tree_on

Use internal /lib function.
Reduce extent size for test to create smaller devices.

tests: more fedora kernels unusable for testing raid456

lvmcache: add mode debug prints

Decorate NULL returns with debug_cache output so the
debug log doesn't contain spurios <bactrace> line without
any reason for it.

Add internal errors when cache is misused.

config: make global/lvdisplay_shows_full_device_path profilable