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

lv_manip: avoid unreleased memory pool(s) message on RAID extend

In case of different PV sizes in a VG, the lvm2 allocator falls short
to define extended segments resiliently asked for 100%FREE RaidLV extension
and a RAID distinct allocation check fails.  Fix is to release a memory pool
on the resulting error path.

Until the lvm2 allocator gets enhanced (WIP) to do such complex (and other)
allocations proper, a workaround is to extend a RaidLV to any free space on
its already allocated PVs by defining those PVs on the lvextend command line
then iteratively run further such lvextend commands to extend it to its
final intended size.  Mind, this may be a non-trivial extension interation.

pages: Update release note tag and timestamp

post-release

pre-release

Revert "dev-cache: move global variables into cmd struct"

This reverts commit 77be3250d926263db0d4daa83a4102078601bac2.

Revert "bcache: move from global variable to cmd struct"

This reverts commit 5bf4efbab5f2c51f0ca1763f47a9aa227dfb915c.

bcache: move from global variable to cmd struct

dev-cache: move global variables into cmd struct

The cmd struct is now required in many more functions, and
it's added as a function arg for most direct dev-cache function
calls.  The cmd struct is added to struct device (dev->cmd) so
that it can be accessed in many other cases where dev-cache
functions are being called from places where getting the cmd
struct is too difficult.

config_file: remove unused code

config_file_read() is only used for regular files, not devices.

devices: isolate fake device setup for config file

Freeing dev-cache from dumb fake device usage.

devices: isolate fake dev use for mirror checks

Freeing dev-cache from dumb fake device usage.

lvmlockd: fix compilation with disabled lvmlockd

With updated functionality for lvmlockd some prototypes
needs to be updated for builds without lvmlockd.

devices: simpler dm devs cache naming

The dm devs cache is separate from the ordinary dev cache,
so give the function names distinct prefixes, using
"dm_devs_cache" to prefix dm devs cache functions.

devices: redo bcache invalidation of PVs on LV

devices: update dm uuid cache naming

replace "dm uuid" with "dm devs" in case it begins
caching more than dm uuids.

Revert "devices: simpler bcache invalidation of PVs on LVs"

This reverts commit 200b05cec0b75ebd1c6860ca7b2233b390cc2024.

Doesn't work to depend on lv name.

doc: release notes for 2.03.25

WHATS_NEW: update

devices: simpler bcache invalidation of PVs on LVs

When a PV is stacked on an LV, the PV needs to be
dropped from bcache before the LV is processed.
The LV can be found in dev-cache using its name
rather than the devno.

dev-cache: unify dm uuid cache struct lifetimes

The list of dm devs was in the cmd struct and had a
different lifetime than the radix trees referencing
those dm devs.  Now the list and radix trees are
created and destroyed together.

devices: update function name to dm_device_is_usable

devices: dm uuid related function names

In the context of dm, 'device' refers to a dm device, but
in the context of lvm, 'device' refers to struct device.
Change some lvm function names to make that difference clearer.

dev_manager_get_device_list() -> dev_manager_get_dm_active_devices()
get_device_list() -> get_dm_active_devices()
device_get_uuid() -> dev_dm_uuid(), devno_dm_uuid()

lvmlockd: vgchange systemid doen't need global lock

The comment explained that the ex global lock was just
used to trigger global cache invalidation, which is no
longer needed.  This extra locking can cause problems
with LVM-activate when local and shared VGs are mixed
(and the incorrect exit code for errors was causing
problems.)

lvmlockd: after restart use existing dlm global lockspace

If lvmlockd is killed and restarted, let it adopt the
existing global lockspace.

lvmlockd: skip LV unlocking with no lvmlockd or no lockspace

vgchange -an vg is permitted when the vg lockspace
is not available, because LVs could still be active
for some reason, and they should be inactive when not
properly locked.  In case lvmlockd was not running, or
the lockspace was not started, the command was
unnecessarily trying and failing to unlock every LV,
printing errors for every LV.  We can skip this when
the lockspace is known to not be available.

lvmlockd: let vgchange lockstop fail with EBUSY for orphans

vgchange --lockstop will fail with EBUSY if orphan locks in the
lock manager prevent stopping the lockspace.  The orphan locks
can then be adopted and released, and the lockspace then stopped
cleanly.

lvmlockd: adopt options

Lock adoption is not part of standard command behavior, but can
be used for manual recovery or cleanup from unexpected failure
cases.  Like other lockopt values, they are hidden options for
--lockopt.  Different lock managers will behave differently.
Adopting locks with lvmlockd -A1 is more accurate and automatic.

--lockopt adoptls
. for vgchange --lockstart
. adopt existing ls, or fail if no existing lockspace is found

--lockopt adoptgl | adoptvg | adoptlv
. for commands using lvmlockd locks
. adopt orphan gl/vg/lv lock, or fail the lock request if
  no orphan lock is found
. will fail if orphan lock exists with a different lock mode
. command may still continue with a failed shared lock request

--lockopt adopt
. for lockstart or any command using lvmlockd locks
. adopt existing lockspace, or start lockspace if none exists
. adopt orphan gl/vg/lv lock, or acquire new lock if no orphan found
. will fail if orphan lock exists with a different lock mode
. command may still continue with a failed shared lock request
. with dlm this option only works for ls

lvmlockd: parse lockopt string into flags

lvmlockd: remove unused paramater

start_init option has never been used

vgchange: improve some lvmlockd lockstart messages

Stop printing "Skipping global lock: lockspace not found or started"
for vgchange --lockstart, since it's generally an inherent limitation
that the global lock isn't available until after locking is started.

Update the start delay warning to "a few seconds".

man: lvmlockd updates

lvmlockd: allow service options

command line options can be set in /etc/sysconfig/lvmlockd, e.g.
OPTIONS='-A1 -o2'

lvmlockd: reduce adopt delays

lvmlockd: update debug messages

Reduce debug messages, and make less verbose.

lvmlockd: disable lvb for sanlock

The lvb is used to hold lock versions, but lock verions are
no longer used (since the removal of lvmetad), so the lvb
is not actually useful.  Disable their use for sanlock to
avoid the extra i/o required to maintain the lvb.

lvmlockd: skip more vgremove steps with lockopt force

vgremove with --lockopt force should skip lvmlockd-related
steps and allow a forced vg cleanup, in addition to using
--nolocking to skip normal locking calls.

lvmlockd: allow forced vgchange locktype from none

vgchange --locktype sanlock|dlm --lockopt force <vgname>
can be used to change the lock type without lvmlockd or
the lock manager involved.