Introduced by commit 059275f6 when the timer is trigger.
Releasing the locks means that job_ptr may point to an element that was
deleted by a different thread in the meantime. Restructuring the code
to advance the iterator prevents this - the iterator itself does not have
this issue as the List structure will manage the position during the
sleep().

While here, move the reservation update handling outside of this
loop to simplify operation. This does not need to piggy-back on the
scan of the job_list - switching to using list_for_each should
mitigate some of the performance loss by needing a second full pass.

Bug 3414.

These were mis-calculated previously, and are internal implementation details
that weren't meant to be exposed.

association GrpWall limit.

limits.

Bug 3476

old ones.

This is cosmetic only, no code change.

Bug 3476

Bug 3472

regcomp() is not safe to use across a fork in older glibc versions.
Reinitialize the keyvalue_re structure after the fork through an atfork()
handler.

Bug 3276.

This is a regression from commit b818dd9d

Basically the first AC_LINK_IFELSE sets whatever compiler we
are using to be that.  Since the above commit removed the BGL/P
code that was linked using C C++ became the compiler since the
next thing was BGQ in configure.ac to test against.

I just grabbed the DATABASES call, but any other one could had
worked.

another one.

Bug 3465

This is the way it is done with the task plugins.  It appears this only
really matters when requesting 1 task with a full socket with exclusive
access.  This code would cyclically allocate sockets to the step instead
of filling up one socket then going to the next.

Fix for job constraint specification with counts, --ntasks-per-node value,
    and no node count.
bug 3470

Task/cray: Treat missing "mems" cgroup with "debug" messages rather than
    "error" messages. The file may be missing at step termination due to a
    change in how cgroups are released at job/step end.

Honor --ntasks-per-node and --ntasks option when used with job constraints
    that contain node counts.
bug 3458

Defer interactive job allocation until ALL allocated nodes are ready
  rather than after PrologSlurmctld (if any) completes.

Bug 3467.

burst_buffer/cray - Do not execute "pre_run" operation until after all nodes
    are booted and ready for use.
bug 3461

partitions.

Insure job does not start running before node is booted and PrologSlurmctld
    is complete.
bug 3446

Ported from 7a4aa7f2.

burst_buffer/cray - Support default pool which is not the first pool
    reported by DataWarp and log in Slurm when pools that are added or removed
    from DataWarp.
bug 3453

This reverts commit fd690a9c.

was there for jobs and steps, but for no real reason.  I couldn't see a
reason anyway.

Jobs preempted with PreemptMode=REQUEUE were incorrectly recorded as
REQUEUED in the accounting.

Bug 3444

bug 3448

Bug 3447

Bug 3444

Fix srun I/O race condtion to eliminate a error message that might be
    generated if the application exits with outstanding stdin.
bug 3166