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

lvmdevices: fix memleak of use_old du

missed in the the prev done_old commit

cleanup: remove lvmcache_has_bad_metadata

cleanup: drop unused code

filtes: remove lvmetad bits from usable filter

Eliminate some dead code from usable filter, that is no longer usable
after removal of lvmetad.

device_id: simplier loop control

device_id: use dm_strncpy

Simplify code.

deviced_id: use zalloc

While moving to dm_strncpy, in this case since pvid is also
passed to crc calc, make sure whole buffer is always initilized.

filter: zero internal structure

Valgrind noticed access to unitialized warned_filter.

lvmdevices: fix memleak of done_old du

'done_old' list missed released.

cov: cast to uint32

cov: validate count is bigger then 0

To avoid teoretical underflow...

tests: better constructor

cov: validate major and minor

Check  major and minor is >= 0.

cov: store 64b flags

Alhtough no target is currently using that many bits for their flags,
preserve the size for loaded segment internally.

cov: validate i before underflow

Although it can't happen in this code, ensure 'i' is not 0
before decrementing it.

cov: remove unnecessary sanity check

This sanity check actually confused in some way Coverity
giving it some assumption about array access.

Since these two checks basically validated compiler's capability
to add and then substract the number from char pointer we likely
don't really need them - as if this base math would not work,
compiler would be having far more troubles...

So drop them - and get rid of report:
Event overrun-call: Overrunning callee's array of size 513 by...

cov: use dm_strncpy

Use better variant.

cov: no overwrite of const qualified fields

cov: reset FILE on error path

Although fclose() fails - we should not be retrying with
already released stucture.

tests: lvmdbusest.py faster test_z_sigint

Reduce #lv and intervals.

We are trying to ensure that the daemon stops while it's busy processing
its internal queues.  Decrease the number of LVs and intervals to allow
this test to complete in less time.

tests: lvmdbustest.py skip if g_tmo != 0

This test doesn't have any additional value when running when g_tmo != 0.

lvmdbusd: Exit faster

Add a global timeout value to be used for the threads to end waiting for
whatever it is they are blocked on.  The values varied from 2-5 seconds,
which is way longer than needed.  Value of 0.5 shows no CPU load when
service is running and is idle.

cleanup: drop unused code

Code was related to long time obsoleted  vgconvert
for lvm1 to lvm2 conversion.
This code likely was missed to be removed.

debug: update some structure members

Makes pahole output look slightly better.

commands: reduce command structure size more

Check for internal limits, if they would ever need to be raised report
error message and fail parsing.

cov: ensure buffer always ends with nul

When we print anything out of buffer, ensure there is \0.

cov: ensure nul ends string

tests: skip some mirror tests for polld and lockd

Leave just lvconvert-mirror-basic-1 test
for polld and lockd testing.

gcc: use uint16_t for counters

gcc: move declaration into ifdef

Prevent unused warning when the ifdef code is not compiled.

gcc: match signed integers

configure: lcov needs -fprofile-update=atomic

To avoid negative counters.
See: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68080

device_id: use dm_basename

Avoid problems for other libc like muslc and use dm_basename.

Prototype for basename has been removed from string.h from latest musl [1]
compilers e.g. clang-18 flags the absense of prototype as error. therefore
include libgen.h for providing it.

[1] https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7

Reported-by: Khem Raj <raj.khem@gmail.com>

dmsetup: refactor getopt_long usage

Shuffle code to avoid using static variable to pass parsed option.
Code is now easier to follow and also number of coverity reports
will go away.
There should be no functional change.

tests: missed reduction of raid check

cov: use proper enum value

cov: ensure namelist is defined

Seems coverity doesn't have good model for scandir.
Let's ensure pointer is seen as defined.

cov: validate return from display_lvname

Ensure strdup will not get NULL.

cov: validate origin pointer

Add internal error for NULL origin here.

cov: drop some unneeded continue

cov: check for syscall resuls

cov: ensure buffer is nul ended

Use dm_strncpy() to ensure ending nul.

tests: fix improper merge within last update

makefiles: ignore negative errors for lcov

tests: check for raid shrink support

Shrink of raid LVs supported from 1.9

cov: check for to_dlm_mode return value

Avoid processing invalid to_dlm_mode in lm_convert_dlm.

cov: check for strftime result

devices_id: add some syscall checks

Add debug tracing for syscall failures.
Also switch some log_error to log_warn when command does not exit
with 'error' result and only warns user.
Easier error path handling.
Initialize some vars at declaration time.

gcc: ensure num has always initilized value

cov: ensure string ends with \0

Use dm_strncpy() that ensures \0 termination.

cov: fix incorrect error check

Check for NULL pointer.
Fix missing release on error path.
Also dir_fd might be just != -1.

