vgimportdevices: skip global lockd locking

Fix commit b65a2c3f3a767 "vgimportdevices: skip lvmlockd locking"
which intended to disable lvmlockd locking, but the lockd_gl_disable
flag was mistakenly set after lock_global() so it wasn't effective.
This caused vgimportdevices to fail unless locking was started.

README: remove old branch names

make: generate

tests: accept new vdoformat output

There is slight change in the printed size number.

args: generate configurable path

tests: quoted DM_DEV_DIR

tests: use single aux lvmconf

Reduce log size as lvmconf can process multiple settings at once.

device_id: use faster dm_strncpy

device_mapper: solve no devices path

Malloc needs to use sane value when names would be NULL.
Also for devices without 'uuid' return empty string instead of NULL.

lvremove: initilize struct list

lvremove: check for handle

Get lvremove_params from non NULL handle.
Introduced in commit 47f8bda05.
TODO: any more thing needed ?

WHATS_NEW: update

make: generate

cov: replace enum report_type_t with unsigned

Another enum used as bitfield mask.

cov: convert some more enternal field masks

Some more 'internal' masks misusing enums for bitfields.

cov: cleanup mask from enum to unsigned

Enums are single 'values' so not a proper type for bitfields.
(Probably better to use such values as defines).

Although here 'daemon_talk()' is part of library API, it's hidden
non-public API call - and moreover 'enum' and 'unsigned' are
using the same size, so linker shouldn't have any issue with
this symbol usage.

For this reason there are no 'versioning' tricks applied.

cov: annotate known use of validated lv name

cleanup: remove blank spaces

cleanup: refactor common code

Refactor shared code from _get_rootvg_dev() between vgchange
and vgimportdevices to get_rootvg_dev_uuid() in toollib.c

tests: add some lvm shell testing

Check lvm shell for autocomplete functionality and error reporting.

cleanup: print MAJOR and MINOR as unsigned

Print directly as %u instead of %d with (int) casting.

libdm: use 'unsigned' for major minor

Although linux uses 'dev_t' as 'unsigned long int' on 64bit,
it's easier to print with just '%u' instead of '%lu'.

cleanup: replace strlen with sizeof

Use the string size from compile time.

cleanup: use const for sizeof values

cleanup: simplier loop condition

Check char for != 0.

cleanup: use correct log_sys_debug

cleanup: struct reorder

Better alignments.

cleanup: some typos

Collection of some typos or invalid uppercase or doublespace cases.

hash: increase some default hash table size

Size of these hashes was quite small, so raise the size of
hashed entries to reduce amount of hash collistion.
Select some unique/unused number for hash_create below 8192.

libdm: drop allocation

Use structure allocated on local stack.

mirror: drop unneeded call

Major,minor,patchlevel are already set from previous call.

device: use device_get_uuid

Replace call to get_dm_uuid_from_sysfs() with use of
device_get_uuid() which gets the same information,
but instead of several syscalls it need either 1 or even 0
when the information is cached with newer kernels.

dev_manager: remove disable_dm_dev flag

Lvm currenly drops cache before first table manipulation
there is no need for this flag ATM.

dev_manager: drop cache for dm tree manipulation

We've got cached DM list before grabbing lock, so there
is some chance, that DM table has changed and we would
need to refresh this info.

TODO: benchmark, whether it would even make sense to refresh cache
and keep it content instead of using individual ioctl() for tree build.

label: cache dm dev list earlier

Cache (when possibly with newer kernel) earlier within label
scanning loop, so the cache information can be also used by
hints file.

device_manager: add device_get_uuid

Function that is working with DM target is located within
lib/activate directory.

This function is able to use cached  dm_device_list when possible
to quickly resolve checks for device's UUID.

Function can fully replace get_dm_uuid_from_sysfs() and instead
of syscalls for open/read/close get the UUID with single ioctl.

When there is cached dm devs list, we can get many UUID from
a single syscall.

device: propagate struct cmd_context

For better use of cached data located within cmd_context,
pass this structure from the top level function.
Also add missing '_' for static _dev_cache_index_devs.
No other change here.

device_mapper: use const for this API

Allow using constified version of devs_list.
Internal loop does not need to modify anythin in this stucture.

device_mapper: add dm_device_list_find_by_dev

Introduce function to find device's name and uuid for
a given major:minor.

This information is cached with dm_device_list which reads all the
info from single ioctl(DM_DEVICE_LIST).
Lvm keeps major:minor name & uuid for active devices in the system.

dev-type: use if..else if...

Compare strings till the first match.

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