test: add some extra tracing command

test: fix race in wait_pvmove_lv_ready with stale lvmpolld entries

The wait_pvmove_lv_ready function could incorrectly accept stale
finished pvmove entries from lvmpolld, causing test timeouts.

When running multiple test iterations, lvmpolld retains finished
polling operations in memory. If a new iteration creates pvmove
LVs with the same names (pvmove0, pvmove1), the wait function
would find these stale entries and consider them "ready" even
though they were from previous iterations with different LVIDs.

Fix by enhancing awk_parse_init_count_in_lvmpolld_dump to only
return init_requests_count for entries with polling_finished=0,
ensuring we wait for actual active polling operations rather
than accepting stale finished entries.

Co-Authored-By: Claude <noreply@anthropic.com>

gitlab: fix typo in test name

Correct test where we ignore failure status
(as kernel is not working correctly).

test: use aux dmsetup with integrated udev_wait

gcc: fix const warnings

New gcc emits new warnings for strstr mismatching constness.

dmeventd: reduce select timeout during shutdown

When dmeventd receives an exit signal, monitoring threads are signaled
to terminate. The threads exit quickly (cleanup completes in ~1ms), but
the main event loop continues sleeping in select() with a 1-second timeout
before it attempts to join and cleanup the terminated threads.

This causes an approximately 1-second delay between thread cleanup
completing and the final DSO/control device release.

Fix by reducing the select() timeout from 1 second to 10ms when exiting.
This allows the main loop to quickly process thread cleanup during shutdown
while maintaining the 1-second timeout during normal operation.

test: add aux tracing

Chasing weird error - so enable tracing of 'aux' functionality here.

test: add some udev synchronization

test: fix missing lvremove

With commit for clvmd removal 492a5a070fb08a869764afec07027abe20f9d8de
we removed 'lvremove' by mistake.

test: rename aux.sh to aux_t.sh

Since lvm2 project seems to be git cloned also on Windows based
machine as submodule avoid using 'aux.sh' as filename in git tree.

We are still using 'aux' within out 'testing' - it's just now
being created from aux_t.sh file instead of aux.sh.

See: https://gitlab.com/lvmteam/lvm2/-/issues/37

Reported-by: Stephan T. Lavavej

gitlab: ignore some test

These test are failing for kernel issue - so ATM
keep them ignored when checking if tests passed.

test: minor updates

Drop sleep before udev_wait.
Use 'sort -u'  over 'sort | uniq'
Avoid some unnecessary 'not'.

test: extend loop

Add few more loop checks before considering repair is not made.

test: add extra write

To ensure there is some scheduled 'write' operation for faster
discovery by dmeventd there is a faulty array member for repair.

test: count with postponning

dmevent may postpostne test when it gets 'error' return code
from failed action.

test: check for lvmdbusd presence

Test checking for presence of running lvm commands would be invalid
if lvmdbusd is running in the system (which could be even unkilled
process started by the test suite itself....)

test: add sync before checkin failure

test: thin-pool is single-segment lv

test: fix and refactor timestamp tracking in TimedBuffer

Refactor TimedBuffer::push() to fix timestamp bugs and improve clarity.

The bug: timestamps were incorrectly updated when processing empty
content at buffer boundaries, causing stale timestamps to persist and
be used for subsequently arriving lines.

The fix: only set timestamp when starting a new line with actual content:
  if ( (b != e) && incomplete.second.empty() && stamp )

Also simplified the code:
- Eliminated intermediate 'now' variable - call gettime() directly
- Converted while loop to for loop for clearer iterator progression
- Reset incomplete buffer with Timespec() instead of current time

This is important for bash 'set -v' output where command text may
arrive in chunks without immediate newlines.
i.e. test like this was giving misleading time stamps:

echo 1
sleep 1
echo 2
sleep 2

Co-Authored-By: Claude <noreply@anthropic.com>

gitignore: update

doc: fix release dates

post-release

pre-release 2.03.37

WHATS_NEW: update

make: generate

lvscan: blockdevice option is unused

test: drop usage of LVM_BINARY

Actually we don't want ot define this variable to better
test real installed lvm2-testsuite package

spec: build rpm with timestamp of current date

lvmpersist: remove unregister error message

