comment and formatting changes

sporadically seeing this problem. Still trying to track down origin.

Correct bad format in log message

Update to commit 02ad8fdd so that locks/unlocks match
if gres.conf not found

gres needs to locally keep the mps_table size rather than use
node_record_count, which gets reset to zero at shutdown.

Check for out of range node index. Not observed, but prevents possible
invalid memory reference.

Copied array without including the array size pointer, so it did
not get freed.

It was failing due to an Epilog, but could also fail when run in
parallel with other jobs.

This includes information about how to get a clean HWLOC report.

Without this change I was able to fairly consistently cause
"scontrol shutdown" to NOT cause the slurmd to exit.
1. Start slurmd and slurmctld
2. Immediately execute "scontrol reconfig" and "scontrol shutdown"

Format changes only

log message and comment format changes

Bug 5851.

# Conflicts:
#	slurm/slurm.h.in

This prevents rebuilding a job's dependency string when it has at least
one invalid (never satisfied) dependency, no matter if such invalid
dependency has already been purged (after MinJobAge) or not. This can
be useful to track down the culprit invalid dependencies even after they
are gone from ctld's in-memory job list.

The flag is cleared upon a successful job dependency update or after
another job in the dependency list has been satisfied if such list is
composed with the '?' symbol (OR'ed).

Bug 5851.

Job dependencies separated by "?" (OR'ed) should make the dependant job
be independent as soon as any of the dependencies are resolved to be
satisfied. Without this patch, if an invalid (non satisfiable)
dependency was resolved before a satisfiable one, then the dependant job
would never become independent, even after the satisfiable one got
eventually resolved.

Bug 5851.

No functional change, just preparement for a following commit with an
actual fix.

Bug 5851.

The response of the XCC raw command is always 16 bytes, we log it and return if
we don't get an answer of this size.

Bug 6743

If GRES configuration data is unavailable from gres.conf, then use the
    node's "Gres=" information slurm.conf. This will eliminate or minimize the
    gres.conf file in many situations.
bug 6761

They were a bit too verbose for my taste

In 0e149092 not setting the variable when job was not requesting any gres
was considered a bug. The cuda API will use all devices if the variable is
not set. If it is set to some unknown or empty value, it will use no devices.
This variable should be used only for testing purposes and ConstrainDevices=yes
in cgroup is recommended.

Bug 6412

gres plugins will setup environment for every gres in the system even if the
job has not requested it.

Bug 6412

than one GRES of the same name but different type"

This reverts f7fca7ba

Bug 6412

initial work needed for bug 6761 support

comment format and change some log messages

this problem was triggered with a configuation of
PrologFlags=Alloc,Contain