Coverity CID 187778

Coverity CID 187927

Coverity CID 187926

This reverts some logic added in commit 01336077 for debugging

"no_send_gids to go back to the original behavior.

Bug 5620

LaunchParameters=send_gids.

Bug 5620

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

Bug 5659

This reverts commit 35285e40.

Bug 5659

Bug 5663

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.