Bug 3253.

If the input value mod 512 == 0, the value would be subject to
unintended rounding. Rework the function to check against this
on each unit promotion.

Bug 3252.

It was causing the loss of node available_features on startup with
  node_features/knl_cray
bug 3241

Check for zonesort file first, to save time over attempting to load
  a module that is already loaded. It may be loaded by default per
  administrator configuration.

Missed from 1aa299ab

This should make a Coverity error go away. The error is being
  generated in error by Coverity, but it isn't smart enough to
  determine the range of values possible from a modulo function
  (if the input is negative, its results can't be less than the
  negative of the divisor). This should clear about a dozen errors.

Leaks reported by Coverity

as reported by Coverity

Plug about 20 memory leaks reported by Coverity

Set per-node HBM availability as a GRES based upon the KNL node's
  MCDRAM state
bug 3171

Caused by race for local_energy which is dynamically allocated. Bail out
of the update if that hasn't been allocated yet.

Bug 3237.

avoid unused variable in front-end mode

Add new node state flag of NODE_STATE_REBOOT for node reboots triggered by
    "scontrol reboot" commands. Previous logic re-used NODE_STATE_MAINT flag,
    which could lead to inconsistencies. Add "ASAP" option to "scontrol reboot"
    command that will drain a node in order to reboot it as soon as possible,
    then return it to service.
bug 3210

bug 3213

select/linear plugin modified to better support heterogeneous clusters when
    topology/none is also configured. Note that use of the select/cons_res
    plugin is strongly recommended for heterogeneous clusters. The use of
    OverSubscribe=exclusive can be used if whole node allocations is
    desired.
bug 3212

Bug 3224.

Different paths were used in various locations. All changed to use
  the /var/spool directory.

If a job is started by the main scheduling logic and requeued while
  the backfill scheduler has locks released, that can result in an
  invalid data structure in select/cons_res. Namely, the backfill
  scheduler's attempt to start the job would clear the job resources
  node_bitmap. That leaves a NULL pointer in the select/cons_res
  plugin generating an abort. (That pointer is needed to clean up
  the job allocation records when the Epilog or Cray Node Health
  Check, NHC, are complete and the resources become available for
  another job.
bug 3230

task and can get quite verbose on a KNL node.

warning fix.

if the pointer is valid

Backup slurmctld will now
1. Not abort due to NULL pointer (needed to move code around on restart)
2. Recover KNL MCDRAM and NUMA modes from state save files if capmc and
   cnselect not available
bug 3241