lvcreate: accept lockopt option

The redundant lvcreate.c implementation of accepted options needs
to be removed.

test: improve for use with older systems

Improve test, so it's also working on systems without delay_dev
and test is actually more 'race' resistant.

raid: remove struct overlap with possible_takeover

After reorganizing elements in `possible_takeover_reshape_type`
(in commit 5b92ce741f6bcb9b3d3c19c0fc13b972f950c560),
it became apparent that the code relied on struct overlap,
which is somewhat unsafe. This commit removes it and ensures
proper `const` qualification for the struct usage.

lvmlockd: lockd_vg return value cleanup

lcov: ignore some errors

While building lcov files - ignore errors from 'negative' counter
(perhaps we can use -fprofile-update=atomic - but it would be another
slowdown of test runs)

Also ignore unexecuted blocks warnings with 'gcov'.

Failure of lcov goal is not supposed to error whole make build.

cleanup: match prototype

cleanup: use full source path to header

test: handle lvmpolling case

When test runs with lvmpolld - we cannot check
messages from pvmove - as those are visible through
output of lvmpolld - so just skip this and
only check LVs are in expected state.

coverity: cleanup model code

gcc: better structure padding

Since we use '.option' to assign struct member - just suffle
structure element for better padding.

gcc: fabsf float based operation

Use 'float' version of 'fabs()' to compare floats.

clang: just use regular final else

There is no need to compare, so just keep 'if' part as the comment.

sanlock: more variable structure part to the end

Move variable part so there is no need to use 'gnu' compiler extension.

mirror: make sure 0 is not clz arg

Just make sure there will be never ever used '0'
as parameter for clz().

lvmcmdline: fix printed error messages

Since long_opt was changed to char[], we were only comparing pointers
that always exist, whereas the original intention of the test was
to verify the presence of a string
(i.e., checking that the first byte is not \0).

raidintegrity: support removal of partial images

vgreduce --removemissing --force replaces a partial image
with an error target.  When that image includes an integrity
layer, that layer needs to first be removed.

WHATS_NEW: update

test: aux gives more time for lvmdbusd start

Try 20s delay to see whether dbus service will be alive...

test: split mirror with opened leg

test: check layout for error and zero

test: pvmove with open temporary volumes

Test pvmove logic when various pvmove mirror volumes are opened
while pvmove is finishing.

gcc: compare same signedness

vdo: correctly size string array

Fix logging of VDO configuration info message which has acutally
printed " and," using next element..
Increase the array element size so it can store >=5 bytes
for " and" + \0.

mirror: splitting handles opened devices

Similar to the pvmove update, enhance error path handling
for scenarios where legs or logs remain open and cannot be
closed during the splitting of a mirror image.

Remove the now obsolete _delete_lv() function,
as it will no longer be needed.

mirror: move status bit masking

Use bit masking in replace_lv_with_error_segment().

debug: update message for pvmove

Add some more consistency to message printed for pvmove.

debug: use more similar skipping messages

debug: reinitialize debug pid after fork

lv_manip: print layout for error and zero

mirror: enhance error path for pvmove finish

When the pvmove operation is completing, it attempts to deactivate
the temporary mirror and remove its mirror legs. However,
if an external tool holds these volumes open, the operation would
previously abort entirely, leaving the LVM2 metadata in a partially
unusable state that required manual administrative fixes.

To improve this, the code has been enhanced to handle such scenarios
more gracefully. It will now complete the pvmove operation even
if some volumes cannot be deactivated, marking them in the metadata
with an error segment. While the command will report errors,
the metadata will remain in a usable state. The administrator
can then remove the orphaned volumes when they are no longer in use.

devices file: fix backup limit

Fix the code that limited the total number of backup files.
It failed, and left excess files, when the file version
number was greated than 9999, exceeding the four digit suffix.

Now, after version 9999, the suffix intentionally grows beyond
four digits as needed, and is not a fixed width, or zero padded.

lvmlockd: correct prototype for sanlock-less build

lvmlockd: compilation fixes

gcc: std=c11 for test suite as well

debug: missed stack tracing

raid: refactor code to _raid_leg_degraded

Move common code into _raid_leg_degraded().

lvmlockd: fix free_lv immediately after init_lv

Fix for recent commit "lvmlockd: free resource structs for LVs"

When a vg_write() fails in lvcreate, lvmlockd sees init_lv()
followed by free_lv().  The LV lock is not acquired prior to
free_lv, and no prior resource struct exists. This wasn't being
handled.

lvmlockd: log correct name in error path

fix for recent "lvmlockd: free resource structs for LVs"

vg_read: correct error path for DM cache update

