Optimize second call to strchr with same parameters

Small optimalization - reusing already known strchr result.

Fallback to full rescan for missing device

Fix bug when NULL could have been passsed as 'data'
to _add_pv_to_list() if 'dev' is NULL.

Now it fallbacks to complete scan.

What's new.

Remove unneeded test for NULL

Remove check for system_id (it is defined as int8_t[], so cannot be NULL).

Test uuid for NULL

Add test for NULL before passing uuid as src argument to memcpy.
As memcpy function is declared as function not accepting NULL.
Though we pass NULL only with zero length so this patch presents
no functional change to the code.

Add stack trace for error path

If dm_task_set_cookie() fails print stack trace, but keep going on.

Add error path stack traces

Check for errors from dm_task_set_name() and dm_task_run().
Add stack traces for error paths.
Return 0 if some error is found.

Remove check for lv is NULL

'lv' is deferenced in the begining of the function so any check
later is not helpful.

Parameters for dev_manager_transien() are marked as nonnull.

Add missing test for failed pool allocation

Add test for NULL from dm_poll_create.
Reorder dm_pool_destroy() before file close and add label out:.
Avoid leaking file descriptor if the allocation fails.

Replace snprintf with dm_snprintf

Use dm_snprintf with known error case return code (-1).

Check reallocated buffer for NULL before use

As *buf is reallocated in case CLVMD_CMD_TEST: test for NULL is needed
before printing status.
(realloc() == NULL and status != 0)

Refactor the percent (mirror sync, snapshot usage) handling code to use
fixed-point values instead of a combination of a float value and an enum.

Fix a failing test (it used a combination of lvconvert parameters that is no
longer permitted.)

Avoid the automatic MISSING_PV recovery path in commands with special
MISSING_PV handling (cmd->handles_missing_pvs is set).

Fix memory leak when VG allocation policy in metadata is invalid.
Ignore unrecognised allocation policy found in metadata instead of aborting.
Fix another missing vg_release() in _vg_read_by_vgid.

Optimize lookup table read

Reread lookup table only when needed.

Remove dead assignment in wait_for_child

'pid' is not used anywhere - remove it.

Remove dead assignment in _step_matcher

'ns' is not used after this assignment and it is reassigned with the following
code, so dropping this assignment.

Remove dead assignment in _mirror_emit_segment_line

Remove unused 'r' assignment.

Fix memory leak in error path

Nicely hidden memory leak in outf macro error path.
This macro is using out_text() and does automagical return_0.
That would leak tag_buffer allocated memory.

As there was same code for tags output - create _out_tags() function.

Use one fprintf call for usage print

Replace multiple fprintf calls with multiline one.

Remove dead assignment in dm_tree_node_add_mirror_target_log

'seg' is never used - remove it.

Remove dead assignment in 'main'

'ret' is never read anywhere - remove it.

Remove unused 'i' in _pv_analyze_mda_raw

'i' is unused in the function - remove it.

Remove dead assignment in lvm2_main

'alias' is not read again in this code path.
Also 'alias' is already equal to 0 in this place.

Remove dead assignment in _lock_for_cluster

'saved_errno' is not read from this initialization and before its
usage is assigned again before _cluster_free_request() call.

Reset vg pointer after release

Set vg to NULL after releasing it as the following memlock() test may
lead to goto for the second call of vg_release() with the already
released vg pointer.

Remove printing of LCK_CACHE

LCK_CACHE is defined as 0x100 so it cannot be passed through
unsigned char parameter - remove it from the sprintf code.

If the LCK_CLUSTER should be printed here - lot of code need
to be reworked - so adding FIXME comment.

Fix check for empty system_dir

Fixing check for zero length system_dir string.

Cleanup remove test for NULL

dm_free is testing NULL itself

Fix "it's" typo to be "its" in lvconvert error message.

Update WHATS_NEW.

Disallow certain lvconvert operations that need to both allocate and free
extents, while physical volumes are specified. Fixes BZ 640051.

All 'size' values of lvm2app properties should be in bytes.
Fix 'seg_size' to return bytes.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add interactive tests for functions to lookup a pv|lv by name|uuid.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

This patch adds helpers to allow users to lookup a lv or pv handle by
uuid (given a vg_t of course).

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

