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.

Explicitely ignore fail error on hash_insert

We cannot do anything better here anyway - we are already in logging function,
so just ignore this issue here - it will most likely stop application later.

Add some log_sys_errors to close() call

Nicer cleanup of excl_uuid hash

Since it on exit path, it's not a big difference,
but makes less noise in analyzer and valgrind.

Check for vg_name existance

Since vg_read() mda ops could be called with NULL vg_name,
check it before derefence also for pool and format1.

Systemd is linux-specific - move the supporting code under the 'ifdef linux'.

Some 'defines' used in this specific code were already under an ifdef so this
patch just completes it.

Fix missing break

Bug introduced with addition of internal error default case.
Seem like this code is not used.
TODO: add coverage test.

Add assert for oldname

Code cannot proceed if oldname would be NULL.
Since lvmetad currently doesn't use logging mechanism of lvm to report
internal errors - stay with current code style of lvmetad which uses
plain asserts for cases like this.

A bit more safe version of sprintf

Use just buffer size limit (it's used for debug only)

Test seg pointer for non-null

As the function accepts NULL for 'seg' parameter,
check for it before dereference.

Move allocation after check for vgid

so there is no mem leak on this error path.
Also actually check if the hash exists.

Make force_t enum usage obvious

Test result of _init_tags.

Explicitely ignore result from sync_unlock()

Make it obvious to analyzer - we can't do anything better here anyway.

Always check result of _set_vg_name()

Drop uname call, it's not used from gulm era.

Check allocation result

Remove test for pvid

Since pvid is char buffer[] and not pointer,
there is no point to check it for NULL.

Ignore result of unlock explicitely

Just reindent with tabs

Use 'killproc' fn in lvmetad init script (fn provided by 'functions' init script library).

Fix a possible SEGV in lvmetad client code, and a minor leak in pvscan
--lvmetad.

A bit of documentation for lvmetad_vg_lookup.

- Improve error reporting on lvmetad connection failure.
- Fix a couple of memory leaks in the lvmetad client code.
- Avoid an error in lvmetad_pv_gone when we aren't using lvmetad.

Improve error handling & reporting in common daemon code.

Add tentative documentation of pvscan --lvmetad to the pvscan manpage.

Also remove intermediate lvmetad init script file.

Use '-p PID_FILE' for the status call since the pidfile is in its own lvm subdir and return proper status code.

The daemon as well as the first scan must be both executed properly, otherwise it's an error!

Add LVMetaD init script.

Add skeleton for lvmetad udev rules - 69-dm-lvm-metad.rules.

Why using the order 69:

  - Storage processing in general happens in 60-persistent-storage.rules,
    including the blkid call that adds some usable information we can use
    for filtering and speedup (these rules are part of upstream udev and
    the order is preserved on most distros)

  - There's still some other storage-related processing done after
    60-persistent-storage.rules in general. These might add some detailed
    storage-related information we might use to filter devices effectively
    (e.g. MD udev rules, ...).

  - We need lvmetad rules to be processed before any consumers can use the
    output - so the metadata cache is ready soon enough (e.g. udisks rules).

  - There's no official (upstream udev) document about assigning the order,
    so this number is chosen in best belief it will suit all scenarios.

Fix server-side leaks in lvmetad.

Clean up the lvmetad state more thoroughly upon shutdown.

Fix an error path in daemon_open.

Make the calls to pvscan --lvmetad in the testsuite slightly more civilised.

Couple of improvements in the daemon (common + lvmetad) code:
- some client-side memory leak fixes
- announce and check protocols and protocol versions

Limit sscanf params with size

Make sure parsed string fits given char buffer.

Unit testing for some string libdm functions

TODO: more functions need to be unit tested.

Introduce dm_strncpy

Should be faster then strncpy - since we could avoid clearing 4KB pages
with each strncpy(...,PATH_MAX).
Also it's easy to check whether string fit - and eventually avoid
to continue working we incomplete string.

Use const for lv

lv_is_active doesn't needs modifiable LV struct so keep it const.

Remove lv_send_message() left bits from code -
they were never released in 2.02.89.