]> git.puffer.fish Git - mirror/frr.git/commitdiff
pimd: Add debug logs to help find problems with stream_read
authoranuradhak <anuradhak@cumulusnetworks.com>
Fri, 2 Dec 2016 21:41:11 +0000 (13:41 -0800)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Thu, 22 Dec 2016 01:26:18 +0000 (20:26 -0500)
Logs only. No functional change
Ticket: CM-13852

Signed-off-by: Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
pimd/pim_msdp_packet.c

index 3f3f9d1de0b40e656e5fd4f5ae1d8650ff512a26..62d0579757817520957e4f449399c3b6e793e5a8 100644 (file)
@@ -586,16 +586,26 @@ pim_msdp_read_packet(struct pim_msdp_peer *mp)
 {
   int nbytes;
   int readsize;
+  int old_endp;
+  int new_endp;
 
-  readsize = mp->packet_size - stream_get_endp(mp->ibuf);
+  old_endp = stream_get_endp(mp->ibuf);
+  readsize = mp->packet_size - old_endp;
   if (!readsize) {
     return 0;
   }
 
   /* Read packet from fd */
   nbytes = stream_read_try(mp->ibuf, mp->fd, readsize);
+  new_endp = stream_get_endp(mp->ibuf);
   if (nbytes < 0) {
+    if (PIM_DEBUG_MSDP_INTERNAL) {
+      zlog_debug("MSDP peer %s read failed %d", mp->key_str, nbytes);
+    }
     if (nbytes == -2) {
+      if (PIM_DEBUG_MSDP_INTERNAL) {
+        zlog_debug("MSDP peer %s pim_msdp_read io retry old_end: %d new_end: %d", mp->key_str, old_endp, new_endp);
+      }
       /* transient error retry */
       return -1;
     }
@@ -604,13 +614,20 @@ pim_msdp_read_packet(struct pim_msdp_peer *mp)
   }
 
   if (!nbytes) {
+    if (PIM_DEBUG_MSDP_INTERNAL) {
+      zlog_debug("MSDP peer %s read failed %d", mp->key_str, nbytes);
+    }
     pim_msdp_peer_reset_tcp_conn(mp, "peer-down");
     return -1;
   }
 
   /* We read partial packet. */
-  if (stream_get_endp(mp->ibuf) != mp->packet_size)
+  if (stream_get_endp(mp->ibuf) != mp->packet_size) {
+    if (PIM_DEBUG_MSDP_INTERNAL) {
+      zlog_debug("MSDP peer %s read partial len %d old_endp %d new_endp %d", mp->key_str, mp->packet_size, old_endp, new_endp);
+    }
     return -1;
+  }
 
   return 0;
 }