tests: update lvconvert-raid.sh

Handle case of removal of orig leg while it's being synchronized.

tests: update raid_leg_status check

Do not wait for progres (non zero processed amount of blocks),
when the raid status already reported 'resync' or 'recover'.

tests: aux avoid discarding created loop devices

Usually it's not a big deal, but couple test creates
large devices and here discard operation just excersice
CPU and consumes time.

tests: reduction needs to ignore fs

Drop 'should' usage and use  --fs ignore when reducing raid LV.

tests: update extend of raid

Here we actually need to slowdown only $dev2 - since repair operation
is only reading data from this device and compares it with origin $dev1,
and if they match there is no write...

commands: refactor memset

Move memset() to the initialization function define_commands().

There is also not much point in clearing memory on command's exit
so drop zeroing of ~2M of RAM.

commands: reduce structure sizes

Reduce slightly RAM usage of running lvm2 and use smaller fields
to count small numbers.

commands: use nul for EOL for compiled-in buffer

Use \0 as EOL in compiled-in syntax description to avoid
unnecessary line copy that just replaced \n with \0.
Also use already splitted lines when possible.

reporting: reuse existing _get_field

Use already existing _get_field() function that also
slightly optimizes 'strlen()' usage.

dev-cache: change locales just once

raid: update dm_get_status_raid

Handle mismatch of reported 'dm raid' status, where the active
raid LV can be actually showing higher numebr of raid leg devices,
that the number of shown status characters.

This can happen if the raid leg is dropped during initial
resynchronization.

udev: create /dev/disk/by-label symlinks for DM devs that have crypto as next layer

We already create /dev/disk/by-uuid symlinks for DM devices which
contain crypto-type as next layer (as identified by blkid).

Also create /dev/disk/by-label symlinks as the labels can be
defined for crypto-type devices too.

Reported and fix suggested by: Patrick Plenefisch <simonpatp@gmail.com>
See also:
https://lore.kernel.org/lvm-devel/CAOCpoWfYjOVNJNt+cnOVXDHiDq2wRogTqBijcUoa7chqOLRa5Q@mail.gmail.com/

WHATS_NEW: update

Commits 48188d7181f735d816e0dac73a7a121ecc667f9b and eb4f744820832aff3d6da1bba14f12e91a5b3535

Upstream db_persist udev rule from dracut

