From 2314b8199fd4209e273277ffbcf0ff88916ab7c0 Mon Sep 17 00:00:00 2001
From: Moe Jette <jette1@llnl.gov>
Date: Thu, 26 Mar 2009 00:55:38 +0000
Subject: [PATCH] modify slurmd to set CPUSETs for tasks before launch.
reporting the wrong value in verbose mode though (some other bug in the
task/affinity plugin).
---
src/slurmd/slurmstepd/mgr.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/slurmd/slurmstepd/mgr.c b/src/slurmd/slurmstepd/mgr.c
index 8f27e440f94..77f4bc8b7d7 100644
--- a/src/slurmd/slurmstepd/mgr.c
+++ b/src/slurmd/slurmstepd/mgr.c
@@ -930,8 +930,8 @@ _fork_all_tasks(slurmd_job_t *job)
writefds[i] = fdpair[1];
}
- error("setting user oom to zero");
set_oom_adj(0); /* the tasks may be killed by OOM */
+ (void) pre_setuid(job);
/* Temporarily drop effective privileges, except for the euid.
* We need to wait until after pam_setup() to drop euid.
@@ -990,7 +990,6 @@ _fork_all_tasks(slurmd_job_t *job)
if (conf->propagate_prio == 1)
_set_prio_process(job);
- (void) pre_setuid(job);
if (_become_user(job, &sprivs) < 0) {
error("_become_user failed: %m");
/* child process, should not return */
--
GitLab