sched: Remove sched_set_*() return value

Ingo suggested that since the new sched_set_*() functions are
implemented using the 'nocheck' variants, they really shouldn't ever
fail, so remove the return value.

Cc: axboe@kernel.dk
Cc: daniel.lezcano@linaro.org
Cc: sudeep.holla@arm.com
Cc: airlied@redhat.com
Cc: broonie@kernel.org
Cc: paulmck@kernel.org
Suggested-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Ingo Molnar <mingo@kernel.org>

diff --git a/drivers/block/drbd/drbd_receiver.c b/drivers/block/drbd/drbd_receiver.c
index 140fd98274b168c62ecb5b46d98d228043ee8a76..280615efef748bcdb0f53d207d259fb159266af7 100644 (file)
--- a/drivers/block/drbd/drbd_receiver.c
+++ b/drivers/block/drbd/drbd_receiver.c
@@ -6020,9 +6020,7 @@ int drbd_ack_receiver(struct drbd_thread *thi)
        int expect   = header_size;
        bool ping_timeout_active = false;
 
-       rv = sched_set_fifo_low(current);
-       if (rv < 0)
-               drbd_err(connection, "drbd_ack_receiver: ERROR set priority, ret=%d\n", rv);
+       sched_set_fifo_low(current);
 
        while (get_t_state(thi) == RUNNING) {
                drbd_thread_current_set_cpu(thi);

diff --git a/drivers/firmware/psci/psci_checker.c b/drivers/firmware/psci/psci_checker.c
index a5279a430274ba56e4c845f0e823a74f52d4b5b3..6fff482847e761d13931f9dc2b388bb2616ee60d 100644 (file)
--- a/drivers/firmware/psci/psci_checker.c
+++ b/drivers/firmware/psci/psci_checker.c
@@ -281,8 +281,7 @@ static int suspend_test_thread(void *arg)
        wait_for_completion(&suspend_threads_started);
 
        /* Set maximum priority to preempt all other threads on this CPU. */
-       if (sched_set_fifo(current))
-               pr_warn("Failed to set suspend thread scheduler on CPU %d\n", cpu);
+       sched_set_fifo(current);
 
        dev = this_cpu_read(cpuidle_devices);
        drv = cpuidle_get_cpu_driver(dev);

diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index 89a8b9c7e044ca45a8ade6fab36d8e2d9094a98f..556cca38487c4aa104d938d42385b2f221e41ecf 100644 (file)
--- a/drivers/gpu/drm/msm/msm_drv.c
+++ b/drivers/gpu/drm/msm/msm_drv.c
@@ -509,10 +509,7 @@ static int msm_drm_init(struct device *dev, struct drm_driver *drv)
                        goto err_msm_uninit;
                }
 
-               ret = sched_set_fifo(priv->event_thread[i].thread);
-               if (ret)
-                       dev_warn(dev, "event_thread set priority failed:%d\n",
-                                ret);
+               sched_set_fifo(priv->event_thread[i].thread);
        }
 
        ret = drm_vblank_init(ddev, priv->num_crtcs);

diff --git a/drivers/platform/chrome/cros_ec_spi.c b/drivers/platform/chrome/cros_ec_spi.c
index c20a43a9704062e0649dec83e39b6aceb6549096..d0926038255086a461d4ac35dc331b3c7c109efa 100644 (file)
--- a/drivers/platform/chrome/cros_ec_spi.c
+++ b/drivers/platform/chrome/cros_ec_spi.c
@@ -725,10 +725,9 @@ static int cros_ec_spi_devm_high_pri_alloc(struct device *dev,
        if (err)
                return err;
 
-       err = sched_set_fifo(ec_spi->high_pri_worker->task);
-       if (err)
-               dev_err(dev, "Can't set cros_ec high pri priority: %d\n", err);
-       return err;
+       sched_set_fifo(ec_spi->high_pri_worker->task);
+
+       return 0;
 }
 
 static int cros_ec_spi_probe(struct spi_device *spi)