This patch adds helpers to allow users to lookup a lv or pv handle by
name (given a vg_t of course).

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add missing destrustion of cmd_context

Lvm1 fallback code missed to destroy cmd_context in error path.

Fix memory leak in error path

Release allocated path buffer in error path.

Fix resource leak of dlopened pointer

Add missing dlclose in _init_formats() error path.
Use return_0 to print stack trace from the call.

Add missing fclose

Fixing resource leak in _umount().

CHECKME: mountpoints with spaces need to be checked

Move arg_vgnames from local scope

As gcc puts probably all vars on stack this bug was not noticed in runtime.
Patch fixes referencing local scope list variable.

Do not call dm_task_destroy with NULL

Add missing  closedir() - fixes resource leak

Move va_end() so it is also used before error path  return

Move va_end(ap)  so we do not leave with return -1 without calling it.
Remove unneeded ';'

Suppress 'No PV label' message when removing several PVs without mdas.

Fix default /etc/lvm permissions to be 0755. (2.02.66)

post-release

pre-release

Fix _output_field crash from field_id free with DEBUG_MEM. (Phillip Susi)

The _free_vg that is created as a placeholder when reporting segments in pvs
was lacking the (vgmem) pool. We now create that pool. There is at least one
more such VG (_dummy_vg) which is pool-less. I am not sure what is the right
way to go about this, but this is currently necessary to fix a segfault
introduced by using vgmem in the reporter in Dave's lvseg lvm2app patches.

Signed-off-by: Petr Rockai <prockai@redhat.com>

Update interactive tests for lvm2app lvseg and pvseg apis.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add lvm2app function to query pvseg properties.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add the macro and specific 'get' functions for pvsegs.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add a new type and function to lvm2app to enumerate pvsegs.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add lvm2app function to query lvseg properties.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add the macro and specific 'get' functions for lvsegs.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add a new type and function to lvm2app to enumerate lvsegs.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Make value.string const char *, in properties.h, to fix a warning introduced by
the previous patch set.

Add vg_set_property to the interactive lvm2app test program.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Implement lvm_vg_set_property() by calling internal 'set' property function.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add generic infrastructure to internal library to 'set' a property.
Similar to 'get' property internal functions.
Add specific 'set' function for vg_mda_copies.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Remove tag length restriction and allow / = ! : # & characters.

WHATS_NEW FIXME

Support repetition of --addtag and --deltag arguments.
Add infrastructure for specific cmdline arguments to be repeated in groups.
Split the_args cmdline arguments and values into arg_props and arg_values.

Preserve const for char pointer

Keep char pointers 'const'  (introduced with cling commit).

Add date configurable variable DATE

Follow the rule to run every command through variable dereference.
Add a runtime check of translated date to seconds.

fsadm fix for downsize of unmounted fs

Fix for the last commit as $MOUNTED is not only used as bool flag,
but also store mounted location for remount - so parsing output
from mount differently then from /proc/mounts.

Prefix calls of 'tunefs' tools with LANG=C to be sure we always do get
some nonlocalized strings.

Avoid using forced 'resize2fs' for cleanly unmounted filesystems and
run regular fsck -f for this case as required by resize2fs.

'fsadm check' uses date difference for extX filesystems between
the last mount and last check of 'fsck -f' execution and if the mount
was later run 'fsck' with -f so resize2fs is happy and user does not
need to pass '-f' flag.

Update fsadm regresion

Scan also 'mount' output for mounted filesystem.

As util-linux package seems to give all the time different names,
try harder to figure out, where is the given lv possible mounted
and scan /proc/mounts and if not found there, test also 'mount' output.

/dev/dm-xxx
/dev/mapper/vg-lv
/dev/vg/lv

All of them could be used different combination in /proc/mount and mount output.

Patch fixes regression for older systems where new detection code failed to
find valid combination.

Extend cling allocation policy to recognise PV tags (cling_by_tags).
Add allocation/cling_tag_list to lvm.conf.

Regenerate configure with 'autoreconf' for --enable-ocf. (2.02.76)

forgotten to regenerate last time configure.in was updated

post-release

pre-release

Fix include commit and switch to use DISTCLEAN_TARGETS

Fixing warning introduced by 'include make.tmpl' commit.
Produced this warning:

