config: fix some settings incorrectly marked as CFG_DEFAULT_COMMENTED instead of CFG_DEFAULT_UNDEFINED and causing segfault

This patch fixes segfault which was caused by incorrectly marking some
settings CFG_DEFAULT_COMMENTED instead of CFG_DEFAULT_UNDEFINED - the
ones which have NULL default value, hence they're really undefined.
A regression caused by a98ceceb1d101f0b19a6115f406e23f13feab5ae.

For example:

$ lvmconfig log/file
file="/a"

Before this patch:

$ lvmconfig --type diff
Segmentation fault (core dumped)

With this patch applied:

$ lvmconfig --type diff
log {
file="/a"
}

The same applies for these settings:

  log/activate_file
  global/library_dir
  global/system_id_file
  <disk_area>/disk_area_id

There were also other settings with NULL default value and marked as
CFG_DEFAULT_COMMENTED instead of CFG_DEFAULT_UNDEFINED, but they were
cfg_array config settings where the NULL value was not causing segfault
(NULL == empty array).

Add a global get_cmd_name()

Which returns the string set by set_cmd_name().

lv_refresh: move the bulk of the function into lib

So that it can be used from other lib code.

man vgchange lvchange: mention activation option s

and improve the existing text about existing
activation options.

man lvm: mention hyphens in option names

Add the 's' activation mode

Just as 'e' means activation with an exclusive lock,
add an 's' to mean activation with a shared lock.

This allows the existing but implicit behavior of '-ay'
of clvm LVs to be specified explicitly.  For local VGs,
asy simply means ay, just like aey means ay.

For local VGs, ay == aey == asy

For clvm VGs,  ay == asy, aey == aey, asy == asy

Ignore hyphens in long option names

The hyphens are removed from long option names before
being read.  This means that:

- Option name specifications in args.h must not include hyphens.
  (The hyphen in 'use-policies' is removed.)

- A user can include hyphens anywhere in the option name.
  All the following are equivalent:
  --vgmetadatacopies,
  --vg-metadata-copies,
  --v-g-m-e-t-a-d-a-t-a-c-o-p-i-e-s-

xlate: add new variants

New variants use the clearer function names from the kernel.

lvmcache: mention lvconvert --cachemode

for changing the cache mode on an existing LV.

debug: better tracing messages

Enhance traced output.

libdm: better debug message

Print reason for failing ioctl if thin pool message fails.

cleanup: drop double const

Second const is unneeded.
Also always create whole array with MAX elements.

dmeventd: better debug

Avoid using 'static' variable within threaded environmnent for debuging.

dmeventd: drop stack

dmeventd: drop pthread_cancel

Drop unused pthread canceling as well as DEBUG printing in signal
handler.

post-release

pre-release

test: Ensure that outdated PVs are wiped just once.

metadata: When outdated PVs are wiped, notify lvmetad about the fact.

metadata: Reject lvmetad metadata extensions when reading from disk.

lvmetad: Make it possible to clear the outdated PVs list for a VG.

metadata: Explain the pvs_outdated field in struct volume_group.

lvmetad, lvmpolld: remove DL_LIBS from Makefile

and rdynamic.  They are not needed.

lvmetad.c: internal err on modifying global handle with open connection

lvmetad_init() should not be called with open connection to the daemon.
Doing so is considered to be an internall error within lvm2 code.

Such coincidence can't occur within current code. Let's assure us it won't
ever happen.

lvmetad.h: rephrase API descriptions

Some of descritpions were misleading at least. Some were completely
off the reality.

lvmetad_init doesn't re-establish or initialise a connection
lvmetad_active and lvmetad_connect_or_warn can do so.

man: lvmthin chunk and metadata sizes

Clear some stale information, and give a suggestion to use
a metadata size of 1GiB.

tests: check for idle only for raid type

lvmetad: missing wrapper for lvmetad less compilation

tests: check for clmvd socket

A bit hacky since it expects PID_DIR == DEFAULT_RUN_DIR for now,
just to check whether it fixes startup clvmd sync problem.

tests: better check for array in sync

Update check for raid array being in sync - getting somewhat complex.
It's another way to fight with problems in:
https://bugzilla.redhat.com/show_bug.cgi?id=1210637

tests: drop debug print

cleanup: gcc warn fix, don't hide pvs()

lvmpolld: terminate error message with a dot and LF

WHATS_NEW: various updates

commit c069aff21b1450923b59e98ab709a5e667cdcda9
commit 8af5f54824f889c438104625dbe159f6e8dedbc4

