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.

Rename liblvm to liblvm-internal.

Make preparation for using liblvm for new application library.

Fix unlocks in clvmd-corosync.

The DLM unlock returns EUNLOCK in the lksb on success, not 0

Fix possible increasing in memory allocation if refreshing_context
(for example when CLVMD_CMD_LOCK_VG for is called during vgscan).

If clvmd calls LV lock, it calls
    /* clean the pool for another command */
dm_pool_empty(cmd->mem);
to clean up memory pool after command.

Unfortunately, do_refresh_cache() do not call this
and because during it operation it allocates some memory,
the pool increases.

Also do_refresh_cache should use lvm_lock
(it manipulates with lvm internal data).

The same applies for lvm_backup command.

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

post-release

vgname_from_mda now tries to parse for vgname even
if rlocn not defined (there is no metadata area).

In most cases it fails in validate_name(),
unfortunately there are situatuions, when
validate_name is ok and later code fails with
checksum error.

Reproducer:

# dd if=/dev/zero of=/dev/loop0

# pvcreate --metadatasize 637k /dev/loop0
  Physical volume "/dev/loop0" successfully created

# pvs /dev/loop0
  /dev/loop0: Checksum error
      PV         VG   Fmt  Attr PSize PFree
        /dev/loop0      lvm2 --   1.00M 1.00M

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

pre-release

Fix last check-ins: seg can be NULL.

.

Attempt cleanup in child before execing new binary in exec_cmd()

.

fsadm cleanups & release prep

autoreconf  (using fedora 10 for this now)

pe_start can be overwritten in VG metadata, so it is not PV label-only field.

Try to avoid full rescan if label scan is enough.

Use pkgconfig to obtain corosync library details during configuration. (kabi)

Fix error returns in clvmd-corosync interface to DLM.

Thanks to Xinwei Hu for spotting this.

Update vgchange and vgmknodes man pages to include --refresh.

Author: Dave Wysochanski <dwysocha@redhat.com>

Fixed bug where lvresize option -t was not properly passed to fsadm.
Using argv[] list in exec_cmd() to allow more params for external commands.
Fsadm does not allow checking mounted filesystem.
Fsadm no longer accepts 'any other key' as 'no' answer to y/n.
Fsadm improved handling of command line options.

Fix include/.symlinks for lvm2.h to lvm.h renaming

Author: Thomas Woerner <twoerner@redhat.com>

Added files lib/lvm.h and lib/lvm_base.c:
New structure lvm (used as an alias to cmd_context), new type definition lvm_t
for the lvm handle. Added functions lvm_create, lvm_destroy and
lvm_reload_config using the new handle.

Modified test/api/test.c:
Use new lvm.h header file and lvm_t handle.

Removed lib/lvm2.h

Author: Thomas Woerner <twoerner@redhat.com>

Fix validation of dataalignment value introduced in previous commit.

Move tools/version.h to lib/misc/lvm-version.h.
Split LVM_VERSION into MAJOR, MINOR, PATCHLEVEL, RELEASE and RELEASE_DATE.

Add system_dir parameter to create_toolcontext() and call it system_dir
everywhere for consistency.

Comment out pvcreate pagesize alignment test for now.

Add --dataalignment to pvcreate to specify alignment of data area.  (mbroz)

This patch is not fully tested and leaves some related bugs unfixed.

Intended behaviour of the code now:

  pe_start in the lvm2 format PV label header is set only by pvcreate (or
vgconvert -M2) and then preserved in *all* operations thereafter.

  In some specialist cases, after the PV is added to a VG, the pe_start
field in the VG metadata may hold a different value and if so, it
overrides the other one for as long as the PV is in such a VG.

  Currently, the field storing the size of the data area in the PV label
header always holds 0.  As it only has meaning in the context of a
volume group, it is calculated whenever the PV is added to a VG (and can
be derived from extent_size and pe_count in the VG metadata).

Fix interrupt unblocking after vgcreate, for example: drop_cached_metadata()
previously left _vg_lock_count incremented.
Other locks are always held during drop_cached_metadata() so there's no
need to increment+decrement it.

Provide da and mda locations in debug message when writing text format label.

Mention the restriction on file descriptors at invocation on the lvm man page.

In testsuite's not.c, print a notice when program dies of a fatal signal.

Fix test output collection in harness.c.

Index cached vgmetadata by vgid not vgname to cope with duplicate vgnames.  (dwyso)

Fix lvm.static build dependence (to properly propagate changes in libdevmapper).

Only fail when tests have failed, but not when they have been just skipped.

If kernel supports only one dm major number, use in place of any supplied.
No longer require kernel and metadata major numbers to match.

Re-implement the test harness in C. This lets us pass through signals and
trigger proper test teardown upon harness interrupt or termination. I also
tweaked the output somewhat while I was at it...

Add a fully-functional get_cluster_name() to clvmd corosync interface.

Remove duplicate cpg_initialize from clvmd startup.

Add option to /etc/sysconfig/cluster to select cluster type for clvmd.

Allow clvmd to start up if its lockspace already exists.

Separate PV label attributes which do not need parse metadata when reporting.

When reporting explicitly label attributes (pv_uuid for example), we do not
need to read metadata.

This patch separate the label fileds and removes scan_vgs_for_pvs
in process_each_pv() if not needed.

(There should be no user visible change in output.)

Remove external dependency on the 'cut' command in fsadm

Fix pvs segfault when pv mda attributes requested for not available PV.

add support for ext4 resize in fsadm

Move locking_type reading inside init_locking().

No functional change.

Rename get_vgs() to get_vgnames() and clarify related error messages.

get_vgs() really returns a list of vgnames.  In the future we will use
get_vgs() to return a list of vg structures, similar to get_pvs().

Allow clvmd to be built with all cluster managers & select one on cmdline.

Mention --with-clvmd=corosync in ./configure

Add as-yet-unused vg_read_error() and vg_might_exist().  (mornfall)

Introduce as-yet-unused replacement vg_read() and vg_read_for_update()
functions.  (mornfall)

 Replace internal vg_check_status() implementation.  (mornfall)

Properly enforce cluster locking in as-yet-unused _vg_lock_and_read. (mornfall)

Introduce as-yet-unused _vg_lock_and_read() and associated header file
definitions.

Rename vg_read() to vg_read_internal(). (mornfall)

post-release

and another one missing

Add stuff people missed.

pre-release

Add a corosync/DLM cluster service to clvmd.

It's not integrated in the configure system yet though.

Use tools.h in for lvm-static.