Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- SLURM protocol is used to initiate communication and
  deliver sender's TCP port number to the receiver.
- After the first message is received all communication
  goes through the direct TCP connections.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- Introduce nitation of endpoint to distinguish between
  broadcast and point-to-point exchanges.
- Fix magic number naming: change all PMIX_ prefixes to PMIXP_
  to avoid conflicts with PMIx library macroses.
- Fix I/O engine cleanup logic as now I/O engines are used more
  than once.
- make a namespace subsystem to return xmalloc'd pointers as opposed
  to malloc'd to avoid confusion as everything else is xmallox'd.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- Introduce data structures reuse to avoid extra malloc/free overhead.
- Introduce direct TCP connection notation (disabled for now)

This commit is intended for testing SLURM protocol portion of the code
to see that nothing is broken.
Direct communication will be enabled in the next commit.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- Provide ability to initialize I/O engine without knowing fd this will
be used when we will be sending connection request using SLURM protocol
and queuing all other send's waiting for connection to be established.
- Simplify some I/O API and I/O internals.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- sort out API: headers and callbacks
- Change "pmix_" prefix to "pmixp_" for previously unused send functions.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

Remove "health check" functionality as it is not needed.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

This field is a stub for now, but will be used to pass the port number
where corresponding stepd is listening for incoming connections.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

- fix some function names
- separate generic I/O from the slurm protocol specific.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>

Coverity CID 45177

Coverity, CID 44795

Coverity CID 171452

All of these were pre-existing Coverity errors, but I changed nearby
code, variable names, etc. so they looked like new errors.

Coverity reported problem, CID 45194

CID 44936

Coverity CID 171494

variables

This restores perl API operation after commit aea2f0e2

Bug 3956.

Add translation code for the RPCs as well.

Bug 3956.

Add pack_job_id and pack_job_offset to accounting database.
Modified sacct to accept pack job ID specification using "#+#" notation.
Modified sstat to accept pack job ID specification using "#+#" notation.

NULL is returned if the token is not found, testing against '\0'
is wrong (although does work okay in older compilers).

Fixes new GCC 7.1 warning.

was matching more than expected.

error messages that aren't really errors.

Related to Bug 3997

Without this change, each component would generate separate
  email at job begin, end, etc.

If the pack job allocation partially failed, properly handle accounting
  and deallocation of burst buffer. Note, this should rarely happen.

Bug 3999

Default to 1, unless set to 0. Allow to be set to 0 even if
GroupUpdateTime was not set before.

Move down to alphabetical position in read_config.c as well.

Bug 3956.