Changeset 02d13cb for tools/tracertstats

Timestamp:

03/01/18 17:10:50 (3 years ago)

Author:

Shane Alcock <salcock@…>

Branches:

cachetimestamps, develop, etsilive, master, rc-4.0.3, rc-4.0.4, ringdecrementfix, ringperformance

Children:

43b773a

Parents:

207e288

Message:

Don't count ERF provenance records during tracestats, tracertstats

Or any other records with 0 wire length for that matter.

This was especially off-putting when using filters as provenance
records automatically match all filters, so you could get some
very misleading results.

File:

• tools/tracertstats/tracertstats.c (modified) (2 diffs)

tools/tracertstats/tracertstats.c

@@ -197 +197 @@
         thread_data_t *td = (thread_data_t *)tls;
         int i;
+        size_t wlen;
 
         if (IS_LIBTRACE_META_PACKET(packet)) {
@@ -212 +213 @@
                                 sizeof(statistic_t) * filter_count);
         }
+        wlen = trace_get_wire_length(packet);
+        if (wlen == 0) {
+                /* Don't count ERF provenance and similar packets */
+                return packet;
+        }
         for(i=0;i<filter_count;++i) {
                 if(trace_apply_filter(filters[i].filter, packet)) {
                         td->results->filters[i].count++;
-                        td->results->filters[i].bytes+=trace_get_wire_length(packet);
+                        td->results->filters[i].bytes+=wlen;
                 }
         }
 
         td->results->total.count++;
-        td->results->total.bytes +=trace_get_wire_length(packet);
+        td->results->total.bytes += wlen;
         return packet;
 }