Fix lvcreate processing of %PVS argument.

- fix missing unlocking of VG
lvcreate -l 100%PVS -n lv1 vg_test
  Please specify physical volume(s) with %PVS
  Internal error: Volume Group vg_test was not unlocked

- if no PVS specified, use all available

Fix segfault if %PVS in lvresize without PVs list.

minor code comment update

Remove old, and now incorrect, vgextend man page description of pvcreate options.

Tidy some uses of arg_count and introduce arg_is_set.

Update for outnl and indent functions

Export functions out_inc_indent(), out_dec_indent() for creating
indented metadata lines.

Macro outnl() is using exported out_newline() instead of direct
call f->fn(), that required the visibility of the internal
struct formatter.

Add fflush for the case the log is redirected to the buffered file.

Without this patch it have not been obvious, why the application
waits on the stdin as the prompt might be still buffered in memory.

Fix hash lookup segfault when keys compared are different lengths.

Rudimentary support for running testsuite in a builddir != srcdir situation.

Rename validate_vg_create_params to vgcreate_params_validate.

Rename fill_vg_create_params to vgcreate_params_set_from_args.

Rename fill_vg_create_params to vgcreate_params_set_from_args and remove
vg_name parameter from function (caller must set before calling function).

Add vgcreate_params_set_defaults().

Add function to set default vgcreate parameters based on existing VG or
internal defaults.  Should be no functional change.

Rename pvcreate_params processing functions to better match <object><action>.

Rename fill_default_pvcreate_params to pvcreate_params_set_defaults.
Rename pvcreate_validate_restore_params to pvcreate_restore_params_validate.
Rename pvcreate_validate_params to pvcreate_params_validate.

More cleanup in udev rules:

 - add copyright notice for 10-dm.rules.in,

 - set DM_UDEV_DISABLE_{DISK, OTHER}_RULES_FLAG in 11-dm-lvm.rules directly
   for inappropriate and non-top-level subdevices in case we use older kernels
   where DM_COOKIE is not used (and therefore there are no flags passed from
   the LVM process itself). This applies for older kernels (version < 2.6.31),

 - remove unnecessary filters in 95-dm-notify.rules - the DM_COOKIE env var
   itself is set for change/remove udev events and for DM devices only so
   there's no need to double-check this.

Update vgsplit to call vg_set_clustered().

Update vgcreate to call vg_set_clustered().

Add vg_set_clustered() - move logic from vgchange.

Similar to other vg_set_* functions, we create a vg_set_clustered() function
which does a few checks and sets a flag.  This is where we check for
any limitations of clusters.

Add vg_mda_count library function.

Add to vgcreate and vgextend nightly tests.

 Insert some missing stack macros into activation code.

Recognise DRBD device part and handle it similar to MD devices.

The DRBD uses underlying device so code should prefer top
device if duplicate is found.

Patch also introduce
        dev_subsystem_part_major and dev_subsytem_name
functions to easily handle all these replication susbystems
and not hardcode md_major call.

See https://bugzilla.redhat.com/show_bug.cgi?id=530881
for full problem description.

post-release

.

add copyright lines

pre-release

Cleanup of previous commit with latest changes in udev support code.

Document --all option in man pages, cleanup {pv|vg|lv}{s|display} man pages.

Option --all is only partially documented currently, so document in all
commands.  Also make {pv|vg|lv}{display|s} man pages consistent with help
output.  Remove ununsed 'disk_ARG' parameter.  Leave --trustcache out of
the man page output.  Update --units argument to show all possible units.

