commit c7eb36f1b71db32bfbbc635d0a9c5d5dd3ccaf42
author Treehugger Robot <treehugger-gerrit@google.com> Tue Feb 07 00:26:17 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue Feb 07 00:26:17 2023 +0000
tree b350b04f415a9d66ac9b6cbab0240f464d417bc3
parent ed5fc4f4b53aba708f6520d29da80beec1a1e7a7
parent 365c8c0b54331486352e0a32abe3d991baf14e62

Merge "Reduce logs in update_verifier" am: f891951d07 am: 365c8c0b54

Original change: https://android-review.googlesource.com/c/platform/bootable/recovery/+/2423949

Change-Id: Ifc7d73d357850874ce2aa296e3716a11c9f08f0f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

update_verifier/update_verifier.cpp

diff --git a/update_verifier/update_verifier.cpp b/update_verifier/update_verifier.cpp
index ea47a65..a0160e2 100644
--- a/update_verifier/update_verifier.cpp
+++ b/update_verifier/update_verifier.cpp
@@ -150,7 +150,6 @@
       static constexpr size_t kBlockSize = 4096;
       std::vector<uint8_t> buf(1024 * kBlockSize);
 
-      size_t block_count = 0;
       for (const auto& [range_start, range_end] : group) {
         if (lseek64(fd.get(), static_cast<off64_t>(range_start) * kBlockSize, SEEK_SET) == -1) {
           PLOG(ERROR) << "lseek to " << range_start << " failed";
@@ -166,9 +165,7 @@
           }
           remain -= to_read;
         }
-        block_count += (range_end - range_start);
       }
-      LOG(INFO) << "Finished reading " << block_count << " blocks on " << dm_block_device;
       return true;
     };
 
@@ -179,8 +176,8 @@
   for (auto& t : threads) {
     ret = t.get() && ret;
   }
-  LOG(INFO) << "Finished reading blocks on " << dm_block_device << " with " << thread_num
-            << " threads.";
+  LOG(INFO) << "Finished reading blocks on partition " << partition_name << " @ " << dm_block_device
+            << " with " << thread_num << " threads.";
   return ret;
 }