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

post-release

pre-release 2.03.29

WHATS_NEW: update

make: generate

configure: autoreconf

lvmlockd: utilize SD_NOTIFY_SUPPORT

Update makefile and code to better utilize SD_NOTIFY_SUPPORT var.

configure.ac: add option disable/enable-sd-notify

Since commit d106ac04ab34 ("configure.ac: use LIBSYSTEMD"),
lvmlockd is not built with SD_NOTIFY by default but depending
on LIBSYSTEMD_LIBS. There are three prerequisites of
nonempty LIBSYSTEMD_LIBS:
NOTIFYDBUS_SUPPORT, SYSTEMD_JOURNAL_SUPPORT and SYSTEMD_JOURNAL_SUPPORT.

If ./configure is called with options ' --disable-systemd-journal
--disable-app-machineid --enable-lvmlockd-sanlock
--disable-notify-dbus', the lvmlockd built is without sd_notify
support which causes hang of start lvmlockd service in notify type.

This commit adds options disable-sd-notify and enable-sd-notify.
The default value is autodetected and when the lvm2 is build with
systemd then sd-notify is enabled.
If systemd/sd-daemon.h is existed, call PKG_CHECK_MODULES libsystemd.

Signed-off-by: Su Yue <glass.su@suse.com>
Modified-by: Zdenek Kabelac <zkabelac@redhat.com>

vgdisplay: remove command def for activevolumegroups

too minor to care about separating

pvdisplay, vgdisplay, lvdisplay: changed listed order

List the common form first, followed by the others.

vgdisplay: add command def for activevolumegroups

This variant does not accecpt positional args like
the others.

vgdisplay: split code for variants using command defs

pvdisplay: split code for variants using command defs

Also fix the use of --all that was mistakenly included
as an accepted option for vgdisplay and two cases of pvdisplay
in commit "tools: enhance lvdisplay vgdisplay pvdisplay"

lvdisplay: split code for variants using command defs

doc: Amend 2.03.27 and 28 release notes' date

tests: make reshape bug more easier to hit

Seems this delay_dev usage exposing some reshape window creation bug.
So make delay bigger to hit the error easier.

tests: do not use  noudevsync with udev system

Remove --noudevsync option - as this breaks synchronization with
udev which is necessary when trying to i.e. create  _rmeta_3
and wipe it - as the symlinks must be present for wiping.

So if there was some other issue  (behind the comment) - we need to
check for the problem elsewhere instead of disabling udev sync.

debug: missing stack

doc: Update WHATS_NEW and Release notes

configure: autoreconf

configure.ac: build lvmlockd with notify

Automatically use --enable-notify-dbus when building lvmlockd
if not configured otherwise by a configure user -
as the lvmlockd.service is notify based.

test: Prevent incorrect value propagating into test

In case dbus-send is not present previous random value of i was tested.

spec: Remove [Build]Requires for cluster

lvmlockd: allow test mode when dlm is not built

man: lvmraid integrity updates

dbus: Add function for running lconvert --repair on RAID LVs

tests: increase mirror size

On fast machines mirror throttling starts to react much later
so try bigger mirror size to fight with race in test.

man: update display fields

The --sort option belongs to  '-C' columned output only.
Also add short commmand description for specialized
vg/pv/lvdisplay command.

make: generate

tools: enhance lvdisplay vgdisplay pvdisplay

Split description for display commands so we can better describe
it's usage and combination of individual options in man page.

Now we can separately describe:

lvdisplay, lvdisplay -c, lvdisplay -C
vgdisplay, vgdisplay -c, vgdisplay -C
pvdisplay, pvdisplay -c, pvdisplay -C

TODO: Drop validation from command code itself.

args: add colon and column for pv/vg/lvdisplay

Better description for --colon and --column option for
pvdisplay, vgdisplay, lvdisplay.

man: clean

Both branches printed new line.