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.

Keep same sign for 's' counter

Drop const modifier from fb

It's not useful here to use const - since we need non-const value for unmap.
Also remove now unneeded cast.

Add explicit cast for time() ret value

To keep all numbers with same sign

Use same signed numbers

Keep unsigned aritmetic.

TODO: we should probably switch dm_split_words() to return unsigned numbers.
(minor API libdm change mostly compatible)

Drop backtrace after log_error

Just a minor change to not give backtrace when log_error has been just
reported.

Remaing code suffling

Move declaration to the front of function to follow coding rules.

Fix regression from cleanup commit

baton is being modified, so needs to be cleared before each use.

post-release

pre-release

Missed dm_free in last commit

Limit number of mem allocs and copies

If we have good enough glibc to return number of needed chars, do not
loop try to reach good size, but use this size directly for allocation,
saving also last strdup.

Since now we start with 16 bytes - skip buffer realloc for shorter string.

Some reformating for lvmetad uddates

cleanup gcc warning,
use PRIu64
header cleanups
const pointer fixes.

Require number of stripes to be greater than parity devices in higher RAID.

Also, add some comments to code that I recently added that may be unclear
otherwise.

Add a vgscan to lvcreate-repair.sh. The old test applied device filter hacks to
make devices invisible to lvm, but the behaviour of those is slightly different
than of actual missing devices. Running vgscan after re-enabling the device
triggers a metadata repair which is not done by vgremove -ff. This is not a
regression, merely an odd behaviour that has been around even before lvmetad.

Use 'd' instead of 'D' in tmpfiles configuration to not overwrite any existing directory.

...that happens when systemd creates it first (e.g. based on a .socket unit)
before the tmpfiles.d content is executed.

Kick out a debugging call to valgrind in lvmcache-exercise.sh.

Add a missed dependency that is required to pull in dameons/common in an early
part of the build.

We need daemons/common now even if the lvmetad server side is not built.

Remove the old systemd socket file, we're using the .in template now.