Fix lv_is_visible to handle virtual origin.

Snapshot is visible if its origin is marked visible,
or if the origin is virtual.

Introduce link_lv_to_vg and unlink_lv_from_vg functions.

link_lv_to_vg and unlink_lv_from_vg are the only functions
for adding/removing logical volume from volume group.

Only these function should manipulate with vg->lvs list.

Tidy format1 import LV function.

Later patch initializes lv->vg after the LV structure is prepared,
so pass through cmd context and do not use vg->cmd here.
Also move LV id calculation (which uses lv->vg too).

Also properly free memory pool if operation fails.

Remove vg->lv_count and use counter function.

This should not cause problems but simplifies code a lot.

(the volumes_count is merged and renamed with lvs_visible
function by following patch.)

Fix snapshot segment import to not use duplicate segments & replace.

The snapshot segment (snapshotX) is created twice
during the text metadata segment processing.

This can cause temporary violation of max_lv count.

Simplify the code, snapshot segment is properly initialized
in init_snapshot_seg function now and do not need to be replaced
by vg_add_snapshot call.

The vg_add_snapshot() is now usefull only for adding new
snapshot and it shares the same initialization function.

The snapshot name is always generated, name paramater can be
removed from function call.

Update test-utils to cope with ":" in device names and allow configurable names.

Do not query nonexistent devices for readahead.

Remove NON_BLOCKING lock flag from tools and set a policy to auto-set.

As a simplification to the tools and further liblvm, this patch pushes
the setting of NON_BLOCKING lock flag inside the lock_vol() call.
The policy we set is if any existing VGs are currently locked, we
set the NON_BLOCKING flag.

Should be no functional change.

better variable name for snapshot counting

Remove snapshot_count from VG and use function instead.

Fix first_seg() call for empty segment list.

The seg variable is temporary variable for list iterator,
code cannot expect that after iteration it remains NULL
(it contains non-NULL pointer here id list is empty).

Patch fixes first_seg function so it now correctly returns NULL
for empty segment list.

Update t-read-ahead.sh to validate lv_read_ahead and lv_kernel_read_ahead.

- check default values
- check active/inactive values

Fix previous commit (scripts/Makefile targets order)

Introduce lvm2_install target.

Buildsystem support device-mapper only install,
but generic install tagret includes both dm+lvm2.

For distribution which uses separate install_device-mapper,
there is no way how to install lvm2 only
(so after installing lvm2 for packaging purposes
built system must remove installed device-mapper files).

Fix it by allowing lvm2_install target, similarily like
install_cluster for clvmd.

(install = install_device-mapper + install_lvm2)

Fix device-mapper static build targets.

dmsetup.static is not built and cleaned properly
if running only device-mapper install/build.

Do not use generic install if running install_device-mapper.

It propagates into subdirs which includes DSOs which requires
lvm2 build (only install_device-mapper should propagate there).

Add test for seg_start, seg_count, seg_start_pe

Update WHATS_NEW wrt a recent patch.

Update tests for region_size.

Add tests to check pv_mda_size and vg_mda_size.

Add tests to check vgcreate --physicalextentsize and field vg_extent_size.

Validate chunksize and originsize for snapshots.

Fix PV datalign when for values starting prior to MDA area.

The dataalign value must always be aligned according
to MDA area.
The currect code checks if calculated value collides with
MDA area but not if the value is so small that it is
located before MDA starts.

Unfortunatelly there can be also MDA in the end of the device.

The patch adds simple check to avoid this miscalculation.
Patch expects that first MDA always starts on <= pagesize boundary
(this is true for all allowed label sector parameters).

Use zalloc in initialization of device manager (to zero pvmove mirror count).

Update columns.h comment.

Fixup whitespace.

Fixup whitespace.

Use liblvm.so instead of .a

Fix error path in vg_make_handle().

Enter the error condition if either of the allocations fail, and
don't use dm_pool_zalloc if dm_pool_create fails.

Fix wrong arg in lv_is_virtual_origin call while creating snapshots.

Add sparse devices: lvcreate -s --virtualoriginsize (hidden zero origin).
Add lvs origin_size field.
Fix linux configure --enable-debug to exclude -O2.

Still a few rough edges, but hopefully usable now:
  lvcreate -s vg1 -L 100M --virtualoriginsize 1T

Fix linux configure --enable-debug to exclude -O2.

Fix pool leak in lvmcache_read_vg error path.
(Introduced in previous patches.)

Fix segfault when using -U, -G and -M options in dmsetup.

Avoid scanning non-PV devices in functional tests, otherwise lvconvert --repair
breaks for some reason -- possibly needs investagation, but this should fix it
in the meantime.