New code for updating DM cache traveled through couple destination
however in this place only 'return_0' is missing unlocking in
error path.

tests: more info from test

dev-type: require 4k only for some topology attrs

Previous commit 5f71cebcbecc37b5291018da35b3efe9636bf6c3 was not
correct. 4k requirement cannot be put on attribute_offset - where
it is valid to have this only 512b aligned.
The rule might get more complicated to recognized invalid values.
For this moment however add more easier requirement - we
impose 4K restriction on minimal and optimal io size if they
are bigger then 1 sector (512B).

WHATS_NEW: update

cov: annotate

cov: ensure buffer does not underflow

Add explicit check for having always positive value.

cov: avoid expression overflow

Keep max memory in MiB  (>> 20).

cov: explicitly ignore return codes

cov: add validation of FID pointer

Ensure NULL theoretical NULL FID pointer is not
dereferenced.

cov: limit buffer size for sscanf

Limit sscanf parser buffer width for string.

cov: use bigger buffer

Since pvck's  _check_vgname_start is actually checking
up-to (NAME_LEN + 2) bytes  (130),
use at least this size for (MAX_LINE_CHECK + 3).

cov: limit log levels to 7

lvm2 is using 7 levels for logging.
(Which might be already probably too much...)

cov: add sscanf width specifiers for lvmlockctl

cov: add space for string 0

Sscan may automatically add 0 after field width mark,
and since it's not exactlu trivial to do a macro calculation
for PATH_MAX - 1, rather make buffer for sscanf results bigger.

Also use matching FSTYPE_MAX as field width specifier.

cov: add extra byte for 0

Since we use 'DM_NAME_LEN' size for sscanf - it may need to store
extra string 0, which is added automatically after 'maximum field
width' specifier.

gcc: updates for use with std=c11

Let gcc -std=c11 pass...
Useful i.e. to get working Coverity scan ATM.
(as gcc15 and stdbool changes makes it problematic.)

debug: drop double '/' in created pathname

Our cmd->dev_dir already has '/dev/' so do not use '/dev//devname'.

tests: update topology test

Check lvm2 create usable VG for optimal_io_size 16776704
(which is not divisible by 4KiB).

dev-types: ignore non-4K divisible topology values

When topology value is bigger then 1 sector (512b)
we require 4K divisibility for them.

config: check for 0 from read

When the read returns 0, it could mean the file
was trimmed or some other type of problem.
In this case abort the potentially endless loop.

vg_read: matching missed empty cache

lvm2 is caching DM nodes with the use of DM_LIST_DEVICES ioctl().
And tried to preserve the cached structure for the same list,
however there was 1 case where cache was empty, and new LIST ioctl
returned some elements - if this DM table change has happened
in the moment of 'scanning' and locking -  lvm2 has then continued
to use 'invalid' empty cache.

Fix by capturing this missed case and update cache properly.

TODO: we could possibly use plain memcmp() with previous ioctl result.

vg_read: rescanning DM cache after taking lock

Since we started to use DM cache now also for basic checks
whether the DM devices is present in DM table, this cache
now needs to be actually refreshed when the LOCK is taken.
This hiddenly happend if there was enabled 'scan_lvs' however
still not at the right place.

Move this explicit cache update call right after the moment
vg_read grabs the lock.

TODO: in the optimal case, we should mark the 'cache invalid'
and later refresh this cache, when the first reader appears,
but since this would be large patch, do this little fix step patch
first and improve performance later.

lvmlockd: fix thin locking case for cow snapshot

The case of lvcreate creating a cow snapshot of a thin volume was
caught by the sanity check for any missed instances of thin locking.

lvmlockd: free resource structs for LVs

Once created, resource structs for LVs were never being freed.
If LVs are activated, then later removed or never used again,
the unused structs waste memory and cause the resource list
to grow.

lvmlockd: thin locking improvements

There was a lot of messy and inefficient locking calls sent from
a command to lvmlockd when working with thin volumes, e.g.
- requesting a lock numerous times that was already held
- releasing a lock numerous times that was already unlocked
- repeating lock/unlock/lock/unlock rather than holding the
  lock until it was no longer needed

Mistakes in the locking could easily hide among all the noise.

The mess was largely because thin-related commands involve a
lot of internal LV manipulations, and lvmlockd calls were done
at the lower level of LV activation/deactivation.  This change
adds locking code that is more specific to the thin command
being run, so it can be more intelligent in acquiring and
releasing locks where needed.

lvmlockd: debug log message improvements

lvmlockd: reject bogus lease offset values

doc: Release Notes for 2.03.30

post-release

pre-release 2.03.30

WHATS_NEW: update

