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

tests: add extra dmsetup table

Strange race seems to be occuring here from time to time...

tests: add another explicit barrier

Just to be sure compiler cannot optimize this out.

tests: update aux for vdo checking

Detect also presence of  'vdoformat' tool.
Fallback to 'kvdo' modprobe only when dm-vdo fails
(removed ugly error message from log)

Also add extra check for scsi model being present
so the test can wait a bit if 'scsi_debug' takes longer time.

No need for 'aux' within aux.

tests: ensure sscan buffer is nul terminated

Avoid reading past the read buffer when sscanf is used.

spec: sanlock 3.7 with version >fc27 and >rh7.4

Decode minor version for rhel.

clang: implicit conversion from 'long' to 'double'

implicit conversion from 'long' to 'double' changes value
from 9223372036854775807 to 9223372036854775808.

So rather compare same integer types.

WHATS_NEW: update

configure: autoreconf

configure.ac: 3.7.0 is new minimal sanlock versioh

Require this minimal version of sanlock for usage with lvmlockd.

dmeventd: use daemon_close_stray_fds

Use common code for close stray descriptors.

libdaemon: implement daemon_close_stray_fds

Refactor existing code from tools/lvmcmdline.c to
libdaemon/server/daemon-stray.h daemon_close_stray_fds()
used to close stray descriptors above some specified Fd.

This is code parses content of /proc dir to minimize 'blind' closing
of all possible descriptors within rlimits range.

As we have the same code in few other places in it's more 'trivial'
version - these were actually sensitive to high amount of descriptors,
which might be configured on some system.

With this patch we effectively resolve this reported gitlab issue:
https://gitlab.com/lvmteam/lvm2/-/issues/5

TODO: Current placement might not be ideal - however considering
existing code base constrains it's not so simple.
ATM it uses lib/misc/lvm-file.h  for custom_fds declaration
and rest of functinality is included in daemon header file.

lvm: use configure proc dir in lvm.conf

Update some code parts to use configurer /proc from lvm.conf
(so i.e. our testing can use some fake values)

configure.ac: define DEFAULT_PROC_DIR in one place

Let's move proc into include/configure.h so this define can
be easily used across the source base.

Configure defines it - but ATM we do not provide any configure
option to change it - there should be no need to ever change it.

dmeventd: typo fix

Fix typo and minor reorder of struct member for better alignment.

lvmcache: fix memleaks on list removal

Free members released from the list.

spec: disable LTO

Currently there is not much gain in using LTO when building
testing rpms

spec: replace exists macro with plain shell test

seems this macro works only with recent rpm.

tests: better handling for dmeventd restart

Handle the case, where we  'kill -9' running dmeventd,
and restarting 'dmeventd -R' manages to still contact this
'zombie' damone and manages to get list of monitored devices
out of this and eventually 'run' new and in this case
unexpected instance of dmeventd.

tests: fallback to normal hostname on older machines

hostname -I  is newer option, add some fallback

tests: explicitely use event_activation=1

Some test do expect event_activation to be set.
So add explicit configuring of this setting in tests,
but also add new default which kind of does it globally
as it's expected default  (yet our testing rpms might
be create with disabled event_activation)
By adding this to each test individually - it's now easy
to locate such tests...

dmeventd: reduce some static variable usage

Move static to main() local vars.

Initilize libdm logging also before starting _restart_dmeventd()
so function there can be also logged.

Move call of _info_dmevent() out of option processing - so some
options like -V are processed with higher priority.

dmeventd: no log when after unlink file is gone

Check for errno ENOENT.

TODO: there should be global wrapper for this...

devmapper-event: add unlock into init_fifo error path

Unlock server path if there is open failure for client_path
and whole init_fifos() function is returning failure.

dmeventd: rework _restart_dmeventd

Use _restart_dmeventd() with return values 0,1,2.
Also let's use already created fifos struct.

