Changeset 7be3cc4 for tools/tracereport/tracereport.c

Timestamp:

04/21/07 15:25:12 (14 years ago)

Author:

Perry Lorier <perry@…>

Branches:

4.0.1-hotfixes, cachetimestamps, develop, dpdk-ndag, etsilive, getfragoff, help, libtrace4, master, ndag_format, pfring, rc-4.0.1, rc-4.0.2, rc-4.0.3, rc-4.0.4, ringdecrementfix, ringperformance, ringtimestampfixes

Children:

4df814d

Parents:

171cc28

Message:

Add start time, end time, duration, packets and packets per second reports.
Fixes #236
Make the default "all reports" instead of "no reports" for less surprise
value.

File:

• tools/tracereport/tracereport.c (modified) (7 diffs)

tools/tracereport/tracereport.c

@@ -84 +84 @@
                         break;
                 }
+                if (reports_required & REPORT_TYPE_MISC)
+                        misc_per_packet(packet);
                 if (reports_required & REPORT_TYPE_ERROR)
                         error_per_packet(packet);
@@ -119 +121 @@
         "-e --error             Report packet errors (e.g. checksum failures, rxerrors)\n"
         "-F --flow              Report flows\n"
+        "-m --misc              Report misc information (start/end times, duration, pps)\n"
         "-P --protocol          Report transport protocols\n"
         "-p --port              Report port numbers\n"
@@ -151 +154 @@
                         { "protocol",           0, 0, 'P' },
                         { "port",               0, 0, 'p' },
+                        { "misc",               0, 0, 'm' },
                         { "tos",                0, 0, 'T' },
                         { "ttl",                0, 0, 't' },
@@ -161 +165 @@
                         { NULL,                 0, 0, 0 }
                 };
-                opt = getopt_long(argc, argv, "f:HeFPpTtOondCs", long_options,
+                opt = getopt_long(argc, argv, "f:HeFmPpTtOondCs", long_options,
                                 &option_index);
                 if (opt == -1)
@@ -194 +198 @@
                                 reports_required |= REPORT_TYPE_SYNOPT;
                                 break;
+                        case 'm':
+                                reports_required |= REPORT_TYPE_MISC;
+                                break;
                         case 'P':
                                 reports_required |= REPORT_TYPE_PROTO;
@@ -213 +220 @@
                 }
         }
+
+        /* Default to all reports, instead of no reports at all.  It's annoying
+         * waiting for 10 minutes for a trace to process then discover you 
+         * forgot to ask for any reports!
+         */
+        if (reports_required == 0)
+                reports_required = ~0;
 
         if (filterstring) {
@@ -227 +241 @@
         }
 
+        if (reports_required & REPORT_TYPE_MISC)
+                misc_report();
         if (reports_required & REPORT_TYPE_ERROR)
                 error_report();