test39.12 was failing with some configurations by double counting
available cores. Moving some code fixes that problem and speeds
up the logic as well.

Specifically log Cores/CPUs related errors using proper name,
not logging bad CPUs information as bad Core information.

Log SLURMD_HOSTNAME on remote host rather than "hostname", which
is more meaningful on a multi-slurmd configuration

Make tests work properly if GPU count exceeds CPU count on a node

The GRES information now contains socket binding information,
necessitating this change to the parsing function

Bug 4479

"Argument with 'nonnull' attribute passed null"

If the Cores count in gres.conf is bad, slurmstepd was generating
a fatal error at time of step launch. Rather than quitting then,
this change logs the error and ignores the Cores value.

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.