cov: unused header file removal

cov: unused header removed
Also ensure library header file with config settings goes first.
Move inclusion of format-text.h into layout.h

bcache: fix memleak on error path

clang: free io on error path.

pvck: avoid memleak of vgname

clang: no vgname buffer leak.

dmsetup: no memleak on failed realocation

clang: keep old buf pointer for release on failing realloc() codepath.

tests: compatible with older kernels

Older kernels just show syscall stacktrace.

tests: move vgsplit writecache

to a different file

pvmove: prevent moving writecache device

vgsplit: handle cachevol

attached to a cache or writecache LV.
Ensure PVs in cachevol are moved with the main LV.

writecache: reject invalid high/low watermark setting

man: lvmcache writecache watermark percent

writecache: report status fields

reporting fields (-o) directly from kernel:
writecache_total_blocks
writecache_free_blocks
writecache_writeback_blocks
writecache_error

The data_percent field shows used cache blocks / total cache blocks.

man: updates to lvmcache

lvmlockd: use ret value in query function

WHATS_NEW: update

tests: add corruption write on PV test

Test a case where PV is readable, but fails on write updating.
Check the failure is reported only for a single PV.

tests: writeerror_dev

Intruduce aux function for easy simulation of disk areas,
that are 'normally' readable, but will fail on write.

vdo: adapt for multi line vdo_format output

Do not close pipeline after 1st. line parsed from vdo_format.
Also reprint the output for a user so new messages from vdo_format
can be seen by users.

raid: more limitted prohibition of stacked raid usage

We actually need to prohibit only reshaping cases which are
running over multiple commands.

man: pvck dump description improvements

add suggestion message for mirror LVs

Currently the error messages are not clear. This very easy to
guide user to execute "--removemissing --force", it is dangerous
and will make the LVs to be destroied.

Signed-off-by: Zhao Heming <heming.zhao@suse.com>

fix corosync.conf: no interface error

systemctl status corosync (version: 2.4.5) report error:
  parse error in config: No interfaces defined

Signed-off-by: Zhao Heming <heming.zhao@suse.com>

WHATS_NEW: update

tests: update for readahead

vdo: avoid running initialization of cache pool vars

Since VDO is also pool, the old if() case missed to know about this,
and executed unnecesserily initialization of cache pool variables.
This was usually harmless when using 'smaller' sizes of VDO pools,
but for big VDO pool size, we were reporting senseless messages
about big cache chunk sizes.

raid: disallow reshape of stacked LVs

Until we resolve reshape for 'stacked' devices, we need to disable it.
So users can no longer reshape i.e. thin-pool data volumes, causing
ATM bad thin-pool problems.

WHATS_NEW: VDO lvmdbusd adds

lvmdbusd: Add function to convert LV into a VDO pool

lvmdbusd: Add VDO enable/disable compress & dedup

Added methods to vdo pool interface to allow enabling and
disabling of VDO:
 * Compression
 * Deduplication

tests: Some lvmdbus tests require larger PVs

tests: VDO detection in dbus tests

drop warnings about missing pvs in foreign vgs

When a foreign VG is ignored, don't print warnings that
it is missing PVs.

pvck: show specific dump option values

tests: improve secure test

Validate we capture core while original task sleeps.

test: fix missing waiting on udev

After device creation we need to wait for a cookie so it's not forgotten
in the system.

test: fail on device create

Correct validation of prepared device and fail if the device can't
be created.

test: aux setup

Avoid endless loop if there was no 'remove' progress.

clenaup: simpler form

vg_read: use else for 3 case

Make it visible we check for ==, >, <  of same var.

debug: enhance messages

Drop 'extra' stack trace where errors are already logged from function.
Add some missing dots in messages.

debug: avoid to slashes in debug message

debug: print VG name in log messages for segment errors

Signed-off-by: Nikhil Kshirsagar <nkshirsa@redhat.com>

libdm: set maj:min while creating and reloading device

Add maj:min to the task structure for RELOAD - which is now
handled in _flatten() and will just skip passing device name.

libdm: support device RELOAD with maj:min and devname set

When devices are created - we were not giving meaning error messages
when the failure happened on 'reload' part of creation.

