Provide more information when drops are seen
PATCH 1/1 Provide more information when drops are seen
This will provide the user with the percentage of drops encountered, allowing them to know how far off they are in being able to capturing all traces. It will also suggest trying to increase _either_ the buffer size (-b) or trying more buffers (-n) when drops are found. We have found that some times the latter is more helpful than the former.
The output would now look something like:
You have 114704 ( 0.7%) dropped events
Consider using a larger buffer size (-b) and/or more buffers (-n)
Signed-off-by Alan D. Brunelle <[email protected]>
--- blktrace.c | 19 ++++++++++++++++--- 1 files changed, 16 insertions(+), 3 deletions(-)
blktrace.c +16 -3
@@ -1321,7 +1321,7 @@ static void show_stats(struct device_information *dips, int ndips, int cpus)struct device_information *dip;struct thread_information *tip;unsigned long long events_processed, data_read; -unsigned long total_drops; +unsigned long total_drops, total_events;int i, j, no_stdout =0 ;if (is_stat_shown ())
@@ -1333,12 +1333,20 @@ static void show_stats(struct device_information *dips, int ndips, int cpus) stat_shown =1 ; total_drops =0 ; + total_events =0 ;__for_each_dip (dip, dips, ndips, i) {if (!no_stdout)printf ("Device: %s\n" , dip->path); events_processed =0 ; data_read =0 ;__for_each_tip (dip, tip, cpus, j) { +/* +* Estimate number of events if not known +*/ +if (tip->events_processed ==0 ) + tip->events_processed = tip->data_read / +sizeof (struct blk_io_trace ); +if (!no_stdout) printf(" CPU%3d: %20lu events, %8llu KiB data\n", tip->cpu, tip->events_processed,
@@ -1347,14 +1355,19 @@ static void show_stats(struct device_information *dips, int ndips, int cpus) data_read += tip->data_read; } total_drops += dip->drop_count; + total_events += (dip->drop_count + events_processed);if (!no_stdout)printf (" Total: %20llu events (dropped %lu), %8llu KiB data\n" , events_processed, dip->drop_count, (data_read +1023 ) >>10 ); } -if (total_drops) -fprintf (stderr,"You have dropped events, consider using a larger buffer size (-b)\n" ); +if (total_drops) { +fprintf (stderr,"\nYou have %lu (%5.1lf%%) dropped events\n" +"Consider using a larger buffer size (-b) " +"and/or more buffers (-n)\n" , total_drops, +100.0 * (double )total_drops / (double )total_events); + } } static struct device_information *net_get_dip(struct net_connection *nc,
Signature · 1 line
1.5.6.3