libceph: move a dout in queue_con_delay()
authorIlya Dryomov <idryomov@gmail.com>
Fri, 2 Oct 2020 12:15:59 +0000 (14:15 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 12 Oct 2020 13:29:27 +0000 (15:29 +0200)
The queued con->work can start executing (and therefore logging)
before we get to this "con->work has been queued" message, making
the logs confusing.  Move it up, with the meaning of "con->work
is about to be queued".

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
net/ceph/messenger.c

index d4d7a0e..8eabcdc 100644 (file)
@@ -2811,13 +2811,13 @@ static int queue_con_delay(struct ceph_connection *con, unsigned long delay)
                return -ENOENT;
        }
 
+       dout("%s %p %lu\n", __func__, con, delay);
        if (!queue_delayed_work(ceph_msgr_wq, &con->work, delay)) {
                dout("%s %p - already queued\n", __func__, con);
                con->ops->put(con);
                return -EBUSY;
        }
 
-       dout("%s %p %lu\n", __func__, con, delay);
        return 0;
 }