If PR was already stopped, stopping PR again would
generate an "sg_persist unregiser error" message
when each sg_persist command returned an error.
The actual result of the lvmpersist stop command
is determined by checking that the key is removed,
not by the sg_persist exit code.  If the key is not
removed from any device, then the command fails with
another higher level error message.

make: generate

revert "debug dmevent"

Unwanted.

debug dmevent

test: umounting by dmeventd

Correct and simplify test.
Use LVM_BINARY for path to lvm binary used by script.

test: aux updates

Ssend eval stderr to null as there is no need for extra trace
of eval values in teardown.

Drop passing args to lvmpoll_dump.

test: show lvmlockd debug info in testing

lvmlockd: reduce require of two SIGTERM signals in shutdown

When lvmlockd received SIGTERM, for_each_lockspace() would signal
lockspace threads to stop and then immediately check if they were
done. Due to scheduling, threads often hadn't finished yet, causing
for_each_lockspace() to return a non-zero count, which reset
daemon_quit to 0. This required a second SIGTERM to actually exit,
adding delays to daemon shutdown in tests.

Two issues were fixed:

1. Fixed mutex synchronization for thread_done flag:
   The lockspace thread was setting ls->thread_done while holding
   only lockspaces_mutex, but for_each_lockspace() was reading it
   while holding only ls->mutex, creating a race condition. Now
   thread_done is set while holding both mutexes in the correct
   order (lockspaces_mutex first, then ls->mutex).

2. Added wait loop during daemon shutdown:
   When daemon_quit is set and threads are signaled to stop,
   for_each_lockspace() now waits up to 100ms (checking every 1ms)
   for threads to actually finish before checking their status.
   This wait only activates during daemon shutdown to avoid adding
   delays to normal lockspace operations.

In practice, threads finish in 1-2ms, so shutdown completes quickly
with a single SIGTERM.

Co-Authored-By: Claude <noreply@anthropic.com>

deviced_ids: extended refresh period

If an automatic device ID refresh is triggered, and does not find
some device IDs, then extend the refresh behavior for a number of
seconds, configured in lvm.conf:

device_ids_refresh=<seconds>  accepts 10-600 seconds

Previous settings remain the same:

device_ids_refresh=0  disables automatic refresh
device_ids_refresh=1  enables one refresh

While the refresh period is extended, a new line appears in
system.devices: REFRESH_UNTIL=<YYYYMMDDHHMMSS>

The timestamp is written by the first lvm command to trigger the
refresh, and is set to current time + device_ids_refresh seconds.
REFRESH_UNTIL is removed when all devices are found, or when the
time expires.

test: tune slowdown

Here we have some tricky case - ATM we are using suspend with
flushing when changing DM tables for cache volume - but this
can take too much time with our delayed device.
So trying to minimize chance to hit this race on very fast machines.

test: improve lvmpolld_talk error messages

Refactor lvmpolld_talk() to provide clearer diagnostics when connection
fails. Previously misreported socket failures as "nc not available".
Now distinguishes between missing tools, missing socket, and connection
errors with actual error details.

Co-Authored-By: Claude <noreply@anthropic.com>

test: enhance _check_size

Grab lv_size as the first thing.
Make tests easier and drop '_not' variants.

cache: add debug message

libdaemon: log which signal triggered daemon shutdown

Capture and log the signal number that triggers daemon shutdown.
This helps diagnose unexpected daemon terminations during testing.

test: add more debug for abort case

Improve logging to better capture lvconvert problem.

dmeventd: drop unused var

fdd no longer needed.

signals: rename interruptible_usleep to sigint_usleep

The interruptible_usleep() function returned 1 on interruption (error)
and 0 on success, opposite to LVM2 convention where 1=success, 0=failure.

Changes:
- Rename to sigint_usleep() to align with sigint_*() function family
- Invert return: 1 if sleep completed, 0 if interrupted
- Update all call sites to use ! operator for error checking

This makes the API more intuitive as ! clearly indicates error path
and return values match standard LVM2 conventions.

Co-Authored-By: Claude <noreply@anthropic.com>

WHATS_NEW

make: generate

lvmconfig: make it possible to print various config types in list mode

For example:

❯  lvmconfig --type diff
report {
compact_output=1
headings=2
}
activation {
polling_interval=1
}

❯  lvmconfig --type diff --list
report/compact_output=1
report/headings=2
activation/polling_interval=1

config: add support for list mode for all cfg_def_tree_t types

