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

man-generator: code reformating

command: refactor to use const command structure

Refactor code so the definitions may become 'static const'
and with configure_command_option_values() we update options
val_enum for actually running command option when used.

Also update _update_relative_opt() which is used for
generating man pages and command help.

Introduce enumeration for lvm2 commands - so we may
use enum cmd_COMMAND instead of string checking.

So running command now does not modified opt_names.

command: embedding strings to structs

Since we will make these struct const, we can also
embedding string content into them to avoid unnecessary
pointer relocations.

cleanup: tab indent and typo

cleanup: unlink passes with ENOENT

cleanup: remove unused code

writecache: reuse some API

Tiny code updates, reuse dm_pool_strdup().

memory: compile with VALGRIND_POOL

Function to _allocate_memory() was not compiled-in when lvm2 was
build with support for better tracking memory pool with valgrind.
Instead now correctly avoid this function only when running
withing valgrind environment.

memory: convert log_error

As the memory locking is not causing command's error - convert
error messages to warnings and debugs.

toolcontext: introduce var for run in valgrind

activation: interruptible sleep when talking to dmeventd

cleanup: drop double _

lvmcmdline: code update

command: more arrays keep static

command: more static const declaration

Use static const for declared arrays.
Access to arrays through get_ functions().

command: use const declaration for arrays

command: refactor common code to command_enum.h

Move shared code to command_enum.h and avoid duplicating code.

lvchange: constify option array

command: using single printf call

Use single string for 1 printf() call with \t tabs.

command: refactor loop test for end of string

command: refactor test for relative opt

Use _update_relative_opt to check for relative option and
update val_enum accordingly.

man-generator: some switch case refactoring

command: split man-generator code

Move code for generating man pages to man-generator.c
which is no longer a symlink to command.c and just
includes this file for the common code.

command: refactor if condition

lvconvert: use cmd_context variable

Do not modify flags field from 'strcut command_name' and
instead control this via cmd_context  get_vgname_from_options.

Flag GET_VGNAME_FROM_OPTIONS is currently used only by lvconvert.

lvmdbusd: its fine if dir already exists

With previous patch, we now need to handle the case
when dir already exists.

dmeventd: use standard reopening mechanism

Apply same code for reopening /dev/null for 0,1,2 when daemonizing.

daemon-stray: correct size limitation for filename

Correct the buffer size for filename printing.

lvmdbusd: create dirs for lock file

Create leading dirs for lvmdbusd lock file, otherwise it fails to start:

| systemd[1]: Starting LVM2 D-Bus service...
| lvmdbusd[1602]: [1602]: Error during creation of lock file(/var/lock/lvm/lvmdbusd): errno(2), exiting!

Signed-off-by: Kai Kang <kai.kang@windriver.com>

WHATS_NEW: update

make: generate

cmdlines: use OO_LVCREATE_RAID

Common options in one OO_

cmdlines: simplify OO_LVCONVER/LVCREATE_THINPOOL

OO_LVCONVERT_THINPOOL always constains OO_LVCONVERT_POOL
OO_LVCREATE_THINPOOL always contains OO_LVCREATE_POOL

More readable description.

tests: resize thin-pool with vdo data

tests: correcting test for invalid commnad

Correting typo in specification of create --typo.
Also removing some unnecessary parts...

makefile: avoid creating shared lib for static only build

vdo: fix setting VDO size when stacked

When VDO is used as data LV for thin-pool, we need
to use correct function to propagate this through
the stack of volumes.

lvmcache: stop saving duplicate pvs info between scans

In the past it was common for a single command to run
multiple lvmcache_label_scan, and this code was a way
to make each call select the same duplicate pvs.  Now
commands run a single lvmcache_label_scan, so this is
not needed.

tests: wait for raid being in sync

cov: unused headers

cov: validation

Add internal error for missing vg.

toolcontext: check runtime valgrind

When reseting stream buffer - check for being run within valgrind
and only in this case skip this code.
Define VALGRIND_POOL was incorrectly used for this logic.

daemon-stray: compile also with VALGRIND_POOL

We can run and build this code just fine with VALGRIND_POOL
and just rely on runtime detection - so we do not close
descriptors also within running valgrind session.

spec: require ncurses only for readline

Hopefully editline can go without ncurses dependency.

makefiles: change condition

Old system do not work well with -l% in findstring,
so use a different strategy to recognize whether -lreadline needs
another library (-ltinfo, -lncourses...)
(So we don't need to solve this via 'configure')

Also for now comment out -Wl,-z,pack-relative-relocs and leave it
for the package maintainer whether they want to use.
Possibly add some 'configure' autodetection for usable switch,
as it's relatively new feature..