Make sure the _systemd_activation variable is properly initialized
from _systemd_handover before calling _restart_dmeventd() as
this variable is used there to decide where   1 or 2 should
be returned - aka either letting systemd to initilize
or restart dmeventd itself.

lvmlockd: use sanlock.h for defines

spec: try with default dependency tracking

Check whether we have now all dependencies right.

spec: when building rpm on test machine disable event_activation

This is initial test how to disable event_activation on
machines, where lvm2-testsuite packages are installed
with its lvm.conf file.

TODO: find more elegant mechanism

spec: update default values

Reverse condition and downgrade setting for older versions.
Also define proper version strings

tests: aux extend lvmdbusd timeout for startup

Unclear yet what's going on here - so try waiting up 10second
to see if our service will get there.
Print task list if not...

tests: use lvname with space

Occasionally this test fails as soemtimes UUID actually
may constain LV[d] string causing it to grep mismatch
UUID and LV name and eventually fail test for wrong reason.

As a simple workaround print the LV name first and
check the name is followed by a space character.

WHATS_NEW: update

tests: update unit-tests

Fix some memleaks.
Drop call of bcache_clear_fd() when bache was not initialized.

tests: update for faster dmeventd

We can drop sleep 6, since dmeventd will quit immediatelly
when no running dmeventd can be restarted.

Also restart itself check more conditions.

tests: add missing vgck

This test missed being adjusted with this updated workflow
so it's been testing something different...

tests: aux fallback to sleep

When dbus-send tool is not present on system, fallback to sleep.

tests: drop test warning

These kernel are not going to be ever fixed, so drop warning
being displayed for these tests.

cov: mask false positive

This code is somewhat complex and involves recursion and pointer
shuffling which confuses coverity here.

Let's add masking comment for this warning as there is no double
free in this code.

cov: pass buffer size

gcc: remove warning about inlining failure

Persuade gcc we want this function to be inline and avoid:
  warning: inlining failed in call to ‘lvmlockd_open.constprop.0’:

cov: use check of snprintf result

Do validation in one go with snprintf() result,
so there is no extra strlen() and uncheck possibly truncating snprintf()
visible to coverity.

cov: use stream ptr in its original form

Let's stop Coverity thinking here we are using freed FILE*
for anything else then comparing numbers.
For this use the original source of old_stream pointer.

cov: use dm_strncpy variant

cov: make obvious how free of memory happens

When there is allocation, it needs also deallocator.
Remove no longer needed coverity warning masking.

cov: use safer version with size limitation

Although there is likely not much risk of having chance of overwritting
given buffers use safer variant.
Use dm_strncpy() when just copying %s.

cov: extra pointer validation

Add few more internal errors to enusure there is no use of NULL pointers
along the code path.

cov: replace strcpy with memcpy

When we know the size we can avoid using strcpy.

cov: use memccpy

When we want to copy string which may not be null terminated,
replace strncpy with more correct memccpy.

dmeventd: restart checks for running daemon

Check whether the pid file is associated with running daemon.
If not there is not much point to wait many seconds in loop.

toolib: skip when there is no vg and error_vg

We must skip even if the skip would be suggesting otherwise
if there is no vg and no error_vg as there is nothing to process.

configure: add --with-default-event-activaion=ON

Add new configurable option for building lvm2 with enable/disable
default autoactivation setting.

Might be useful for building i.e. rpms for systems where
this event_activation is not desired.

reporter: correcting failure return value

This function do use ECMD_ return values,
so use ECMD_FAILED as initial value - used for internal error paths

gitignore: add more ignored suffixes

makefiles: add libdm as deps for unit-test

When using just 'make check' ensure we make also dmsetup
so it's not needed to run explicit 'make' to get all the tools.

tests: include date and hostname IPs in test log

When our testing machines are going wild with NetworkManager,
this might be slightly helping.

tests: add timestamp to kernel line

Parse timestamps included in kernel kmsg line and add current system
time to the messsage as well - this makes it easier to look over,
when chasing some messages in journal after some time...

tests: check for supported dBus interface