Implement, test and document (first iteration of) lvconvert --repair.

Refuse adding missing PVs in create_pv_list in toollib when allocatable_only is
requested.

A more thorough PV equality test (that also copes better with MISSING_PVs) in
_is_mirror_image_removable.

Do not include MISSING_PVs in allocation maps.

Update columns.h comment to describe macro args.

fix typo reported by "A. Costa" <agcosta@gis.net>

Fix vgreduce --removemissing failure exit code.

Remove some trailing whitespace so git won't complain.

Clean a lot of extra extra whitespaces.

Fix remote metadata backup for clvmd

Run backup of metadata on remote nodes in the
same place like local node - when calling backup().

Introduce backup_locally() which calls only
local backup if needed.

Remote backup is now trigerred by LCK_VG_BACKUP flag
combination (special VG lock).

This lock type will call check_current_backup()
(including backup_locally() call) and updates
metadata on all nodes.

(Patch fixes non-functional remote backup,
current call during VG lock never triggers.)

Alloc PV internal structure from VG mempool if possible.

- Updating cluster log with latest code changes/bug fixes before
  altering to new kernel structures.

Move metadata backup call after vg_commit.

The backup() call store metadata from memory.

But in cluster backup() call performs
remote nodes metadata backup and it reads data from disk.

For metadata backup consistency,
patch moves all backup() calls after vg_commit.

(Moreover, some tools already do that this way.)

Tidy lv_hash[_lock] use inside clvmd.

 - Rename unlock_all to destroy_lvhash,
this function is called in cluster shutdown
unlocks everything and clean up allocated info space.

 - Tidy lv_hash_lock use
.
Except adding free(lvi) in lv_has destructror
there is no functional change.

Fix pvseg report for orphan PVs and other devices.

If user requests report attribute from PVSEG type
and PV is orphan (or all devices is set), the report
is empty.

Try for example (when only orphan PV are present)
 #pvs
 #pvs -o +devices
# pvs /dev/sdb1
  PV         VG   Fmt  Attr PSize  PFree
    /dev/sdb1       lvm2 --   46.58G 46.58G
# pvs -o +devices /dev/sdb1
(no output)

The problem is caused by empty pv->segments list.