With this patch we are now able to report both name and major:minor.

Enhancment is most visible with 'crypto' devices,
which are using 'secure' memory erase bit.

man pvck: describe settings

update option description for settings

pvck: deal with coverity warnings

post-release

pre-release

build: make generate

pvck: use zalloc in more places

pvck: repair headers and metadata

To write a new/repaired pv_header and label_header:

  pvck --repairtype pv_header --file <file> <device>

This uses the metadata input file to find the PV UUID,
device size, and data offset.

To write new/repaired metadata text and mda_header:

  pvck --repairtype metadata --file <file> <device>

This requires a good pv_header which points to one or two
metadata areas.  Any metadata areas referenced by the
pv_header are updated with the specified metadata and
a new mda_header. "--settings mda_num=1|2" can be used
to select one mda to repair.

To combine all header and metadata repairs:

  pvck --repair --file <file> <device>

It's best to use a raw metadata file as input, that was
extracted from another PV in the same VG (or from another
metadata area on the same PV.)  pvck will also accept a
metadata backup file, but that will produce metadata that
is not identical to other metadata copies on other PVs
and other areas.  So, when using a backup file, consider
using it to update metadata on all PVs/areas.

To get a raw metadata file to use for the repair, see
pvck --dump metadata|metadata_search.

List all instances of metadata from the metadata area:
  pvck --dump metadata_search <device>

Save one instance of metadata at the given offset to
the specified file (this file can be used for repair):

  pvck --dump metadata_search --file <file>
    --settings "metadata_offset=<off>" <device>

pvck: dump functions cleanup args and return vals

pvck: dump show most recent metadata

pvck: allow disk locations to be specified

using --settings:

mda_offset=<offset> mda_size=<size> can be used
in place of the offset/size that normally come
from headers.

metadata_offset=<offset> prints/saves one instance
of metadata text at the given offset, in
metadata_all or metadata_search.

pvck: move some arg processing

scan: add simple scan to find a pvid

metadata: add vg_from_config_tree

Add cmd/fmt args to import functions so that
they can be used without the fid arg which.

Revert "cov: use zalloc"

This reverts commit 9af1d63b4d96facb0aa055a44c2f61eac01e13f2.

fixes folded into subsequent pvck commit

Revert "pvck: check result of dev_get_size"

This reverts commit 1f4968289c9c0e2a1e850317ef173df11938b7f3.

fixes folded into subsequent pvck commit

Revert "cov: fix mem leaking buffer"

This reverts commit d67ce9e140b5691a3f75a69d6e735eec30603670.

fixes folded into subsequent pvck commit

Revert "cov: avoid passing NULL to strstr function"

This reverts commit 0bad3977df40a70c68778ff0514d0ca712fee402.

fixes folded into subsequent pvck commit

Revert "cov: check for retvalue"

This reverts commit 153e55c20e9a3810a1862314a6465179e1b53931.

fixes folded into subsequent pvck commit

scanning: use bool type for _scan_text_mismatch

tests hints: update check for io count

Running a reporting command on a VG now includes one
additional read to check the mda_header for any change
to the vg between scan and lock.

Revert "lvs: disable scanning optimization"

This reverts commit 7474440d3b540d20eb4f997efeb31b881cc6ac8e.

lvs can use the scanning optimization again since it has
been changed in:
"scanning: optimize by checking text offset and checksum"

scanning: optimize by checking text offset and checksum

After the VG lock is taken for vg_read, reread the mda_header
and compare the metadata text offset and checksum to what was
seen during label scan.  If it is unchanged, then the metadata
has not changed since the label scan, and the metadata does not
need to be reread under the lock for command processing.

For commands that do not make changes (e.g. reporting), the
mda_header is reread and checked on one mda to decide if the
full metadata rereading can be skipped.  For other commands
(e.g. modifying the vg) the mda_header is reread and checked
from all PVs.  (These could probably just check one mda also.)

bcache: add invalidate_bytes function

lvcreate: ensure striped raid region size is at least stripe size

The kernel MD runtime requires region size to be larger than stripe size
on striped raid layouts, thus the dm-raid target's constructor rejects
such request.