Instead of blindly relying on sleeping, that dBus understands our
dBus interface, query the info with dbus-send.

tests: query dmeventd instead of sleep

Checking status of running instance of dmeventd should be a better 'sleep'.

dmeventd: check for running dmeventd for status

Before initiation fifo communication, check whether there is
running dmeventd - in case there is no running instance, this
would be just blocked for 5 seconds trying to open fifo.

tests: use non inlined copy constructor

gcc would like to see code without trying to inline large
copy constructor used for std::vector

makefiles: avoid using # with awk

Also awk got annoyed by this \# char sequence which is reported
as incorrect, however older rpm builder were failing without this.
So let's just try other variant.

makefiles: use SHOW

gcc: clear some complains

Use dm_strncpy() were possible to get rid of gcc compile warnings.

WHATS_NEW: update

tests: add sleep

mdadm may eventually be too slow...

pools: no checking for lv/vgremove deactivation

Thin-pool and cache-pool targets got already quite stable so let's try
to remove checking of pools when using lvremove or vgremove commands.

This skips checking pools when they are going to be removed, but it
also when removing thin volume that was the only user of a thin-pool.
In this case thin-pool will be still there and could be activated
again with another thin volume and thin_check will be executed
in this moment. In this case it can delay discovery of metadata damage.

thin-pool: avoid reactivation on new thin-pool

Shortens processing of 'lvcreate -L -V -T' command and
avoid deactivation and its activation with thin_check of the empty
created thin-pool that will be used for the new thin volume
made with a single lvcreate command.

vdo: modprobe dm-vdo for kernels >= 6.9

Modprobe "dm-vdo" with new kernels >= 6.9 and go for
'kvdo' with older Linux kernels.

WHATS_NEW: update

tests: skip convertions tests for usptream driver

tests: add correct condition for skip of lvmpolld tests

Check when flavour want to test  lvmlockd & lvmpolld tests and
in for case ignore SKIP_WITH_LVMPOLLD from test script.

vdo: use stats msg parser for also segment status

Shuffle code to parsing VDO message also for lvs segment status
so it can report correctly data usage for VDO LVs.

For this change move code and also change its API to use just mempool.

Fixes usage with upstream 6.9 vdo target driver.

vdo: fix status message parsing

When using message API for parsing VDO stats info, 0 was wrongly
used for fallback for trying the old sysfs API.

Switch to use  ULLONG_MAX for values that could not have been obtained
through the message call.

Fixes lvdisplay info for freshly created VDO volume with 0 used data
blocks.

tests: updated check for dm-vdo module

With new kernels dm-vdo is now correct name,
so use standard loading and just try extra for kvdo.

tests: inittest fix skipping condition

dev_manager: improve readability

Make a seperate function to decode which ID should be user
for cvol meta or data volume - also avoids duplication of code.
As a result it's now also easier to see how the lvid is build.

device_id: add comment for trim

lvmlocks: rework dm_strncpy

Fix cutting away 1 character via incorrect usage of dm_strncpy
introduced in last batch of commits and use sizeof(buffer) to
get proper sizes.

In case of use strcpy_name_len() the replacement was invalid,
so we need to restore this case since sanlock uses buffer without
nul ending, so we would strip one more character from the buffer.

Also start to use dm_strncpy() without (void) for unchecked usage.

refactor: update dm_strncpy to _dm_strncpy

For checked versions of dm_strncpy use inline wrapper _dm_strncpy.

refactor: remove (void) from dm_strncpy usage

Start to use dm_strncpy() as unchecked version within source tree.

util: add _dm_strncpy

Add internal inline function wrapper for dm_strncpy().
Use it for calls where we test the result.
Avoids emitting warnings in Coverity for unchecked usage.

util: swap header file load order

Ensure libdevmapper is included before util.h since
we are going to add function that is using it.
TODO: maybe add include directly here...

cov: missing initializer

Ensure there is no code path with uninitialized takes_arg.

lvmcache: free list of add_cache_devs