diff options
author | Amin Hassani <ahassani@google.com> | 2017-12-06 13:47:52 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-12-12 13:50:58 -0800 |
commit | d8b67f49d7f8ae291cd4fa901b2d3767137704e7 (patch) | |
tree | 2d039015afcd03433da0dd8860a83ce3ce2a65cf /payload_generator/delta_diff_utils.cc | |
parent | 06c6d088b89656a120d40c4b578200c87af5c4c8 (diff) |
update_engine: Remove the duplicate BlocksInExtents
This patch removes the duplicate BlocksInExtents from extent_utils.h and fixes
the remainder of the code to reflect this change.
BUG=none
TEST=unittests pass;
Change-Id: I76f5106f75072b20cd8f41f081b2f2b07aeac9a8
Reviewed-on: https://chromium-review.googlesource.com/812009
Commit-Ready: Amin Hassani <ahassani@chromium.org>
Tested-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Alex Deymo <deymo@google.com>
Reviewed-by: Sen Jiang <senj@chromium.org>
Diffstat (limited to 'payload_generator/delta_diff_utils.cc')
-rw-r--r-- | payload_generator/delta_diff_utils.cc | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/payload_generator/delta_diff_utils.cc b/payload_generator/delta_diff_utils.cc index 13027ab2..fbb6066d 100644 --- a/payload_generator/delta_diff_utils.cc +++ b/payload_generator/delta_diff_utils.cc @@ -234,7 +234,7 @@ void FileDeltaProcessor::Run() { TEST_AND_RETURN(blob_file_ != nullptr); LOG(INFO) << "Encoding file " << name_ << " (" - << BlocksInExtents(new_extents_) << " blocks)"; + << utils::BlocksInExtents(new_extents_) << " blocks)"; if (!DeltaReadFile(&file_aops_, old_part_, @@ -248,7 +248,7 @@ void FileDeltaProcessor::Run() { version_, blob_file_)) { LOG(ERROR) << "Failed to generate delta for " << name_ << " (" - << BlocksInExtents(new_extents_) << " blocks)"; + << utils::BlocksInExtents(new_extents_) << " blocks)"; } } @@ -367,9 +367,9 @@ bool DeltaReadPartition(vector<AnnotatedOperation>* aops, old_unvisited = FilterExtentRanges(old_unvisited, old_visited_blocks); } - LOG(INFO) << "Scanning " << BlocksInExtents(new_unvisited) - << " unwritten blocks using chunk size of " - << soft_chunk_blocks << " blocks."; + LOG(INFO) << "Scanning " << utils::BlocksInExtents(new_unvisited) + << " unwritten blocks using chunk size of " << soft_chunk_blocks + << " blocks."; // We use the soft_chunk_blocks limit for the <non-file-data> as we don't // really know the structure of this data and we should not expect it to have // redundancy between partitions. @@ -492,7 +492,7 @@ bool DeltaMovedAndZeroBlocks(vector<AnnotatedOperation>* aops, blob_file)); } LOG(INFO) << "Produced " << (aops->size() - num_ops) << " operations for " - << BlocksInExtents(new_zeros) << " zeroed blocks"; + << utils::BlocksInExtents(new_zeros) << " zeroed blocks"; // Produce MOVE/SOURCE_COPY operations for the moved blocks. num_ops = aops->size(); @@ -555,7 +555,7 @@ bool DeltaReadFile(vector<AnnotatedOperation>* aops, brillo::Blob data; InstallOperation operation; - uint64_t total_blocks = BlocksInExtents(new_extents); + uint64_t total_blocks = utils::BlocksInExtents(new_extents); if (chunk_blocks == -1) chunk_blocks = total_blocks; @@ -676,8 +676,8 @@ bool ReadExtentsToDiff(const string& old_part, InstallOperation operation; // We read blocks from old_extents and write blocks to new_extents. - uint64_t blocks_to_read = BlocksInExtents(old_extents); - uint64_t blocks_to_write = BlocksInExtents(new_extents); + uint64_t blocks_to_read = utils::BlocksInExtents(old_extents); + uint64_t blocks_to_write = utils::BlocksInExtents(new_extents); // Disable bsdiff, and puffdiff when the data is too big. bool bsdiff_allowed = |