]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
msg/async/rdma: fix bad message that went to the user 12558/head
authorAvner BenHanoch <avnerb@mellanox.com>
Tue, 6 Dec 2016 09:38:36 +0000 (09:38 +0000)
committerAdir Lev <adirl@mellanox.com>
Sun, 18 Dec 2016 16:52:12 +0000 (16:52 +0000)
1. the printed value of "bad length" was incorrect, because 'r' was changed before the log line.
2. log appeared to user as error even though everything was calm; hence, reducing its severity

issue: 914840

Change-Id: I41520d27fc53fbbe07b61dd915e16448b87769d5
Signed-off-by: Avner BenHanoch <avnerb@mellanox.com>
src/msg/async/rdma/Infiniband.cc

index 68a4eb22161b5d3eea2909696c193d592fc884fe..591afeb080cad54f45f0c457ba0fb445ebcd57b8 100644 (file)
@@ -343,8 +343,8 @@ int Infiniband::recv_msg(CephContext *cct, int sd, IBSYNMsg& im)
   } else if (r == 0) { // valid disconnect message of length 0\r
     ldout(cct, 10) << __func__ << " got disconnect message " << dendl;\r
   } else if ((size_t)r != sizeof(msg)) { // invalid message\r
+    ldout(cct, 1) << __func__ << " got bad length (" << r << "): " << cpp_strerror(errno) << dendl;\r
     r = -EINVAL;\r
-    lderr(cct) << __func__ << " got bad length (" << r << "): " << cpp_strerror(errno) << dendl;\r
   } else { // valid message\r
     sscanf(msg, "%x:%x:%x:%x:%s", &(im.lid), &(im.qpn), &(im.psn), &(im.peer_qpn),gid);\r
     wire_gid_to_gid(gid, &(im.gid));\r