Continuation of 2764f3fd

Bug 6589

Continuation of 9a243a1a

Bug 6592

Many comments were modified to follow Linux kernel standard
Many log messages were using the old function name and now
  print __func__ instead
A few log messages lacked the function name and those were added

No change in any logic

This adds logic to validate the count of GRES by device Type and
not just Name and modifies the data structures as needed for
consistency within slurmctld.

Removed linear search, replaced with direct record references and a hashmap.
This is faster and avoids potential collisions between assoc id's and user id's.
Bug 4811

Fixed existing usages as well.
Bug 4811

Coverity CID 197448
bug 6303

Remove reference to REQUEST_SIGNAL_PROCESS_GROUP in slurmstepd.
It has been defunct since July 2013

sort the expected and actual output for GRES APIs irrelevant.
Depending upon the GRES plugins loaded (specifically gres/gpu plus
gres/mps), the GRES records can be sorted by File name to insure
the GRES records line up (the same position in both lists should
refer to the same device file).

Bug 6750.

This logic could allocate to a job a GRES device with
an availability count of zero.

This should only happen if there is flawed logic somewhere, but
avoiding an abort is better than not.

If the count of GPUs configured in slurm.conf and gres.conf differ
and FastSchedule>=1 then the bitmap identifying the GPU allocation
sent from slurmctld to slurmd will differ. Previously this resulted
in CUDA_VISIBLE_DEVICES being set to NULL. Now it will be set correctly.

bug 6725

If slurmd finds GRES with files and slurmctld can't use them
(i.e. slurm.conf has a GRES count of 0), then avoid trying to
create zero length bitmaps in the GRES data structure.
bug 6725

This makes the gres bitmap size equal to the number of records for
shared gres (i.e. gres/mps), otherwise it is the gres count (i.e.
gres/gpu).

bug 6733

if the device files for gres/gpu are out of order or grouped
in an unordered fashion (e.g. "Name=gpu Files=/dev/nvidia[2,8,10]")
then split the gres/gpu records to one record per file and make
sure the gres/mps records are in an identical order. Required for
matching gres/gpu and gres/mps records (one GPU can be allocated
either as gres/gpu or as gres/mps, but not both, so we need to
be able to match records in slurmctld).

Coverity CID 197447

Bug 6710.

Bug 6590.

Make some tests better able to work with CR_ONE_TASK_PER_CORE

More testing required. This configuration is still disabled in
select_cons_tres.c

Add --ntasks-per-core option to execute line as needed

Without this change, test7.17 was failing

Cosmetic change
No change in logic

cosmetic change only

This can happen on node failure

Use stored pointer rather than pointer to pointer for
cleaner code. No change in logic.

Workaround involves specifying the index name when modifying an
existing index.
Bug 6303

The correct use of correct_query and query is clarified here.
Also adds saving the index name to parsing so correct_query
can be set to the correct "drop <index name>" for future query.
Bug 6303

This relocates a variable in order to move some common code into
one place rather than repeat it in several locations. No changes
to functionality, but simpler/less code.

Bug 6680