diff options
| author | Ilya Dryomov <idryomov@gmail.com> | 2020-10-02 14:15:59 +0200 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2020-10-12 15:29:27 +0200 |
| commit | 5a5036c89f345769c0049f4af04cc6647c0df058 (patch) | |
| tree | 752bece11812893ed47290e8c9ddd7b9c3f887d8 /net/ceph/messenger.c | |
| parent | c74d79af9002a9cb179f7950a2c266f0de964abe (diff) | |
| download | lwn-5a5036c89f345769c0049f4af04cc6647c0df058.tar.gz lwn-5a5036c89f345769c0049f4af04cc6647c0df058.zip | |
libceph: move a dout in queue_con_delay()
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>
Diffstat (limited to 'net/ceph/messenger.c')
| -rw-r--r-- | net/ceph/messenger.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c index d4d7a0e52491..8eabcdc2af56 100644 --- a/net/ceph/messenger.c +++ b/net/ceph/messenger.c @@ -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; } |