Replace free_vg with release_vg

Move the free_vg() to  vg.c  and replace free_vg  with release_vg
and make the _free_vg internal.

Patch is needed for sharing VG in vginfo cache so the release_vg function name
is a better fit here.

Remove INCONSISTENT_VG flag

As this flag could not have been set by the current code - removing it.

Note: because of the wrong code logic this call:

lvmcache_update_vg(correct_vg, correct_vg->status & PRECOMMITTED &
   (inconsistent ? INCONSISTENT_VG : 0));

had always passed '0' - now after flag removal it's passing
PRECOMMITTED flag in - this present functinal change in this patch.

To match the original functionality - 0 had to be always passed.
More testing is needed here.

Fix compiler warning.

Compiler complaining that meta_lv could be used uninitialized.  (Not true
because it is protected by 'clear_metadata'.)  I switched to using 'lv->vg',
as it makes no difference to vg_[write|commit].

If anything bad happens and unlocking fails
(here clvmd crashed in the middle of operation),
lock is not removed from cache - here is one example:

locking/cluster_locking.c:497       Locking VG V_vg_test UN (VG) (0x6)
locking/cluster_locking.c:113   Error writing data to clvmd: Broken pipe
locking/locking.c:399         <backtrace>
locking/locking.c:461         <backtrace>
  Internal error: Volume Group vg_test was not unlocked

Code should always remove lock info from lvmcache and update counters
on unlock, even if unlock fails.

Initialise clvmd locks before lvm context to avoid open descriptor leaks.

(At least /dev/mapper/control is open after lvm context init.)

autoreconf

Remove obsoleted GULM clvmd cluster locking support.

Remove support for the original dm ioctl interface version 1.
Leave the basic support for multiple versions in case we have a new version
in future.

Suppress low-level locking errors and warnings while using --sysinit.