This causes e.g. an 'lvcreate --type raid10 -i3 -I4096 -R2048 -n lv vg' to fail.

Avoid failing late in the kernel by enforcing region size to be
larger or equal to stripe size.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1698225

pvcreate,pvremove: fix reacquiring global lock after prompt

When pvcreate/pvremove prompt the user, they first release
the global lock, then acquire it again after the prompt,
to avoid blocking other commands while waiting for a user
response.  This release/reacquire changes the locking
order with respect to the hints flock (and potentially other
locks).  So, to avoid deadlock, use a nonblocking request
when reacquiring the global lock.

writecache: modprobe dm-writecache

tests: update to md dev name

Restore WAIT_MD_DEV in teardown.

NOTE: The name of MD device may have changed.

(cherry picked from commit c2ff8876f97a5840cd9d6fcda1bec55ef6220dc3)

tests: Find md name using lsblk

After stopping MD device and rescanning the leg, it is created with
different name.

Partial revert "tests: update to md dev name"

This partially reverts commit c2ff8876f97a5840cd9d6fcda1bec55ef6220dc3.

Not all MD devices are stopped. Something is missing there...

test: Fix handling leftovers from previous tests

teardown fails current PREFIX is prefix of previously failed test with
leftovers in dmtable.

lvs: disable scanning optimization

The scanning optimization can produce warnings from
'lvs' when run concurrently with commands modifying LVs,
so disable the optimization until it can be improved.

Without the scanning optimization, lvs will always
read all PVs twice:

1. read metadata from all PVs, saving it in memory
2. for each VG
3. lock VG
4. reread metadata from all PVs in VG, replacing metadata
   saved from step 1
5. run command on VG
6. unlock VG

The optimization would usually cause step 4 to be skipped,
and PVs would be read only once.

Running the command in step 5 using metadata that was not
read under the VG lock is usually fine, except for the
fact that lvs attempts to validate the metadata by comparing
it to current dm state.  If other commands are modifying dm
state while lvs is running, lvs may see differences between
metadata from step 1 and dm state checked during step 5,
and print warnings.

(A better fix may be to detect the concurrent change and
fall back to rereading metadata in step 4 only when needed.)

tests: add more tracing info

tests: reduce amount of written date

Since we reduced created LV to 4M - dd also just 4M.

revert "dmeventd: vdo plugin link lvm library"

This reverts commit cbabdf2fca6131660cfb5525ed9edb3f7a41525a.
and add extra comment why this code may look unused, but
in runtime is necessary.

Revert "hints: rewrite function"

This reverts commit 70fb31b5d6863248b5adfb2581b706cbb158b30e.

Revert "debug: enhance debug messages"

This reverts commit e92d3bd1f75d335fba5303c433516ea4ebe5cab1.

tests: enusure lib is initilized

debug: enhance debug messages

hints: drop unneeded memset

strncpy will zero buffer itself.

hints: no need to check for NULL before free

free() itself checks for NULL.

cov: use zalloc

Instead of malloc() memset() -> zalloc()

cov: add explicit ret value ignoring

We don't need to check for any error result codes here.

cov: remove unused headers

cov: more checks for failing syscalls

dmeventd: vdo plugin link lvm library

Since we fixed linking of proper version of 'libdevmapper' with
linking lvm2 plugin correctly - we already have correct function
available linked with internal lvm library.
So drop unneeded include of parsing function.

cov: inline _build_desc_write

Embed function into the code, since the function is actually
simpler written this as there are no memleak troubles
with failing allocation error path.

hints: check for _touch_hints

Exit when !_touch_hints().

hints: fix mem leaking buffers

hints: validate allocation result

hints: validate sscanf results

hints: allocate hint only when needed

Avoid mem leaking hint on every loop continue and
allocate hint only when it's going to be added into list.

Switch to use 'dm_strncpy()' and validate sizes.

hints: rewrite function

pvck: check result of dev_get_size

Don't use garbage value for later computations.

cov: fix mem leaking buffer

Free allocated buffer on function's exit.
Also check for fwrite() results.

cov: avoid passing NULL to strstr function

When 'str1' would be NULL, there is no point to run 2nd. strstr().

cov: check for retvalue