Added new field to partition_info data structure
Break up some long lines and minor format changes
Move some definitions and statements into alphabetic order

This will happen if the plugin is configured and the DynAllocPort
configuration parameter is not set

Hi everybody!

Here attached is a patch that enables partition based SelectType (currently CR_Socket/CR_Core) in select/cons_res.

The patch requires that CR_ALLOCATE_FULL_SOCKET is enabled to work and also this patch from master branch: https://github.com/SchedMD/slurm/commit/cdf679d0158a246e7389a15b62f127e5142003fe

It should however be easy to change it to use the old #define if you want to.

We are currently testing this in our development system but will go into production later this spring based on needs from some of our users.

One thing that I noticed during the development of this is that if a new option is added to the slurm.conf that is not supported with an earlier version of slurm programs/libs that are compiled with the earlier version stops working due to complaining of errors in slurm.conf.

We have the CR_ALLOCATE_FULL_SOCKET patch in our production system and some programs linked with openmpi stop working for some of our users.

It might we wise to try require less reading of the slurm.conf from the core parts of slurm and try to put more reading/parsing of the config file from the plugins (and other modular parts of slurm).

Best regards,
Magnus

1. Removed the job_submit and job_modify functions from the plugin, they are not required for the "slurmctld" plugin type
2. Renamed the new parameter from "JobSubmitDynAllocPort" to "DynAllocPort" and renamed the variable (You need to change this in your slurm.conf file)
3. Added logic so you can see the DynAllocPort value using "scontrol show config" or "sview"
4. I made some minor formatting changes, mostly for lines that were too long
5. Added #ifdef to the msg.h header file
6. Changed the #ifdef variables in the header files to start with "DYNALLOC_", perhaps not needed, but it should safer, especiallly with some common names like "INFO_H"
7. I re-wrote much of info.c. There was no need to get a copy of the node information and process the copy. We can just work directly with the data structures.

These provide support for MapReduce+

Make core reservation test a bit more flexible in terms of node names

Fix arithmetic for core reservation count return for old RPCs
Update NEWS and RELEASE_NOTES

Remove trailing spaces from recent scontrol man page changes

Core based reservations in an asymmetric cluster should use a nodelist
with a corecnt array which overcomes the last limitation when using
these kind of reservations.
Next work will be to allow updating a core based reseravtion.

Old functions were completely wrong, but not used on most systems

Old logic uses some vestigial power management logic that no longer
applies. It creates a job allocation, but the job step blocks
indefinitely.

Once the first complete job is found, there is no need to check for
its dependencies as they will not stop another job from starting

Added for MR+ operation

Conflicts:
	NEWS

Conflicts:
	META

since left the system.

to compile correctly on newer compilers

Signed-off-by: Danny Auble <da@schedmd.com>

If job involved in dependency completes and is purged the logic
used to test for circular dependencies can use the invalid pointer
and generate an invalid memory reference before the pointer is
cleared from the dependency list data structure.

Without this change, allocations less than a whole node can result
in an incorrect task binding for power7 processors

Without this change, the testing only happens with task/affinity