doc: Try to test limit of release notes

This works with newer ikiwiki when building locally, but not on the
gitlab.

memlock: skip locking unused libs

Skip memlocking libraries unused in critical section.

WHATS_NEW: update

configure: autoreconf

configure.ac: notify_dbus works fine with older version

Picking version 218 for notify_dbus support - maybe older works too,
but not being tested...

configure.ac: report 'none' when nvme_wwid is missing

cov: check syscall

cov: just be sure nsid is defined

cov: remove unused header

cov: some error path updates

tests: cleanup typo

man: fix double words

This patch fixes double words found in man.

Signed-off-by: Masanari Iida <standby24x7@gmail.com>

tests: test unsupported lvresize of origin vdo lv

tests: check memory locking

Check reserved memory and stack is able to disable mlocking.

lvresize: vdo origin resize is unsupported

Add explicit error message for not (yet) supported resize
of origin VDO volume.

lvmcmdline: memlock init after config parsing

Since we now support disabling memory locking by setting
reserved memory or stack to 0 - it would be useful if this would
work also with cmdline --config option.

TODO: rework creation and usage of cmdtool context so we avoid
several places in the code which do try to initialized something...

memlock: check for proper reserved size

Fix regression introduced with commit:
964012fdb924076e9ab97fabe00e759ddbf7c3bd
that effectively disabled memory locking before suspending volumes.
From merging/testing there remained wrong condition
as we really want to check for 0 memory reservation value
for both checked settings.

WHATS_NEW: update

tests: adjust lvresize-xfs tests for recent lvextend changes

Because now, we are doing the fsinfo check before extending an LV and if
that check fails, we do not proceed to the LV extension itself and the
lvextend command bails out immediatelly.

filesystem: add new_size_bytes to struct fs_info

It seems we need new_size_bytes in places where struct fs_info is also
passed. Store the new_size_bytes inside the struct fs_info so we
can just pass that one to all the functions we call and hence make
the code a bit cleaner and easier to follow.

lv_manip: check fs resize is supported before LV extension

This avoids a situation where we would extend an LV and then we would
not do anything to the FS on it because the FS info check failed for some
reason, like the type was not supported (e.g. swap) or we could not resize
the FS unless being in some supported state (e.g. XFS to be mounted for
the xfs_growfs to work).

Before this patch (LV resized, FS not resized):

❯  lvextend --fs resize -L+4M vg/swap
  Size of logical volume vg/swap changed from 32.00 MiB (8 extents) to 36.00 MiB (9 extents).
  File system extend is not supported (swap).
  File system extend error.
  Logical volume vg/swap successfully resized.

With this patch (LV not resized, FS not resized):

❯  lvextend --fs resize -L+4M vg/swap
  File system extend is not supported (swap).

WHATS_NEW: update

make: generate

args: missed option all for vgdisplay

tests: check conversion of in-use volume

Thin-pool conversion fails early when trying to convert
volume which is in use  (simulated by sleep <)

thin: deactivate converted volume early

Deactivate converted volume to pool early, so the conversion
exits early and does not leave some already created metadata
volumes that needed manual cleanup by user after command
aborted its conversion operation when the converted volume
was actually in-use  (i.e. when user tried to convert
a mounted LV into a thin-pool, 2 extra volumes needed removal).

configure: autoreconf

configure.ac: add support for libnvme

Add 2 new options for linking  libnvme with lvm2.
Option  --without-libnvme, --disable-nvme-wwid

device_id: nvme devices may use alternate wwids

Device quirks may cause sysfs wwid file to change what it
displays, from a bogus eui... string to an nvme... string.

The old wwid may be saved in system.devices, so recognizing
the device requires finding the old value from libnvme.

After matching the old bogus value using libnvme, system.devices
is updated with the current sysfs wwid value.

doc: Fix limit for inlined release notes

udev: create /dev/disk/by-diskseq/<DISKSEQ> symlink for public DM devices

All block devices have a disk sequence number assigned (an ever-increasing 64 bit
sequence number) since kernel v5.15 (February 2021). The number is exported through
/sys/block/<disk>/diskseq property and also as DISKSEQ udev event variable.
The diskseq helps with referencing a device throughout its existence in
race-free way.

By default, the /usr/lib/udev/rules.d/60-persistent-storage.rules set
/dev/disk/by-diskseq/<diskseq> symlink for each block device. However,
these rules do not apply for DM devices because we manage the symlinks
ourselves in 13-dm-disk.rules where it properly follows the
DM_UDEV_DISABLE_DISK_RULES flag, among other things.

Add a rule to 13-dm-disk.rules to create the /dev/disk/by-diskseq/<diskseq>.