Update slurm.spec and slurm.spec-legacy as well.

Bug 6029.

Duplicate file names will cause problems for gres/mps,
which needs to make 1-to-1 to gres/gpu devices

Support undocumented "Files=" in addition to "File=".
Note that multiple file name can be used as an argument
and this minor change eliminates some possible confusion.

bug 5520

bug 5520

without this, the jobs were being assigned the wrong
CUDA_VISIBLE_DEVICES value

The cpu frequency set by the user is not exact with current kernels.
There seems to be a fair variation depending upon timing and other
events. This is resulting in test1.76 failing sporatically. This
changes the logic to retry if the frequency differs by more than
10 percent rather than failing immediately.

The device numbers are set using the same mechanism used to set
CUDA_VISIBLE_DEVICES

bug 5520

Add step_unconfigure_hardware() to GRES plugin API
Update test39.18 regarding links.
Update GRES docs.
Update docs related to links.
Document GPU frequency resetting behavior.
Specify what the default is for GpuFreqDef.
Move NVML init and shutdown to configure() and unconfigure().
Get rid of superfluous `!= 0`-style statements.
Print note when GPU index != minor number.
Clean up various formatting and other errors.

bug 5520

No functional change.

Due to upcoming changes in the X11 forwarding subsystem, support
for older-style X11 tunnels will be removed. Older client commands
cannot support the newer style. Rather than have the tunnel fail,
request the job allocation request up front.

Bug 3647.

Also tweak the one info() message here to match these others.

New X11 forwarding code will only support forwarding back to
salloc or an allocating srun command.

Using this option within sbatch was always hit-or-miss. If the
user submitting was disconnected from the alloc host for any
reason their xauth credentials would likely fail even if they
managed to get assigned the same local TCP port for forwarding.

Bug 3647.

Bug 6029

< PAM_MAX_MSG_SIZE (which as of this date is 512)

In some systems there can be multiple user accounts for uid 0, therefore
the check for literal user name "root" might be insufficient.

Bug 6184

Using memcpy, an amount of undefined data from the stack will be copied
into the target buffer. While pam_conv probably doesn't evalute the
extra data it still unclean to do that. It could lead up to an
information leak somewhen.

This pam module is tailored towards running in the context of remote ssh
logins. When running in a different context like a local sudo call then
the module could be influenced by e.g. passing environment variables
like SLURM_CONF.

By limiting the module to only perform its actions when running in the
sshd context by default this situation can be avoided. An additional pam
module argument service=<service> allows an Administrator to control
this behavior, if different behavior is explicitly desired.

Bug 6184

Modify the functions in gres/mps to match changes implemented
in commit 5d40d5dc863446ff6

Add step_configure_hardware() to GRES plugin API.
Call step_configure_hardware() from stepd and make sure it's root.
Sort frequencies returned by NVML.
Update GRES-related docs.
Document step_configure_hardware() and get_devices().
Test the bubble sort algorithm.
Update test 39.9 to work with new code.
Remove vestigial references to volts in the docs.

Only print a warning for 18.08. If a user has SLURM_MEM_PER_CPU or
SLURM_MEM_PER_NODE environment variables set for some reason this
situation could be happening by accident, and we don't want to prevent
the srun command from launching steps at this point.

Bug 6058.

Bug 5648.