Use slurmctld_conf.state_save_location instead.

burst_buffer/cray - Fix race condition that could cause multiple batch job
    launch requests resulting in downed nodes.
bug 3366

This reverts commit b9bff82f.

The dbd_list_msg_t was uninitialized in a bunch of places and the
  return_code variable in the structure was being referenced,
  resulting in valgrind error reports

No change in logic, just added values to comments in a long enum
  to more easily identify their values

First noticied by Brian.

It was leaking memory otherwise

Not used here.

Commit 63b7e3a8 changed the --mem limit to 1MB for the job if
not using a memory SelectType, but this can cause the job to fail
if the JobAcctGatherFrequency is frequent enough to notice that the
"sleep" command is using more than 1MB of resources.

Refactor test to avoid specifying job memory. Use --wrap to avoid
creating a temporary job script as well.

Commit 63b7e3a8 changed the --mem limit to 1MB for the step if
not using a memory SelectType, but this can cause the job to fail
if the JobAcctGatherFrequency is frequent enough to notice that the
"sleep" command is using more than 1MB of resources.

Refactor test to avoid specifying job memory. Use --wrap to avoid
creating a temporary job script.

Commit 63b7e3a8 changed the --mem limit to 1MB for the step if
not using a memory SelectType, but this can cause the job to fail
if the JobAcctGatherFrequency is frequent enough to notice that the
"sleep" command is using more than 1MB of resources.

Refactor test to avoid specifying memory memory; and since only one
step is checked for, only run a single step in the job. Use --wrap
to avoid creating a temporary job script.

This bug was introduced in commit 93adc329
Symptom is failure of test 2.7

Some portions of tests 21.30 and 21.34 failed with accounting and
priority basic. These changes disable portions of those tests as
needed based upon configuration.

FreeBSD requires this to build; overlooked in 2d9e999f.

Update information about modified functions and data structures.

It was leaking memory otherwise

_allocate_sc() is heavily used within the scheduler, change to
stack allocation from heap to avoid constant churn on xmalloc()/xfree().

Bug 3420.

test1.63 was failing periodically due to a race condition. A signal
  was being sent to srun before the signal handler thread was spawned.

Too closely related to working_cluster_rec.

CID 16088[1-4]

Could be used in bit_ffs and bit_fls functions rather than
existing for loops.