]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
Revert "ctdb-common: Use SCHED_RESET_ON_FORK when setting SCHED_FIFO"
authorAmitay Isaacs <amitay@gmail.com>
Mon, 24 Oct 2016 07:24:54 +0000 (18:24 +1100)
committerKarolin Seeger <kseeger@samba.org>
Mon, 2 Jan 2017 10:56:50 +0000 (11:56 +0100)
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12392

Feature SCHED_RESET_ON_FORK is completely broken on RHEL6 and RHEL7
distributions.  So do not rely on SCHED_RESET_ON_FORK for now.

This reverts commit 1be8564e553ce044426dbe7b3987edf514832940.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Autobuild-User(master): Martin Schwenke <martins@samba.org>
Autobuild-Date(master): Tue Oct 25 11:28:28 CEST 2016 on sn-devel-144

(cherry picked from commit 71b69b0169dc6e2843325f1567f64b6acd43e6b8)

ctdb/common/system_util.c

index e6c4f170b3559a604683c3802ceec1e203118de5..738bf9cf8d376e89714a6741566b7232a01430d4 100644 (file)
@@ -67,14 +67,10 @@ bool set_scheduler(void)
 #else /* no AIX */
 #if HAVE_SCHED_SETSCHEDULER
        struct sched_param p;
-       int policy = SCHED_FIFO;
 
        p.sched_priority = 1;
 
-#ifdef SCHED_RESET_ON_FORK
-       policy |= SCHED_RESET_ON_FORK;
-#endif
-       if (sched_setscheduler(0, policy, &p) == -1) {
+       if (sched_setscheduler(0, SCHED_FIFO, &p) == -1) {
                DEBUG(DEBUG_CRIT,("Unable to set scheduler to SCHED_FIFO (%s)\n",
                         strerror(errno)));
                return false;
@@ -107,7 +103,6 @@ void reset_scheduler(void)
 #endif
 #else /* no AIX */
 #if HAVE_SCHED_SETSCHEDULER
-#ifndef SCHED_RESET_ON_FORK
        struct sched_param p;
 
        p.sched_priority = 0;
@@ -116,7 +111,6 @@ void reset_scheduler(void)
        }
 #endif
 #endif
-#endif
 }
 
 void set_nonblocking(int fd)