dev-type: detect mixed dos partition with gpt's PMBR

Detect when we have mixed dos partition with gpt's PMBR partition.

This is not a sane configuration, but detect it anyway, just in case
someone configures such partition layout manually and forcefully and
incorrectly defines one of the partition types to be the GPT's PMBR.

For example:

  ❯  fdisk -l /dev/sdc

  Device     Boot Start    End Sectors Size Id Type
  /dev/sdc1        2048  67583   65536  32M 83 Linux
  /dev/sdc2       67584 262143  194560  95M ee GPT

Before:
(The partition filter passes even though there's real existing dos
partition - the empty GPT PMBR overrides it.)

  ❯  pvcreate /dev/sdc
  WARNING: PMBR signature detected on /dev/sdc at offset 510. Wipe it? [y/n]:
  Wiping PMBR signature on /dev/sdc.
  Physical volume "/dev/sdc" successfully created.

With this patch applied:
(The GPT PMBR does not override the existence of the dos partition.)

  ❯  pvcreate /dev/sdc
    Cannot use /dev/sdc: device is partitioned

post-release

pre-release

WHATS_NEW and release-notes update

WHATS_NEW: update

[WIP] Disable dlm for RHEL10

lvmdbusd: Fix Python script

The thread does not contain field "damon" but "daemon".

Actually found with codespell.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

filesystem: add note about swap devices

tests: also test swap devices in lvresize-fs.sh

lv_manip: handle swap devices in fs-related checks for lvreduce/lvextend

This provides better hints when trying to resize the fs on top of an LV.
Also needs a3f6d2f593a4d278daf9ba3a1ba30bae38d8396a for proper operation.

❯  lvs -o name,size vg/swap
  lv_name lv_size
  swap     60.00m

Before:

❯  lvextend -L72m vg/swap
  Size of logical volume vg/swap changed from 60.00 MiB (15 extents) to 72.00 MiB (18 extents).
  Logical volume vg/swap successfully resized.

❯  lvreduce -L60m vg/swap
  File system swap found on vg/swap.
  File system device usage is not available from libblkid.

❯  lvreduce -L50m vg/swap
  Rounding size to boundary between physical extents: 52.00 MiB.
  File system swap found on vg/swap.
  File system device usage is not available from libblkid.

After:

❯  lvextend -L72m vg/swap
  Size of logical volume vg/swap changed from 60.00 MiB (15 extents) to 72.00 MiB (18 extents).
  Logical volume vg/swap successfully resized.

❯  lvreduce -L60m vg/swap
  File system swap found on vg/swap.
  File system size (60.00 MiB) is equal to the requested size (60.00 MiB).
  File system reduce is not needed, skipping.
  Size of logical volume vg/swap changed from 72.00 MiB (18 extents) to 60.00 MiB (15 extents).
  Logical volume vg/swap successfully resized.

❯  lvreduce -L50m vg/swap
  Rounding size to boundary between physical extents: 52.00 MiB.
  File system swap found on vg/swap.
  File system size (60.00 MiB) is larger than the requested size (52.00 MiB).
  File system reduce is required and not supported (swap).

dev-type: get swap device size from blkid using FSSIZE

blkid does not report FSLASTBLOCK for a swap device. However, blkid
does report FSSIZE for swap devices, so use this field (and including
the header size which is of FSBLOCKSIZE for the swap) instead to
set the "filesystem last block" which is used subsequently for
further calculations and conditions.

tests: add pvcreate-partition.sh

dev-type: add prefix to differentiate msdos and gpt constants

filter: partitioned: also detect non-empty GPT partition table

We already detect msdos partition table. If it is empty, that is, there
is just the partition header and no actual partitions defined, then the
filter-partitioned passes, otherwise not.

Do the same for GPT partition table.

make: generate

cov: annotate

cov: potentially overflowing expression

Use 64bit arithmentic.

lvmlockd: configurable sanlock lease sizes on 4K disks

New config setting sanlock_align_size can be used to configure
the sanlock lease size that lvmlockd will use on 4K disks.

By default, lvmlockd and sanlock use 8MiB align_size (lease size)
on 4K disks, which supports up to 2000 hosts (and max host_id.)

This can be reduced to 1, 2 or 4 (in MiB), to reduce lease i/o.
The reduced sizes correspond to smaller max hosts/host_id:

