tequilaOS/platform_bootable_recovery
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Reduce logs in update_verifier

Browse source 
update_verifier logs to kernel log, reduce unnecessary log lines to
avoid throttling.

Test: th
Bug: 265715753
Change-Id: I57b5dcfbf04eebc62fdf96abf0c91d7df5e0f0ea
This commit is contained in:
Kelvin Zhang 2023-02-06 12:20:53 -08:00
parent e3ebedd279
commit 27c731c706
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
update_verifier/update_verifier.cpp
View file

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