command: better const usage for struct cmd_name

No need to store _CMD string - as it's not used anywhere.
Use LVM_COMMAND_COUNT as end of array.

command: reduce struct size

Reduce memory usage for parser.
If the sizes would need to be bigger, man-generator warns.
For struct command_name  use just uint8_t to keep 1|0.

command: no need to duplicate const mem

No need to duplicate const char* here as
we refer already constant string in .rodata segment.

command: futher move to single struct

Refactor code to not allocate memory for rule->opts,
instead use uint16_t array of MAX_RULE_OPTS within cmd_rule.

Also print more info if array would not be enough (>= 8).

tests: check vg fields with one command

tests: keep more LVs inactive

tests: mask result from kill command

tests: wait for running dmeventd

Add slight delay for waiting until 'started' dmeventd is
responsing to other 'dmeventd -i' command.

TODO: race observed here is somewhat unclear, might need some more
details

tests: kill process in flock group

There is no reason to wait for 10sec when removing 'vg' at test
exit - we just need to kill 'sleep 10' process forked from flock.

(Not using 'flock -F' as this flag is not across all machines used
for testing)

tests: skip test on systems without delay_dev

On such systems this test only sleeps for 30sec.

libdm: reapply "fix condition"

This reverts commit d16a8f80e9dd21d97b10056858a21b7e8fbd8c94.
So the correction was OK, however we missed to fix also
cut&paste bug here - as the second check should be
actually checking  field->type.

WHATS_NEW: update

make: generate

tests: vdo conversion updates

commands: move tabs to eol

Make the source code a bit easier to read.

commands: enable more options for vdo conversion

Update definitions to add support for some more VDO options
when converting volumes to be used as thin-pool with vdo data volume.

Split some option in existing OO_LVCONVERT_VDO to OO_LVCONVERT_VDO_POOL
and reused then with OO_LVCONVERT_THINPOOL.

lvconvert: vdo support conversion options

When convert already existing vdopool to be used as
thin-pool backend and user is passinng option for VDO configuration
process them - as we know converted LV is offline, we can do such
change easily instead of telling user to run separate lvchange later.

lvconvert: vdo control for signature wiping

When converting volumes to be used for thin-pool with VDO, allow
users to control wipesingature behaviour.
By default volumes should be checked against signature, and if
they are present, we promt user whether he wants to process with
conversion and lose i.e. filesystem present on such volume.

Users that want to bypass prompt in script can use either --yes
or they can disable wipe signature -Wn.

man: lvmthin more extend options

tests: runner updates line with last timestamp

clean: use struct initializer

dmsetup: const string_args array

Convert _string_args to const char*.
Do not modify options read from getopt().

dmsetup: use tab for indent

Use \t for help indention.

dmsetup: more const strings

More strings into .rodata segment.

dmsetup: reduce parser code

Switch set upfront.
Only add code, which is not common for every switch.
Use just 16bit for switch counting.

libdm: fix invalid conversion

Previous commit missed to also add one to added 'len'.

libdm: stats uses overflowing counter

Here we need to annotate counter as it starts with UINT64_MAX
and overflows to 0 is expected.

clenaup: update local static with _

const: tests

man-generator: remove unused declarations

cov: remove dead code

cov: eliminate dead code

Deduplicate code and also drop some warn about dead code.

cov: avoid TOCTOU

Use just opendir() call and produce error message from errno state.

cov: enum mapping cache format

Cleanly map cache metadata format enumeration.

cov: enum health status

Cleanly map health char status to enum.

cov: enum matching type

cov: enum correct prototype

Cleanup enum usage.

cov: cast to matching prototype

Match function prototype with casting.

cov: check for overlow math

Add some extre protection to avoid integer overflow type of problems.

cov: annotate to silent some coverity warning

cov: use literal for printf

Use literal for printf and just empty string if needed.

cov: use literal for printf format

Prepare suffix ahead and use literal for printf().

cov: sanitize input buffer

Make coverity aware the input line is legit.
Eventually add more sanitize functionality.

cov: sanitize string for file path

cov: wrap pvck call

