dmeventd: enhance time waiting loop

dmeventd is 'scanning' statuses in loop (most usually in 10sec
intervals) - and meanwhile it sleeps within:
pthread_cond_timedwait()

However this function call tends to wakeup sometimes a short amount of
time sooner - and our code still believe the 'right time' has not yet
arrived and basically for a moment 'busy-looped' on calling this
function - so for systems with 'clock_gettime()' present we obtain
time and we go 10ms to the future second - this avoids unneeded
repeated invocation of our time scheduling loop.

TODO: monitoring during 1 hour 'time-change'...

pvck: use dm_config_parse_without_dup_node_check

instead of dm_config_parse.  Some strange case could
cause dm_config_parse to print duplicate warnings about
all the metadata fileds.

tests: reduce sizes in pvck-dump and improve checks

Smaller devs can be used so tests can be run on small vms.
Improve checks.

tests: pvck dump from larger metadata areas

pvck: allow dump from file

pvck: fix reading large mda1

When mda_size is larger than io_memory_size, reading
the entire mda fails unless the previous read of the
label has been invalidated.

pvck: improve mda_offset mda_size choices

Attempt to calculate an offset or size if one only
value was specified in the settings.

Use header values when available.

pvck: print longer command description

pvck: ensure text lines are terminated

hints: free hint list in error exit path

man: lvmcache raid1 references

tests: validate vdo slab_size

New vdoformat can print this size - so check we pass proper bit count
matching preset value.

vdo: fix slab size bits calculation

When formating VDO volume, the calculated amound of bits
for 'vdoformat --slab-bits' parameter was shifted by 2 bits
(calculated size was making 2MiB vdo_slab_size_mb value appear like if
user would be specifying only 512KiB)

Fixed by properly converting internal size_mb value to KiB.

writecache: check watermark value

writecache: allow removing wcorig lv

like removing corig

writecache: fix watermark error message

writecache: working real dm uuid suffix for wcorig lv

writecache: drop real dm suffix

fixes the problem of adding writecache to an active LV

thin: don't use writecache for poolmetadata

writecache: check if cachevol is writable

before trying to initialize it (since wipe_lv
does not return an error if it fails to write.)

cachevol: stop dm errors with uncaching cache with cachevol

Fix the anoying kernel message reported:
device-mapper: cache: 253:2: metadata operation 'dm_cache_commit' failed: error = -5
which has been reported while cachevol has been removed.
Happened via confusing variable - so switch the variable to commonly user '_size'
which presents a value in sector units and avoid 'scaling' this as extent length
by vg extent size when placing 'error' target on removal path.

Patch shouldn't have impact on actual users data, since at this moment
of removal all date should have been already flushed to origin device.

m

post-release

pre-release

vdo: fix vdoformat when -V is specified

The previous patch improved read of pipe when lvm2 was looking
for default logical size, but we clearly must read pipe also
for -V case, when the logical size is already defined.

writecache: skip zeroing in test mode

writecache: check for invalid cachevol

writecache: fix return value

raid: better place for blocking reshapes

Still the place can be better to block only particular reshape
operations which ATM cause kernel problems.

We check if the new number of images is higher - and prevent to take
conversion if the volume is in use (i.e. thin-pool's data LV).

writecache: prevent snapshots

there appear to be problems with taking a snapshot
of an LV with a writecache, so block it until that
is understood or fixed.

writecache: fix splitcache when origin is raid

WHATS_NEW: update

generate: remake

Regen man page.

lv_manip: add extra check for existin origin_lv

clang: it's supposedly impossible path to hit, as we should always
have origin_lv defined when running this path, but adding protection
isn't a big issue to make this obvious to analyzer.

raid: add internal error for no segment

clang: capture internal error when data_seg would not be defined.
(invalid LV with no areas)

lv_manip: add error handling for _reserve_area

Since _reserve_area() may fail due to error allocation failure,
add support to report this already reported failure upward.

FIXME: it's log_error() without causing direct command failure.

command: validate reporting of previous argument

When reporting parsing error, report 'previous' argument
only when there is one.

dmeventd: nicer error path for reading pipe

When _daemon_read()/_client_read() fails during the read,
ensure memory allocated withing function is also release here
(so caller does not need to care). Also improve code readbility a bit
a for same functionality use more similar code.

lvmlockctl: use inline initilizers

clang: ensure r_name[] is in all possible paths defined.

lvmlockctl: ensure result value is always defined

Ensure passed pointer gets predefined value (instead of random stack
value).

lvmlockd: move eval of ENOENT

To avoid logging 'errors' for no real error state (ENOENT),
move this evaluation upward in the code.

cov: check error code from mutex init

cov: widen before calculating min_chunk_size

Although we expect min_chunk_size to be 32bit value, for
large size of caches it might be useful to do calcs 64bit.
So to avoid doing shift as signed 32bit - use unsigned 64bit
from the start.

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