Several changes to udev support code:

 - we have these levels when the udev rules are processed:
   10-dm.rules --> [11-dm-<subsystem>.rules] --> [12-dm-permissions.rules] -->
   13-dm-disk.rules --> [...all the other foreign rules...] --> 95-dm-notify.rules

 - each level can be disabled now by
   DM_UDEV_DISABLE_{DM, SUBSYSTEM, DISK, OTHER}_RULES_FLAG

 - add DM_UDEV_DISABLE_DM_RULES_FLAG to disable 10-dm.rules

 - add DM_UDEV_DISABLE_OTHER_RULES_FLAG to disable all the other (non-dm) rules.
   We cutoff these rules by using the 'last_rule', so this one should really be
   used with great care and in well-founded situations. We use this for lvm's
   hidden and layer devices now.

 - add a parameter for add_dev_node, rm_dev_node and rename_dev_node so it's
   possible to switch on/off udev checks

 - use DM_UDEV_DISABLE_DM_RULES_FLAG and DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG
   if there's no cookie set and we have resume, remove and rename ioctl.
   This could happen when someone uses the libdevmapper that is compiled with
   udev_sync but the software does not make use of it. This way we can switch
   off the rules and fallback to libdevmapper node creation so there's no
   udev/libdevmapper race.

Rename 11-lvm.rules to 11-dm-lvm.rules and 12-dm-disk.rules to 13-dm-disk.rules.

Update lvcreate/lvconvert man pages to explain PhysicalVolume parameter.

Addresses rhbz 500177.

Permit snapshots of mirrors.  (brassow)

Minor fix to vgcreate/vgextend man pages for PHYSICAL DEVICE OPTIONS.

Cleanup mimagetmp LV if allocation fails for new lvconvert mimage.

Fix clvmd segfault when refresh_toolcontext fails.   (brassow, bz 506986)

Remember to clear 'global lock held during cache refresh' state after use.

Add manpage entry for dmsetup udevflags.

Use udev flags in the rules and cleanup the rules:
 - remove default permissions set in 95-dm-notify.rules (and add a hint in 12-dm-permissions.rules to set it by the user directly)
 - add multipath DM_ACTION=="PATH_FAILED" filter
 - remove unnecessary filters in the headers of the rules (we can simply use DM_UDEV_RULES_VSN instead)
 - fix symlink priorities in /dev/disk/ (snapshot volumes have low priority for FS UUID symlinks so it will not overwrite symlinks for the origin)

Use udev flags support in LVM and provide dm_tree_add_new_dev_with_udev_flags wrapper for dm_tree_add_new_dev.

Add udev flags support in libdevmapper and provide 'dmsetup udevflags' command to decode them.

.

Only announce mirror monitoring to syslog after initialisation succeeds.

Fix the "const discarded" warnings introduced in the unknown segment patch.

Clarify message:
"  Mirror status: 1/2 images failed."

Half of an image?  Half of the images?  No, 1 of 2 images.

Handle metadata with unknown segment types more gracefully.

I saw this in a bug report:
[root@xxxx-01 ~]# lvconvert -m 1 --corelog VG/cmirror
  Unable to convert the log of inactive cluster mirror cmirror

I've tried to clean-up the message a little more, so the name
of the mirror stands out more while preserving the sense that
it's not a problem with the specific device, but the fact that
it is inactive that is causing the problem.

New msg:
  Unable to convert the log of an inactive cluster mirror, cmirror

configure.in: don't set default file owner and group

Per discussion on lvm-devel mailing list and part of debian patch set,
don't set defaults for owner and group, since nobody seems to use them, and
still allow override.

Attempt to build dmeventd.static.

More build cleanups (linker parameter ordering).

Disable realtime support by default.

This code is used only in a under development feature and it's not used
anywhere in the tree.

Allow to build it conditionally.

Make clvmd return 0 on success rather than 1.

Merge Debian patch 05 debian: keep libdm-abi consistent.

This appears to be the only user visible feature that can change libdm ABI
at build time.

Thanks to Bastian Blank for the patch.

Extra libs must be appended, not defined first.

linking parms missing from o->so conversion.

Cleanup comment and some whitespace.

Add --pvmetadatacopies as a synonym for --metadatacopies in various commands.

