Bug 3953

This removes several define statements with different names in
  various functions

Fix for possible slurmctld abort with use of salloc/sbatch/srun
    --gres-flags=enforce-binding option.
bug 4008

Update from commit b40bd8d3

Clarify --immediate option.

By removing the real locks we can get into a race condition where the prolog
starts and finishes before we get here and then we end up waiting forever.

Making the mutex a static seemed to help in many cases, but didn't
completely close the window.  Changing slurm_cond_wait to
slurm_cond_timedwait fixed the scenario where we would hit the window, but
not degrade performance the original commit provides.

There were also spots where if the job or step didn't exist it wouldn't
signal the conditional also providing a spot this could get stuck not
starting the job.

Fix regression from commit 52ce3ff0

Bug 3977

Fix for debugger setup bug introduced in commit
f1110568

Avoid interleaving labels and output from various components of
   a pack job

This allocates an array to the proper size (based upon all tasks
  to be launched). Work still needed to populate the data structure
  properly for all tasks.

No change in logic

The debugger symbol in srun is not being properly handled today.
  This change does the malloc once, even for a pack-job, makes
  the array over-sized, and range checks before writes. Suitable
  for making srun progress without memory errors (writing off end
  of allocated memory in an array).

Code provided by Ole Nielsen <Ole.H.Nielsen@fysik.dtu.dk>

Bug 3985

Major re-write of task state container logic to support for list of
  containers rather than one container per srun command.

Modify all daemons to re-open log files on receipt of SIGUSR2 signal.  This
    is much than using SIGHUP to re-read the configuration file and rebuild
    various tables.
bug 3070

debug.

This is a regression from commit fec995e0.

It turns out using tok here was erroneous for situations where the gres had
a type and name and potentially a count (i.e. network:gigabit:1)

_get_gres_req_cnt() would alter the incoming char *config which is what tok
was.  So when we print it back to the requested string it would only have
what was there to the first ':'.  As we didn't need to \0 out the first char
as we skip over it anyway I just kept track of what the replaced \0 was for
the number portion and put it back when we are done copying it.

Related to bug 3521

No changes to logic

bug 3979

Bug 3967