Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed
  • tools/tracereport/tracereport.c

    r051618c r7be3cc4  
    8484                        break;
    8585                }
     86                if (reports_required & REPORT_TYPE_MISC)
     87                        misc_per_packet(packet);
    8688                if (reports_required & REPORT_TYPE_ERROR)
    8789                        error_per_packet(packet);
     
    119121        "-e --error             Report packet errors (e.g. checksum failures, rxerrors)\n"
    120122        "-F --flow              Report flows\n"
     123        "-m --misc              Report misc information (start/end times, duration, pps)\n"
    121124        "-P --protocol          Report transport protocols\n"
    122125        "-p --port              Report port numbers\n"
     
    151154                        { "protocol",           0, 0, 'P' },
    152155                        { "port",               0, 0, 'p' },
     156                        { "misc",               0, 0, 'm' },
    153157                        { "tos",                0, 0, 'T' },
    154158                        { "ttl",                0, 0, 't' },
     
    161165                        { NULL,                 0, 0, 0 }
    162166                };
    163                 opt = getopt_long(argc, argv, "f:HeFPpTtOondCs", long_options,
     167                opt = getopt_long(argc, argv, "f:HeFmPpTtOondCs", long_options,
    164168                                &option_index);
    165169                if (opt == -1)
     
    194198                                reports_required |= REPORT_TYPE_SYNOPT;
    195199                                break;
     200                        case 'm':
     201                                reports_required |= REPORT_TYPE_MISC;
     202                                break;
    196203                        case 'P':
    197204                                reports_required |= REPORT_TYPE_PROTO;
     
    213220                }
    214221        }
     222
     223        /* Default to all reports, instead of no reports at all.  It's annoying
     224         * waiting for 10 minutes for a trace to process then discover you
     225         * forgot to ask for any reports!
     226         */
     227        if (reports_required == 0)
     228                reports_required = ~0;
    215229
    216230        if (filterstring) {
     
    227241        }
    228242
     243        if (reports_required & REPORT_TYPE_MISC)
     244                misc_report();
    229245        if (reports_required & REPORT_TYPE_ERROR)
    230246                error_report();
Note: See TracChangeset for help on using the changeset viewer.