post-release

.

pre-release

Code refactoring

Properly test for dm_asprintf result.
Keep unlocking of mutex in the same function and do not spread lock and
unlock over functions.

Just space moving

Don't leave space on EOL.

List _thread_registry missed mutex

Operation on _thread_registry needs to be covered by mutex.
Cosmetic move a die code after free for valgind short leak list.

a fixme

Support 16GB for thin pool metadata

Add some hack math to allow 16GB devices to be passed as thinpool metadata.
Since kernel has put in limit to not allow which are just bigger then
some predefined constant in kernel but not matching 16GB so any device bigger
is rejected.

FIXME: Current code still might need more tweaks to be more generic.

fix non-lvmetad pvscan macro

Add support for thin check

Use libdm callback to execute thin_check before activation
thin pool and after deactivation as well.

Supporting thin_check_executable which may pass in extra options for
the tool.

Purge remaining trim bits from code

missing reply frees

Pass struct device around internally rather than dev_t.
Add 3rd daemon return state "unknown" for lookups that are carried out
successfully but don't find the item requested.
Avoid issuing error messages when it's expected that a device that's
being looked up in lvmetad might not be there.

Merge metadata size checking

Move the code for poolmetadatasize operation into one place.
Report override for minimum and maximum size.
Drop _read_thin_params function its error reporting is handled elsewhere.

Change pvscan --lvmetad to pvscan --cache.

Added dm_tree_node_set_callback() for preload and deactivation hooks

Run users hook after preload for the node is finished,
or after the node has been deactivated.

Fix estimation of pool metadata device size

If no size was give the later added minimal size check efectively
disable this code. Also the argument for size now must be kept
in sector_size, so adding division by SECTOR_SIZE (moved into
a const expression)

Allow multiple device names with pvscan --lvmetad.
Hold global lock in pvscan --lvmetad.  (This might need refinement.)
Add PV name to "PV gone" messages.
Adjust some log message severities.  (More changes needed.)

Remove support for TRIM message

It's been unsupporte for now - and it's not going to be
implemented for thin pool kernel driver - so dropping
appearence of TRIM from libdm and lvm.

Reinstate error if e.g. daemon comms fails.

Wait for mirror in sync

1s could be too short

Check for error from dup2

Do not take correct fd as error result.

Add traceback for failpath

Skip zero length messages

In case of zero length message, there would be a memory leak on
return path from _do_process_request.

Check allocated pointers

Test pointers from allocation against NULL.
Error paths should be checked, some of them probably need
some extesions.

Improve logging

Use %m for strerror.
Switch to use 0 for fail return code.

Check pointer before deref in debug message

Explicitely ignore fail from this label_read

Add some more pointer validation

Ensure _display_name() and _add_dep() would not deref NULL names.
Switch to use internal dm_basename().

Test alloc fail

Replace goto with just log_debug

FIXME: temporal quick hack to make things green
There might be a better reaction.

Check for alloc error

Simplify segtype_str usage and check for NULL segtype.

Log sys error for lseek

Debug log for hold_lock failure

Log sys errors

Backtrace for failing drop_cached_metadata

Check for udev_get_dev_dir result

Don't use NULL return value.

Correct enum type

Using debug_t and some forgetten alloc_policy_t, force_t from past commit.

Few more close and dev_close trace

Adding (void) where we cannot really report an error.

Use 64 bit integers for device size & label sector in _pv_populate_lvmcache in
lvmetad client code. Fixes RHBZ 798267.

Add a regression test for the PSize bug in pvs/lvmetad.

Use 64 bit integers whenever extracting numbers from daemon replies.

s/CPG_/CS_: Various CPG constants are going away, even though CPG itself stays

F17 is getting rid of OpenAIS libraries (and checkpointing).  While the
CPG stuff is staying, some if its constants are being removed.  So, we
must adjust and use the remaining constants which the CPG constants were based on.

[~]# egrep 'CPG_DISPATCH_ALL|CPG_OK' /usr/include/*/*
corosync/corotypes.h:#define CPG_DISPATCH_ALL     CS_DISPATCH_ALL
corosync/corotypes.h:#define CPG_OK               CS_OK

Missed ;

More useful debug message

Add _rimage as reserved suffix

Add cast for ptrdiff_t - cleanup gcc warning

Improve error logging

Log errors instead of plain return 0.
Check for f->private strdup result.

Add stack trace for lv_reduce

Use common code call with stack trace.

TODO: maybe the release_lv_segment_area()
should be actually able to return error code to upper level.

Check for version string buffer

Since lvm seems to call driver_version(NULL, 0)  this would lead
to crash. Though the combination of the code is probably very hard to hit.
If the user doesn't supply version buffer, just skip printing to buffer.

Check for allocation error

return ENOMEM when malloc fails.

Log fail of pthread_join

Syslog error outside of mutex section.

Fix leak of FID structure

Wipe initial 4KiB for non-zeroed thin volumes

If the thin pool has disabled zeroing  (created with -Zn), we at least
clear initial 4KiB of such thin volume (provisions 1st block).

