From b621f8fa7b4dda9478049c3f2397649c3c95d950 Mon Sep 17 00:00:00 2001
From: Morris Jette <jette@schedmd.com>
Date: Wed, 7 Nov 2012 13:53:24 -0800
Subject: [PATCH] Energy Accounting bug, prevent divide by zero for step with
no allocated CPUs
---
src/slurmctld/step_mgr.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/slurmctld/step_mgr.c b/src/slurmctld/step_mgr.c
index 932bd817463..a2a7d8e85bb 100644
--- a/src/slurmctld/step_mgr.c
+++ b/src/slurmctld/step_mgr.c
@@ -2708,7 +2708,8 @@ extern int step_partial_comp(step_complete_msg_t *req, uid_t uid,
/* we have been adding task average frequencies for
* jobacct->act_cpufreq so we need to divide with the
* total number of tasks/cpus for the step average frequency */
- step_ptr->jobacct->act_cpufreq /= step_ptr->cpu_count;
+ if (step_ptr->cpu_count)
+ step_ptr->jobacct->act_cpufreq /= step_ptr->cpu_count;
if (!step_ptr->exit_node_bitmap) {
/* initialize the node bitmap for exited nodes */
--
GitLab