command: reorder vals order

Keep most frequently used val names in front.

command: store val_name with its size

Compile length of val_name during compilation.

dev-cache: skip scanning known blockless dirs

Skip scan and stat() for dirs and nodes within known /dev/ paths,
where no block devices are located.

Also strlen(_cache.dev_dir) just once.

TODO: add more dirs to _no_scan (configurable via lvm.conf ?)

dev-cache: collapsing slashes counts string size

_collapse_slashes() already goes through the whole string,
so just return now already known string length.

cleanup: use single array for idtype string

Share string array for de/coding idtype.

error: try with modprobe

If the error target would be built as module,
ask modprobe to load it if the target is not present.

cleanup: relocate section

Move _features[] structure from .data.rel.ro to .rodata.

cleanup: use ALTERNATIVE_EXTENTS

Instead of comparing string of command names,
introduce ALTERNATIVE_EXTENTS command flags.

cleanup: use ARG_LONG_OPT

Just like with _VAL strings, also _ARG strings do not need to
be present - as we can easily check for LONG opt version just
by adding attribute.

With attribute ARG_LONG_OPT string arg name[] becomes unused
and can be safely removed.

Also within _find_command_id_function() we do not need to handle
'command_enum == CMD_NONE' as separate case and just use single loop.

cleanup: do not store _VAL name string

String enum_name[] is unused so remove is with
all generated strings.

Reoder struct members, so there are no holes.

libdm: parse only block section for proc_devices

Check for begining of 'B'lock section in /proc/devices file
before starting to sscanf() for major.

lvmcmdline: use known lvm_command_enum

lvm command is already decoded (when found),
so use lvm_command_enum directly and avoid
comparing all strings again...

command: return proper unsigned value

Use better 'unsigned' value, so we don't have any 'casting' troubles.

lvm: fix shell completion

Previous commit 82617852a4d3c89b09124eddedcc2c1859b9d50e
introduce bug in complession - as the rl_completion_matches()
needs to always advance to next element where the index
is held in static variable.

Add comment about this usage.

lvremove: remove device_id for PVs on LVs

When PVs are created on LVs, remove the devices file entries
for the PVs when the LVs are removed.  In general, the devices
file entries should be removed with lvmdevices --deldev when
the LVs are removed (lvremove is the equivalent of detaching
a device from the system when layering PVs on LVs.)
This change is effectively an automatic lvmdevices --deldev
command that is built into lvremove when the LV has a PV on it.

Allow system.devices to be automatically created on first boot

An OS installer can create system.devices for the system and
disks, but an OS image cannot create the system-specific
system.devices.  The OS image can instead configure the
image so that lvm will create system.devices on first boot.

Image preparation steps to enable auto creation of system.devices:
- create empty file /etc/lvm/devices/auto-import-rootvg
- remove any existing /etc/lvm/devices/system.devices
- enable lvm-devices-import.path
- enable lvm-devices-import.service

On first boot of the prepared image:
- udev triggers vgchange -aay --autoactivation event <rootvg>
- vgchange activates LVs in the root VG
- vgchange finds the file /etc/lvm/devices/auto-import-rootvg,
  and no /etc/lvm/devices/system.devices, so it creates
  /run/lvm/lvm-devices-import
- lvm-devices-import.path is run when /run/lvm/lvm-devices-import
  appears, and triggers lvm-devices-import.service
- lvm-devices-import.service runs vgimportdevices --rootvg --auto
- vgimportdevices finds /etc/lvm/devices/auto-import-rootvg,
  and no system.devices, so it creates system.devices containing
  PVs in the root VG, and removes /etc/lvm/devices/auto-import-rootvg
  and /run/lvm/lvm-devices-import

Run directly, vgimportdevices --rootvg (without --auto), will create
a new system.devices for the root VG, or will add devices for the
root VG to an existing system.devices.

post-release

pre-release

pages: Use gitlab pages and ikiwiki for doc

doc: Update TESTING file

makefiles: use properly defined varitable

Variable used for tracking dependencies is USE_TRACKING.

tests: add basics for dm_hash unit testing

