commit 5a02a9bab9db55ac02ab3eafd28e325233583a31
author Matthew Duggan <mduggan@google.com> Wed May 18 17:05:54 2022 +0900
committer Matthew Duggan <mduggan@google.com> Wed May 18 17:05:54 2022 +0900
tree 6e4792de43abbd31e513898bc827ba9060168fdb
parent 3a728ac7c63f58ad05401cd3450d25a3f805d490

storaged: Avoid divide-by-zero in performance measurement

The code previously assumed that write() took a non-zero number of clock
ticks.  This is mostly true, but can't be guaranteed.  Add a check to
avoid crashing if it's 0.

Bug: 233008287
Test: Compile
Change-Id: Idec0052f534dc4abb81a414ca76f6c11f4a4dded

storaged/storaged.cpp

diff --git a/storaged/storaged.cpp b/storaged/storaged.cpp
index 8cc8b59..cefef6e 100644
--- a/storaged/storaged.cpp
+++ b/storaged/storaged.cpp
@@ -333,7 +333,7 @@
             first_write = false;
         }
 
-        if (benchmark_size) {
+        if (benchmark_size && benchmark_time_ns) {
             int perf = benchmark_size * 1000000LLU / benchmark_time_ns;
             storage_info->update_perf_history(perf, system_clock::now());
         }