1 MiB = 250 hosts
2 MiB = 500 hosts
4 MiB = 1000 hosts
8 MiB = 2000 hosts (default)

(Disks with 512 byte sectors always use 1MiB leases and support
2000 hosts/host_id, and are not affected by this.)

WHATS_NEW: update

cleanup: replace use of alloced with allocated

cleanup: typos in libdm

cleanup: more typos in WHATS_NEW

cleanup: fix typos

Typos found with codespell.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

memlock: use value of 0 to disable memory locking

In cases user is sure he is not using his 'rootfs' or 'swap' on LVs
managed with his command - it possible to completely bypass pinning
process to RAM which may eventually slightly speedup command execution,
(however at the risk the process can be eventually delayed by swapping).
Basicaly use this only at your risk...

TODO: add some dmeventd support for this.

vdo: depcreate unused settings

lvmlockd: use lvmlock LV size

Previously, lvmlockd detected the end of the lvmlock LV
by doing i/o to it until an i/o error was returned.
This triggered sanlock warning messages, so use the LV
size to avoid accessing beyond the end of the device.

Previously, every lvcreate would refresh the lvmlock LV
in case another machine had extended it.  This involves
a lot of unnecessary work in most cases, so now compare
the LV size and device size to detect when a refresh is
needed.

lvmlockd: fix previous thin locking fix

Restore the original lvremove locking for non-thin LVs
that were changed in the thin locking fix.

vg: remove unused hostnames hash table

The hash table of lv creation hostnames was not used for anything,
so remove it.

lvmlockd: fix locking for thin

lvremove of a thin lv while the pool is inactive would
leave the pool locked but inactive.

lvcreate of a thin snapshot while the pool is inactive
would leave the pool locked but inactive.

lvcreate of a thin lv could activate the pool to check
a threshold before the pool lock was acquired in lvmlockd.

tests: remove superfluous -a option for df used in lvresize-fs.sh

metadata: use lv_hash in segment-specific metadata parsing

The lv_hash wasn't being passed to the seg-specific text import
functions, so they were doing many find_lv() calls which consumes
a lot of time when there are many LVs in the metadata.

args: readonly description update

Include reference to --permission r.

WHATS_NEW_DM: update

libdm: do not fail if GETVAL semctl fails for udev sync inc and dec

While performing udev sync semaphore's inc/dec operation, we use the
result from GETVAL semctl just to print a debug message with current
value of that sempahore, nothing else.

If the GETVAL fails for whetever reason while the actual inc/dec
completes successfully, just log a warning message about the GETVAL
(and print the debug messages without the actual semaphore value)
and return success for the inc/dec operation as a whole.

libdm: clean up udev sync semaphore on fail path during its creation

Clean up udev sync semaphore on fail path during its creation, otherwise
the caller will have no handle returned to clean it up itself and the
semaphore will keep staying in the system. The only way to clean it up
would be to call `dmsetup udevcomplete_all` which would destroy all
udev sync semaphores, not just the failed one, which we don't want.

libdm: add 'cookie create/inc/dec' log prefix if GETVAL fails for udev sync ops

The same message is printed while performing create/inc/dec operation and
the GETVAL semctl fails. Add a prefix so we know exactly in which of
these functions the issue actually happened.

scripts: Fix ConditionPathExists in unit files

WHATS_NEW: update

configure: autoreconf

make generate

cleanup: typos in configure.ac

cleanup: typos in test comments

cleanup: typos in test logging

cleanup: typos in doc

cleanup: typos man pages

cleanup: typos in WHATS_NEW

cleanup: typos in comments

Collection of typos in code comments.
Should have no runtime effect.

cleanup: typos in logging

Fixes various typos in printed/logged messages.

cleanup: typo dirctory to directory

Rename internal lm_idm_scsi_find_block_directory() function.

cleanup: typo premable to preamble

Correct preable, seems even unused.

libdm: _DOESNT_ to _DOES_NOT_

Rename internal define DM_NAME_LIST_FLAG_DOESNT_HAVE_UUID
to DM_NAME_LIST_FLAG_DOES_NOT_HAVE_UUID
(currently unused)

lvmlockd: retrive to retrieve

Replace retrive with retrieve for internal function.

libdaemon: daemonise to daemonize

Replace daemonise() with daemonize().

lvconvert: use pvmove as part of renamed LV name

