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

dmsetup: always close fd when leaving _stats_update_file() (coverity)

dmsetup: do not start dmfilemapd if region creation fails (coverity)

libdm: do not attempt filemap cleanup if dm_stats_list() fails (coverity)

libdm: abort filemap update if pool allocation fails (coverity)

libdm: do not leak FIEMAP extent buffer if pool allocation fails (coverity)

dmfilemapd: free region table at end of _update_regions() (coverity)

dmfilemapd: fix off-by-one in fd comparison (coverity)

The function _filemap_monitor_check_file_unlinked() attempts to
test whether a fd value should be closed by comparison to zero:

        if ((fd > 0) && close(fd))
                log_error("Error closing fd %d", fd);

The test should be '>=' since 0 is a valid file descriptor.

dmfilemapd: always close link check file descriptor (coverity)

Similar to 40fb91a, but for the file descriptor opened using the
link path reported by /proc/<self>/fd/<fd>.

The daemon opens a new file descriptor from /proc/<self>/fd when
checking for an unlinked file with mode=inode. Ensure that it is
always closed even if the same file test fails.

dmfilemapd: always terminate link buffer after readlink(2) (coverity)

dmfilemapd: always close check file descriptor (coverity)

The daemon opens a new file descriptor from fm->path when checking
for an unlinked file with mode=inode. Ensure that it is always
closed even if the same file test fails.

dmfilemapd: use *argv[0] when testing for absolute path (coverity)

The path argument check in dmfilemapd incorrectly tests for:

  if (argv[0] == '/')

Rather than testing the 1st character in the string pointed to by
argv[0].

dmfilemapd: pass correct dirp to closedir()

dmfilemapd: do not closdir(pid_d) unless it is open

dmfilemapd: log closedir errors with log_sys_error()

dmstats: Fix path to static dmstats

man: ranges/LV names/symbolic use regexp

man: simplify by avoiding different headers on de-duplication of lvmconfig, lvm-config and lvm-dumpconfig

man: de-duplicate lvmconfig, lvm-config and lvm-dumpconfig manuals

spec: Add dmfilemapd

dmfilemapd: Fix install target

man: regexp readability / use $(SED)

post-release

pre-release

Removing some unused new lines and changing some incorrect "can't
release until this is fixed" comments.  Rename license.txt to make
it clear its merely an included file, not itself a licence.

man: pre-generated files weren't committed

man: fix / typo

man: enhance man postprocessing regexp

tests: update to better fit

Die is automatic on 'error' result
Cleanup everything on 'regular' code path.

cleanup: use more common FMTd64 type

We use 'd' for plain singed integers.

build: fix x32 arch

This patch fixed lvm2 compilation running on x32 arch.
(Using 64bit x86 cpu features but running on 32b address space,
so consuming less mem in VM).

On x32 arch 'time_t' is 64bit while 'long' is 32bit.

man-generator/man/help: simplify hyphen escaping

Commits a29bb6a14b5e920092495fb9a1623e527d22daac
    ... 5c199d99f4dc9911d62673baba7643ea649ffe64
narrowed down on addressing the escaping of hyphens
in the dynamic creation of manuals whilst avoiding
them in creating help texts.  This lead to a sequence
of slipping through hyphens adrressed by additional
patches in aforementioned commit series.

On the other hand, postprocessing dynamically man-generator
created and statically provided manuals catches all hyphens
in need of escaping.

Changes:
- revert the above commits whilst keeping man-generator
  streamlining and the detection of any '\' when generating
  help texts in order to avoid escapes to slip in

- Dynamically escape hyphens in manaual pages using sed(1)
  in the respective Makefile targets

- remove any manually added escaping on hyphens from any
  static manual sources or headers

lvchange: reject setting all raid1 images to writemostly

raid1 doesn't allow to set all images to writemostly because at
least one image is required to receive any written data immediately.

The dm-raid target will detect such invalid request and
fail it iwith a kernel error message.

Reject such request in uspace displaying a respective error message.

man: a few more missed '-' to escape

man-generator: more escaped '-'

man-generator: fix buffer length calculation

make: Don't hard-code SHELL as /bin/sh

man/help: avoid escaping of '-' with --help

WHATS_NEW: man-generator escape '-'

man-generator: cleanup escape '-'

man-generator: emit escaped '-'

man: escape all single '-'

man: revert erouneous '-' escapes in Makefine.in

man: escape all double '-'

tools: Reinstate lvm script processing.

We check for a script if the command isn't recognised (ENO_SUCH_CMD).
(Also added a few comments and fixed some whitespace.)

libdm: typo

commands: improve error message for unknown command

when running "lvm foo".

commands: fix commands with run with path basename

The recent command definitions commit took the command
name from argv[0] without applying basename to the value,
so a pathname, e.g. /usr/sbin, would cause lvm to not
recognize the command name.

tools: Show configuration command line in lvm version.

Also update configure.in with some items recently added to the tree.

raid: adjust to misordered raid table line output

This commit supersedes reverted 1e4462dbfbd2bbe3590936df24b3ccd83110b158
to avoid changes to liblvm and the libdm API completely.