Before, we printed only CFG_DEF_TREE_LIST as a list instead of
structured tree. But we can also print all the other types as lists
if needed which might be more suitable.

This patch adds a 'listmode' switch to struct config_def_tree_spec
so that the config printing functions (_out_prefix_fn, _out_line_fn,
_out_suffix_fn) can then decide on the actual format.

test: use 4k blocks for fs

test: correcting wait shell vars

test: just skip test if there are suspended devices

test: use should for a while

test: aux type

Removed extra '/'.

test: missed removal of clvmd testing

dmfilemap: stray close code reuse

Reuse header file from libdaemon directory here.
It has better code to close stray descriptors - since there
are systems with very high numbers of supported opened descriptors.

In case project would ever split header file would need to be copied.

Also use >=  check for valid file descriptor.

WHATS_NEW: update

test: aux rename wait variable

test: rework bare wait commands

Replace bare 'wait' commands with specific PID waiting to prevent
test hangs when base shell runs process in the background.

When tests use 'wait' without arguments, it waits for ALL background
processes. This may causes tests to hang indefinitely.
Fix by capturing background job PIDs and waiting
only for those specific processes.

Pattern changed from:
  some_command &
  wait

To:
  some_command &
  SOME_PID=$!
  wait "$SOME_PID" || true

The '|| true' handles cases where the process has already exited.

Co-Authored-By: Claude <noreply@anthropic.com>

test: aux kill_sleep_kill

test: remove clvmd testing infrastructure

Remove all clvmd-related test code since clvmd was removed from lvm2.
This includes prepare_clvmd(), SKIP_WITH/WITHOUT_CLVMD variables,
LOCAL_CLVMD conditional blocks, and outdated comments. Tests now run
unconditionally with non-cluster behavior.

Co-Authored-By: Claude <noreply@anthropic.com>

test: simplify corrupt_dev() function

Replace strings+grep with grep -oba for byte pattern search. This is
more direct and efficient - grep finds the byte offset, we extract it
with cut, and use dd to overwrite at that position. Also add
conv=notrunc to prevent truncation.

test: aux easier code for building config

For older bash use more simple code.

Co-Authored-By: Claude <noreply@anthropic.com>

daemonize: fix file descriptor leaks on error paths

Fix resource leaks in daemonization code across multiple daemons by
properly closing /dev/null file descriptor on all error paths.

Changes across all daemonizing code:
- Skip dup2() when fd already equals target (no-op optimization)
- Close fd on error paths before exit (prevents leak)
- Add Coverity annotations on both error and success paths
- Use consistent == -1 pattern for dup2() error checks
- Remove explicit close(0/1/2) in cmirrord (dup2 handles atomically)

This eliminates resource leak warnings from static analyzers while
maintaining correct daemonization behavior.

lvmpolld: signal child processes before pthread_cancel

Fix lvmpolld shutdown to properly terminate child lvpoll processes
before cancelling monitoring threads.

Previously, pthread_cancel was called on monitoring threads while
their child processes were still running. This could leave orphaned
lvpoll processes and cause shutdown delays.

Now send SIGTERM to child lvpoll processes first, then immediately
cancel the monitoring threads which will reap the children during
their cleanup.

This significantly reduces shutdown time in tests with lvmpolld.

Co-Authored-By: Claude <noreply@anthropic.com>

libdaemon: improve graceful shutdown of client threads

Optimize daemon shutdown to allow client threads to exit quickly
without blocking on socket operations, and refactor code to improve
maintainability.

Functional improvements:

1. Close listening socket BEFORE waiting for client threads
   - Prevents new connections during shutdown
   - Unlinks socket file earlier in shutdown sequence

2. Shutdown all client sockets with SHUT_RDWR
   - Interrupts blocking read() calls in client threads
   - Allows threads to exit immediately instead of hanging
   - Uses shutdown() to wake blocked threads, not just close()

3. Add early exit check in client thread loop
   - Check _shutdown_requested at start of while loop
   - Allows threads to exit gracefully when shutdown is initiated

Code organization improvements:

4. Eliminate goto statement
   - Replace "goto out" with direct function calls
   - Improves code readability and control flow clarity
   - Both daemon_main and daemon_init paths now follow same pattern

5. Extract _shutdown_sockets() helper function
   - Centralizes all socket shutdown logic in one place
   - Handles both listening socket close and client socket shutdown
   - Respects _systemd_activation flag for socket file unlinking
   - Includes comprehensive logging for shutdown operations