LV with pvmove_ prefix is not allowed to be created by user
so bigger chance our selected name will never exist.

TODO: probably add code to get generic unused LV name...

libdm: check for queue name

Fix typo and check for queue string within sysfs kernel name.

integrity: typo  interleave_sectors

Correct emitted parameter name for interleave_sectors for DM table line.

make: generate

man: lvmreport: add note about log/command_log_selection="all"

config: update description for log/command_log_selection

lvresize: ignore given stripe and stripe size args for raid0

toollib: do not accept duplicate device name on pvcreate input

post-release

pre-release

lvmlockd: increase dump buffer size

WHATS_NEW: update

make: generate

gcc: single type initializers

Older gcc doesn't really like complex types (buffer, struct) to be
initialized without extra {} around such type.
So pick any other 'single type' var from a struct and set it to 0,
rest will do the compiler without emitting a warning.

cleanup: drop extra space

tests: check pvmove on single PV

validation: support empty lv segments

Revert 373372c8ab3749bc76ced37cec04b00aae6e5979 and instead update
our validation code to handle LVs with empty segment - currently
we should need this only for pvmove operation, thus such LV should
have name  'pvmove%u'.

This fixes a problem where user tried i.e. pvmove on a VG with single
PV - as reported: https://github.com/lvmteam/lvm2/issues/148

Reported-by: bob@redhat.com

WHATS_NEW: update

tools: vgscan: accept --refresh option

tools: vgscan: accept --refresh option

tools: args: update description for --refresh

tools: vgmknodes: use display_lvname

tools: vgmknodes: handle /dev/mapper content without LV refresh only if udev not running

tools: vgmknodes: log error message if LV refresh fails

tools: vgmknodes: wait for udev first with --refresh, then check /dev

man: vgmknodes: add note about udev and --refresh

gitignore: update

The .cache and compile_commands.json is used by popular source crawling and
indexing clang tools which in turn may be integrated with source code editors.

We may reuse the .cache directory for for other caches and temporary
files.

The /doc/.ikiwiki and /public are related to the ikiwiki.

make: generate

Revert "tests: use put_time"

This reverts commit b4af2571dbee9dab5717bb166ed63df5ae63974b.
std::put_time is a relatively new feature so breaks builds on older
systems.

tests: add some udev waits

Avoid possible udev race - since dmsetup create is
not using the same cookie logic as lvm2 commands,
try to avoid racing on some systems with udev scanning.

tests: use put_time

tests: search for delimiter in bounded buffer

cov: annotate known case

cov: validate list emptiness

Emit internal error for empty list.

cov: validate altdev list is not empty

cov: check for dev_iter_create result

dev_iter_create() may fail in malloc so check
for its error code.

cov: drop unused header inclusion

cov: avoid expression overflow

Use 64bit math to multiply extent_size.

integrity: add --integritysettings for tuning

The option can be used in multiple ways (like --cachesettings):

--integritysettings key=val
--integritysettings 'key1=val1 key2=val2'
--integritysettings key1=val1 --integritysettings key2=val2

Use with lvcreate or lvconvert when integrity is first enabled
to configure:
journal_sectors
journal_watermark
commit_time
bitmap_flush_interval
allow_discards

Use with lvchange to configure (only while inactive):
journal_watermark
commit_time
bitmap_flush_interval
allow_discards

lvchange --integritysettings "" clears any previously configured
settings, so dm-integrity will use its own defaults.

lvs -a -o integritysettings displays configured settings.

devices: fix dev_dm_uuid

If a non-dm device is passed, the dm code doesn't fail and returns
some dm uuid.

WHATS_NEW: update

tests: add tests for autoswitching to JSON format for log messages

make: generate

reporter: simplify checking output format setting in report_format_init

config_settings: fix typo

config: use default log/command_log_report=1 for json/json_std output format

log/command_log_report config setting defaults to 1 now if json or json_std
output format is used (either by setting report/output_format config
setting or using --reportformat cmd line arg).

This means that if we use json/json_std output format, the command log
messages are then part of the json output too, not interleaved as
unstructured text mixed with the json output.

If log/command_log_report is set explicitly in the config, then we still
respect that, no matter what output format is used currently. In this
case, users can still separate and redirect the output by using
LVM_OUT_FD, LVM_ERR_FD and LVM_REPORT_FD so that the different types
do not interleave with the json/json_std output.

WHATS_NEW