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..

tests: avoid race check

tests: accept also closing status

makefiles: comman escape needs this trick

makefiles: add as-needed only for newer systems

Add some -Wl flags separatly and avoid their duplication.
Also add --as-needed only when system is using 'newer' readline
library - on these older system the usage of '--as-needed'
seems to be causing some hard to solve problem - so avoid it.

tests: correct linking order for RT_LIBS

configure: autoreconf

configure.ac: update ldflags args

Attach -Wl,-z,relro,-z,now,-z,pack-relative-relocs,--as-needed
to LDFLAGS, but only if LDFLAGS already doesn't contain 'relro'
(so it's not given repeatedly).

Also start to use -z,now linkage also when building libraries
with default compilation - this avoid calling symbol resolver
while library function are using function needing resolving.

Note: Fedora or RHEL rpm building is using:
CFLAGS=$(rpm --eval %{build_cflags})
LDFLAGS=$(rpm --eval %{build_ldflags})

Also split -DUSE_SD_NOTIFY into DEFS from CFLAGS.

configure.ac: leave LDFLAGS resolving on make time

Use LDFLAGS separately with every use of CLDFLAGS and leave
this flag only for handling versioning.

This will reflect any LDFLAGS setting use during make.

tests: update dmsecuretest

Look not only for whole 64byte sequence,
but seek also 32byte, 16byte and 8byte parts of the
key.

Currently to pass memcpy ZMM problems add possible
workaround in the form of GLIBC_TUNABLES setting.

make: generate

tests: look also for 16byte sequences

tests: add explicit call to lvs

Avoid using 'lvs' from  'get' shell - as that would wait until
whole group of processes is finished.

TODO: rethink what would be the point of starting 'dmeventd' with lvs.
It seems to break some rules.

spec: define rhel_version_file for rhel only

Do not define rhel_minor_version on non-rhel systems.

libdevmapper: _do_event waits for working dmeventd

Refactor _start_daemon and add synchronization delay waiting
untill new forked dmeventd actually open fifos and is ready
to process messages.

This closes some race window in testing.

lvmlockd: check for multiple lock managers running

When no lock manager for the global lock had been set yet,
and the first global lock request was received, and both
dlm and sanlock were running, lvmlockd would assume it
should use the dlm for the global lock, and would start
the "lvm_global" dlm lockspace automatically.  That's
not always correct, so don't assume the dlm should be used,
fail the lock request, and wait until a VG with a specific
lock type is started to determine the lock manager to use.

lvcreate: allow raidintegrity option with implicit raid type

Allow "lvcreate -m1 --raidintegrity y" when raid1 is used, but
not explicit on the command line.

spec: correct condition

Test needs to make sure that it's either <RHX or <FXX.
However they are set to 0 when on different system.
So use '&&'.