select/cray: whitespace fixes and removal of unused code

select/cray: whitespace fixes and removal of unused code
Patch 10_Cray_COSMETICS-whitespace.diff from Gerrit Renker, CSCS

src/plugins/select/cray/basil_interface.c

@@ -745,9 +745,8 @@ extern int do_basil_confirm(struct job_record *job_ptr)
 
 		rc = basil_confirm(resv_id, job_ptr->job_id, pagg_id);
 		if (rc == 0) {
-			debug2("confirmed ALPS resId %u for JobId %u, "
-				"pagg %"PRIu64"",
-				resv_id, job_ptr->job_id, pagg_id);
+			debug2("confirmed ALPS resId %u for JobId %u, pagg "
+			       "%"PRIu64"", resv_id, job_ptr->job_id, pagg_id);
 			return SLURM_SUCCESS;
 		} else if (rc == -BE_NO_RESID) {
 			/*

src/plugins/select/cray/select_cray.c

@@ -139,7 +139,7 @@ extern int init ( void )
 	/*
 	 * FIXME: At the moment the smallest Cray allocation unit are still
 	 * full nodes. Node sharing (even across NUMA sockets of the same
-	 * node) is, as of CLE 3.1 (summer 2010) still not supported, i.e.
+	 * node) is, as of CLE 3.x (Summer 2011) still not supported, i.e.
 	 * as per the LIMITATIONS section of the aprun(1) manpage of the
 	 * 3.1.27A release).
 	 * Hence for the moment we can only use select/linear.  If some

src/slurmd/slurmstepd/mgr.c

@@ -277,7 +277,6 @@ static int _call_select_plugin_from_stepd(slurmd_job_t *job, uint64_t pagg_id,
 
 	fake_job_record.job_id		= job->jobid;
 	fake_job_record.job_state	= (uint16_t)NO_VAL;
-	fake_job_record.alloc_sid	= getsid(0);
 	fake_job_record.select_jobinfo	= select_g_select_jobinfo_alloc();
 	select_g_select_jobinfo_set(fake_job_record.select_jobinfo,
 				    SELECT_JOBDATA_RESV_ID, &job->resv_id);