dmsetup: zero errno in before strtoul call

Testing errno value without explicitly setting to
zero in before the strtoul call may lead to
unexpected failures.

lvmpolld: zero errno in before strtoul call

Testing errno value without explicitly setting to
zero in before the strtoul call may lead to
unexpected failures.

lvconvert.c: fix whitespace mess

libdm: Add dm_task_get_errno to return ioctl errno.

There are reports of unexplained ioctl failures when using dmeventd.
An explanation might be that the wrong value of errno is being used.

Change libdevmapper to store an errno set by from dm ioctl() directly
and provide it to the caller through a new dm_task_get_errno() function.

[Replaced f9510548667754d9209b232348ccd2d806c0f1d8]

WHATS_NEW: update

update for commit: f8bf6410954fcf82bf28852e0dba015c6b7f19dc

tests: add test for pvscan --cache --background

regression test for a segfault in pvscan --cache --background
bug fixed by commit:
f8bf6410954fcf82bf28852e0dba015c6b7f19dc

lvmetad.c: ignore lvmetad global handle on disconnect

do not unset lvmetad global handle on disconnect. This is
hotfix for issue described in:
https://www.redhat.com/archives/linux-lvm/2015-May/msg00008.html

Reported-by: Christian Hesse <list@eworm.de>

lvconvert: change how to get failed mirrors number

Commit  b00711e3128c9cb38da90f303a3b64cf3a30fc9b improperly
convert _area_missing() replacment and moved check for
AREA_PV seg_type() into same if() section.

Signed-off-by: Lidong Zhong <lzhong@suse.com>

scripts: activation generator: do not use --sysinit if lvmpolld used

If lvmetad is not used, we generate lvm2-activation{-early,-net}.service
systemd services to activate any VGs found on the system. So far we used
--sysinit during this activation as polling was still forked off of the
lvm activation command.

This has changed with lvmpolld - we have proper lvmpolld systemd
service now (activated via its socket unit). As such, we don't need
to use --sysinit anymore during activation in systemd environment
as polling was the only barrier to remove the need for --sysinit.

lvmpolld-client.c: debug print when querying progress

polldaemon.c: modify log levels in report_progress

There's a race when asking lvmpolld about progress_status and
actually reading the progress info from kernel:

Even with lvmpolld being used we read status info from
LVM2 command issued by a user (client side from lvmpolld perspective).
The whole cycle may look like following:

1) set up an operation that requires polling (i.e. pvmove /dev/sda)
2) notify lvmpolld about such operation (lvmpolld_poll_init())
3) in case 1) was not called with --background it would continue with:
4) Ask lvmpolld about progress status. it may respond with one of:
   a) in_progress
   b) not_found
   c) finished
   d) any low level error

5) provided the answer was 4a) try to read progress info from polling LV
(i.e. vg00/pvmove1). Repeat steps 4) and 5) until the answer is != 4a).

And now we got into racy configuration: lvmpolld answered with in_progress
but it may be the that in_between 4) and 5) the operation has already
finished and polling LV is already gone or there's nothing to ask for.
Up to now, 5) would report warning and it could print such warning many
times if --interval was set to 0.

We don't want to scary users by warnings in such situation so let's just
print these messages in verbose mode. Error messages due to error while
reading kernel status info (on existing, active and locked LV) remained
the same.

format_text: Parse (optional) outdated_pvs section in VG metadata.

lvmetad: Set up lvmcache & PV structs for outdated_pvs.

lvmetad: Provide entire pvmeta sections for outdated_pvs.

metadata: Add pvs_outdated to struct volume_group.

This is a list of PVs that should have their MDAs wiped because they carry
outdated metadata (that used to belong to the VG they are attached to).

metadata: Factor _wipe_outdated_pvs() PVs out of _vg_read().

lvmetad: Attach an outdated_pvs list to vg_lookup replies.

lvmetad: Clear the vgid_to_outdated_pvs hash on shutdown.

lvmetad: Maintain info about outdated PVs.

makefiles: use bash subshell