6. Extract _daemon_cleanup() helper function
   - Common cleanup path for both daemon_main and normal execution
   - Handles daemon_fini callback, logging, pidfile removal
   - Eliminates code duplication between execution paths

This allows daemon to shutdown cleanly without client threads
blocking on socket operations, enabling faster test execution and
cleaner shutdown sequences.

Co-Authored-By: Claude <noreply@anthropic.com>

cov: missing free

With recent commit f7a308d2e71e332a9cbc01d917182ba562943b77
we missed to relase allocated idname.

dmeventd: fix missing lock when reading _timeout_thread_id

Acquire _timeout_mutex before reading _timeout_thread_id to match
the locking pattern used when writing this variable. While the
variable is only written once and never changed, static analyzers
flag the inconsistent lock usage.

Release mutex before pthread_join() to avoid holding locks during
blocking operations.

make: generate

test: we need fs on origin as well

Last commit made fs only on snapshot - but it's needed also on origin.

test: bash clenaups

Various improvemetns to make shellcheck less complaining.

test: check simplified

We do not need to use 5 second sleep here for checking
opened device cannot be removed - just check with mounted
snapshot - which will also be seen as opened device - but
with known holder - thus 5sec retry loop is avoided.

test: require newer target

man: update pvmove for pvmove_max_segment_size_mb

dmeventd: cleanup

Show help options in alphabetic order.

vgchange: limit persist stop in lockstop

"vgchange --lockstop --persist stop" should skip the
persist_stop() on a VG without PR required, otherwise
it hits errors attempting to stop PR on VGs where it's
not used.

test: select-report: fix checking of failure scenarios

man: update lvmdevices refresh info

test: fix lots of not usage

Added lots of FIXME - since these tests
are not working properly - but ATM it's unclear
what they are supposed to test.

test: fix use of not mimages|mirrorlog....

test: fix use of not is_lv_opened_

Correct the test and wait till dmeventd really umounts directory.

test: fix typo group to grep

We actually want to use 'not grep'.

test: fix use of not sel

Looks like we need to fix 101%.

test: fix use of not test

Fixing 'not ' and making it more readable.
Handle blkid error code 2.

test: fix use of not fill

make: generate

config: stop logging that a default value is used

default values are not interesting to see

lvmlockd: fix lock_args flags usage with dlm

lockd_lockargs_get_meta_flags() only applies to sanlock, and
would cause dlm lockspaces to fail beause it doesn't recognize
the cluster name in a dlm VG lock_args string.

lvmlockd: fix to allow for no vg_args

man: update lvmdevices

lvmdevices: update force option

When lvmdevices --update doesn't see any reason to update
system.devices, it doesn't update.  Let the user force an
update in case they see a reason to do it.

device_id: prevent incorrect dm uuid idtype

If a dm-uuid-based device was added to system.devices, and
the wrong dm-based type was specified, then the system.devices
entry would have the correct idname but the wrong idtype. e.g.

lvmdevices --adddev /dev/mapper/mpatha --deviceidtype crypt_uuid
would add an entry with: IDTYPE=crypt_uuid IDNAME=mpath-<uuid>
rather than the correct: IDTYPE=mpath_uuid IDNAME=mpath-<uuid>

This mistaken type would not affect lvm, and the device would
be entirely usable.  The dm-based types (mpath, crypt, lvmlv)
were interchangable because they all read the device id from
the same dm/uuid sysfs file.

Fix this by adding a missing check that the dm uuid prefix is
correct for the specified idtype.  Also, accept an existing
system.devices file which contains this mistake.

lvmdevices: new option --listids shows id types and names for dev

lvmdevices: new option --delid to delete device by device_id

lvmdevices: new option --addid to add device by device_id

test: aux improve daemon killing

When daemon takes multiple pids, handle them via array.
(as recently made with lvmlockd)

makefile: indent

gcc: fix fp complain about use of unit

Just quiting warning.

vdo: add missing synchronization

After wiping, formating and deactivating vdopool volume,
we should wait on udev before we continue and activate
same volume again with different UUID.

revert "test: update for new vg attr field"

This reverts commit 2855526824fd815bb0394eda3973d34cc4b985f7
for incorrect merging.
We already have 85bef39ed00dd4f657cb7781f2754f5bb5c5ac86.