cov: fix memory leak

Reapply 23cc7ddc50e2800a6dc248de897a4c88c1514160 to internal version
of libdm.

fix dev_unset_last_byte after write error

dev_unset_last_byte() must be called while the fd is still valid.
After a write error, dev_unset_last_byte() must be called before
closing the dev and resetting the fd.

In the write error path, dev_unset_last_byte() was being called
after label_scan_invalidate() which meant that it would not unset
the last_byte values.

After a write error, dev_unset_last_byte() is now called in
dev_write_bytes() before label_scan_invalidate(), instead of by
the caller of dev_write_bytes().

In the common case of a successful write, the sequence is still:
dev_set_last_byte(); dev_write_bytes(); dev_unset_last_byte();

Signed-off-by: Zhao Heming <heming.zhao@suse.com>

WHATS_NEW: update

tests: add test of resize of different segtypes

lvextend: fix resizing volumes of different segtype

When resizing 2 volumes like  thin-pool and it's metadata and they
would be of a different type - command would be actually expecting
both LVs being of a same segtype - and would throw an error in
case they are different.

This patch fixes is by setting a new segtype from last segment of
2nd. extented device.

Also it fixes the possible 'percentage' extension setup that
might have been used for 'primary' volume - while the 'secondary'
LV always goes with direct size - as we do not support 'percentage'
setup for them

This affects maily usage of thin-pool where the extension of
thin-pool data size may also lead to extension of metadata size.

tests: add extra settle

To avoid removing, while 'add' might not have been processed yet.
(when emulating reboot in pvmove-restart)

tests: skip unneeded status check

If 'remove' was succesful - we can break loop immediatelly.

daemons: check for non-zero thread_id

Do not call pthread_join if thread_id would be 0.

daemon: better error path handling for shutdown

Report errors for open in better order.
Ensure descriptors are not leaked.

cov: ensure read_ahead is available

Make sure read_ahead pointer is not NULL when quering for RA.

cov: missing checks of syscalls

Check for sigprocmask errors

gcc: avoid declaration shadowing

dev_name is global in device.h

man: adjust 'disks' to 'devices' as used throughout

test: Fix metadata-zero-space with long VG names

tests: be happy with less then 90 percent

Thin metadata evolve between kernel version, so it's not always
precisely predictible its usage - so let's met test happy,
when it gets bellow 90%.

tests: skip test if scsi_debug is not available

tests: slowdown delay of raid

Slowdown 'delay' more.

tests: conversion only of exclusive lv

We can 'cache' only exclusively active LV in cluster.

tests: avoid checking command result in cluster

When running cluster test with clvmd, the actual 'monitoring'
happens in cluster - so the 'already monitored' message
is also logged within clvmd code and the command cannot
see such effect.

clvmd was incapable to report this information back to command
so it cannot be displayed this way.

Add 'lvs -o+seg_monitor' validation which also works in clustered mode.

WHATS_NEW: update

activation: drop removed declaration

Seems this function has been removed long time ago with:
3e781ea446bb7ddc9a494cbba6b6104dd51c3910

tests: reduce space requirements

Test well runs on smaller test machines.

raid: drop internal error

Fix some internal error reports and debug trace returns

manip: optimize lvs_using_lv

Instead of checking all LVs in a VG - do just a direct copy of LVs
from the existing list ->segs_using_thin_lv.

TODO: maybe it could be better to expose seg_list to /tools...

mirror: remove unused code

mirror: directly activate updated mirror

mirror: fix leg splitting

Enhance lv_info with lv_info_with_name_check.
This 'variant' not only check existance if UUID in DM table
but also compares its  DM name  whether it's matching expected LV name.
Otherwise activation may 'skip' activation with rename in case the
DM UUID already exists, just device is different name.

This change make fairly easier manipulation with i.e. detached mirror
leg which ATM is using same UUID - just the LV name have been changed.

Used code was not able to run 'activation' (and do a rename) and just
skipped the call. So the code used to do a workaround and 'tried'
to deactivate such LV firts - this however work only in non-clvmd case,
as cluster was not having the lock for deactivated LV.

