From e98b8ce204d6ff2efc95eacd550d32fe6e02b158 Mon Sep 17 00:00:00 2001
From: Moe Jette <jette1@llnl.gov>
Date: Tue, 9 Mar 2010 17:24:56 +0000
Subject: [PATCH] corrections to comments only. no changes in logic

---
 src/plugins/select/cons_res/job_test.c | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/plugins/select/cons_res/job_test.c b/src/plugins/select/cons_res/job_test.c
index b63237a2bc0..076a1596865 100644
--- a/src/plugins/select/cons_res/job_test.c
+++ b/src/plugins/select/cons_res/job_test.c
@@ -1099,8 +1099,7 @@ static int _eval_nodes(struct job_record *job_ptr, bitstr_t *node_map,
 					avail_cpus = rem_cpus;
 
 				total_cpus += avail_cpus;
-				/* check to make sure we don't over
-				   step the max_cpus limit */
+				/* enforce the max_cpus limit */
 				if((job_ptr->details->max_cpus != NO_VAL) &&
 				   (total_cpus > job_ptr->details->max_cpus)) {
 					debug2("1 can't use this node "
@@ -1135,8 +1134,7 @@ static int _eval_nodes(struct job_record *job_ptr, bitstr_t *node_map,
 					avail_cpus = rem_cpus;
 
 				total_cpus += avail_cpus;
-				/* check to make sure we don't over
-				   step the max_cpus limit */
+				/* enforce the max_cpus limit */
 				if((job_ptr->details->max_cpus != NO_VAL) &&
 				   (total_cpus > job_ptr->details->max_cpus)) {
 					debug2("2 can't use this node "
@@ -1178,8 +1176,7 @@ static int _eval_nodes(struct job_record *job_ptr, bitstr_t *node_map,
 					avail_cpus = rem_cpus;
 
 				total_cpus += avail_cpus;
-				/* check to make sure we don't over
-				   step the max_cpus limit */
+				/* enforce the max_cpus limit */
 				if((job_ptr->details->max_cpus != NO_VAL) &&
 				   (total_cpus > job_ptr->details->max_cpus)) {
 					debug2("3 can't use this node "
@@ -1382,8 +1379,7 @@ static int _eval_nodes_topo(struct job_record *job_ptr, bitstr_t *bitmap,
 					avail_cpus = rem_cpus;
 
 				total_cpus += avail_cpus;
-				/* check to make sure we don't over
-				 * step the max_cpus limit */
+				/* enforce the max_cpus limit */
 				if((job_ptr->details->max_cpus != NO_VAL) &&
 				   (total_cpus > job_ptr->details->max_cpus)) {
 					debug2("4 can't use this node since "
@@ -1524,8 +1520,7 @@ static int _eval_nodes_topo(struct job_record *job_ptr, bitstr_t *bitmap,
 				avail_cpus = rem_cpus;
 
 			total_cpus += avail_cpus;
-			/* check to make sure we don't over
-			 * step the max_cpus limit */
+			/* enforce the max_cpus limit */
 			if((job_ptr->details->max_cpus != NO_VAL) &&
 			   (total_cpus > job_ptr->details->max_cpus)) {
 				debug2("5 can't use this node since it "
@@ -1866,7 +1861,7 @@ extern int cr_job_test(struct job_record *job_ptr, bitstr_t *bitmap,
 	 * let's factor in the existing allocations and seek the optimal set
 	 * of resources for this job. Here is the procedure:
 	 *
-	 * Step 1: Seek idle nodes across all partitions. If successful then
+	 * Step 1: Seek idle CPUs across all partitions. If successful then
 	 *         place job and exit. If not successful, then continue. Two
 	 *         related items to note:
 	 *          1. Jobs that don't share CPUs finish with step 1.
-- 
GitLab