Makefile:29: warning: overriding commands for target `distclean'
../make.tmpl:366: warning: ignoring old commands for target `distclean'

Clarify error messages when activation fails due to activation filter use.

Add OCF support

Updated patch from Florian Haas from Linux-HA project.

User needs to 'configure --enable-ocf' to get file installed
by 'make install' target by default.
User can also use 'make install_ocf' to get only ocf files installed.
With disabled (default) ocf support - no ocf files are installed.

FIXME: ocf installation path needs to be kept in sync with pacemaker.
find better way and possible also better location.

Add given user prefix to make target install_initscripts

Avoid files to be written into the live system if lvm was
configured with different --prefix.

Use initdir for install target path.

Use include make.tmpl

Makes easier to use recursive targets and simplifies Makefile.

lost line

Fix regex optimiser not to ignore RHS of OR nodes in _find_leftmost_common.

Use new status code from fsadm check

Patch updates exec_cmd() and adds 3rd parameter with pointer for
status value, so caller might examine returned status code.
If the passed pointer is NULL, behavior is unmodified.

Patch allows to confinue with lvresize if the failure from fsadm check is
caused by mounted filesystem as many of filesystem resize tools do support
online filesystem resize. (originally user had to use flag '-n' to bypass
this filesystem check)

Add DIAGNOSTICS section to fsadm man page.

Return different status code for fsadm check of mounted filesystem

Return status code 3 for fsadm check of mounted filesystem - used later with
lvresize update patch to better support online filesystem resize.

Also makes a more consistent user interruption and returns status code 2
in this case.

Use dm_strdup/dm_free instead of strdup/free.

Allocate buffer for reporting functions dynamically to support long outputs.
Fix memory leak of field_id in _output_field function.

There's been a patch added recently to use dynamic allocation for metadata
tags buffer to remove the 4k limit (for writing metadata out). However, when
using reporting commands like vgs and lvs, we still need to fix libdm reporting
functions themselves to support such long outputs. So the buffer used in those
reporting functions is dynamic now.

The patch also includes a fix for field_id memory leak which was found in
the _output_field function.

Update VG metadata only once in vgchange when making multiple changes.
Allow independent vgchange arguments to be used together.
(Still more inconsistencies to iron out here.)

Update WHATS_NEW.

Add code to the dmeventd snapshot plugin to automatically unmount snapshots
that have been invalidated.

Alasdair correctly pointed out that if the two closes are concurrent (I haven't
checked, so they *might*), there is still a race possibility with the last
fix. This patch fixes that.

Add a missed test for dmeventd -R.

Update WHATS_NEW.

Fix a double close in clvmd.

The management threads (main_loop, the socket thread) could close a single fd
twice in a row sometimes. At least one other thread can be running at the same
time as the threads doing the double close. That one running thread also
happens to do some IO (namely, open /proc/devices, read from it, close it). If
there was enough "demand" for the local socket, this could happen:

- a connection to clvmd is about to finish, let's say the fd is 13 (it often
  happens to be in my test script, don't ask why)
- the local_sock thread calls close(13)
- the lvm thread calls open("/proc/devices"...) and gets 13
- the main_loop thread calls close(13) [OOPS!]
- new connection arrives, and is accept'd by a (new) local_sock thread
- the accept gives an fd of 13 (since it's the lowest free fd at this point)
- the lvm thread gets around to read from it's /proc/devices handle... 13,
  again
- the lvm thread hangs forever trying to read from the socket instead of
  /proc/devices

Signed-off-by: Petr Rockai <prockai@redhat.com>
Reviewed-by: Milan Broz <mbroz@redhat.com>

Use a more generous timeout for dmeventd in t-lvconvert-repair-dmeventd.

Let's rely on the exit code and do not test for empty output.

Add missing return for NULL passed buffer

Function pull_stateo() checks for NULL 'buf' - but return for this error
path was missing.  cmirror code never calls this function with NULL 'buf',
so this fix has no effect on current code base, but makes clang happier.

Macro uninitialized_var gives warnings in static analysis

Deactivate uninitialized_var() macro for clang static analysis.

Hotfix usage of __builtin_unreachable()

It's quite new feature which is not supported by older compilers.
So until some better macros are introduced into LVM code - hotfix current
compilation problems and compile this code only for __clang__ defining compilers.