pre-release

autoconf

make generate

tests: use egrep to check for message

Seems we may get different '' around option on different nixes.

tests: aux defines fsadm path

raid: use log_error

Turn log_print into log_error for error path.

lvconvert: return error without conversion

lvconvert parameters not causing a conversion (i.e. no type,
number of stripes, stripesize or regionsize changes) will
remove any allocated reshape space in which case the command
returns success.  If reshape space does not exist though,
return error.

tests: test for new supported commands

New tests for:

lvconvert --regionsize | --swapmetadata | --startpoll |
          --mergesnapshot | mergethin

tests: restore some test changes

Restore existing tests and separate new tests
for new options later.

Keep coverage for existing syntax

Reverts bits from commits:

a801b92b2c81213dddccc9f9a09794208f92bfc9.
d71aaca07b2c14049130ab368221510c65236af7
fa2a728a395639a32707bb75e6bb4197b02769de
1e2420bca85da9a37570871cd70192e9ae831786
46abc28a480069b3e8b2c5c7c3448ffa25ec17ba
0e3e611a137a264a343335d18d500fce7af8e0f7

fsadm: support configurable full path

Just like with other tools lvm2 is using allow to define
fully configurable path.

Default is selected by $PREFIX/sbin/fsadm

thin: properly check for status for max sizes metadata

When metadata LV size was over DM_THIN_MAX_METADATA_SIZE sectors,
the info() routine was incorrectly trying to match bigger size,
while we do never pass any bigger device.

Fixing a case, where lvs should be displaying status for metadata
LV with 16GB size.

cmdline: ensure command struct is defined

Make it clear to Coverity cmd will be defined.

libdm: set min,maj

Reset derefered pointers instead of pointers.

dmfilemapd: free path

Cleanup resources before exit.

dmfilemapd: keep 1 byte for \0

Coverity noticed the buffer should have 1 byte for \0.

commands: improve error message for option rules

When a cmd def RULE fails because of a disallowed
combination of options, improve the error message
to show the option combination, not just the options
that broke the rule.

WHATS_NEW: add items since last release

commands: add rule for lvconvert commands that create pools

For an lvconvert command that creates a thin pool or cache pool,
--poolmetadata cannot be used with --readahead, --stripes or --stripesize.

WHATS_NEW: add recent RAID entries

lvconvert: better message on --regionsize

Enhance message on "lvconvert --regionsize size RaidLV".
in case the regionsize does not change and return error.

WHATS_NEW_DM: add recent dmsetup and dmfilemapd entries

Typo: change loose to lose

loose (v): set free; release
lose (v) : be deprived of or cease to have or retain

We 'lose' redundancy or 'lose' meaning.

lvconvert: fix failing valid regionsize change

Reshape check failed when regionsize changed and current raid type
was provided with no other change requested (stripes or stripesize).

E.g. "lvconvert --type raid6 --regionsize 256K" on a raid6 LV
with != 256K regionsize.

Enable --type in test script.

lvconvert: add segment type raid10_near

Introducing this alias for "raid10", avoid allocating
reshape space when converting between them.

Resolves: rhbz1441347

raid: handle insufficent PVs on takeover from striped/raid0

Remove any newly allocated sub LV (pair) remnants in case
allocation fails due to lag of (parallel) free PV space
and keep initial raid type.

Resolves: rhbz1438013

man lvmraid: wrap long lines

Break lines in examples to keep line length under 80.

raid_manip: fix typo warning message

man lvmraid: fix line wrapping in examples

man lvmraid: use same style as generated pages

lvconvert: require different LV for pool

When convering an LV to a cache or thin LV, require
that an LV being converted to cache pool or thin pool
at the same time be a different LV.

lvconvert: prevent duplicate use of metadata lv

The LV being processed cannot be used as the
pool metadata LV.

man lvmraid: improve text layout

lvconvert: validate vg name prefix for pool metadata name

man lvcreate/lvconvert: minor clean up rewording

commands: check required option value when matching command

A command def can include a specific constant option value,
but the value was not being checked for optional opts.

e.g. this is an incorrect command and does not match any
command defs:

lvconvert --type cache --cachepool vg/lv

However, it was mistakely being matched to this cmd def,
where the required options match, but the optional options
do not:

lonvert --cachepool LV_linear_striped_raid_cachepool
OO: --type cache-pool, ...

The optional options were mistakely considered matching
because 'cache' and 'cache-pool' were not being compared.

signals: fix comparison flaw

commit 9a689fb8f0a6b1675a4e819f4837bf2e0269832f
fixed SIGINT blocking but introduced a comparision
flaw to sigint_restore().

Related: rhbz1440766

signals: fix SIGINT blocking flaw causing inconsistent metadata

SIGINT isn't blocked properly after a sigint_allow(),
sigint_restore() cycle leading to illicit interruptable
metadata updates.  These can leave corrupted metadata behind.

Issues addressed in this commit:

sigint_allow() fails to set _oldmasked[] members properly due
to an offset by one bug on indexing the members of the array.

