sizeof(char) is defined to be 1, and Slurm is already heavily
reliant on that throughout. Stop pretending Slurm would work
on a hypothetical system in which this was not true.

(C99 Standard - 6.5.3.4.3.)

Strip off unnecessary casts while here as well.

Replace with xstrfmtcat(), and resolve this overrun warning.

Cannot return NULL, it will abort() before then.
Remove the unnecessary cast while here.

Add configuration parameter "GpuFreqDef" to control a job's default GPU
    frequency.
Also Coverity CID 187908

Spaces need to be built into each key definition, otherwise it'll
all run together.

Bug 5657.

Otherwise assoc_mgr_fill_in_user() will abort on a failed lock annotation.

Convert to designated initializer format while fixing this.

Bug 5653.

Further work will be needed to track these independently.

Bug 5256.

Move the trigger_backup_ctld_fail() call into ping_controllers(),
and use the ping_all boolean as a standin for whether we're the active
controller or not.

Bug 5256.

The handling for response_time was a bit odd, as care was needed to make
the timestamp values match up. Meaning they're not really time values,
just a complicated way of signaling whether the remote controller is
responding. So, simplify this by just indicating through a bool whether
a given controller is responding or not.

Bug 5256.

If true, ping everyone except outselves. If false, continue to ping
only the higher priority controllers.

Also move the ctld_ping xmalloc and xfree in here so we don't need
to recreate them in the controller code later.

Bug 5256.

Both the primary and backup need to provide similar ping capabilities.
This implementation - with multiple threads launched in parallel - is
the more advanced one, so start reworking it to be able to use it in
the primary controller as well instead of _ping_backup_controller().

Add a boolean option to indicate whether this is running in the active
controller or the backup. This will control whether all other controllers
are pinged (needed by the primary) or only the higher priority controllers
(as done in the backups).

Bug 5256.

with some configurations test39.1 would fail. This was due to
not considering use of GRES that are non-local to available
CPUs, even without enforced binding. This was due to always
trying to enforce binding on the first pass. If that fails,
then consider use of non-local GRES. The problem in the logic
is it would never try non-local GRES if avail_res_array->avail_res_cnt
was zero. This patch sets its value to the node's CPU count,
which may not be ideal, but is a reasonable option.

Bug 5651.

The correct syntax is .\" not .\ which was causing these to appear in the
man2html formatted version of the man page. As these options do not and will
not exist in 18.08, remove them rather than re-commenting them with the
correct syntax.

The correct syntax is .\" not .\ which caused these to appear in the man2doc
formatted version of the page by accident.

Update slurm.spec and slurm.spec-legacy as well.

And slurm.spec-legacy. Missed in commit ee281387.

Bug 4771.

Asterisk '*' was being used in 2.4, and replaced with colon ':' in 2.5.
But liense_list_to_string was missed, and was still using an asterisk
until now. Since we've internally used both up until 18.08, license parsing
code will need to respect both delimiters until two versions past 18.08.

Bug 5630.

Will be needed to distinguish between which backup controller is resuming.

Bug 5256.

No changes to RPC yet, identical to prior RPC version.

Bug 5256.

This should solve a clang/scan-build issue.

Bug 5256.

starting.

Bug 5565