Going forward, we would like to allow users to specify the total
number of metadatacopies in a VG rather than on a per-PV basis.  In
order to facilitate that, introduce --pvmetadatacopes to replace
--metadatacopies everywhere.  We still allow --metadatacopies for
pv commands, but require --pvmetadatacopies for vg commands.
Eventually we will introduce --vgmetadatacopies.  Once we do that,
we should either deprecate --metadatacopies or make it a synonym based
on the command (pvmetadatacopies for pv commands, and vgmetadatacopies
for vg commands).  The latter option would likely just require a simple
'strncpy' check against cmd->command->name to qualify the merge_synonym
call.

Update nightly tests to cover the pvmetadatacopies synonym.

Note that this patch is the result of an eariler review comment for
the implicit pvcreate patches.  Should apply cleanly on top of the
implicit pvcreate patches (I applied after patch 10/10 in that series).

NOTE: This patch will require --pvmetadatacopies for vgconvert as
--metadatacopies is no longer accepted.

Add --pvmetadatacopies as a synonym for --metadatacopies in various commands.

Going forward, we would like to allow users to specify the total
number of metadatacopies in a VG rather than on a per-PV basis.  In
order to facilitate that, introduce --pvmetadatacopes to replace
--metadatacopies everywhere.  We still allow --metadatacopies for
pv commands, but require --pvmetadatacopies for vg commands.
Eventually we will introduce --vgmetadatacopies.  Once we do that,
we should either deprecate --metadatacopies or make it a synonym based
on the command (pvmetadatacopies for pv commands, and vgmetadatacopies
for vg commands).  The latter option would likely just require a simple
'strncpy' check against cmd->command->name to qualify the merge_synonym
call.

Update nightly tests to cover the pvmetadatacopies synonym.

Note that this patch is the result of an eariler review comment for
the implicit pvcreate patches.  Should apply cleanly on top of the
implicit pvcreate patches (I applied after patch 10/10 in that series).

NOTE: This patch will require --pvmetadatacopies for vgconvert as
--metadatacopies is no longer accepted.

Update help messages for vgextend / vgcreate to mention PHYSICAL DEVICE OPTIONS.

Add implicit tests for implicit pvcreate during vgcreate and vgextend.

Update tests and WHATS_NEW for implicit pvcreate support.

Update vgcreate and vgextend man pages to clarify implicit pvcreate.

vgcreate and vgextend now implicitly call pvcreate on devices not
currently initialized for LVM use.  Previously these commands would
fail with an error, so clarify the new behavior in the man page.

Add implicit pvcreate support to vgcreate and vgextend.

Adds implicit pvcreate support when calling vgcreate or vgextend with
device paths that are not yet PVs.  This changes the behavior of vgcreate
and vgextend from failing with an error message to implicitly pvcreating.

Move pvcreate_validate_params into toollib to allow calling from mutiple tools.

For implicit pvcreate support, we need to call this from vgcreate and vgextend,
so move it into toollib.

Refactor vgcreate/vgextend validation of vgname/pvname(s).

Decrement argc and increment argv in a consistent way to allow for later
code-sharing.  Should be no functional change.

Refactor pvcreate - split pvcreate_validate_params into recovery/non-recovery.

Split pvcreate_validate_params into recovery and non-recovery parameters.
This is necessary so we can call the non-recovery validate function from
vgextend / vgcreate.  Note in the pvcreate tool case, we must call the
recovery validation function first (see treatment of pe_start and --zero),
and that we add a call to fill_default_pvcreate_params before the validation
functions.

Allow calling fill_default_pvcreate_params from tools.

We need defaults for pvcreate_params at a higher level - this will
allow us to use a common function from the tools to take defaults,
then fill in any non-defaults from the commandline.

Future patches will refactor vgcreate/vgextend to call this function
if one or more pvcreate parameters are given on the commandline.

Add pvcreate_params to vg_extend.

