drbd: Eliminated the user of drbd_task_to_thread()
authorPhilipp Reisner <philipp.reisner@linbit.com>
Mon, 7 Feb 2011 10:14:38 +0000 (11:14 +0100)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Wed, 28 Sep 2011 08:26:45 +0000 (10:26 +0200)
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
drivers/block/drbd/drbd_int.h
drivers/block/drbd/drbd_main.c
drivers/block/drbd/drbd_receiver.c
drivers/block/drbd/drbd_worker.c

index 7beb374451b3d3823a78352e0d6b03be8bfd5e77..9a351a2cab7c1edb536f73e12df55210633b4168 100644 (file)
@@ -1133,10 +1133,10 @@ extern int  drbd_thread_start(struct drbd_thread *thi);
 extern void _drbd_thread_stop(struct drbd_thread *thi, int restart, int wait);
 extern char *drbd_task_to_thread_name(struct drbd_conf *mdev, struct task_struct *task);
 #ifdef CONFIG_SMP
-extern void drbd_thread_current_set_cpu(struct drbd_conf *mdev);
+extern void drbd_thread_current_set_cpu(struct drbd_conf *mdev, struct drbd_thread *thi);
 extern void drbd_calc_cpu_mask(struct drbd_conf *mdev);
 #else
-#define drbd_thread_current_set_cpu(A) ({})
+#define drbd_thread_current_set_cpu(A, B) ({})
 #define drbd_calc_cpu_mask(A) ({})
 #endif
 extern void drbd_free_resources(struct drbd_conf *mdev);
index 852a3e3fbb76914b2c056367abaac56f759bd00a..ae995ed0e6f917eaadaa7fa119d6090355c37b94 100644 (file)
@@ -635,17 +635,15 @@ void drbd_calc_cpu_mask(struct drbd_conf *mdev)
 /**
  * drbd_thread_current_set_cpu() - modifies the cpu mask of the _current_ thread
  * @mdev:      DRBD device.
+ * @thi:       drbd_thread object
  *
  * call in the "main loop" of _all_ threads, no need for any mutex, current won't die
  * prematurely.
  */
-void drbd_thread_current_set_cpu(struct drbd_conf *mdev)
+void drbd_thread_current_set_cpu(struct drbd_conf *mdev, struct drbd_thread *thi)
 {
        struct task_struct *p = current;
-       struct drbd_thread *thi = drbd_task_to_thread(mdev, p);
 
-       if (!expect(thi != NULL))
-               return;
        if (!thi->reset_cpu_mask)
                return;
        thi->reset_cpu_mask = 0;
index 3a9cd31e094bfd3c482a65ced7533c35c0feb37c..dfb59671ff1c0a38575cc0d6c58490218c8277ca 100644 (file)
@@ -3773,7 +3773,7 @@ static void drbdd(struct drbd_conf *mdev)
        int rv;
 
        while (get_t_state(&mdev->tconn->receiver) == RUNNING) {
-               drbd_thread_current_set_cpu(mdev);
+               drbd_thread_current_set_cpu(mdev, &mdev->tconn->receiver);
                if (!drbd_recv_header(mdev, &cmd, &packet_size))
                        goto err_out;
 
@@ -4564,7 +4564,7 @@ int drbd_asender(struct drbd_thread *thi)
        current->rt_priority = 2;    /* more important than all other tasks */
 
        while (get_t_state(thi) == RUNNING) {
-               drbd_thread_current_set_cpu(mdev);
+               drbd_thread_current_set_cpu(mdev, thi);
                if (test_and_clear_bit(SEND_PING, &mdev->flags)) {
                        if (!drbd_send_ping(mdev)) {
                                dev_err(DEV, "drbd_send_ping has failed\n");
index f13d56c2bf0516f4ba6627b54272cdc352e2e228..0dbd20ca630653c21081b1cbee4dfc861f7bf4c9 100644 (file)
@@ -1621,7 +1621,7 @@ int drbd_worker(struct drbd_thread *thi)
        sprintf(current->comm, "drbd%d_worker", mdev_to_minor(mdev));
 
        while (get_t_state(thi) == RUNNING) {
-               drbd_thread_current_set_cpu(mdev);
+               drbd_thread_current_set_cpu(mdev, thi);
 
                if (down_trylock(&mdev->tconn->data.work.s)) {
                        mutex_lock(&mdev->tconn->data.mutex);