Bug 5680

and other_cons_res.

continuation of previous commit.

Bug 5680

In preparation for next commit.

Bug 5680

Blessed by Tim.

Still respect "cray" as the keyword here for now, but add "datawarp" as
the documented form instead.

Bug 6440.

Since the slurmd will change an older protocol version to the current
version we need to make sure if an old srun sends a message we handle
things correctly.

Otherwise non-FairTree systems will have a decay period delay before
the updated value makes it into the sum.

Bug 5202.

Usagefactor matches the documentation and now multiplies TRES time
limits and usage.

Bug 5435

This is very coarse-grained locking, but as the initial implementation
did not anticipate concurrent access this is the safest approach for now.

Bug 5638.

Rather than repeat the #ifdef block through five locations, use a common
function to handle this and print a consistent error message if there
is an issue.

From getrlimit(2):
	setrlimit() now returns with errno set to EINVAL in places that
	historically succeeded. It no longer accepts "rlim_cur =
	RLIM_INFINITY" for RLIM_NOFILE. Use "rlim_cur = min(OPEN_MAX,
	rlim_max)".

(This was causing spurious error messages to appear for sbatch/srun.)

macOS does not permit the symbol overloading trick that was previously
used here. Sending function pointers in as part of slurm_opt_t should
work around this.

Avoids a dereference of slurmctld_primary, which is a weak symbol,
and is not available when the select plugins are loaded in the user
commands.

(Weak symbols on macOS cannot have local definitions as well - so
any reference to them that does not resolve causes the process to
crash.)

macOS always has these available without setting -pthread, and ACX_PTHREAD
does not work properly on macOS at the moment.

These should ideally be split off from the task plugin interface since
they're tied to a specific implementations.

And the core_spec code itself does not really belong directly in
slurmd.c.

There is no equivalent to cpuset_t.

Remove initializations - these are not permitted on weak_import'd
symbols.

This was cobbled together from a few different references.
All it's doing is forcing a symbol to be created with the slurm_*
name, and setting the symbol address to that of the unaliased symbol.

Bug 3745.

Using an index value equal to the number of elements puts you one past
the end of the array, modify conditional to >= instead.

CID 197759.