Better code coverage.

gcc: clean cast with -Wbad-function-cast

Makes gcc happier.

gcc: same signess

gcc: use proper commnent

gcc: better initilizer for older gcc compilers

So use  { { } }.

activation: log_debug_activation

Correcting debug class.

tests: reduce number of created LVs

lvm2 for a while already optimizes 'vgremove' operation to
a single commit when possible if all LVs can be
easily deactivated.

So the number of LVs doesn't matter much - but the tested
case 'test_delete_non_complete_job' seems to be taking
some time anyway to capture the exception.

So just reducing the running time of the test significatanly
as we don't need to create 64LVs for 4 'execution mode' runs.

cov: ensure pointer has defined value

Add initialization of 'fin' pointer.

dmfilemapd: assign 0 to integer

Avoid mixing enumeration assign with 0, so just pick some integer
from filemap_monitor,  stucture will be fully zeroed anyway....

libdevmapper-event: correct switch break

Replace 'continue' within 'switch' statement with correct break.

man: lvmlockd: remove dash in option name

for consistency with other man pages

lvmcmdline: check for NULL

Althought this code likely never is hit with NULL, just add
check to be sure.

cov: enusure _table doesn't leak

If someone would enter _table multiple times by mistatek,
there is 'teroretical' leak - although doesn't matter much
in this case...

make: generate

lv_types: sort order of types

Order is used for man page generation (although not completely).
So place  'zero & error' to the end of list.
Keep  linear,striped,snapshot in front.
For the rest use alphabetic order.

cleanup: local const arrays

No need for relocation entries for locally used arrays.

cleanup: static missed _

cleanup: remove unuable code path

Eliminate some ancient unreachable code
for historical version of libdm.

cleanup: some missed public symbols static

label: make scan_bcache local

Seems there are no users of scan_bcache, so make it local.
TODO: such variable should be ideally in cmd_context.

cleanup: drop unused declared struct

cleanup: use static vars

Drop global visibility.

cleanup: add static _

lv_type: simplier macro usage

Use more simple macros - as there is likely not going to happen
any macro replacement - so use more effient structure layout.

lv_prop: simplier macro usage

Use more simple macros - as there is likely not going to happen
any macro replacement - so use more effient structure layout.

command: validation code only for man-generator

Don't waste time with string order validation within command,
check within  'man-generator -c' is enough.

command: refactor inner loop

Move part of the 'inner' loop which is would be otherwise
always production same results for all  'opt_enum' values
out of the loop, so it can be evaluated just once.

command: replace strcmp with enum

Here we already know 'enum' value, so compare
just those instead of calling strcmp().

lvmcmdline: runtime function resolving

Instead of resolving and storing 'command_fn'
withing 'struct command' use just funtion enum
and resolve function pointer just in place,
where it is really needed - first try to resolve
'new style' and fallback to 'old style' named.

command: use command_id_to_enum

Instead of storing command_id as string, direcly
translate string to enum index and use 'command_enum()'
to get string when needed for printing.

This way we can easily detect error in the structure
while parsing it - and we can later avoid separate
'translation' loop.

command: refactor code for simplier lookup

We can more efficiently use command_name struct to
lookup for  lvm_command_enum and avoid many repeated
command name searches since we already know
the enum index that is now stored in 'struct command'.

lvmcmdline: use strcut as const

Constity members in cmdline_context, would be nice, to replace
this static struct with couple function calls.

Also replace some 'while' loops with for loops, so code
is more readable.

command: return if already factored

We can directly return here, when we spot already counted varitants.

command: refactor struct command_name

Split struct command_name to the constant part (keep the name)
and new 'struct command_name_args' which holds runtime computed
info.   To get to the _args part - we can easily use
lvm_command_enum as equivalent index.

Constified part 'struct command_name' is now fully stored
in .data.rel.ro segment, while command_name_args part goes
to .bss segment.

Code will be further reduced with next refactoring.

command: better const usage for struct cmd_name

No need to store _CMD string - as it's not used anywhere.
Use LVM_COMMAND_COUNT as end of array.