It bails out prematurely comparing to MAX_SIGINTS causing nesting
depths to be one less than MAX_SIGINTS.  Fix the comparision.

Correct the related comparison flaw in sigint_restore().

Initialize all sig_atomic_t variables consequently.

Resolves: rhbz1440766

lvconvert: cleanup prompting

lvconvert: fixe conversion message

When selecting a convenience RAID type only display
the selected type when it changed.  Display proper
current raid type when prompting.

lvconvert: raid1 -> linear prompt

Avoid 2 prompts when downconverting raid1 to linear
(related commit 0f65d7ec3a7325165af1a9f722cb760fd363e932).

lvconvert: avoid error message on raid1 -> raid4 conversion

Avoid error message
"Logical Volume *_rimage_0 already exists in volume group,,,"
on takeover conversion from a 2-legged raid1 to raid4
(aiming to reshape it adding images).

Resolves: rhbz1439398

lvconvert: prompt on raid1 image changes

Don't change resilience of raid1 LVs without --yes.

Adjust respective tests.

raid: reload on removing images

Requesting _raid_remove_images() to commit the
metadata missed to reload the origin causing a
kernel takeover error converting a 2-legged raid1
(with previously removed images) to raid5.

man: lvmraid.7 raid6_n_6 fix

raid: tidying

lvconvert: allow --type with --regionsize

Allow the combination of both arguments keeping
the raid level but changing the regionssize
(e.g. "lvconvert --type raid1 --regionsize 1M RaidLV"
 on an existing raid1 LV).

Resolves: rhbz1438396

lvchange/lvconvert: fix missing lvmlockd LV locks

lvchange/lvconvert operations that do not require the LV
to already be active need to acquire a transient LV lock
before changing/converting the LV to ensure the LV is not
active on another host.  If the LV is already active,
a persistent LV lock already exists on the host and the
transient LV lock request is a no-op.

Some lvmlockd locks in lvchange were lost in the cmd def
changes.  The lvmlockd locks in lvconvert seem to have
been missed from the start.

lvchange: tidy switch code in _lvchange_properties_single()

lvchange: fix missing return value

A return value from lvchange_persistent_cmd() was missed
in commit 1c41898c07ad750820fb39770355fded8e9b030a

lvmconf: fix systemd unit existence check for lvmconf --services --startstopservices

We have to unset the LoadState variable from previous use when we check
for systemd unit state. We use this variable to check if systemd services
are loaded or not and if it is loaded, we issue systemctl commands to
enable/disable and start/stop the service. We don't issue these commands
if the unit is not loaded to avoid error messages which may confuse users.

commands: fix misspelling of option value

(The cmd def parser really needs to catch mistakes
like this as it did when it was external.)

lvchange: fix --poll value when set from option

The actual value specified by the --poll y|n option was not
being used.  The way the --poll value is used is hidden
through an indirection where the value is stored in a
global variable at the start of the command, and then the
value is read from there later.  Setting the global
variable early in the command had been lost with the
cmd def changes.

vgchange/lvchange: fix poll and monitor use

Fill in some gaps where old versions of lvm allowed
--poll and --monitor in combination with other operations,
but those combinations had been lost since the cmd def work.
(The new cmd def code also added some combinations that
had been missed by the old code.)

Changes:

lvchange --activate: add poll and monitor options, and
                     add calls to them in implementation.

lvchange --refresh: add monitor option (poll already there),
                    and call to monitor in implementation.

lvchange <metadata ops>: add poll and monitor options, and
                         add calls to them in implementation.

vgchange <metadata ops>: add poll option (call to poll
                         already in implementation).

vgchange --refresh: remove monitor option (not used by code)

lvchange --persistent y: add poll and monitor options, and
                         add calls to them, and to activate
                         in the implementation. (Making it
                         match the main lvchange metadata
                         command.)

Summary of current usage:

lvchange --activate: monitor, poll
vgchange --activate: monitor, poll

lvchange --refresh: monitor, poll
vgchange --refresh: poll

lvchange --monitor: ok
lvchange --poll: ok
lvchange --monitor --poll: ok
vgchange --monitor: ok
vgchange --poll: ok
vgchange --monitor --poll: ok

lvchange <metadata ops>: monitor, poll
vgchange <metadata ops>: poll

lvchange: enhance avoiding multiple metadata updates/reloads/backups

Enhance commit 25b5915c9b5260c59d627bd1f6db8220bd4ad61e
to process options requiring immediate metadata commits
and reloads after those we can group together doing just
one commit and an optional reload for the whole group.

Backup metadata after processing options successfully.

Related: rhbz1437611

man: '-pool' regexp

lvconvert: fix converting hidden lv to pool

A hidden LV can not be converted to a thin pool or cache pool.

help: print all options in abbreviated help output

Don't abbreviate the --help output quite as much
when there are many command defs.  Print all the
options in the cmd defs that are shown.  --longhelp
output is unchanged and includes everything.