With this extended lv_info code will run 'activation' and will
synchronize the name to match expected LV name.

Patch extends _lv_info() with new paramter 'with_name_check',
which is later translated into 'name_check' argument for
_info_run() which in case of name mismatch evaluates the
check as if device does not exists.

Such call is only used in one place _lv_activate() which then
let activation run.  All other invocation of _info() calls
are left intact.

TODO: fix mirror table manipulation (and raid)....

Experimental VDO lvmdbusd support

lvmdbustest.py: Use local data instead of fetching

Avoid making more dbus calls to get information we already have.  This
also avoids us getting an error where a dbus object representation is
being deleted by another process while we are trying to gather information
about it across the wire.

lvmdbustest.py: Improve concurrent test handling

Filter out LVs too, so that we can run more than 1 instance of the
unit test at the same time.

lvmdbustest.py: Add tests for LV interface

Add tests for all the different LV types with the standard LV dbus
interface.  These tests shook out a couple of new bugs.

lvmdbusd: Debug msg. improvements.

lvmdbusd: Fix model inconsistency when LV loses interface

When a LV loses an interface it ends up getting removed and recreated.
This happens after the VGs have been processed and updated.  Thus when
this happens we need to re-check the VGs.

lvmdbusd: Bug fix for activate/deactivate

Prevent the daemon from stalling when it gets stuck on a y/n prompt.

testlib.py: Add interface instance vars.

testlib.py: WS corrections

lvmdbustest.py: Add basic vdo test

testlib.py: Correct dbus signature verification

This allows us to fully verify introspection data matches what we are
getting.

lvmdbusd: VDO Pool LV representation

VDO pool LVs are represented by a new dbus interface VgVdo.  Currently
the interface only has additional VDO properties, but when the
ability to support additional LV creation is added we can add a method
to the interface.

lvmdbustest.py: Create common func. _create_cache_lv

lvmdbustest.py: Add nested helper function major_minor

lvmdbustest.py: WS corrections

lvmdbusd: Prevent running --nojson with VDO support

lvmdbustest.py: Add cache LV rename test

lvmdbusd: Add VgVdo class & assoc. interface

When VDO support is available we will create VG object instances
which will allow the API user to create VDO pool LVs.

lvmdbusd: Add cfg.vdo_support

Will be used to add vdo interfaces on demand.

lvmdbustest.py: Remove 2 TODOs

This issue has been resolved, sizes > 2**32-1 not supported.

lvmdbustest.py: Add func. _pv_scan

lvmdbustest.py: Use existing _create_lv

lvmdbustest.py: Add func. _create_thin_lv

lvmdbustest.py: Add func. _all_pv_object_paths

This is needed in a number of places.

lvmdbustest.py: Add function for lv path check

lvmdbusdtest.py: Use common function for tag add

lvmdbustest.py: Remove duplicate setup code

Remove the same copy & pasted code which simply creates a VG to
use.

man lvmvdo: Correct spellings

man: Include '_vdata' as reserved name

lvmdbusd: Add check for reserved name '_vdata'

Added for vdo support.

lvmdbustest.py: Add blurb about scan_lvs = 1

When developing and testing on a local system, to get the unit
test to pass the test_nesting test, editing the test conf will achieve
the success too.

lvmdbusd: Add d and D to type map for VolumeType

These were added for vdo integration.

lvmdbusd: Remove use of tmp variables

We can use tuple expansion from the command handler functions
directly.

lvmdbusd: Remove duplicate error handling code

vg, lv, pv code had the same function for handling command execution.
Move to utility function and abstract the difference.

[bcache] bcache_invalidate_fd, only remove prefixes on success.

[bcache] reverse earlier patch.

It broke some unit tests, for v. little benefit

[bcache] pass up the error from io_submit rather than using generic -EIO

Author: Heming Zhao

[bcache] add unit test

abort-forces-read

[label] Use bcache_abort_fd() to ensure blocks are no longer in the cache.

The return value from bcache_invalidate_fd() was not being checked.

