From fc08a0b45e3f574b811910f1b37630c5c0715c78 Mon Sep 17 00:00:00 2001
From: Danny Auble <da@schedmd.com>
Date: Thu, 1 Dec 2011 15:47:06 -0800
Subject: [PATCH] BGQ - cleaner code to exit out of loop when we already know
we are only looking for 1 midplane, no reason to go through any more than
that.
---
src/plugins/select/bluegene/ba_bgq/block_allocator.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/plugins/select/bluegene/ba_bgq/block_allocator.c b/src/plugins/select/bluegene/ba_bgq/block_allocator.c
index 610b2de099b..e257b5a5cd3 100644
--- a/src/plugins/select/bluegene/ba_bgq/block_allocator.c
+++ b/src/plugins/select/bluegene/ba_bgq/block_allocator.c
@@ -1191,7 +1191,7 @@ extern int ba_sub_block_in_record_clear(
error("ba_sub_block_in_record_clear: "
"didn't have the units_used bitmap "
"for some reason?");
- continue;
+ break;
}
bit_not(jobinfo->units_used);
@@ -1222,6 +1222,7 @@ extern int ba_sub_block_in_record_clear(
xfree(tmp_char3);
FREE_NULL_BITMAP(total_bitmap);
}
+ break;
}
list_iterator_destroy(itr);
--
GitLab