blob: b4f97d210076168549e698c26802040cd3daabf3 [file] [log] [blame]
Kelvin Zhang94f51cc2020-09-25 11:34:49 -04001//
2// Copyright (C) 2020 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
16
17#include "update_engine/payload_consumer/vabc_partition_writer.h"
18
Kelvin Zhang76f10b82021-06-25 18:45:46 -040019#include <algorithm>
20#include <map>
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040021#include <memory>
Kelvin Zhang3f60d532020-11-09 13:33:17 -050022#include <string>
Kelvin Zhange52b6cd2021-02-09 15:28:40 -050023#include <utility>
Kelvin Zhang9b10dba2020-09-25 17:09:11 -040024#include <vector>
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040025
Akilesh Kailash3e6e7df2021-11-18 23:29:15 +000026#include <android-base/properties.h>
Kelvin Zhang76f10b82021-06-25 18:45:46 -040027#include <brillo/secure_blob.h>
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040028#include <libsnapshot/cow_writer.h>
29
Kelvin Zhang9b10dba2020-09-25 17:09:11 -040030#include "update_engine/common/cow_operation_convert.h"
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040031#include "update_engine/common/utils.h"
Kelvin Zhang76f10b82021-06-25 18:45:46 -040032#include "update_engine/payload_consumer/extent_map.h"
Kelvin Zhang7a265752020-10-29 15:51:35 -040033#include "update_engine/payload_consumer/file_descriptor.h"
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040034#include "update_engine/payload_consumer/install_plan.h"
Kelvin Zhang9b10dba2020-09-25 17:09:11 -040035#include "update_engine/payload_consumer/snapshot_extent_writer.h"
Kelvin Zhang76f10b82021-06-25 18:45:46 -040036#include "update_engine/payload_consumer/xor_extent_writer.h"
Kelvin Zhanga37aafc2021-06-14 13:21:37 -040037#include "update_engine/payload_generator/extent_ranges.h"
38#include "update_engine/payload_generator/extent_utils.h"
39#include "update_engine/update_metadata.pb.h"
Kelvin Zhang94f51cc2020-09-25 11:34:49 -040040
41namespace chromeos_update_engine {
Kelvin Zhang52cb1d72020-10-27 13:44:25 -040042// Expected layout of COW file:
43// === Beginning of Cow Image ===
44// All Source Copy Operations
45// ========== Label 0 ==========
46// Operation 0 in PartitionUpdate
47// ========== Label 1 ==========
48// Operation 1 in PartitionUpdate
49// ========== label 2 ==========
50// Operation 2 in PartitionUpdate
51// ========== label 3 ==========
52// .
53// .
54// .
55
56// When resuming, pass |next_op_index_| as label to
57// |InitializeWithAppend|.
58// For example, suppose we finished writing SOURCE_COPY, and we finished writing
59// operation 2 completely. Update is suspended when we are half way through
60// operation 3.
61// |cnext_op_index_| would be 3, so we pass 3 as
62// label to |InitializeWithAppend|. The CowWriter will retain all data before
63// label 3, Which contains all operation 2's data, but none of operation 3's
64// data.
65
Kelvin Zhanga37aafc2021-06-14 13:21:37 -040066using android::snapshot::ICowWriter;
67using ::google::protobuf::RepeatedPtrField;
68
Kelvin Zhang76f10b82021-06-25 18:45:46 -040069// Compute XOR map, a map from dst extent to corresponding merge operation
70static ExtentMap<const CowMergeOperation*, ExtentLess> ComputeXorMap(
71 const RepeatedPtrField<CowMergeOperation>& merge_ops) {
72 ExtentMap<const CowMergeOperation*, ExtentLess> xor_map;
73 for (const auto& merge_op : merge_ops) {
74 if (merge_op.type() == CowMergeOperation::COW_XOR) {
75 xor_map.AddExtent(merge_op.dst_extent(), &merge_op);
76 }
77 }
78 return xor_map;
79}
80
Kelvin Zhange52b6cd2021-02-09 15:28:40 -050081VABCPartitionWriter::VABCPartitionWriter(
82 const PartitionUpdate& partition_update,
83 const InstallPlan::Partition& install_part,
84 DynamicPartitionControlInterface* dynamic_control,
Kelvin Zhanga37aafc2021-06-14 13:21:37 -040085 size_t block_size)
Kelvin Zhange52b6cd2021-02-09 15:28:40 -050086 : partition_update_(partition_update),
87 install_part_(install_part),
88 dynamic_control_(dynamic_control),
Kelvin Zhange52b6cd2021-02-09 15:28:40 -050089 block_size_(block_size),
90 executor_(block_size),
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -070091 verified_source_fd_(block_size, install_part.source_path) {
92 for (const auto& cow_op : partition_update_.merge_operations()) {
93 if (cow_op.type() != CowMergeOperation::COW_COPY) {
94 continue;
95 }
96 copy_blocks_.AddExtent(cow_op.dst_extent());
97 }
Kelvin Zhang7dd5a5e2023-05-11 15:30:38 -070098 LOG(INFO) << "Partition `" << partition_update.partition_name() << "` has "
Kelvin Zhang67144e52023-04-03 18:17:07 -070099 << copy_blocks_.blocks() << " copy blocks";
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700100}
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500101
Kelvin Zhangd7958752023-03-23 12:23:47 -0700102bool VABCPartitionWriter::DoesDeviceSupportsXor() {
103 return dynamic_control_->GetVirtualAbCompressionXorFeatureFlag().IsEnabled();
104}
105
106bool VABCPartitionWriter::WriteAllCopyOps() {
107 const bool userSnapshots = android::base::GetBoolProperty(
108 "ro.virtual_ab.userspace.snapshots.enabled", false);
109 for (const auto& cow_op : partition_update_.merge_operations()) {
110 if (cow_op.type() != CowMergeOperation::COW_COPY) {
111 continue;
112 }
113 if (cow_op.dst_extent() == cow_op.src_extent()) {
114 continue;
115 }
116 if (userSnapshots) {
117 TEST_AND_RETURN_FALSE(cow_op.src_extent().num_blocks() != 0);
118 TEST_AND_RETURN_FALSE(
119 cow_writer_->AddCopy(cow_op.dst_extent().start_block(),
120 cow_op.src_extent().start_block(),
121 cow_op.src_extent().num_blocks()));
122 } else {
123 // Add blocks in reverse order, because snapused specifically prefers
124 // this ordering. Since we already eliminated all self-overlapping
125 // SOURCE_COPY during delta generation, this should be safe to do.
126 for (size_t i = cow_op.src_extent().num_blocks(); i > 0; i--) {
127 TEST_AND_RETURN_FALSE(
128 cow_writer_->AddCopy(cow_op.dst_extent().start_block() + i - 1,
129 cow_op.src_extent().start_block() + i - 1));
130 }
131 }
132 }
133 return true;
134}
135
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400136bool VABCPartitionWriter::Init(const InstallPlan* install_plan,
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400137 bool source_may_exist,
138 size_t next_op_index) {
Kelvin Zhang1c4b9812022-04-06 17:29:00 -0700139 if (dynamic_control_->GetVirtualAbCompressionXorFeatureFlag().IsEnabled()) {
140 xor_map_ = ComputeXorMap(partition_update_.merge_operations());
141 if (xor_map_.size() > 0) {
142 LOG(INFO) << "Virtual AB Compression with XOR is enabled";
143 } else {
144 LOG(INFO) << "Device supports Virtual AB compression with XOR, but OTA "
145 "package does not.";
146 }
147 } else {
148 LOG(INFO) << "Virtual AB Compression with XOR is disabled.";
149 }
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400150 TEST_AND_RETURN_FALSE(install_plan != nullptr);
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700151 if (source_may_exist && install_part_.source_size > 0) {
152 TEST_AND_RETURN_FALSE(!install_part_.source_path.empty());
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500153 TEST_AND_RETURN_FALSE(verified_source_fd_.Open());
154 }
Kelvin Zhang3f60d532020-11-09 13:33:17 -0500155 std::optional<std::string> source_path;
156 if (!install_part_.source_path.empty()) {
157 // TODO(zhangkelvin) Make |source_path| a std::optional<std::string>
158 source_path = install_part_.source_path;
159 }
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400160
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400161 // ===== Resume case handling code goes here ====
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400162 // It is possible that the SOURCE_COPY are already written but
163 // |next_op_index_| is still 0. In this case we discard previously written
164 // SOURCE_COPY, and start over.
David Andersona4b7ba62023-05-10 21:41:37 -0700165 std::optional<uint64_t> label;
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400166 if (install_plan->is_resume && next_op_index > 0) {
167 LOG(INFO) << "Resuming update on partition `"
168 << partition_update_.partition_name() << "` op index "
169 << next_op_index;
David Andersona4b7ba62023-05-10 21:41:37 -0700170 label = {next_op_index};
171 }
172
173 cow_writer_ =
174 dynamic_control_->OpenCowWriter(install_part_.name, source_path, label);
175 TEST_AND_RETURN_FALSE(cow_writer_ != nullptr);
176
177 if (label) {
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400178 return true;
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400179 }
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400180
181 // ==============================================
Kelvin Zhanga37aafc2021-06-14 13:21:37 -0400182 if (!partition_update_.merge_operations().empty()) {
Kelvin Zhang5d74b722021-09-29 15:24:26 -0700183 if (IsXorEnabled()) {
184 LOG(INFO) << "VABC XOR enabled for partition "
185 << partition_update_.partition_name();
Kelvin Zhangd7958752023-03-23 12:23:47 -0700186 }
187 // When merge sequence is present in COW, snapuserd will merge blocks in
188 // order specified by the merge seuqnece op. Hence we have the freedom of
189 // writing COPY operations out of order. Delay processing of copy ops so
190 // that update_engine can be more responsive in progress updates.
191 if (DoesDeviceSupportsXor()) {
192 LOG(INFO) << "Snapuserd supports XOR and merge sequence, writing merge "
193 "sequence and delay writing COPY operations";
Kelvin Zhang5d74b722021-09-29 15:24:26 -0700194 TEST_AND_RETURN_FALSE(WriteMergeSequence(
195 partition_update_.merge_operations(), cow_writer_.get()));
Kelvin Zhangd7958752023-03-23 12:23:47 -0700196 } else {
197 LOG(INFO) << "Snapuserd does not support merge sequence, writing all "
198 "COPY operations up front, this may take few "
199 "minutes.";
200 TEST_AND_RETURN_FALSE(WriteAllCopyOps());
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700201 }
Kelvin Zhanga37aafc2021-06-14 13:21:37 -0400202 cow_writer_->AddLabel(0);
Kelvin Zhangab3ce602021-02-24 14:46:40 -0500203 }
Kelvin Zhang7a265752020-10-29 15:51:35 -0400204 return true;
205}
206
Kelvin Zhanga37aafc2021-06-14 13:21:37 -0400207bool VABCPartitionWriter::WriteMergeSequence(
208 const RepeatedPtrField<CowMergeOperation>& merge_sequence,
209 ICowWriter* cow_writer) {
210 std::vector<uint32_t> blocks_merge_order;
211 for (const auto& merge_op : merge_sequence) {
212 const auto& dst_extent = merge_op.dst_extent();
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700213 const auto& src_extent = merge_op.src_extent();
Kelvin Zhanga37aafc2021-06-14 13:21:37 -0400214 // In place copy are basically noops, they do not need to be "merged" at
215 // all, don't include them in merge sequence.
216 if (merge_op.type() == CowMergeOperation::COW_COPY &&
217 merge_op.src_extent() == merge_op.dst_extent()) {
218 continue;
219 }
Akilesh Kailash3e6e7df2021-11-18 23:29:15 +0000220
221 const bool extent_overlap =
222 ExtentRanges::ExtentsOverlap(src_extent, dst_extent);
223 // TODO(193863443) Remove this check once this feature
224 // lands on all pixel devices.
225 const bool is_ascending = android::base::GetBoolProperty(
226 "ro.virtual_ab.userspace.snapshots.enabled", false);
227
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700228 // If this is a self-overlapping op and |dst_extent| comes after
229 // |src_extent|, we must write in reverse order for correctness.
Akilesh Kailash3e6e7df2021-11-18 23:29:15 +0000230 //
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700231 // If this is self-overlapping op and |dst_extent| comes before
232 // |src_extent|, we must write in ascending order for correctness.
Akilesh Kailash3e6e7df2021-11-18 23:29:15 +0000233 //
234 // If this isn't a self overlapping op, write block in ascending order
235 // if userspace snapshots are enabled
236 if (extent_overlap) {
237 if (dst_extent.start_block() <= src_extent.start_block()) {
238 for (size_t i = 0; i < dst_extent.num_blocks(); i++) {
239 blocks_merge_order.push_back(dst_extent.start_block() + i);
240 }
241 } else {
242 for (int i = dst_extent.num_blocks() - 1; i >= 0; i--) {
243 blocks_merge_order.push_back(dst_extent.start_block() + i);
244 }
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700245 }
246 } else {
Akilesh Kailash3e6e7df2021-11-18 23:29:15 +0000247 if (is_ascending) {
248 for (size_t i = 0; i < dst_extent.num_blocks(); i++) {
249 blocks_merge_order.push_back(dst_extent.start_block() + i);
250 }
251 } else {
252 for (int i = dst_extent.num_blocks() - 1; i >= 0; i--) {
253 blocks_merge_order.push_back(dst_extent.start_block() + i);
254 }
Kelvin Zhangd1f90bc2021-09-15 21:12:34 -0700255 }
Kelvin Zhanga37aafc2021-06-14 13:21:37 -0400256 }
257 }
258 return cow_writer->AddSequenceData(blocks_merge_order.size(),
259 blocks_merge_order.data());
260}
261
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400262std::unique_ptr<ExtentWriter> VABCPartitionWriter::CreateBaseExtentWriter() {
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400263 return std::make_unique<SnapshotExtentWriter>(cow_writer_.get());
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400264}
265
266[[nodiscard]] bool VABCPartitionWriter::PerformZeroOrDiscardOperation(
267 const InstallOperation& operation) {
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400268 for (const auto& extent : operation.dst_extents()) {
269 TEST_AND_RETURN_FALSE(
270 cow_writer_->AddZeroBlocks(extent.start_block(), extent.num_blocks()));
271 }
272 return true;
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400273}
274
275[[nodiscard]] bool VABCPartitionWriter::PerformSourceCopyOperation(
276 const InstallOperation& operation, ErrorCode* error) {
Kelvin Zhangb5bd0752022-03-10 15:45:25 -0800277 // COPY ops are already handled during Init(), no need to do actual work, but
278 // we still want to verify that all blocks contain expected data.
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700279 auto source_fd = verified_source_fd_.ChooseSourceFD(operation, error);
280 TEST_AND_RETURN_FALSE(source_fd != nullptr);
281 std::vector<CowOperation> converted;
282
283 const auto& src_extents = operation.src_extents();
284 const auto& dst_extents = operation.dst_extents();
285 BlockIterator it1{src_extents};
286 BlockIterator it2{dst_extents};
Kelvin Zhangd7958752023-03-23 12:23:47 -0700287 const bool userSnapshots = android::base::GetBoolProperty(
288 "ro.virtual_ab.userspace.snapshots.enabled", false);
Kelvin Zhang67144e52023-04-03 18:17:07 -0700289 // For devices not supporting XOR, sequence op is not supported, so all COPY
290 // operations are written up front in strict merge order.
291 const auto sequence_op_supported = DoesDeviceSupportsXor();
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700292 while (!it1.is_end() && !it2.is_end()) {
293 const auto src_block = *it1;
294 const auto dst_block = *it2;
295 ++it1;
296 ++it2;
297 if (src_block == dst_block) {
298 continue;
299 }
Kelvin Zhangd7958752023-03-23 12:23:47 -0700300 if (copy_blocks_.ContainsBlock(dst_block)) {
Kelvin Zhang67144e52023-04-03 18:17:07 -0700301 if (sequence_op_supported) {
302 push_back(&converted, {CowOperation::CowCopy, src_block, dst_block, 1});
303 }
Kelvin Zhangd7958752023-03-23 12:23:47 -0700304 } else {
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700305 push_back(&converted,
306 {CowOperation::CowReplace, src_block, dst_block, 1});
307 }
Kelvin Zhangdc122bc2022-03-15 14:19:04 -0700308 }
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700309 std::vector<uint8_t> buffer;
310 for (const auto& cow_op : converted) {
Kelvin Zhangd7958752023-03-23 12:23:47 -0700311 if (cow_op.op == CowOperation::CowCopy) {
312 if (userSnapshots) {
313 cow_writer_->AddCopy(
314 cow_op.dst_block, cow_op.src_block, cow_op.block_count);
315 } else {
316 // Add blocks in reverse order, because snapused specifically prefers
317 // this ordering. Since we already eliminated all self-overlapping
318 // SOURCE_COPY during delta generation, this should be safe to do.
319 for (size_t i = cow_op.block_count; i > 0; i--) {
320 TEST_AND_RETURN_FALSE(cow_writer_->AddCopy(cow_op.dst_block + i - 1,
321 cow_op.src_block + i - 1));
322 }
323 }
324 continue;
325 }
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700326 buffer.resize(block_size_ * cow_op.block_count);
327 ssize_t bytes_read = 0;
328 TEST_AND_RETURN_FALSE(utils::ReadAll(source_fd,
329 buffer.data(),
330 block_size_ * cow_op.block_count,
331 cow_op.src_block * block_size_,
332 &bytes_read));
333 if (bytes_read <= 0 || static_cast<size_t>(bytes_read) != buffer.size()) {
334 LOG(ERROR) << "source_fd->Read failed: " << bytes_read;
335 return false;
336 }
337 TEST_AND_RETURN_FALSE(cow_writer_->AddRawBlocks(
338 cow_op.dst_block, buffer.data(), buffer.size()));
339 }
340 return true;
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400341}
342
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500343bool VABCPartitionWriter::PerformReplaceOperation(const InstallOperation& op,
344 const void* data,
345 size_t count) {
346 // Setup the ExtentWriter stack based on the operation type.
347 std::unique_ptr<ExtentWriter> writer = CreateBaseExtentWriter();
348
349 return executor_.ExecuteReplaceOperation(op, std::move(writer), data, count);
350}
351
Tianjie8e0090d2021-08-30 22:35:21 -0700352bool VABCPartitionWriter::PerformDiffOperation(
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500353 const InstallOperation& operation,
354 ErrorCode* error,
355 const void* data,
356 size_t count) {
357 FileDescriptorPtr source_fd =
358 verified_source_fd_.ChooseSourceFD(operation, error);
359 TEST_AND_RETURN_FALSE(source_fd != nullptr);
Kelvin Zhang76f10b82021-06-25 18:45:46 -0400360 TEST_AND_RETURN_FALSE(source_fd->IsOpen());
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500361
Kelvin Zhang76f10b82021-06-25 18:45:46 -0400362 std::unique_ptr<ExtentWriter> writer =
363 IsXorEnabled() ? std::make_unique<XORExtentWriter>(
Kelvin Zhanga3a68a92023-04-05 13:17:18 -0700364 operation,
365 source_fd,
366 cow_writer_.get(),
367 xor_map_,
368 partition_update_.old_partition_info().size())
Kelvin Zhang76f10b82021-06-25 18:45:46 -0400369 : CreateBaseExtentWriter();
Tianjie8e0090d2021-08-30 22:35:21 -0700370 return executor_.ExecuteDiffOperation(
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500371 operation, std::move(writer), source_fd, data, count);
372}
373
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400374void VABCPartitionWriter::CheckpointUpdateProgress(size_t next_op_index) {
375 // No need to call fsync/sync, as CowWriter flushes after a label is added
376 // added.
Kelvin Zhang6a4d1ec2021-02-04 16:28:48 -0500377 // if cow_writer_ failed, that means Init() failed. This function shouldn't be
378 // called if Init() fails.
379 TEST_AND_RETURN(cow_writer_ != nullptr);
Kelvin Zhang52cb1d72020-10-27 13:44:25 -0400380 cow_writer_->AddLabel(next_op_index);
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400381}
382
Kelvin Zhangec205cf2020-09-28 13:23:40 -0400383[[nodiscard]] bool VABCPartitionWriter::FinishedInstallOps() {
384 // Add a hardcoded magic label to indicate end of all install ops. This label
385 // is needed by filesystem verification, don't remove.
Kelvin Zhang6a4d1ec2021-02-04 16:28:48 -0500386 TEST_AND_RETURN_FALSE(cow_writer_ != nullptr);
Kelvin Zhang9e5e1ed2021-09-28 14:19:16 -0700387 TEST_AND_RETURN_FALSE(cow_writer_->AddLabel(kEndOfInstallLabel));
388 TEST_AND_RETURN_FALSE(cow_writer_->Finalize());
David Andersona4b7ba62023-05-10 21:41:37 -0700389
390 auto cow_reader = cow_writer_->OpenReader();
391 TEST_AND_RETURN_FALSE(cow_reader);
392 TEST_AND_RETURN_FALSE(cow_reader->VerifyMergeOps());
Kelvin Zhang02fe6622021-11-01 16:37:58 -0700393 return true;
Kelvin Zhangec205cf2020-09-28 13:23:40 -0400394}
395
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400396VABCPartitionWriter::~VABCPartitionWriter() {
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500397 Close();
398}
399
400int VABCPartitionWriter::Close() {
Kelvin Zhang6a4d1ec2021-02-04 16:28:48 -0500401 if (cow_writer_) {
Kelvin Zhangd11e2fc2022-10-24 15:40:30 -0700402 LOG(INFO) << "Finalizing " << partition_update_.partition_name()
403 << " COW image";
Kelvin Zhangb1c05242023-12-14 13:49:53 -0800404 if (!cow_writer_->Finalize()) {
405 return -errno;
406 }
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500407 cow_writer_ = nullptr;
Kelvin Zhang6a4d1ec2021-02-04 16:28:48 -0500408 }
Kelvin Zhange52b6cd2021-02-09 15:28:40 -0500409 return 0;
Kelvin Zhang9b10dba2020-09-25 17:09:11 -0400410}
Kelvin Zhang94f51cc2020-09-25 11:34:49 -0400411
412} // namespace chromeos_update_engine