So I've introduced a little function, _invalidate_fd() that always
calls bcache_abort_fd() if the write fails.

[bcache] add bcache_abort()

This gives us a way to cope with write failures.

WHATS_NEW

tests: explicit testing of thin snapshot

Check merging of old snapshot of thin LV.

snapshot: use single merging sequence

The resume of 'released' 'COW' should preceed the resume of origin.
The fact we need to do the sequence differently for merge was
cause by bugs fixed in 2 previous commits - so we no longer need
to recognize 'merging' and we should always go with single
sequence.

The importance of this order is - to properly remove  '-real' device
from origin LV. When COW is activated as 2nd. '-real' device is
kept in table as it cannot be removed during 1st. resume of origin,
and later activation of COW LV no longer builds tree associated
with origin LV.

snapshot: fix checking of merged thin volume

When merging of thin snapshot is taking place, the origin target will
be of thin type.

snapshot: correctly check device id of merged thin

When checking device id of a thin device that is just being
merged - the snapshot actually could have been already finished
which means  '-real' suffix for the LV is already gone and just LV
is there - so check explicitely for this condition and use
correct UUID for this case.

lvmlockd: fix cachevol locking

When a cachevol LV is attached, have the LV keep it's lock
allocated.  The lock on the cachevol won't be used while
it's attached.  When the cachevol is split a new lock does
not need to be allocated.  (Applies to cachevol usage by
both dm-cache and dm-writecache.)

tests: lvmlockd-lv-types handle new cpool renaming

man lvmthin: change wording about mounting xfs

post-release

pre-release

cov: Fix memory leak

build: make generate

improve command definition matching using type

When a user includes "--type foo" in a command, only
look at command definitions with matching type, as
opposed to using matching/mismatching --type as a
vote for/against a given command def.  This means a
command with --type foo will prioritize a command def
with --type foo over other command defs that have
more matching options but an unmatching type.  This
makes it more likely that a closely matching command
def will be recommended.

cache: _cpool is protected suffix now

tests: enable with 6.2.1

Package vdo-6.2.2.18  goes with  kvdo version 6.2.1
Hopefully will get fixed soon.
Meanwhile to get test coverage...

dm: fix compilation of dmsetup

Fix: 889c88e9dab33195efc4dc0400a4b0aaa3383948
Use correct enum DM_DEVICE_GET_TARGET_VERSION.

tests: more suffixes

gcc: older version can't see udev is always set

vgchange: remove bogus option restriction

for -A with -a

cachevol: use cachepool code for metadata size

Based on a more detailed calculation, but because of
extent size rounding, the final result is about the
same.

man: lvmcache naming updates

gcc: avoid warning: declaration of xxx shadows a global declaration

Fix some gcc complaints again shadowing global declarations

gcc: all paths will set ret

Set success on common path.
Fixes random failure on writecache uncaching path.

gcc: older compiler is happier with this initilizer

tests: used cache_pool adds _cpool

cache: use _cpool for used cache-pools

When LV gets cached and uses cache-pool - such cache-pool
will now get _cpool suffix automatically.

Thus 'Pool' column for cached LV will now show either _cvol
or _cpool LV.

cache: drop _cpool suffix from unused cache-pool

Drop _cpool prefix if present and cache-pool is going to be unused.

lvconvert: use new functions

lvm-string: add drop_lvname_suffix

Internal function to drop suffix out of lvname.

lv_manip: add lv_uniq_rename_update

Add function to rename LV to either passed name or if
the name is already in use, generate new lvol% name.

pvck: dump metadata search

Improve the implementation of extracting all text metadata
copies from the metadata area.  Use this for the existing
metadata_all dump option.

Add a new metadata_search dump option which does not use
lvm headers to find metadata, but looks in standard
locations.  This is useful if headers are damaged and
can't be used to locate metadata.

Adding '-v' to metadata_all or metadata_search will add
the description and creation_time to the printed list of
metadata instances that are found.

tests: cachevol needs FORMAT2

FORMAT2 is mandatory with cachevols...

tests: skip when raid is not present