command: reduce struct size

Reduce memory usage for parser.
If the sizes would need to be bigger, man-generator warns.
For struct command_name  use just uint8_t to keep 1|0.

command: no need to duplicate const mem

No need to duplicate const char* here as
we refer already constant string in .rodata segment.

command: futher move to single struct

Refactor code to not allocate memory for rule->opts,
instead use uint16_t array of MAX_RULE_OPTS within cmd_rule.

Also print more info if array would not be enough (>= 8).

tests: check vg fields with one command

tests: keep more LVs inactive

tests: mask result from kill command

tests: wait for running dmeventd

Add slight delay for waiting until 'started' dmeventd is
responsing to other 'dmeventd -i' command.

TODO: race observed here is somewhat unclear, might need some more
details

tests: kill process in flock group

There is no reason to wait for 10sec when removing 'vg' at test
exit - we just need to kill 'sleep 10' process forked from flock.

(Not using 'flock -F' as this flag is not across all machines used
for testing)

tests: skip test on systems without delay_dev

On such systems this test only sleeps for 30sec.

libdm: reapply "fix condition"

This reverts commit d16a8f80e9dd21d97b10056858a21b7e8fbd8c94.
So the correction was OK, however we missed to fix also
cut&paste bug here - as the second check should be
actually checking  field->type.

WHATS_NEW: update

make: generate

tests: vdo conversion updates

commands: move tabs to eol

Make the source code a bit easier to read.

commands: enable more options for vdo conversion

Update definitions to add support for some more VDO options
when converting volumes to be used as thin-pool with vdo data volume.

Split some option in existing OO_LVCONVERT_VDO to OO_LVCONVERT_VDO_POOL
and reused then with OO_LVCONVERT_THINPOOL.

lvconvert: vdo support conversion options

When convert already existing vdopool to be used as
thin-pool backend and user is passinng option for VDO configuration
process them - as we know converted LV is offline, we can do such
change easily instead of telling user to run separate lvchange later.

lvconvert: vdo control for signature wiping

When converting volumes to be used for thin-pool with VDO, allow
users to control wipesingature behaviour.
By default volumes should be checked against signature, and if
they are present, we promt user whether he wants to process with
conversion and lose i.e. filesystem present on such volume.

Users that want to bypass prompt in script can use either --yes
or they can disable wipe signature -Wn.

man: lvmthin more extend options

tests: runner updates line with last timestamp

clean: use struct initializer

dmsetup: const string_args array

Convert _string_args to const char*.
Do not modify options read from getopt().

dmsetup: use tab for indent

Use \t for help indention.

dmsetup: more const strings

More strings into .rodata segment.

dmsetup: reduce parser code

Switch set upfront.
Only add code, which is not common for every switch.
Use just 16bit for switch counting.

libdm: fix invalid conversion

Previous commit missed to also add one to added 'len'.

libdm: stats uses overflowing counter

Here we need to annotate counter as it starts with UINT64_MAX
and overflows to 0 is expected.

clenaup: update local static with _

const: tests

man-generator: remove unused declarations

cov: remove dead code

cov: eliminate dead code

Deduplicate code and also drop some warn about dead code.

cov: avoid TOCTOU

Use just opendir() call and produce error message from errno state.

cov: enum mapping cache format

Cleanly map cache metadata format enumeration.

cov: enum health status

Cleanly map health char status to enum.

cov: enum matching type

cov: enum correct prototype

Cleanup enum usage.

cov: cast to matching prototype

Match function prototype with casting.

cov: check for overlow math

Add some extre protection to avoid integer overflow type of problems.

cov: annotate to silent some coverity warning

cov: use literal for printf

Use literal for printf and just empty string if needed.

cov: use literal for printf format

Prepare suffix ahead and use literal for printf().

cov: sanitize input buffer

Make coverity aware the input line is legit.
Eventually add more sanitize functionality.

cov: sanitize string for file path

cov: wrap pvck call

Resolve reported leak by renaming existing pckk() to pvck_mf()
and wrapping pvck() over this. This function can easily
free allocated buffer within the subfunction.