The libdevmapper interface compares existing table line retrieved from
the kernel to new table line created to decide if it can suppress a reload.
Any difference between input and output of the table line is taken to be a
change thus causing a table reload.

The dm-raid target started to misorder the raid parameters (e.g. 'raid10_copies')
starting with dm-raid target version 1.9.0 up to (excluding) 1.11.0.  This causes
runtime failures (limited to raid10 as of tests) and needs to be reversed to allow
e.g. old lvm2 uspace to run properly.

Check for the aforementioned version range in libdm and adjust creation of the table line
to the respective (mis)ordered sequence inside and correct order outside the range
(as described for the raid target in the kernels Documentation/device-mapper/dm-raid.txt).

Revert "raid: adjust to misordered raid table line output"

This reverts commit 1e4462dbfbd2bbe3590936df24b3ccd83110b158
in favour of an enhanced solution avoiding changes in liblvm
completetly by checking the target versions in libdm and emitting
the respective parameter lines.

commands: fix combined thin pool and vol create defs

Fixes command defs related to creating a new thin pool and
then a new thin lv in the new pool.

1. lvcreate --size --virtualsize --thinpool
   Needs a cmd def, it was missing.
   The def is unique by the three required
   options: size, virtualsize and thinpool.

2. lvcreate --size --virtualsize --thinpool VG
   Needs a cmd def, it was missing.
   The def is unique by the three required
   options: size, virtualsize and thinpool,
   and one required positional arg: VG.

3. lvcreate --thin --virtualsize --size LV_new|VG
   This existing def should not accept an optional
   --type thin, which if used makes it indistinct
   from another def.

4. lvcreate --size --virtualsize VG
   This existing def should not accept an optional
   --type thin or --thin, which if used makes it
   indistinct from other defs (e.g. 3)

Revert "lvcreate: continue to accept --thinpool with -L and -V but not -T"

This reverts commit 642d682d8ddcc3152f68b3ec8518902a0ef448ac.

Using the thinpool option with this cmd def makes it
indistinct from other cmd defs where thinpool is a
required option.

metadata: cleanup flags definition to be consistent

Use shift bitops throughout segtype.h.

WHATS_NEW: adjust to misordered raid parameters

raid: adjust to misordered raid table line output

The libdevmapper interface compares existing table line retrieved from
the kernel to new table line created to decide if it can suppress a reload.
Any difference between input and output of the table line is taken to be a
change thus causing a table reload.

The dm-raid target started to misorder the raid parameters (e.g. 'raid10_copies')
starting with dm-raid target version 1.9.0 up to (excluding) 1.11.0.  This causes
runtime failures (limited to raid10 as of tests) and needs to be reversed to allow
e.g. old lvm2 uspace to run properly.

Check for the aforementioned version range and adjust creation of the table line
to the respective (mis)ordered sequence inside and correct order outside the range
(as described for the raid target in the kernels Documentation/device-mapper/dm-raid.txt).

lvcreate: continue to accept --thinpool with -L and -V but not -T

lvcreate --thinpool POOL1 -L 100M --virtualsize 100M snapper_thinp

https://bugzilla.redhat.com/1434027

(The general rule is that a command is accepted if it is unambiguous.
The combination -L -V --thinpool uniquely identifies the operation.)

man-generator: Remove unused variable.

man-generator.c:2976:6: warning: variable "sep" set but not used

lvmdbusd: Rename ee to got_external_event

This variable is global, make it more descriptive.

lvmdbusd: Update state during pv move

Periodically update the state during pv move so that all the different
dbus objects reflect something close to reality during the process.

lvmdbusd: Limit state refreshes for udev events

Udev events can come in like a flood when something changes.  It really
doesn't do us any good to refresh the state of the service numerous times
when 1 would suffice.  We had something like this before, but it was
removed when we added the refresh thread.  However, we have since learned
that we need to sequence events in the correct order and block dbus
operations if we believe the state has been affected, thus udev events are
being processed on the main work queue.  This change limits spurious
work items from getting to the queue.

lvmdbusd: Call add_no_notify for *move commands

Missed this in change when "add_no_notify" was added.  This was causing
extra external events to process when we did moves.

lvmdbusd: Only disable notify_dbus after getting external event

If we always disable the sending of notify dbus events then in the case
where all the users are lvm dbus users we will be in udev handling mode
until at least 1 external lvm command occurs.  Instead we will not disable
notify dbus until after we get at least 1 external event.  This makes the
service get into the correct mode of operation faster.

tests: early detect leaking error dev

lvconvert should not leak 'error' device.

(This patch is not fix the problem, just makes it more easily visible
instead of more confusing 'clvmd' trace).

man: advise against mirrored mirror log

help: align option list in pv/lv/vgchange cases

Align one-required options like is done for
optional options.

raid: check target version for shrink support

Starting with dm-raid target version 1.9.0 shrinking of mapped devices is supported.
Check for support being present in lvresize and lvreduce.

Related: rhbz1394048

raid: conditionally reject convert to striped/raid; fix

Fix a logic flaw introduced in commit 17bee733d16e
preventing e.g. striped -> raid5 conversions.

Related: rhbz1191935
Related: rhbz1366296