If lvcreate is executed with '-an' command will abort (same way like we for
normal LV - however for normal LV option -Zn may skip clearing completely,
for thin volumes this option is not supported (applies only for pools).

Allow cluster mirrors to handle the absence of the checkpoint lib (libSaCkpt).

The OpenAIS checkpoint library is going away; therefore, cmirrord must
operate without it.  The algorithms the handle the timing of when to send
a checkpoint, the determination of what to send, and which ongoing cluster
requests are relevent with respect to the checkpoints are unaffected.  We
need only replace the functions that actually perform the storing/transmitting
and retrieving/receiving of the checkpoint data.  Rather than store the
checkpoint data in an OpenAIS checkpoint file, we simply transmit it along
with the message that notifies the incoming node that the checkpoint is
ready.

Pass 'single_device' parameter down to suppress 'Can't find uuid' messages
when reading VG text metadate and called from pvscan --lvmetad.

(Longer-term, that check needs moving outside of that code.)

Fix a whitespace bug in the last checkin.

Attempt a fix for lvm shell accumulating copies of orphan PVs with each "pvs"
invocation.

expose new include files to build

Reflect new file locations, include file updates etc.

remove old makefile

move daemon files

new makefiles

Series of commits to move daemon code into separate top-level dir and
fix build deps etc.

Invert some return codes (which used to get ignored anyway) in cases when
lvmetad is not being used.

Bypass lvmetad fns when not configured to use lvmetad.

Add LVMETAD_SUPPORT

If lvmetad fails, don't set _has_scanned

Define LVMETAD_SUPPORT in source code

spacing

I don't think -u is meant to be used with lvmetad

Check return values after calling new lvmetad fns
(Haven't checked error path handling though)

Fix empty string warning logic in _find_config_str. (1.02.68)

pvcreate gives
WARNING: Ignoring unsupported value for metadata/pvmetadataignore.

It was warning if there is no config file entry instead of only if the node
exists but is empty.

Check for failing dm_asprintf

Using enum types for enums

alloc_policy_t, dm_string_mangling_t, percent_range_t, sign_t

Revert free of allocated segtype

lvm_register_segtype takes ownership of segtype and call destructor
for it in error path.

LVMetaD socket operates in non-blocking mode.

Add support for systemd socket handover for common daemon-server code and also add support for new OOM killer adjustment interface.

This code is already a part of dmeventd, but it's modified slightly to check
sockets instead of FIFOs.

Test dm_hash_insert() failures mem failures

Check error from _lv_each_dependency

_lv_mark_if_partial_collect cannot fail, however it's good to keep
checking here as we do in all other cases.

Check result of export_vg_to_buffer

and avoid usage of failed buffer.

Add stacktrace for init_lvmcache_orphans fail path

Ensure clvmd message is always \0 terminated

Drop whole buffer clearing (most messages at <100 bytes).
Just make sure we have always \0 terminated string for strlen() operations.
(before for PIPE_BUF sized messages this was not set).

Better detection of missing dmeventd fifo connection

Missed to properly merge patch for definition of MAX_MISSING_LEN

(fix previous commit)

Add log_sys_error for close

Add some stack traces for dev_close error paths

Explicitely ignore errors in label_read iteration.

We don't want stack traces here, so mark as void.

Explicitely check list size of segments

instead of checking for NULL from last_seg and first_seg.

Duplicate standard in/out descriptors for daemon

Addressing somewhat tricky bug here.
Since stdin,stdout,stderr were closed it's been occasionally possible to
see some unexpected messages to be flowing into a clvmd and generating some
randomly sized allocation of many megabytes. Since the message was not
being generated by standard send_message() construction, after some more
testing it apperead to be a debug log message - thus something has flown
to local socket opened on strandard out descriptor.

To fix the issue - use standard file descriptor duplication code for daemons.

For making easier debugging of polling daemon - developer might want to recompile
without modifition of standard file descriptors.

Limit max size of clvmd message

This could be seen as some sort of simple validation - it's not easy to
recognize a valid message for now - but we definitely do not want to
allocate a lot of megabytes in  clvmd memory locked daemon when broken
message gets in.

Size of 8000 is just selected for now - possibly there could be much
lower value put in.

Do not send uninitilised bytes

Use struct initalizers to fill struct members and at the same time have
all unspecified members set to 0.

Fix dm_task_set_name to properly resolve path to dm name.

Wrong variable was checked in _dm_task_set_name_from_path.

lvmetad.socket

Do not hide deallocation of buffer

As API is passing structures by value, do not leave
the function which created buffer and keeps valid pointer
look like it would be some memory leak and move
free of buffer from inner function - makes more obvious,
how is the memory management handled.

Add FIXME for misuse in case of dm_event_mask enum.

Using enum for bitmask is not proper solution here.

Use unsigned type for bitmask

Using report_type_t for bitmask is not correct, since we have not defined types
for all  bit combinations - so switching to unsigned type, since values of
report_type_t enum are unsigned.

Just code move of hash initialization in front of function

Make sure both hash tables are initialized before _read_sections() call.
Presents no functional change (since PV scan phase was not adding LV hashes),
but makes the code easier to handle mem failing case, and static analyzer is
hapier as well.

Add stack traces for lock_vol failures

Adding at least stack traces with some FIXMEs for cases,
where we might want to do something cleaver - maybe fail command
or give user hints something is not going well ?

For remote_backup is stack probably 'good' enough for now.

Check id_write_format result

Currently we never fail with 64byte uuid buffer,
but just stay consitent with rest of the code and check for result.