Thin forgotten initialisation of pointer to NULL

Since the code may go in error path, set to defined NULL.

Tiny cleanup

Just remove double braces from conditions when they are not really needed.
(So it doesn't look like an assignment and comparison).

Thin cleanup

Reorder condition so the code is better readable (and shorter).

Drop hack in segtype reporting

Since striped name function knows when to report 'linear' instead of
'stripe' type name - drop it from this place.

This fixes problem when reporting segtype e.g. for thin-pool which
is also using area_count=1 to store thin data device reference.

It also returns properly strduped memory instead of badly casted const char*.

Thin extend lvcreate man page about snapshot support

.

improve comment

Preserve exclusive activation of cluster mirror when converting.

This patch to the suspend code - like the similar change for resume -
queries the lock mode of a cluster volume and records whether it is active
exclusively.  This is necessary for suspend due to the possibility of
preloading targets.  Failure to check to exclusivity causes the cluster target
of an exclusively activated mirror to be used when converting - rather than
the single machine target.

Drop unimplemented

Drop unused variable

Thin indent update

Thin disable snapshot creation when pool is over the threshold.

Since snapshot needs to suspend origin - it might lead to pool userspace
deadlock (as the pool will wait for new space in case it would be overfilled,
but dmeventd would not be able to resize it, as the lvcreate operation would
have kept the VG lock.)
To minimize the risk of such scenario - we prevent to create new snapshot
in case we are over the threshold - but beware, there is still small timewindow,
so keep threshold at some reasonable level!

Thin add new display field for  lvs

New field Data% is able to display info about
thin_pool, thin, snapshot and has generic meaning here.

Simple Time/Host field are here to display host and time creation.

Add support to keep info about creation time and host for each LV

Basic support to keep info when the LV was created.
Host and time is stored into LV mda section.

FIXME: Current version doesn't support configurable string via lvm.conf
and used fixed version strftime "%Y-%m-%d %T %z".

Thin add function to read thin volume percent

This value returns percentage of 'mapped' size compared with total LV size.
(Without passed seg pointer it return highest mapped size - but it's
not used yet.)

Thin updated support for thin pool percent

Support to check also for metadata percent
(By checking whether seg pointer is set)

Thin rename seg var  pool_metadata_lv to metadata_lv

Better fits the code.

Thin handle empty thin volume case

Report both values as 0 in case the volume is unused.

Thin use consistentely metadata

Do not shortcut to 'meta' and stay with 'metadata'
Also matches kernel doc for dm API then.

Thin rename local static

Use '_' for local const char.

Rename internal macro to match signess

Since _read_int64 called dm_config_get_uint64, rename it to
less confusing _read_uint64.

Update for gcc old-style

Add dmsetup 'wipe_table' to replace table with one that uses error target.

Move dm_task_set_newname from libdm-iface.c to libdm-common.c

Beef up the lvmetad code with more functionality and a bunch of bugfixes. There
used to be a few mis-ordered memory accesses (release and access in the next
block). Fix that set_flag could have sometimes corrupted the flags being
modified.

A few issues with metadata tracking are sorted out as well now, and there are
only a few problems remaining before we can integrate lvmetad, mostly on the
client side:

- metadata areas need to be tracked in lvmetad (most likely to be addressed
  through an extension of metadata, meaning no special support in lvmetad would
  be needed)
- non-udev scanning code needs to be taught about telling lvmetad about device
  disappearance (pvscan most importantly)
- this last item also needs to mesh with metadata inconsistencies and
  suddenly-incomplete volume groups (aux disable_dev in tests); udev-based
  scanning should address this separately and more elegantly

Fix a boundary condition in read_buffer in daemon-shared.c.

Fix prototypes for daemon_send_simple (const char).

Unfortunately, blank lines are sometimes produced by config serializer, and
this interferes with their role as message separator in the lvmetad
protocol. Switch to using "##" on an otherwise blank line as a separator.

Fix a couple of resource leaks in daemon-common server code -- filehandles and
unjoined threads were leaked for each connection.

Make error message hit when preallocated memlock memory exceeded clearer.

Show read-only activation in display tools.

Missing const.

"warning: assignment discards 'const' qualifier..."

Add activation/read_only_volume_list to override LV permission in metadata.

Add activation/read_only_volume_list to override LV permission in metadata.

Give priority to emcpower devices with duplicate PVIDs.

Support different device name types on output of dmsetup deps, ls and info -c command.

Add 'blkdevname' and 'blkdevs_used' field to dmsetup info -c -o.
Add 'blkdevname' option to dmsetup ls --tree to see block device names.
Add '-o options' to dmsetup deps and ls to select device name type on output.

Add dm_device_get_name to get map name or block device name for given devno.

This is accomplished by reading associated sysfs information. For a dm device,
this is /sys/dev/block/major:minor/dm/name (supported in kernel version >= 2.6.29,
for older kernels, the behaviour is the same as for non-dm devices).

For a non-dm device, this is a readlink on /sys/dev/block/major:minor, e.g.
  /sys/dev/block/253:0 --> ../../devices/virtual/block/dm-0.
The last component of the path is a proper kernel name (block device name).

One can request to read only kernel names by setting the 'prefer_kernel_name'
argument if needed.

Add dm_uuid_prefix/dm_set_uuid_prefix for non-lvm users to override hard-coded
LVM- prefix.

Try harder not to leave stray empty devices around (locally or remotely) when
reverting changes after failures while there are inactive tables.

Check for error code in _adjust_policy_params

If error is detected in _adjust_policy_params, break further command processing.

Minor oldstyle warning update

Improve readahead in dmsetup man page

Use sysfs to set/get of read-ahead

If we know major:minor number of device (which is known after resume) we will
try to use  sysfs to set/get read ahead parameters of device.
This avoid potential problem of blocking commands like 'dmsetup info' awaiting
for device being usable for open/close - i.e. overfilled thin pool may block
such command.

Support rounding of percentage upward

We want to keep this logic -
when LV is extend - extend the LV by at least given amount,
when LV is reduced - reduce the LV by at most given amount.

So for this the rounding needs to be used.
Current logic which seems to satisfy give rule is to round up all
extent values for LV resize upward except for values with '-' sign
that are round downward.

This patch also fixes the problem when  lvextend --use-polices tried
to extend LV the by i.e. 20% - but the resulting 20% were smaller
the extent size thus before this patch no extension happened.

Use new dmeventd_lvm2_command function in dmeventd plugins.

For snapshot, prepare whole command in front into private buffer.
Add also some missing '\n' for syslog messages.
For raid and mirror only convert creation of command line string.
This should avoid any unbound growth of mempool for dm_split_names.

Thin use helper function

Fix some minor outstading issue from thin plugin introduction -
Call dmeventd_lvm2_exit() in failpath for registration.
Add some missing '\n' in syslog messages.

Add helper function dmeventd_lvm2_command().

Since this code is in all plugins - create a common helper function.

Updated documentation for dmeventd.

Update man page style.
Mention raid and thin plugins.
Update help text printed by command to match man page.

Thin man page update

Thin update test case

Use thin-pool instead of thin_pool type.

Exported functions from thin plugin

Drop extra stat before open of device

Since the !(dev->flags & DEV_REGULAR) code path just called
dev_name_confirmed() which has just called 'stat()' inside,
remove duplicate second stat() call here.

Do not lstat common path prefix

When both path have identical prefix i.e. /dev/disk/by-id
skip  2 x lstat() for /dev  /dev/disk /dev/disk/by-id
and directly lstat() only different part of the path.

Reduces amount of lstat calls on system with lots of devices.

Add common initialization code for struct device

Avoid duplicate code and add _dev_init() where all common
member values are initialized.

Always zalloc device structure

Since there is zalloc behind the macro, put 'z' into the name.
Make the 'use_malloc' code path also using zalloc() call,
so it also give zeroed area.

Thin automatic policy based extension

Thin add lv_thin_pool_percent

Thin add dev_manager_thin_pool_percent

dev manager function to read percent info from thin pool.

Thin add dmeventd support

This is basic version with still few unresolved issue mainly in case,
when the pool resize is failing.

Fix missing thread list manipulation

For manipulation with thread list to avoid race with timeout thread,
take also _timeout_mutex.

Check lv pointer for NULL before derefence.

Thin remove unused code

Thin move layer suffix into local static const

Thin rename internal thin pool segment

Use matching name as kernel target - useful when function like
_percent is using this for validation.

Add Thin API for parsing thin status

Add dm_get_status_thin_pool and dm_get_status_thin functions to
parse 'params' argument which is received via dm_get_next_target.

Returns filed structure allocated from given mempool.

Allow empty strings for description and creation_host config fields

Add dm_config_find_str_allow_empty

Support empty string values.

Reinstate support for format1 snapshots, but issue deprecated warning.

I anticipate removing support for snapshots with lvm1-formatted metadata in a
future release.

Add a unit test for config cascade (overlay). Needs expanding though.

Fix up lvmetad for the minor API change in dm_config_create.

Make a cleaner split between config tree and config file functionality. Move
the latter out of libdm.

Use also normalized output for vgchange command in lvm2-monitor init script.

Cleanup test compile warning

Add some declaration and cast to cleanup gcc warnings.
Add missing dm_config_destroy() to cleanup pool leak report.

Fix a compiler warning.

Use a more canonic regex escaping in the default lvm.conf file for testing.

Move dm_config_write out of libdm, back to lib/config, as config_write.

Add a shell test for config parsing and merging (lvm dumpconfig).

Autoreconf.

Add a unit test for dm_config_clone_node.

Start a rudimentary unit test for the dm_config_* interface.

In the dm_config_*get_* functions, make the actual value retrieval optional
(useful for just checking that a given key is of a given type).

Use pkg-config to look for CUnit (needed for systems where it's not on the
standard include path).

Nowadays, lvm-version.h is generated from lvm-version.h.in, and lives in the
build directory, not the source one -> fix test/shell/000-basic.sh.

update FIXMEs

When dropping the VG metadata from the cache during a VG revert, we must
reset the 'precomitted' flag, just as we'd clear if we next did a 'commit'.

Only use built-in stack size in clvmd - ignore lvm.conf.

Test for LCK_CLUSTER_VG directly in args[0].
Drop unused LCK_LOCAL from debug msg.

Don't pass LCK_LOCAL to clvmd - it's unused.
Pass LCK_CLUSTER_VG in args[0] instead of args[1].

Update comments.  CLUSTER_VG belongs firmly to args[0].

reinstate !first_time check
(recovery from first_time failure would need different code)

tag uninit var

Fix lvm2-monitor init script to use normalized output when using vgs.

Add policy based automated repair of RAID logical volumes

The RAID plug-in for dmeventd now calls 'lvconvert --repair' to address failures
of devices in a RAID logical volume.  The action taken can be either to "warn"
or "allocate" a new device from any spares that may be available in the
volume group.  The action is designated by setting 'raid_fault_policy' in
lvm.conf - the default being "warn".

Fix FIXME and comment :-)

Switch locking bits to match RHEL5 version.

FIXME:
There is a problem with overloaded bit 0x80 in locking flag,
the bit flags array must be extended or changed.

update old comment

Don't allow two images to be split and tracked from a RAID LV at one time

Also, don't allow a splitmirror operation on a RAID LV that is already tracking
a split, unless the operation is to stop the tracking and complete the split.
Example:
~> lvconvert --splitmirrors 1 --trackchanges vg/lv /dev/sdc1
# Now tracking changes - image can be merged back or split-off for good
~> lvconvert --splitmirrors 1 -n new_name vg/lv /dev/sdc1
# ^ Completes split ^

If a split is performed on a RAID that is tracking an already split image and
PVs are provided, we must ensure that
 1) the already split LV is represented in the PVs
 2) we are careful to split only the tracked image

  Don't allow size change of RAID LV that is tracking changes for a split image
  Don't allow size change of RAID sub-LVs independently

Do not allow users to change the name of RAID sub-LVs or the name of the
RAID LV if it is tracking changes for a split image.

WHATS_NEW for previous commit.

Do not allow users to change permissions on RAID sub-LVs.