Skip to content

Commit 5a4f954

Browse files
committed
fixup! core+proto: Generalize histogram utilities
1 parent a9e1039 commit 5a4f954

File tree

3 files changed

+28
-28
lines changed

3 files changed

+28
-28
lines changed

bessctl/conf/port/latency.bess

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,13 @@ while True:
6969
(ret.timestamp - start_time,
7070
diff_pkts / 1e6,
7171
diff_bits / 1e6,
72-
ret.latency.avg_ns / 1e3,
73-
ret.latency.min_ns / 1e3,
74-
ret.latency.percentile_values_ns[0] / 1e3,
75-
ret.latency.percentile_values_ns[1] / 1e3,
76-
ret.latency.max_ns / 1e3,
77-
ret.jitter.avg_ns / 1e3,
78-
ret.jitter.min_ns / 1e3,
79-
ret.jitter.percentile_values_ns[0] / 1e3,
80-
ret.jitter.percentile_values_ns[1] / 1e3,
81-
ret.jitter.max_ns / 1e3))
72+
ret.latency_ns.avg / 1e3,
73+
ret.latency_ns.min / 1e3,
74+
ret.latency_ns.percentile_values[0] / 1e3,
75+
ret.latency_ns.percentile_values[1] / 1e3,
76+
ret.latency_ns.max / 1e3,
77+
ret.jitter_ns.avg / 1e3,
78+
ret.jitter_ns.min / 1e3,
79+
ret.jitter_ns.percentile_values[0] / 1e3,
80+
ret.jitter_ns.percentile_values[1] / 1e3,
81+
ret.jitter_ns.max / 1e3))

bessctl/measurement_utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,8 +194,8 @@ def next(self):
194194
rtt = dict()
195195
jitter = dict()
196196
for i, p in enumerate(self.rtt_percentiles):
197-
rtt[p] = mstats.latency.percentile_values_ns[i]
198-
jitter[p] = mstats.jitter.percentile_values_ns[i]
197+
rtt[p] = mstats.latency_ns.percentile_values[i]
198+
jitter[p] = mstats.jitter_ns.percentile_values[i]
199199

200200
if self.rate:
201201
now = time.time()

bessctl/module_tests/timestamp.py

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -60,28 +60,28 @@ def test_timestamped_and_measured(self):
6060
# timestamp: 1517529594.72
6161
# packets: 64807616
6262
# bits: 43550717952
63-
# latency = {
63+
# latency_ns = {
6464
# count: 64807616
65-
# min_ns: 400
66-
# avg_ns: 513
67-
# max_ns: 295600
68-
# total_ns: 33273840000
69-
# percentile_values_ns: [400, 500, 500, 500, 295600]
70-
# resolution_ns: 100
65+
# min: 400
66+
# avg: 513
67+
# max: 295600
68+
# total: 33273840000
69+
# percentile_values: [400, 500, 500, 500, 295600]
70+
# resolution: 100
7171
# }
72-
# jitter = {
72+
# jitter_ns = {
7373
# count: 3238279
74-
# avg_ns: 15
75-
# max_ns: 295100
76-
# total_ns: 48577800
77-
# resolution_ns: 100
74+
# avg: 15
75+
# max: 295100
76+
# total: 48577800
77+
# resolution: 100
7878
# }
7979
print()
80-
print('min ns =', stats.latency.min_ns)
81-
print('avg ns =', stats.latency.avg_ns)
80+
print('min ns =', stats.latency_ns.min)
81+
print('avg ns =', stats.latency_ns.avg)
8282
# these two should be approximately equal - within about 1%
83-
a = stats.latency.avg_ns * stats.latency.count
84-
b = stats.latency.total_ns
83+
a = stats.latency_ns.avg * stats.latency_ns.count
84+
b = stats.latency_ns.total
8585
diff = abs(a - b) / float(b) * 100.0
8686
self.assertLessEqual(diff, 1.0)
8787

0 commit comments

Comments
 (0)