Fix it by providing fake segment here (similar to fake structures
in _pvsegs_sub_single() calls.

Fix pvs -a for segmented output

 # pvs -a -o devices
   Volume group name (null) has invalid characters
   Skipping volume group (null)

...
_pvsegs_sub_single creates fake vg, we need to check
that pv is real here.

Add libdevmapper to test/api building.

Add MMC device type to filters.

Fix dmsetup.static build.

Properly release VG memory pool in all CLI tools.

Properly release VG memory pool in metadata manipulation code.

Properly release VG memory pool in archiver code.

Properly release VG memory pool in activation code and clvmd.

Introduce memory pool per volume group.

Since now, all code reading volume group is responsible for releasing
the memory allocated by calling vg_release(vg).
(For simplicity of use, vg_releae can be called for vg == NULL,
the same logic like free(NULL)).

Also providing simple macro for unlocking & releasing in one step,
tools usualy uses this approach.

The global memory pool (cmd->mem) should be used only for global
physical volume operations.

This patch have to be applied with all subsequent patches to complete
memory pool per vg logic.

Using separate memory pool has quite bit memory saving impact when
using large VGs, this is mainly needed when we have to use
preallocated and locked memory (and should not overflow from that
memory space).

Helper function to catch memory pool leaks.

Properly copy the whole pv structure for later use.

The all_pvs list, used in vg_read, should make its own private
copy of pv structures, otherwise (when vg will use its own pool)
it will point to released memory pool.
The same applies for get_pvs() call.

Patch adds pv_list copy helper and adds explicit memory pool
parameter into _copy_pv.

(Please note that all these helper functions cannot guarantee that
vg related fields are valid - proper vg read & lock must be used
if it is requested.)

Always return exit error status when locking of volume group fails.

Fix mirror log convert validation question.

Avoid referencing files from DESTDIR during build process

Do not create some dm and lvm static librarie when they are not requested
by configure option
Add dependency for static dmeventd library

Enable use of cached metadata for pvs & pvdisplay.

Currently PV commands, which performs full device scan, repeatly
re-reads PVs and scans for all devices.

This behaviour can lead to OOM for large VG.

This patch allows using internal metadata cache for pvs & pvdisplay,
so the commands scan the PVs only once.
(We have to use VG_GLOBAL otherwise cache is invalidated on every
VG unlock in process_single PV call.)

Add missing 'device-mapper' internal subdir build dependency.

Use pv from newly read_vg to avoid possible use of not initialized memory.

If the vg in process_each_segment_in_pv is NULL, the pv struct
can be incomplete (for example lv_segs are not copied in get_pvs()
call).

We need use the new pv from just read-in volume group.

(The same code is in pvdisplay already.)

Fix memory pool leak.

Call the alloc_destory call always after finishing operation
with handle otherwise it will leak a memory pool.

Also fix return code in lv_extend.

Fix whitespace in t-mdata-strings.sh

Author: Dave Wysochanski <dwysocha@redhat.com>

Save and restore the previous logging level when log level is changed.

Fix error message when archive initialization fails.

Fix debug pool grow object to properly support delta=0

(It prints garbage for some reports)

Allocate new pv->vg_name from pool, it can be destroyed later.

(The mempool rename will be used later by vg private mempools)

Do not use pointer from released memory pool (cmd->cmd_line).

Make sure clvmd-corosync releases the lockspace when it exits.

patch from Xinwei Hu

Add some more special chars for device name test.

fix some issues when compiling with -D DEBUG_POOL

- fix compilation issues
- fix wrong pool object maipulation (lvm dumpconfig triggers assert)
- second iteration in loop _log_parallel_areas operates on non-existing object

Fix segfault for vgcfgrestore on VG with missing PVs.

Block SIGINT & SIGTERM in clvmd subthreads so they don't delay shutdown.
Patch from Xinwei Hu, Thanks

Restore log_suppress state when metadata backup file is up-to-date.

Author: Takahiro Yasui <tyasui@redhat.com>

Remove old metadata backup file after renaming vg.

Author: Takahiro Yasui <tyasui@redhat.com>

Fix size and error message of memory allocation at backup initialization.

Author: Takahiro Yasui <tyasui@redhat.com>

Fix error message when adding metadata directory to internal list fails.

Author: Takahiro Yasui <tyasui@redhat.com>

Missed file from previous checkin.

Some extra (paranoid) checks on dev_is_{md,swap} result.

Detect and wipe swap signatures in pvcreate.

Fix some clean rules, fix previous distclean checkin.

In libdm/Makefile.in, we need to cleanup the symlink properly.
Adding to CLEAN_TARGETS seemed like the simplest way to do this
in the current build framework.  We could redo dependencies for
VERSIONED_SHLIB, but for now just add to CLEAN_TARGETS.

For scripts/Makefile.in, we should be adding to DISTCLEAN_TARGETS.
The generic rule in make.tmpl.in takes care of the cleanup.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>
Author: Takahiro Yasui <tyasui@redhat.com>

Fix some distclean rules.

By gnu coding stds, 'distclean' should remove all files generated
by ./configure in addition to what 'clean' does.

Author: Takahiro Yasui <tyasui@redhat.com>

Fix warning in previous commit.

Fix lv_count when manipulating with snapshots and max_lv is set.

Patch fixes these problems:
 - during the snapshot creation process, it needs create 2 LVs,
   one is cow, second becomes snapshot.
   If the code fails in vg_add_snapshot, code lvcreate will not remove
   LV cow volume.

 - if max_lv is set and VG contains snapshot, it can happen that
   during the activation lv_count is temporarily increased over the limit
   and VG metadata are not properly processed
   see https://bugzilla.redhat.com/show_bug.cgi?id=490298

 - vgcfgrestore alows restore with max_lv set to lower valuer that actual
   LV count. This later leads to situation that max_lv is completely ignored.

 - vgck doesn't call vg_validate(). It should at least try:-)

Signed-off-by: Milan Broz <mbroz@redhat.com>

Remove unnecessary includes in lvm_base.c.

We would like to declare our handles pv_t, vg_t, and lv_t in
the external library header lvm.h.  However, these are already
defined in metadata-exported.h for the use of some of the
in-progress liblvm APIs.  Thus, we cannot both define
them in lvm.h and include metadata-exported.h in the external
library C files.  We could use preprocessor tricks (#ifndef)
but for now we just avoid the include.

Fix clvmd build after liblvm commit.

Fix error messages when PV uuid or pe_start reading fails.

Author: Takahiro Yasui <tyasui@redhat.com>
Committer: Dave Wysochanski <dwysocha@redhat.com>

Add missing liblvm/.exported_symbols

Fix liblvm version symlink.

Remove unnecessary linker flags for liblvm.

Add DSO generation for new LVM application library.

Fix test/api/test build.

Move lvm.h from lib to liblvm.

Add new liblvm build directory and move lvm_base.c.

The original liblvm.a has been moved to liblvm-internal.a.
We now use liblvm.a for the new application library and build
it inside liblvm directory.

Change dependencies so tools depend on liblvm application library,
and application library depends on liblvm internal.