diff --git a/include/linux/sched.h b/include/linux/sched.h
index b792b8f0f4cf80ca5efa80fa4b9bde7ebce1fe8f..ae7664492af2d7a5fc941b011d251f3bb0597a88 100644 (file)
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -1653,9 +1653,9 @@ extern int idle_cpu(int cpu);
 extern int available_idle_cpu(int cpu);
 extern int sched_setscheduler(struct task_struct *, int, const struct sched_param *);
 extern int sched_setscheduler_nocheck(struct task_struct *, int, const struct sched_param *);
-extern int sched_set_fifo(struct task_struct *p);
-extern int sched_set_fifo_low(struct task_struct *p);
-extern int sched_set_normal(struct task_struct *p, int nice);
+extern void sched_set_fifo(struct task_struct *p);
+extern void sched_set_fifo_low(struct task_struct *p);
+extern void sched_set_normal(struct task_struct *p, int nice);
 extern int sched_setattr(struct task_struct *, const struct sched_attr *);
 extern int sched_setattr_nocheck(struct task_struct *, const struct sched_attr *);
 extern struct task_struct *idle_task(int cpu);

diff --git a/kernel/rcu/rcutorture.c b/kernel/rcu/rcutorture.c
index bbc3c8a9e7b95c9ebb429cdbc0eb166cc662777e..b4c1146de414af1e00092b0f3785a1c00b73f4eb 100644 (file)
--- a/kernel/rcu/rcutorture.c
+++ b/kernel/rcu/rcutorture.c
@@ -893,10 +893,7 @@ static int rcu_torture_boost(void *arg)
        VERBOSE_TOROUT_STRING("rcu_torture_boost started");
 
        /* Set real-time priority. */
-       if (sched_set_fifo_low(current) < 0) {
-               VERBOSE_TOROUT_STRING("rcu_torture_boost RT prio failed!");
-               n_rcu_torture_boost_rterror++;
-       }
+       sched_set_fifo_low(current);
 
        init_rcu_head_on_stack(&rbi.rcu);
        /* Each pass through the following loop does one boost-test cycle. */

diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index f882d3d74dad03af6dd9442e922d7d95262c8151..41d3778ea80e60f21f2e038f3e803f5f992b9f3c 100644 (file)
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -5183,30 +5183,30 @@ int sched_setscheduler_nocheck(struct task_struct *p, int policy,
  * The administrator _MUST_ configure the system, the kernel simply doesn't
  * know enough information to make a sensible choice.
  */
-int sched_set_fifo(struct task_struct *p)
+void sched_set_fifo(struct task_struct *p)
 {
        struct sched_param sp = { .sched_priority = MAX_RT_PRIO / 2 };
-       return sched_setscheduler_nocheck(p, SCHED_FIFO, &sp);
+       WARN_ON_ONCE(sched_setscheduler_nocheck(p, SCHED_FIFO, &sp) != 0);
 }
 EXPORT_SYMBOL_GPL(sched_set_fifo);
 
 /*
  * For when you don't much care about FIFO, but want to be above SCHED_NORMAL.
  */
-int sched_set_fifo_low(struct task_struct *p)
+void sched_set_fifo_low(struct task_struct *p)
 {
        struct sched_param sp = { .sched_priority = 1 };
-       return sched_setscheduler_nocheck(p, SCHED_FIFO, &sp);
+       WARN_ON_ONCE(sched_setscheduler_nocheck(p, SCHED_FIFO, &sp) != 0);
 }
 EXPORT_SYMBOL_GPL(sched_set_fifo_low);
 
-int sched_set_normal(struct task_struct *p, int nice)
+void sched_set_normal(struct task_struct *p, int nice)
 {
        struct sched_attr attr = {
                .sched_policy = SCHED_NORMAL,
                .sched_nice = nice,
        };
-       return sched_setattr_nocheck(p, &attr);
+       WARN_ON_ONCE(sched_setattr_nocheck(p, &attr) != 0);
 }
 EXPORT_SYMBOL_GPL(sched_set_normal);