pre-release

pre-release

Cosmetic change use return macro with implicit stack

Set missing header define

Fix leak of hash table

Minor leak on command initialization.

Instrument code that pointer are already released

Set pointers to NULL since on the function exit they are no longer valid.

Set to a defined value vars used after error path

Static analyzer noticed this vars are used even when error is reported
back thus their state is undefined - set to 0 for this case.

Test for uname result

in fail path initialize to 0.

Fix missing dmt destructor

Also always initialize maj,min,patchlevel when success is returned.

Free allocated segment type in error path

Limit alignment to 32bit values

to get the same behavior on 32/64 machines.

Ensure whole info is initialised

Since _create_dm_tree_node is copying whole structure,
make sure all members are initialized.

Fix resource leak of file handle

Introduces when added dm_device_get_name.
Close file handle in all error paths.

Check for correctness of uint64 value if exists

Add breaks for cases

Ensure reply struct has all fields defined

Reply is returned by value.

Check and print perror for syscalls

& -> &&

Fix compilation with disabled devmapper

During release preparation things has changed, so making sure
we are compilable with --disable-devmapper.

Thin read  percent info from layered device

Since we only need layered device - check for tpool device presence,
so user doesn't need to activate thin pool explicitely for percent info.

Rename  origin_only to more generic use_layer flag

Since now we have more layered devices i.e. thin volumes - support
selection of layer via flag.

Clean var declarations to the front of the function

Update verbose lvs  to print metadata_percent info

Update lvs  -o fields in WHATS_NEW.

Thin clear stacked message for thin pool

Before removing thin pool LV always make sure, stacked message
for previous run are cleared - but allow to remove any
device that should have been created
(i.e. creation of snapshot failed - so the message for snapshot creation
may be replaced with delete message within unfinished transaction).

Also commit messages after lv remove - so free space is released in pool.

Make commented out code more obvious

Thin skip activation when there are no thin message

If the list with thin messages is empty, do not touch thin pool device.

Thin correct activation order

When the message is passed only in resume path the order needs
to be corrected.

Thin use  suspend/resume_lv_origin

Use origin_only support for thin volume when thin snapshot is created.

Thin use origin_only for thin pools as well

Extend the usage of origin_only flag to allow resume of thin pool LV
(when it's active) to pass only the messages.

origin_only flag will skip detection of already resumed tree for thin_pool,
so we do not need to suspend the tree and we just send messages.

Thin add support for origin_only suspend of thin volumes

Pass in the origin_only flag also for thin volumes - but curently the flag
is not used to its best.

FIXME: achieve the state where only  thin volume snapshot origin is
suspended without its childrens -  let's explore whether this may
happen automatically inside libdm (might be generic for other targets).
So the code would not need to annotate the node for this.

Thin add messages only for activation tree

Extend lv_activate_opts with bool flag to know for which purpose
dtree is created - and add message only for activation tree
(since that's the only place that may send them).

Extend validation check for thin snapshot creation and test whether
active snapshot origin is suspended before its snapshot is created
(useful in recover scenarios) -  in this case also detect, whether
transaction has been already completed and avoid such suspend check
failure in that case.

Thin check for lv_thin_pool_percent error status

Check has been missing.

Update lv_info whats_new

lv_info using -real layer only for origin_only LV

If the origin_only flag is passed for non lv_is_origin LVs,
the extension is not added.

Thin volumes may also use origin_only flag.

Thin prevent removal of its data and metadata LVs

LVs cannot be removed while there are linked to thin pool.
(Gives better error message, than validation).

Thin fix transaction_id incrementation and code refactoring

Add pool_has_message and use it in attach_pool_message.
Also update header to make more obvious which segment type is
expected as parameter.
Rename  'read_only' to  'no_update' (no auto update transaction_id)
to better fit how it's used.
Fix problem when there was only one stacked message replaced with delete
message that caused unwanted transaction_id increase.

Comment cleanups

Move comment where it applies and remove unused attribe when the var
is actually used.

Thin dependency scan support

Go through pool_lv and metadata_lv LVs when doing recursive scan.

Thin add lv_thin_pool_transaction_id

Easy function to get transaction_id status value.

Thin send messages on activation resume code path

Using PRELOAD part would lead to problems when the problem
would happen before vg_write and vg_commit.
Also this change is necessary for snapshot creation sequence.

Caller is still entitled to reference an LV that's unlinked, so don't
tamper with struct contents.

Use suspend|resume_origin_only when up-converting RAID LVs, as mirrors do.

Failure to do so results in "Performing unsafe table load while X device(s) are
known to be suspended" errors.  While fixing the problem in this way works and
is consistent with the way the mirror segment type does it, it would be nice
to find a solution that uses the generic suspend/resume calls.

Also included in this check-in are additions to the test suite that perform
conversions on RAID LVs under a snapshot.  These tests are disabled for the
time being due to a kernel bug that is yet to be tracked down.

Properly show LV removal message.

(Fix regression in commit 6e181ba96dc1f806afc2d7fe825a8d3dd8093052)

Use chunk_size consistently for thin_pool within LVM.

Fix the way RAID meta LVs are added to the dependency tree.

Similar to the "mirror" segment type's log device, _add_dev_to_dtree should
be called and not _add_lv_to_dtree when adding metadata sub-LVs to the deptree.
Since _add_lv_to_dtree was being called, 'origin_only' could be set if a
snapshot sits on top of the RAID device.  This would cause the actual device
that needed to be added to be skipped in favor of the non-existant device,
"<foo>-real".

Fix metad help.

add const

Reorder fns in libdm-deptree.
Tweak dm_config interface and remove FIXMEs.

Add CLVMD_FLAG_REMOTE to skip processing on local node.

Attempt to improve clustered 'lvchange -aey' behaviour to try local node before
remote nodes and address some existing anomalies.

Prompt if request is made to remove a snapshot whose "Merge failed".

Allow removal of an invalid snapshot that was to be merged on next activation.

Don't allow a user to merge an invalid snapshot.

Use m and M lv_attr to indicate that a snapshot merge failed in lvs.

snapshot (m)erge failed, suspended snapshot (M)erge failed

Differentiate between snapshot status of "Invalid" and "Merge failed".

Lookup snapshot usage percent of origin when a snapshot is merging.

Update lvdisplay to show more info about thin LVs

Reformat name and path how the LV is represented with lvm1 compatible option,
to switch to the old way - which had number of  problem - i.e. many links
do not exist - since for private devices we are not creating them.
Add more info about thin pools and volumes.

Minor cleanup

Simplier and more readable char pointer math.

Thin until proper vgcfgrestore for thin is implementad, disable restore.

Since it may probably do more harm to leave it enabled - add extra test
for presence of thin volumes in VG, and in this case disable restore.

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.