commands: missing comma in cmd def option list

Does not affect parsing, but keep it consistent.

help: include cmd def info in verbose output

It can sometimes be important to correlate a
command with its cmd def entry, so include
the cross reference info in help -vv output.

commands: recognize ignored options

commands: ignore partial option in most cases

These days --partial is only used with activation in
lvchange/vgchange.  It probably had another meaning
at some point in history which is no longer used,
so ignore it in those cases.

When included in the OO list, the option is advertised in
help/man output, implying it is meaningful to the command,
when in fact the command never uses it.
The IO list means the option won't cause an error if it's
used, but is not displayed as an valid option for the command.
If the option is not included in either OO or IO lists,
using the option would cause a command error, which would cause
problems for anyone is using the option for historical reasons.

man: fix description of --partial option

lvchange: refresh command missing activationmode option

lvchange: avoid multiple metadata updates/reloads/backups

_lvchange_properties_single() processes multiple command
line arguments in a loop causing metadata updates and/or
backups per argument.

Optimize to only perform one update and/or backup
(but necessary interim ones; e.g. for --resync)
per command run.

Related: rhbz1437611

tests: fix LD_LIBRARY_PATH

Use only selected paths for finding .so in builddir.
So if builddir constains some embeded subdirs with some more
occurences of project (i.e.  'make rpm' build subdir)
those library paths will not get into list.

cache: scan kallsyms for kernel symbols

With monolithic kernels we can't actually modprobe
for cache modules as they are already compiled-in
and policy modules do not export version symbol.

Reported issue on list:
https://www.redhat.com/archives/dm-devel/2017-March/msg00061.html

Fix will try to look for explicit kernel symbols first before
calling modprobe.

lvchange: persistent major/minor processing regression

Optimized change.

Resolves: rhbz1437611

lvchange: persistent major/minor processing regression

Fix a regression introduced by new command line processing
(see starting commit 1e2420bca85da9a37570871cd70192e9ae831786)
causing activation to be rejected in combination with setting
persistent minor device numbers.

Resolves: rhbz1437611

man: pregen file change

man-generator: shutdown stdout buffer

Flush stdout and switch it back to line buffering before exit.

man-generator: enhance performance

Set block buffering on stdout to
save ~30% time generating manuals.

man-generator: don't ignore description file errors

In case we provide a description file it has to
succeed opening and reading.  Dynamically allocate
buffer space for it whilst on it.

raid: more coverity issues

raid: favour dm_list_first()

raid: address coverity issues

man-generator: add '.' to all messages

man: add minor regexp

dmsetup: replace (f)printf("\n") with putchar/putc

libdm: use correct FMTu32 format specifier for uint32_t

libdm: fix incorrect format specifier in _stats_resize_group()

The bitmap size in dm_bitset_t[0] is a uint32_t, not uint64_t: use
%u instead of FMTu64.

dmsetup: Flush ouptut when using --interval

libdm: use uint64_t where possible in filemap routines

Use uint64_t where possible and explicit casts elsewhere to avoid
compiler warnings when converting between int, int64_t and
uint64_t.

libdm: use size_t for buflen in _stats_group_file_regions()

libdm: make _stats_resize_group() num_regions argument uint64_t

libdm: make _find_extent() nr_extents argument uint64_t

dmfilemapd: cast return value of strto* functions when needed

dmfilemapd: use log_sys_error in _filemap_monitor_set_notify

Since _filemap_monitor_set_notify() is only called after daemon
start up it should not use the _early_log() macros. Instead, use
log_sys_error to log errors from the two syscalls in the function.

dmfilemapd: cast sysconf() return value to int

dmfilemapd: fix len type in _filemap_monitor_check_file_unlinked()

Make 'len' a size_t to match the return type of readlink(2).

dmfilemapd: make filemap_monitor->nr_regions uint64_t

libdm: use DM_STATS_GROUP_* constants in dm_stats_create_regions_from_fd

dmsetup: simplify branching in _stats_update_file()

The fallback branch in _stats_update_file() is redundant (since the
branch taken when the daemon starts successfully must jump to the
'out' label anyway): remove it and re-order the conditions to
improve readability.

man: raid cleanup

man: Copyright

man: use variable in Makefile / regexp

man-generator: catch missing command name

tests: fix typo in makefile

Use properly $(SORT)

man: fix type

With commit 4af09a94bd44693e5a40a4f895599601d8233d69 unwanted rename  of
target happened - restore it.

vgrename: use long enough buffer for path

Use PATH_MAX when creating buffers for rename.

libdm: use log_sys_error if dmfilemapd exec fails

Use log_sys_error rather than log_error if execvp() fails:

  /mnt/redhat/xdoio.13752.XIORQ: Created new group with 2 region(s) as group ID 0.
  # execvp() failed.

vs:

  /var/lib/libvirt/images/rhel7-vm1.qcow2: Created new group with 884 region(s) as group ID 0.
  dmfilemapd: execvp failed: No such file or directory