Merge remote-tracking branch 'volth/release-1.1pre16-rtt' into 1.1
authorGuus Sliepen <guus@tinc-vpn.org>
Wed, 13 Jun 2018 20:23:27 +0000 (22:23 +0200)
committerGuus Sliepen <guus@tinc-vpn.org>
Wed, 13 Jun 2018 20:26:49 +0000 (22:26 +0200)
Also, reformat the code and fix a compiler warning.

1  2 
src/info.c
src/net_packet.c
src/tincctl.c

diff --cc src/info.c
index 09e65805d901c19d6e07337c4dfae9162c191ac3,4ca67ec4fd8a60c2e3502d2b730f60e829b78e85..f0dce8c66c860dc0866a2ecf202f5f0550fb0fe8
@@@ -143,6 -145,8 +145,10 @@@ static int info_node(int fd, const cha
  
        if(status.udp_confirmed) {
                printf(" udp_confirmed");
 -              if(udp_ping_rtt != -1)
 -                      printf(" (rtt %ld.%03ld)", udp_ping_rtt / 1000, udp_ping_rtt % 1000);
++
++              if(udp_ping_rtt != -1) {
++                      printf(" (rtt %d.%03d)", udp_ping_rtt / 1000, udp_ping_rtt % 1000);
++              }
        }
  
        printf("\n");
index 8ade0576227d5436099ad69d59e10fbf32efdd95,ee2b316b6cd12d9459ac9a18adc0393ecdb9271d..ebf2938acb8bc826f2dd517d37b700f0a41718bf
@@@ -151,7 -152,15 +152,15 @@@ static void udp_probe_h(node_t *n, vpn_
                len = ntohs(len16);
        }
  
-       logger(DEBUG_TRAFFIC, LOG_INFO, "Got type %d UDP probe reply %d from %s (%s)", DATA(packet)[0], len, n->name, n->hostname);
 -      if (n->udp_ping_sent.tv_sec != 0) { // a probe in flight
++      if(n->udp_ping_sent.tv_sec != 0) {  // a probe in flight
+               gettimeofday(&now, NULL);
+               struct timeval rtt;
+               timersub(&now, &n->udp_ping_sent, &rtt);
 -              n->udp_ping_rtt = rtt.tv_sec*1000000 + rtt.tv_usec;
++              n->udp_ping_rtt = rtt.tv_sec * 1000000 + rtt.tv_usec;
+               logger(DEBUG_TRAFFIC, LOG_INFO, "Got type %d UDP probe reply %d from %s (%s) rtt=%d.%03d", DATA(packet)[0], len, n->name, n->hostname, n->udp_ping_rtt / 1000, n->udp_ping_rtt % 1000);
+       } else {
+               logger(DEBUG_TRAFFIC, LOG_INFO, "Got type %d UDP probe reply %d from %s (%s)", DATA(packet)[0], len, n->name, n->hostname);
+       }
  
        /* It's a valid reply: now we know bidirectional communication
           is possible using the address and socket that the reply
diff --cc src/tincctl.c
index cf19d4818554d641191dba148b4d9c355d972d86,d8df67c91c280bd9d91918870e8e751bb4b99c63..12e5ead920b5a3209030adcf855b4bb09b3989c8
@@@ -1331,8 -1333,11 +1333,14 @@@ static int cmd_dump(int argc, char *arg
                                        continue;
                                }
  
-                               printf("%s id %s at %s port %s cipher %d digest %d maclength %d compression %d options %x status %04x nexthop %s via %s distance %d pmtu %d (min %d max %d)\n",
-                                      node, id, host, port, cipher, digest, maclength, compression, options, status_int, nexthop, via, distance, pmtu, minmtu, maxmtu);
+                               printf("%s id %s at %s port %s cipher %d digest %d maclength %d compression %d options %x status %04x nexthop %s via %s distance %d pmtu %d (min %d max %d) rx %"PRIu64" %"PRIu64" tx %"PRIu64" %"PRIu64,
+                                      node, id, host, port, cipher, digest, maclength, compression, options, status_int, nexthop, via, distance, pmtu, minmtu, maxmtu, in_packets, in_bytes, out_packets, out_bytes);
 -                              if (udp_ping_rtt != -1)
++
++                              if(udp_ping_rtt != -1) {
+                                       printf(" rtt %d.%03d", udp_ping_rtt / 1000, udp_ping_rtt % 1000);
++                              }
++
+                               printf("\n");
                        }
                }
                break;