Today, we use "suppress_messages" flag (set internally in init_locking fn based
on 'ignorelockingfailure() && getenv("LVM_SUPPRESS_LOCKING_FAILURE_MESSAGES")'.
This way, we can suppress high level messages like "File-based locking
initialisation failed" or "Internal cluster locking initialisation failed".

However, each locking has its own sequence of initialization steps and these
could log some errors as well. It's quite misleading for the user to see such
errors and warnings if the "--sysinit" is used (and so the ignorelockingfailure
&& LVM_SUPPRESS_LOCKING_FAILURE_MESSAGES environment variable). Errors and
warnings from these intermediary steps should be suppressed as well if requested.

This patch propagates the "suppress_messages" flag deeper into locking init
functions. I've also added these flags for other locking types for consistency,
though it's not actually used for no_locking and readonly_locking.

Remove unused 'origin' variable in lv_remove_single function.

Add missing check for allocation failure

Static analyzer discovered missing check.

Remove unused inconsistent_seqno

Last usage was removed in Petr's commit related to VG mda repair fix
where relaxed check starts to ignore inconsistencies coming from
PVs that are marked MISSING - thus removing unused variable.

Add 'empty' command

Using empty command ';' here makes the code look nicer.

Remove meaningless const type qualifiers on cast type

Static analyzis noticed we do not really need them - so removing.

Remove self assigment which has no effect

Just a tiny code cleanup found by analyzer.

Minor memory leak fix

Defer the test of the function return value after the string memory is released.
Otherwise in this error path the string would present memory leak.
(Thought in this case we are already out of memory...)

Add test for fcntl error in singlenode client code.

Static analyzer noticed this check could be handy.

Remove --force option from lvrename manpage.

Add missing new line in lvrename help text.

Add basic RAID segment type(s) support.

Implementation described in doc/lvm2-raid.txt.

Basic support includes:
- ability to create RAID 1/4/5/6 arrays
- ability to delete RAID arrays
- ability to display RAID arrays
Notable missing features (not included in this patch):
- ability to clean-up/repair failures
- ability to convert RAID segment types
- ability to monitor RAID segment types

Change DEFAULT_UDEV_SYNC to 1 so udev_sync is used even without any config.

This should be set by default! Normally we have "activation/udev_sync = 1"
in lvm.conf (example.conf.in). But if we use lvm2 without any config file
(or without a definition within '--config' option) the DEFAULT_UDEV_SYNC
is used instead. Together with verify_udev_operations=0 (when we rely on
udev fully), this can cause races as the node could be missing when needed.

(See also https://bugzilla.redhat.com/show_bug.cgi?id=723144)

Add support for systemd file descriptor handover in dmeventd.

Systemd preloads file descriptors for us and passes them in for
newly spawned daemon when using on-demand fifo (or socket)
based activation.

This patch adds checks for file descriptors preloaded by
systemd and uses them instead of opening the FIFOs again
to properly support on-demand FIFO-based activation.

(We'll change FIFOs to sockets soon - but still this
part of the code will stay almost the same.)

Add support for new oom killer adjustment interface (oom_score_adj).

The filename to adjust the oom score was changed in 2.6.36.
We should use oom_score_adj instead of oom_adj (which is still
there under /proc, but it's scheduled for removal in August 2012).

New oom_score_adj uses a range from -1000 (OOM_SCORE_ADJ_MIN,
disable oom killing) to 1000 (OOM_SCORE_ADJ_MAX).

autoreconf

Configure and makefile changes for newly added systemd unit files.

Add systemd unit files for dmeventd.

lvmetad: Edit the MISSING_PV flags only after making a "reply" copy of the
metadata, which is then serialised and discarded. This fixes a couple of
outstanding TODO items about handling the MISSING flags correctly.

lvmetad: Check integrity of multiple metadata copies, i.e. ensure that seqno
equality implies metadata equality. Signal error in response to any update
requests that try to overwrite metadata without providing a higher seqno.

lvmetad: A couple of TODOs, and fix a few trivial memory leaks.

Add simple test for read-only table reload suppression

Just a small test for previous libdm commit.

Fix read-only identical table reload supression.

Compare also file size to detect changed config file

Clvmd detects modifed config file before it takes lv_lock.
If the config file is changed rapidly - the change was ignored within
a seocnd ranged.  This patch adds also compare of file size.
So change like some flag for 0 to 1 would pass unnoticed - but
it's quick fix for failing test suite.

FIXME: Implement inotify solution.

lvmetad: Obliterate vg_status by returning the same information from
update_pv_status, saving a dozen lines of code and execution time of one
walkthrough of the PV list.

lvmetad: Fix a possible infinite loop in vg_status.

lvmetad: Robustify update_pv_status and remove an useless lookup.

First stab at making lvmetad-core threadsafe. The current design should allow
very reasonable amount of parallel access, although the hash tables may become
a point of contention under heavy loads. Nevertheless, there should be orders
of magnitude less contention on the hash table locks than we currently have on
block device scanning.

lvmetad: Avoid stale PV -> VG mappings on metadata update.

Optimise PV -> VG lookups by using a UUID (hash) map.

Free up allocated memory before exiting, in lvmetad.

Fix two small (but eventually unbounded) leaks in daemon-server.c.

Can't have a global memory pool in lvmetad (that would constitute an ongoing
memory leak) => remove it (it's been unused anyway).

Make lvmetad also report VGID in reply when adding a PV without MDAs (this
obviously only works for VGs that already had at least some MDA discovered).

Add code to lvmetad's testclient that scans an actual physical device,
effectively emulating a future "pvscan --lvmetad" command.

Make lvmetad report the VG ID and status (complete, partial) in reply to pv_add
requests.

Towards MISSING (PV) flag management in lvmetad.

Make it possible to represent type-correct single-item arrays in config trees.

Add an API to config.h for creating values in the config_tree mempool.

Work out some more details in metadata update in lvmetad.

Remove and unneeded parameter from build_parallel_areas_from_lv()

Fix potential null ptr deref in 'origin_from_cow'

return NULL rather than segfaulting if lv->snapshot is not set

More work on cache maintenance code in lvmetad: keep track of PV status.

Start filling in the core LVMetaD functionality and the corresponding
testclient bits.

Various improvements to the daemon-common code, including automated response
formatting from config trees provided by the daemon implementation.

Improve format_buffer in daemon-shared.c, adding block formatting in addition
to string/integer (this propagates to the *simple* family of request/response
functionality).

Revert the #include changes. Need to fix this at the #include site for now, and
eventually refactor the way we structure #includes in the all of the library.

Slightly refactor the config code to allow better reuse (no functional change).

Add a missing libdevmapper.h #include to lvm-types.h (for dm_list).

Add a couple missing #includes in uuid.h.

Better version of failing 'should' test

where the naming is left completely on lvm.
(Commited code has been different version of test).
So here it should be able to figure out new free name and create a new LV.

Adding test for repair when confusing metadata are found

LVM2 RAID design doc

Fix t-vgreduce-usage to stop relying on the persistent cache not seeing a
device that has been brought back from the dead: this sometimes fails with
clvmd (the cache is updated "too soon"). Instead, force a pvscan and rely on an
up-to-date cache as usual.

post-release

remove unnecessary lvconvert.h

pre-release

Downgrade error message - it isn't strictly an internal error in the
library, and the known cause within lvm2 got fixed.

move doc to doc dir

missing ifdef when new fn unused

 Remove dev name prefix from dmsetup line output if exactly one dev requested.

Ignore activation/verify_udev_operations if dm kernel driver vsn < 4.18.

Fix warning for pvcreate over MD linear.

If MD linear device has set rounding (overload chunk size attribute),
the pvcreate command prints this warning:

  /dev/md0 sysfs attr level not in expected format: linear

Need to snapshot lookup by uuid instead of name in case it's renamed.

Log cookie values when incrementing/decrementing to give clues about races.

Move snapshot removal activation logic into lib/activate.

Move snapshot deactivation logic into lib/activate, fixing the
teardown sequence.  (Previously the snapshot was deactivated while its
origin was active and before its removal was committed to disk, so
restarting after a crash at the point would leave corruption.)

report which dev was not found

Increase timeout for udev settle

For some unknown reason (atm) udev settle takes longer time on recent rawhide.
To pass the test - inrease the settle timeout.

Cope with a PV only discovered missing when creating deptree.

Abort operation if dm_tree_node_add_target_area fails.

Always perform preload logic before suspending - not only in the case when we
have precommitted metadata.  (Necessary to avoid loading tables
while suspend in lvchange --refresh.)

decode cookie flags in debug msgs

Snapshots LVs are never loaded in their own right, only along with their
origin.

Fix conditions using no_merging: only those using lv_is_merging_cow() should
have been converted, not pure lv_is_cow ones.
(Merging has no impact on how the pre-merged cow segment itself is loaded.)

first attempt to fix test

reinstate accidentally-removed lines to fix pvmove again

Report internal error if suspending a device using an already-suspended dev.
This catches the recent pvmove problem trapping I/O between layers.

RAID updates for gdbinit file

Add framework for validation of ioctls.  Doesn't do any checks yet.
dmsetup --checks
libdevmapper: dm_task_enable_checks()
lvm.conf: activation/checks=1

update pv_attr check for new missing attr

When suspending, automatically preload newly-visible existing LVs
Let's find out if this makes things better or worse overall...

Report internal error when parameters are missing on table load

When some target is passing empty parameters to some dm target,
report this as an internal error to better catch some broken
table construction (some mirror conversions seem to be doing
this for now).

use remove -f

teardown stray filesystems too

Use -f to remove device

Since some test may leave devices in suspend mode which would require
carefull order of resume operation - use '-f' to replace them with
error targets

For disable_dev  - when  'error' target is used for open count - treat
return code as ok (|| true) to avoid breaking futher test processing.

Try to force remove dm devices in after test fails.

Try to remove any stray loop devices left behind by earlier aborted tests.

test attempt to remove stray loop devs

Differentiate the request and response format, in daemon/common.

Add age filter to dmsetup udevcomplete_all to minimise concurrency problems.

make udev teardown conditional on using udev