Resolve reported leak by renaming existing pckk() to pvck_mf()
and wrapping pvck() over this. This function can easily
free allocated buffer within the subfunction.

cov: strcpy mempcy

cov: replace strcat with safe snprintf

cov: replace strcpy with memcpy

cov: use sscanf buffer size limits

String parsing with buffer size limit.

cov: check for valid raid0 LV segment

Add internal error to quiet Coverity fear of NULL pointer deref.

raid: easier prompt message

Use  printf capability of 'prompt()' funtions so use literal
when building this message.

lvmlockd-dlm: log_error message with literal

Use literals for printf() message construction.

libdm: unlink checks for ENOENT

libdm: use literal for error message

Use literals for printf messages.

const: daemon server logging backend struct

const: lvmpolld long_options

Rework usage of long_options so it can be constified,
and also correct use of 'enum action_index'.

const: rework conversion to enum

Easier code with string and also avoids mixing enums.

const: array of const values

const: libdm err_fmt as string

man: lvmthin update

major refresh of text and organization

dmsetup: fix invalid loop test

Last patcheds had incorrect merging bug - fix loop test.

Revert "libdm: fix condition"

This reverts commit 2a1e200f7af3d4c52afbb4627c8cf87f98baa71e.

Breaks some test - more analysis is needed.

WHATS_NEW: update

make: generate

Update for --wipesignatures for lvconvert.

dmsetup: use getopt defines

Replace 0/1 with getopts defines no/required_argument

const: dmsetup long_options

const: dmsetup string

const: cmirrord array

const: lvmlockctl options

const: dmfilemapd strings

const: array for _vdo_split

const: report_name

const: use common error path

Instead of defining  _field_selection_value_alloc_failed_msg[],
use common error path code for printing field_id.

const: log level string

const: libdm cmd_data_v4

const: libdm tok_op

const: suffixes list

dm_tree_set_optional_uuid_suffixes() is safe to use with overcasted
array to (const char**)

const: suffixes

const: _reserved

const: use arrays of strings 2

Next set of changes.

const: use arrays of strings

Such string is stored directly in '.rodata' section.

const: drop static keyword

We don't need relocatable space to store 'const' number.

const: static array

Such array can be stored  in   .rodata   section.

const: config items

const: raid takover arrays

const: _fns poll_function

const: _ops text_vg_version

const: _ops labeler _text_ops

const: _ops segtype handler

const: _ops text handler

Making sure these structures ends in '.data.rel.ro' section.
(instead of plain '.data' section).

libdm: fix condition

When we switch supported_reserved_types_with_range to const
gcc repots this problem:

 warning: ‘and’ of mutually exclusive equal-tests is always 0
 !(iter->type & supported_reserved_types_with_range))) {

It's not clear from the history what was the actual intention of this
internal error test, let's assume the check wanted to make sure
that when DM_REPORT_FIELD_RESERVED_VALUE_RANGE is set,
some other fields from supported_reserved_types_with_range set
are also selected.

libdm: ensure suffixes list elements are const

This was rather API mistake - the internal of libdm
do handle suffixes list as const string, just the API
was only using  'const char **'.

So the user may pass safely casted 'const char * const`.

lvonvert: support control over wipingsignature

When converting volume to a thin-pool allow control over
signature wiping.

TODO: likely should become more commonly supported option...

vdo: enhance error path

When the conversion to VDO fails, try to restore previous VG state
and also remove orphan DM node from table.

lvcreate: --yes option for thin-pool vdo creation

Correct typo and accept proper --yes option instead
of misplaced --force option.

command: correct lvresize

Code for supporting size reduction of pool metadata is not yet present
so don't pretend the option can work as pushed in previous commit.

WHATS_NEW: update

make: generate

Update lvresize options with +/-.

command-lines: improve spec for thin snapshot

man-generate --check actually noticed 2 same definitions
for snapshot create  with 'lvreate -T [--snapshot]'
and 'lvcreate --snapshot [-T]'.
So drop the '-T' from second alternative variant as
thin type is already implied here.

use LVM_COMMAND_COUNT