Avoid using  make's $(shell invocation since the eval order is
then somewhat different and use $$(  subshell.

This also fixes a problem when more then one symbol is found,
since target shell has been given separate word list
so the 'R' assignment would need "" around it.

configure: fix missing [

WHATS_NEW: various updates

polldaemon.c: do not report error when LV not found

currently in wait_for_single_lv() fn trying to poll missing pvmove LV
is considered success. It may have been already finished by another
instance of polldaemon. either by another forked off polldaemon
or by lvmpolld.

Let's try to handle the mirror conversion and snapshot merge the same
way.

lvconvert.c: drop get_vg_lock_and_logical_volume fn

pvmove.c: code cleanup

polldaemon: remove get_copy_vg and get_copy_lv wrappers

These wrappers have been replaced by direct calls
to vg_read() and find_lv() in previous commits.

This commit should have no functional impact since
all bits were already unreachable.

polldaemon.c: call find_lv directly

lvconvert.c: call find_lv directly

polldaemon.c: call vg_read directly

replace calls in wait_for_single_lv() and report_progress() fns

lvconvert.c: call vg_read directly

pvmove.c: call vg_read directly

polldaemon: move dev_close_all out of poll_get_copy_vg

let's call dev_close_all() only before we're about to 'sleep'
for at least one second during the polling.

(it's questionable whether to call dev_close_all() at all in
polldaemon code. Natural extension would be to drop it completely)

tests: add test for duplicate pvs

configure: update localedir

Previous patch incorrectly skipped replace of @LOCALEDIR@.

The standard option is --localedir  so use --with-localedir
as backward compatible option and set localedir if it's not
yet been set (if the could ever happen).

Use double-eval to translate $datarootdir to $prefix to real dir.

configure: LOCALEDIR needs evaluated value

Also fix typo.

tests: fix calcucaltion

Code works properly.

tests: thin_restore  not needed

Few more test which could live without thin_restore.

makefiles: drop LVM_SHARED_PATH

We already have LVM_PATH define used everywhere else
to access LVM binary so stay with one name.

makefiles: better clean

More exact clean of library exported symbols files.

Also use $(firstword) test to check for empty string
so 'make clean' has now cleaner condensed look.

Clean also created include links.

makefiles: use := for shell calls

makefiles: use single target

Possibly  easier to follow - to have just a single dependency line
and use  if() within rule.

Also replace $(words) with $(firstword) which is more commonly used.

configure: move DEFS to configure.h

tests: no warn if test does not need thin_repair

Set LVM_TEST_THIN_REPAIR_CMD to /bin/false for test which
doesn't need it.

This way - even if on the system there is no such tool present,
test will not result with warning about missing tool.

Also remove from Makefile settings of TEST vars which are set in
through /lib/paths  - this also allows to override them in test.

post-release

pre-release

mirror: use proper 64bit constants

ed2a08bf25e3b4e6a88c55402ab5dc11061ad68e missed to use 64bit
constants.

spec: new man page

config: remove UNDEFINED from cache_pool_cachemode

Replace UNDEFINED with COMMENTED because the code
requires a value to be returned from the config.

config: remove UNDEFINED from thin_pool_discards and thin_pool_zero

Replace UNDEFINED with COMMENTED for these two since
undefined seems to break things.

lvmpolld.8.in: man page rewrite

- add client functionality
- fill in long option variants

lvmpolld: don't return success on invalid option

lvmpolld: add long option variants for all short ones

lvmpolld: introduce client functionality

as of now lvmpolld works as client utility for
querying running instance of lvmpolld server
on metadata, state, etc.

Currently the only request implemented is the '--dump'.
It prints out full lvmpolld state (mimics lvmdump -p command).

pvmove.c: relocate id components extraction

we don't want to fail properly set pvmove after metadata
update. failure to copy id components could end with dangling
mirror moving PV segments but no monitoring from lvmpolld or
classical polldaemon.

lvpoll.c: harden the checks for proper LV name

lvpoll now process passed LV name properly. It respects
LVM_VG_NAME env. variable and is able to process LV name
passed in various formats:

- VG/LV
- LV name only (with LVM_VG_NAME set)
- /dev/mapper/VG-LV
- /dev/VG/LV

lvpoll.c: replace arg_count with arg_is_set

didn't need to count the occurence but check if set

example.conf.in: apply previous improvements to example.conf

config_settings.h: improve lvmpolld config description

lvm.8.in: add reference to lvpoll built-in command

lvmdump.8.in: describe lvmpolld related option

commands.h: reorder lvpoll options

lvm-lvpoll.8.in: man page for built-in command

lvmpolld.8.in: hide origin of lvmpolld man page

:)

lvmpolld-client.c: explain known return codes

lvmpolld returns few well known return codes. Explain
these to users and suggest reading lvmpolld log files
for more details

lvmpolld: label known return codes

so that lvmpolld-client can decode and describe these
return codes properly

libdaemon: fprintf(stderr...) -> ERROR()

log data structures are ready. so why not pass
error messages through ERROR()