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.

cov: strcpy mempcy

cov: replace strcat with safe snprintf

cov: replace strcpy with memcpy

cov: use sscanf buffer size limits

String parsing with buffer size limit.

cov: check for valid raid0 LV segment

Add internal error to quiet Coverity fear of NULL pointer deref.

raid: easier prompt message

Use  printf capability of 'prompt()' funtions so use literal
when building this message.

lvmlockd-dlm: log_error message with literal

Use literals for printf() message construction.

libdm: unlink checks for ENOENT

libdm: use literal for error message

Use literals for printf messages.

const: daemon server logging backend struct

const: lvmpolld long_options

Rework usage of long_options so it can be constified,
and also correct use of 'enum action_index'.

const: rework conversion to enum

Easier code with string and also avoids mixing enums.

const: array of const values

const: libdm err_fmt as string

man: lvmthin update

major refresh of text and organization

dmsetup: fix invalid loop test

Last patcheds had incorrect merging bug - fix loop test.