Setting db_persist is required for dm devices so that their properties
are carried over on switch-root from the initrd to the rootfs. This
logic has always lived in dracut
(https://github.com/dracutdevs/dracut/blob/master/modules.d/90dm/11-dm.rules).
However, this means that other initramfs generators each have to
implement and maintain the same rule which leads to unnecessary
duplication.

Instead, let's make the rule part of the upstream lvm rules, which
will ensure that generated initramfses will just work if they make
sure the lvm udev rules are installed, without having to figure out
that they have to add an extra rule themselves on top.

Identical rule in Arch Linux's lvm2 package: https://gitlab.archlinux.org/archlinux/packaging/packages/lvm2/-/blob/main/11-dm-initramfs.rules?ref_type=heads

udev: change action check from ACTION!="add|change" to ACTION=="remove"

For DM devices, the add/change/remove can appear as action for genuine
udev events.

However, there are more action types (bind, unbind, move, online, offline)
which never appear as actions for genuine DM udev events, but they can
still be synthesized (e.g. by writing "<action>" to "/sys/.../uevent" file
or by calling "udevadm trigger --action=<action>").

Let's also process these extra action types so that the udev-related content
is not lost completely, keeping all the symlinks and udev db entries just like
this was a synthetic udev event with "change" action.

Related to https://gitlab.com/lvmteam/lvm2/-/issues/4.

WHATS_NEW: update

Commits 6f44e1093c4a06214831d439c5db9e885b70f072 .. 038f9254d9554654197b59c160e3f775af27cdb1

10-dm.rules: bump DM_UDEV_RULES_VSN to 3

Bump the rules version in order to indicate that upper level rules
should consume DM_UDEV_DISABLE_OTHER_RULES_FLAG rather than DM_NOSCAN
and DM_SUSPENDED.

Also update the comments at the top of the file that describe the
exported properties, and add a note about internal device-mapper
properties.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

dm udev rules: don't export and save DM_NOSCAN

DM_NOSCAN is not an official API any more and doesn't have to be
restored from the udev db. Rename it to .DM_NOSCAN.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

dm udev rules: don't export and save DM_SUSPENDED

DM_SUSPENDED is a device-mapper internal flag, which is not intended to be
used by other rules, and which is determined by 10-dm.rules from sysfs for
every uevent. Rename it to ".DM_SUSPENDED", so that it won't be saved in the
udev database.

Known consumers of DM_SUSPENDED are 66-kpartx.rules (from multipath-tools) and
99-systemd.rules (from systemd). These will have to be adapted.
11-dm-mpath.rules will be changed to use .DM_SUSPENDED.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

11-dm-lvm.rules: don't restore DM_UDEV_DISABLE_OTHER_RULES_FLAG from db

DM_UDEV_DISABLE_OTHER_RULES_FLAG is used as the "output" flag of the
device-mapper rules, to be consumed by non-dm rules. It is a logical OR of
several conditions that might make dm devices inaccessible. 10-dm.rules
calculates it for every uevent, whether it's genuine or spurious.

DM_SUBSYSTEM_UDEV_FLAG0 is just another flag that needs to be or'd in. We
don't need to restore the previous state of DM_UDEV_DISABLE_OTHER_RULES_FLAG.
Actually, doing so is wrong if the flag has previously been set because the
device was suspended, and the device isn't suspended anymore.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

10-dm-rules: don't restore DM_UDEV_DISABLE_OTHER_RULES_FLAG from db

We use DM_UDEV_DISABLE_OTHER_RULES_FLAG to tell upper non-DM layers
to keep their hands off the device in question, for any reason.
One possible reason is that the device is supended; another is that
the cookie carries the flag of the same name.

DM_SUSPENDED is not restored from the db, but evaluated anew for every
uevent. Therefore DM_UDEV_DISABLE_OTHER_RULES_FLAG shouldn't be
restored, either. Use a new variable DM_COOKIE_DISABLE_OTHER_RULES_FLAG
to save and restore the original value from the cookie.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

10-dm.rules: test DISK_RO after importing properties

DISK_RO is set in the environment of a block-device uevent if and only if
the read-only (ro) attribute of the device just changed (the kernel
function set_disk_ro() was called). It is not synoymous with the "ro" sysfs
attribute; the device could very well be write-protected if DISK_RO is not
set. Device mapper-level probing is possible for DISK_RO events, but it makes
little sense, because the device propreties haven't changed as far as dm is
concerned. But we should import possible previously set device properties
to avoid confusing follow-up rules. We should do this for both DISK_RO=1
and DISK_RO=0 events.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

13-dm-disk.rules: import ID_FS_TYPE

ID_FS_TYPE is the most important udev property for most follow-up
rules. It must be imported from the udev db if blkid can't be run.

Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Peter Rajnoha <prajnoha@redhat.com>

WHATS_NEW: update

configure: autoreconf with new version

Add  AC_PROG_GREP & AC_PROG_EGREP.
Rebuild with newer version of auto tools packages.

tests: check cachevol makes one table line for -cmeta

When using cache with cachevol, make sure the subdevice -cmeta
creates just one line.

dev_manager: pending traces only existing LV

Before adding LV to the dtree, check whether it's already
in dtree.

dev_manager: open_count not needed

We do not need to check for open_count in these _info calls.

dev_manager: implement _cached_dm_tree_node

dev_manager: fix table for cache with cachevol

When using cached LV with cachevols (so not with cachepool),
the loaded table could have been using more then one mapping line
for sub devices - resulting into data corruption in some cases
when i.e. taking snapshot of such cached LV with and instead of
single line - 2 lines were generated into DM table as the code
skipped protection again repeated addition.

vg-fast_cvol-cdata: 0 16384 linear 253:2 16384
vg-fast_cvol-cdata: 16384 16384 linear 253:2 16384

New code is also refactoring to use _add_new_cvol_subdev_to_dtree
(similar _add_cvol_subdev.. ) and also the addition of subdev has
been moved after check for already processed node.
Also the cachevol sub devices are now added with the insertion
of cachevol with cached LV.

dev_manager: relocate code

Use better place to search for cvol subdev mappings.
Add them with cachevol that is used for cache LV.

dev_manager: refactor

Use struct initilizer to setup lvid.

debug: print names of tracked LVs

Debug was printing passed in LV name, but it's more usable to see
names of LVs 'tracked' for removal.

man: lvm.conf mention missing array brackets

lvconvert: fix displayed lv name in convert to snapshot

tests: add more sleep time

Give some more waiting time to testing loop.

cmdline: some simple optimization

Some cheap parser init speedups cutting away some cpu cycles...
Use cheaper tests in if ( && )
Avoid unnecessary strchr calls.

makefiles: add missing cleanup targer

WHATS_NEW: update

make: generate

tests: external origin chaining

Add tests for stacked usage of external origins.

man: lvmthin update for external origin usage

Document usage of chaining external origins as it is now possibly to
create thinA LV in poolA which use thinB LV in poolB as it external
origin and user could create chain of such LV.

debug: enhace device_mapper debug log for dm tree

debug: trace when ignoring origin_only suspend

Add log debug message when origin_only suspend would be ignored.

lvconvert: enhance external origin conversion