Another refactoring for implicit pvcreate support.  We need to get
the pvcreate parameters somehow to the vg_extend routine.  Options
seemed to be:
1. Attach the parameters to struct volume_group.  I personally
did not like this idea in most cases, though one could make an
agrument why it might be ok at least for some of the parameters
(e.g. metadatacopies).
2. Pass them in to the extend routine.  This second route seemed
to be the best approach given the constraints.

Future patches will parse the command line and fill in the actual
values for the pvcreate_single call.
Should be no functional change.

Add pvcreate_params to vg_extend_single_pv.

Should be no functional change.  If this parameter is set to NULL, just fail
the extend if the device is not already a PV.  If non-NULL, try pvcreate_single
before failing.  Note that pvcreate_single() handles the log_error in case
of failure so we just return 0 if pvcreate_single() fails.

Refactor vg_extend - add vg_extend_single_pv.

Simple refactor to setup future changes related to implicit pvcreates.
Should be no functional change.

More makefile cleaning up and fixing.  (gentoo)

Correct example.conf to indicate that lvm2 not lvm1 is the default format.
Remove an unused stray LVM1_SUPPORT ifdef.

Only include selinux libs in libdevmapper.pc when selinux build enabled.

Allow for a build directory separate from the source.

Update distclean target for rename clogd to cmirrord. (2.02.52)

Remove daemons/clogd from distclean dirs (replaced with cmirrord)

Run distclean also for daemons/cmirrord

Add LCK_CONVERT flag I missed on the last checkin.

Stop clvmd from automatically doing lock conversions. Now, if a lock
is granted at one mode and an attempt to convert it wthout the LCK_CONVERT
flag set then it will return errno=EBUSY.

This fixes a pretty bad bug in which an LV could be activated exclusively on
one node and lvchange -ay on another would convert it to shared!

It might break some things in other areas, but I doubt it.

Add percent_range to copy_percent too.

Introduce percent_range_t and centralise snapshot full/mirror in-sync checks.

Make poll_mirror_progress a function that can be replaced.

Factor out poll_mirror_progress and introduce progress_t.

Update nightly tests to deal with unit changes.

Now uppercase letters imply Si units, so use lowercase everywhere.
We could stay with uppercase, but then we'd have to deal with rounding, etc.
Also, some output / error messages change slightly (instead of "GB" we're
now saying "GiB").
One test enhancement might be to add some new tests for the units changes
but for now let's just get the test back to passing.

Distinguish between powers of 1000 and powers of 1024 in unit suffixes.

Just a cleanup from previous commit. We don't need pvname local var in _activate_lvs_in_vg anymore...

look up pvmove by pvmove LV when PVMOVE flag is set

Don't attempt to restart pvmoves when deactivating LVs in vgchange.
Restart lvconverts in vgchange by sharing lv_spawn_background_polling.

Generalise polldaemon code by changing mirror-specific variable names.

Don't attempt to deactivate an LV if any of its snapshots are in use.

Return fail if lv_deactivate did not removed device from kernel.

lv_deactivate now returns always success, because tree deactivation
functions (see dm_tree_deactivate_children) always returns success.

Because code should return failure in lv_deactivate at least,
fix it by checking for device existence after real deactivation call.

(After discussion this was prefered solution to dm tree function rewrite
which affects snapshots and mirrors.)

Trivial cleanup to lvcreate man page - use virtualsize in example.

virtualsize replaced virtualoriginsize but the example was never updated.

Provide alternative implementation of obsolete siginterrupt().

Consolidate LV allocation into alloc_lv().

Treat input units of both 's' and 'S' as 512-byte sectors.  (2.02.49)

's' and 'S' are special suffixes representing sectors and are always 512 bytes,
regardless of whether you're using powers of 1000 or 1024.

Add global/si_unit_consistency to enable cleaned-up use of units in output.
Add configure --enable-units-compat to set si_unit_consistency off by default.

Use standard output units for 'PE Size' and 'Stripe size' in pv/lvdisplay.

post-release

Fix log fn prototype.

missing dm_snprintf

pre-release

pre-release

ensure dm_strdup succeeds

remove unused var & rename fn