diff options
author | Alex Elder <elder@inktank.com> | 2013-05-29 12:19:00 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-07-03 18:32:40 -0400 |
commit | 4974341eb99861720d54db9337bf1fe78eb8b9d0 (patch) | |
tree | 7f6a6d03c26f45251823b7e38320c6891a070952 /net/ceph/osd_client.c | |
parent | 08f75463c15e26e9d67a7c992ce7dd8964c6cbdd (diff) |
libceph: print more info for short message header
If an osd client response message arrives that has a front section
that's too big for the buffer set aside to receive it, a warning
gets reported and a new buffer is allocated.
The warning says nothing about which connection had the problem.
Add the peer type and number to what gets reported, to be a bit more
informative.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'net/ceph/osd_client.c')
-rw-r--r-- | net/ceph/osd_client.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index e0abb83b520e..61147fe70181 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c | |||
@@ -2454,8 +2454,10 @@ static struct ceph_msg *get_reply(struct ceph_connection *con, | |||
2454 | ceph_msg_revoke_incoming(req->r_reply); | 2454 | ceph_msg_revoke_incoming(req->r_reply); |
2455 | 2455 | ||
2456 | if (front > req->r_reply->front.iov_len) { | 2456 | if (front > req->r_reply->front.iov_len) { |
2457 | pr_warning("get_reply front %d > preallocated %d\n", | 2457 | pr_warning("get_reply front %d > preallocated %d (%u#%llu)\n", |
2458 | front, (int)req->r_reply->front.iov_len); | 2458 | front, (int)req->r_reply->front.iov_len, |
2459 | (unsigned int)con->peer_name.type, | ||
2460 | le64_to_cpu(con->peer_name.num)); | ||
2459 | m = ceph_msg_new(CEPH_MSG_OSD_OPREPLY, front, GFP_NOFS, false); | 2461 | m = ceph_msg_new(CEPH_MSG_OSD_OPREPLY, front, GFP_NOFS, false); |
2460 | if (!m) | 2462 | if (!m) |
2461 | goto out; | 2463 | goto out; |