Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // |
| 2 | // Copyright (C) 2012 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 | // |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 16 | |
| 17 | #include "update_engine/payload_state.h" |
| 18 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 19 | #include <algorithm> |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 20 | #include <string> |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 21 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 22 | #include <base/logging.h> |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 23 | #include <base/strings/string_util.h> |
| 24 | #include <base/strings/stringprintf.h> |
Alex Deymo | a259179 | 2015-11-17 00:39:40 -0300 | [diff] [blame] | 25 | #include <metrics/metrics_library.h> |
Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 26 | #include <policy/device_policy.h> |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 27 | |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 28 | #include "update_engine/common/clock.h" |
| 29 | #include "update_engine/common/constants.h" |
Alex Deymo | e88e9fe | 2016-02-03 16:38:00 -0800 | [diff] [blame] | 30 | #include "update_engine/common/error_code_utils.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 31 | #include "update_engine/common/hardware_interface.h" |
| 32 | #include "update_engine/common/prefs.h" |
| 33 | #include "update_engine/common/utils.h" |
Sen Jiang | 255e22b | 2016-05-20 16:15:29 -0700 | [diff] [blame] | 34 | #include "update_engine/connection_manager_interface.h" |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 35 | #include "update_engine/metrics_reporter_interface.h" |
Alex Deymo | 38429cf | 2015-11-11 18:27:22 -0800 | [diff] [blame] | 36 | #include "update_engine/metrics_utils.h" |
Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 37 | #include "update_engine/omaha_request_params.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 38 | #include "update_engine/payload_consumer/install_plan.h" |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 39 | #include "update_engine/system_state.h" |
Jae Hoon Kim | 5e8e30b | 2020-05-06 14:59:06 -0700 | [diff] [blame] | 40 | #include "update_engine/update_attempter.h" |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 41 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 42 | using base::Time; |
| 43 | using base::TimeDelta; |
| 44 | using std::min; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 45 | using std::string; |
| 46 | |
| 47 | namespace chromeos_update_engine { |
| 48 | |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 49 | using metrics_utils::GetPersistedValue; |
| 50 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 51 | const TimeDelta PayloadState::kDurationSlack = TimeDelta::FromSeconds(600); |
| 52 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 53 | // We want to upperbound backoffs to 16 days |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 54 | static const int kMaxBackoffDays = 16; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 55 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 56 | // We want to randomize retry attempts after the backoff by +/- 6 hours. |
| 57 | static const uint32_t kMaxBackoffFuzzMinutes = 12 * 60; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 58 | |
Colin Howes | 0e452c9 | 2018-11-02 13:18:44 -0700 | [diff] [blame] | 59 | // Limit persisting current update duration uptime to once per second |
| 60 | static const uint64_t kUptimeResolution = 1; |
| 61 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 62 | PayloadState::PayloadState() |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 63 | : prefs_(nullptr), |
Jae Hoon Kim | 5e8e30b | 2020-05-06 14:59:06 -0700 | [diff] [blame] | 64 | powerwash_safe_prefs_(nullptr), |
| 65 | excluder_(nullptr), |
David Zeuthen | bb8bdc7 | 2013-09-03 13:43:48 -0700 | [diff] [blame] | 66 | using_p2p_for_downloading_(false), |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 67 | p2p_num_attempts_(0), |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 68 | payload_attempt_number_(0), |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 69 | full_payload_attempt_number_(0), |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 70 | url_index_(0), |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 71 | url_failure_count_(0), |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 72 | url_switch_count_(0), |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 73 | rollback_happened_(false), |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 74 | attempt_num_bytes_downloaded_(0), |
| 75 | attempt_connection_type_(metrics::ConnectionType::kUnknown), |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 76 | attempt_type_(AttemptType::kUpdate) { |
| 77 | for (int i = 0; i <= kNumDownloadSources; i++) |
| 78 | total_bytes_downloaded_[i] = current_bytes_downloaded_[i] = 0; |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 79 | } |
| 80 | |
| 81 | bool PayloadState::Initialize(SystemState* system_state) { |
| 82 | system_state_ = system_state; |
| 83 | prefs_ = system_state_->prefs(); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 84 | powerwash_safe_prefs_ = system_state_->powerwash_safe_prefs(); |
Jae Hoon Kim | 5e8e30b | 2020-05-06 14:59:06 -0700 | [diff] [blame] | 85 | excluder_ = system_state_->update_attempter()->GetExcluder(); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 86 | LoadResponseSignature(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 87 | LoadPayloadAttemptNumber(); |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 88 | LoadFullPayloadAttemptNumber(); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 89 | LoadUrlIndex(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 90 | LoadUrlFailureCount(); |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 91 | LoadUrlSwitchCount(); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 92 | LoadBackoffExpiryTime(); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 93 | LoadUpdateTimestampStart(); |
| 94 | // The LoadUpdateDurationUptime() method relies on LoadUpdateTimestampStart() |
| 95 | // being called before it. Don't reorder. |
| 96 | LoadUpdateDurationUptime(); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 97 | for (int i = 0; i < kNumDownloadSources; i++) { |
| 98 | DownloadSource source = static_cast<DownloadSource>(i); |
| 99 | LoadCurrentBytesDownloaded(source); |
| 100 | LoadTotalBytesDownloaded(source); |
| 101 | } |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 102 | LoadNumReboots(); |
David Zeuthen | a573d6f | 2013-06-14 16:13:36 -0700 | [diff] [blame] | 103 | LoadNumResponsesSeen(); |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 104 | LoadRollbackHappened(); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 105 | LoadRollbackVersion(); |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 106 | LoadP2PFirstAttemptTimestamp(); |
| 107 | LoadP2PNumAttempts(); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 108 | return true; |
| 109 | } |
| 110 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 111 | void PayloadState::SetResponse(const OmahaResponse& omaha_response) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 112 | // Always store the latest response. |
| 113 | response_ = omaha_response; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 114 | |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 115 | // Compute the candidate URLs first as they are used to calculate the |
| 116 | // response signature so that a change in enterprise policy for |
| 117 | // HTTP downloads being enabled or not could be honored as soon as the |
| 118 | // next update check happens. |
| 119 | ComputeCandidateUrls(); |
| 120 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 121 | // Check if the "signature" of this response (i.e. the fields we care about) |
| 122 | // has changed. |
| 123 | string new_response_signature = CalculateResponseSignature(); |
| 124 | bool has_response_changed = (response_signature_ != new_response_signature); |
| 125 | |
| 126 | // If the response has changed, we should persist the new signature and |
| 127 | // clear away all the existing state. |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 128 | if (has_response_changed) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 129 | LOG(INFO) << "Resetting all persisted state as this is a new response"; |
David Zeuthen | a573d6f | 2013-06-14 16:13:36 -0700 | [diff] [blame] | 130 | SetNumResponsesSeen(num_responses_seen_ + 1); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 131 | SetResponseSignature(new_response_signature); |
| 132 | ResetPersistedState(); |
| 133 | return; |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 134 | } |
| 135 | |
Sen Jiang | 97eba34 | 2017-05-22 14:34:11 -0700 | [diff] [blame] | 136 | // Always start from payload index 0, even for resume, to download partition |
| 137 | // info from previous payloads. |
| 138 | payload_index_ = 0; |
| 139 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 140 | // This is the earliest point at which we can validate whether the URL index |
| 141 | // we loaded from the persisted state is a valid value. If the response |
| 142 | // hasn't changed but the URL index is invalid, it's indicative of some |
| 143 | // tampering of the persisted state. |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 144 | if (payload_index_ >= candidate_urls_.size() || |
| 145 | url_index_ >= candidate_urls_[payload_index_].size()) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 146 | LOG(INFO) << "Resetting all payload state as the url index seems to have " |
| 147 | "been tampered with"; |
| 148 | ResetPersistedState(); |
| 149 | return; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 150 | } |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 151 | |
| 152 | // Update the current download source which depends on the latest value of |
| 153 | // the response. |
| 154 | UpdateCurrentDownloadSource(); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 155 | } |
| 156 | |
David Zeuthen | bb8bdc7 | 2013-09-03 13:43:48 -0700 | [diff] [blame] | 157 | void PayloadState::SetUsingP2PForDownloading(bool value) { |
| 158 | using_p2p_for_downloading_ = value; |
| 159 | // Update the current download source which depends on whether we are |
| 160 | // using p2p or not. |
| 161 | UpdateCurrentDownloadSource(); |
| 162 | } |
| 163 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 164 | void PayloadState::DownloadComplete() { |
| 165 | LOG(INFO) << "Payload downloaded successfully"; |
| 166 | IncrementPayloadAttemptNumber(); |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 167 | IncrementFullPayloadAttemptNumber(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 168 | } |
| 169 | |
| 170 | void PayloadState::DownloadProgress(size_t count) { |
| 171 | if (count == 0) |
| 172 | return; |
| 173 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 174 | CalculateUpdateDurationUptime(); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 175 | UpdateBytesDownloaded(count); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 176 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 177 | // We've received non-zero bytes from a recent download operation. Since our |
| 178 | // URL failure count is meant to penalize a URL only for consecutive |
| 179 | // failures, downloading bytes successfully means we should reset the failure |
| 180 | // count (as we know at least that the URL is working). In future, we can |
| 181 | // design this to be more sophisticated to check for more intelligent failure |
| 182 | // patterns, but right now, even 1 byte downloaded will mark the URL to be |
| 183 | // good unless it hits 10 (or configured number of) consecutive failures |
| 184 | // again. |
| 185 | |
| 186 | if (GetUrlFailureCount() == 0) |
| 187 | return; |
| 188 | |
| 189 | LOG(INFO) << "Resetting failure count of Url" << GetUrlIndex() |
| 190 | << " to 0 as we received " << count << " bytes successfully"; |
| 191 | SetUrlFailureCount(0); |
| 192 | } |
| 193 | |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 194 | void PayloadState::AttemptStarted(AttemptType attempt_type) { |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 195 | // Flush previous state from abnormal attempt failure, if any. |
| 196 | ReportAndClearPersistedAttemptMetrics(); |
| 197 | |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 198 | attempt_type_ = attempt_type; |
| 199 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 200 | ClockInterface* clock = system_state_->clock(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 201 | attempt_start_time_boot_ = clock->GetBootTime(); |
| 202 | attempt_start_time_monotonic_ = clock->GetMonotonicTime(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 203 | attempt_num_bytes_downloaded_ = 0; |
David Zeuthen | b281f07 | 2014-04-02 10:20:19 -0700 | [diff] [blame] | 204 | |
| 205 | metrics::ConnectionType type; |
Sen Jiang | 255e22b | 2016-05-20 16:15:29 -0700 | [diff] [blame] | 206 | ConnectionType network_connection_type; |
| 207 | ConnectionTethering tethering; |
Alex Deymo | f6ee016 | 2015-07-31 12:35:22 -0700 | [diff] [blame] | 208 | ConnectionManagerInterface* connection_manager = |
| 209 | system_state_->connection_manager(); |
Alex Deymo | 3053450 | 2015-07-20 15:06:33 -0700 | [diff] [blame] | 210 | if (!connection_manager->GetConnectionProperties(&network_connection_type, |
David Zeuthen | b281f07 | 2014-04-02 10:20:19 -0700 | [diff] [blame] | 211 | &tethering)) { |
| 212 | LOG(ERROR) << "Failed to determine connection type."; |
| 213 | type = metrics::ConnectionType::kUnknown; |
| 214 | } else { |
Alex Deymo | 38429cf | 2015-11-11 18:27:22 -0800 | [diff] [blame] | 215 | type = metrics_utils::GetConnectionType(network_connection_type, tethering); |
David Zeuthen | b281f07 | 2014-04-02 10:20:19 -0700 | [diff] [blame] | 216 | } |
| 217 | attempt_connection_type_ = type; |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 218 | |
| 219 | if (attempt_type == AttemptType::kUpdate) |
| 220 | PersistAttemptMetrics(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 221 | } |
| 222 | |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 223 | void PayloadState::UpdateResumed() { |
| 224 | LOG(INFO) << "Resuming an update that was previously started."; |
| 225 | UpdateNumReboots(); |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 226 | AttemptStarted(AttemptType::kUpdate); |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 227 | } |
| 228 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 229 | void PayloadState::UpdateRestarted() { |
| 230 | LOG(INFO) << "Starting a new update"; |
| 231 | ResetDownloadSourcesOnNewUpdate(); |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 232 | SetNumReboots(0); |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 233 | AttemptStarted(AttemptType::kUpdate); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 234 | } |
| 235 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 236 | void PayloadState::UpdateSucceeded() { |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 237 | // Send the relevant metrics that are tracked in this class to UMA. |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 238 | CalculateUpdateDurationUptime(); |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 239 | SetUpdateTimestampEnd(system_state_->clock()->GetWallclockTime()); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 240 | |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 241 | switch (attempt_type_) { |
| 242 | case AttemptType::kUpdate: |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 243 | CollectAndReportAttemptMetrics(ErrorCode::kSuccess); |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 244 | CollectAndReportSuccessfulUpdateMetrics(); |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 245 | ClearPersistedAttemptMetrics(); |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 246 | break; |
| 247 | |
| 248 | case AttemptType::kRollback: |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 249 | system_state_->metrics_reporter()->ReportRollbackMetrics( |
| 250 | metrics::RollbackResult::kSuccess); |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 251 | break; |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 252 | } |
David Zeuthen | a573d6f | 2013-06-14 16:13:36 -0700 | [diff] [blame] | 253 | |
| 254 | // Reset the number of responses seen since it counts from the last |
| 255 | // successful update, e.g. now. |
| 256 | SetNumResponsesSeen(0); |
Sen Jiang | 97eba34 | 2017-05-22 14:34:11 -0700 | [diff] [blame] | 257 | SetPayloadIndex(0); |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 258 | |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 259 | metrics_utils::SetSystemUpdatedMarker(system_state_->clock(), prefs_); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 260 | } |
| 261 | |
David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 262 | void PayloadState::UpdateFailed(ErrorCode error) { |
| 263 | ErrorCode base_error = utils::GetBaseErrorCode(error); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 264 | LOG(INFO) << "Updating payload state for error code: " << base_error << " (" |
| 265 | << utils::ErrorCodeToString(base_error) << ")"; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 266 | |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 267 | if (candidate_urls_.size() == 0) { |
| 268 | // This means we got this error even before we got a valid Omaha response |
| 269 | // or don't have any valid candidates in the Omaha response. |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 270 | // So we should not advance the url_index_ in such cases. |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 271 | LOG(INFO) << "Ignoring failures until we get a valid Omaha response."; |
| 272 | return; |
| 273 | } |
| 274 | |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 275 | switch (attempt_type_) { |
| 276 | case AttemptType::kUpdate: |
| 277 | CollectAndReportAttemptMetrics(base_error); |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 278 | ClearPersistedAttemptMetrics(); |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 279 | break; |
| 280 | |
| 281 | case AttemptType::kRollback: |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 282 | system_state_->metrics_reporter()->ReportRollbackMetrics( |
| 283 | metrics::RollbackResult::kFailed); |
David Zeuthen | 96197df | 2014-04-16 12:22:39 -0700 | [diff] [blame] | 284 | break; |
| 285 | } |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 286 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 287 | switch (base_error) { |
| 288 | // Errors which are good indicators of a problem with a particular URL or |
| 289 | // the protocol used in the URL or entities in the communication channel |
| 290 | // (e.g. proxies). We should try the next available URL in the next update |
| 291 | // check to quickly recover from these errors. |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 292 | case ErrorCode::kPayloadHashMismatchError: |
| 293 | case ErrorCode::kPayloadSizeMismatchError: |
| 294 | case ErrorCode::kDownloadPayloadVerificationError: |
| 295 | case ErrorCode::kDownloadPayloadPubKeyVerificationError: |
| 296 | case ErrorCode::kSignedDeltaPayloadExpectedError: |
| 297 | case ErrorCode::kDownloadInvalidMetadataMagicString: |
| 298 | case ErrorCode::kDownloadSignatureMissingInManifest: |
| 299 | case ErrorCode::kDownloadManifestParseError: |
| 300 | case ErrorCode::kDownloadMetadataSignatureError: |
| 301 | case ErrorCode::kDownloadMetadataSignatureVerificationError: |
| 302 | case ErrorCode::kDownloadMetadataSignatureMismatch: |
| 303 | case ErrorCode::kDownloadOperationHashVerificationError: |
| 304 | case ErrorCode::kDownloadOperationExecutionError: |
| 305 | case ErrorCode::kDownloadOperationHashMismatch: |
| 306 | case ErrorCode::kDownloadInvalidMetadataSize: |
| 307 | case ErrorCode::kDownloadInvalidMetadataSignature: |
| 308 | case ErrorCode::kDownloadOperationHashMissingError: |
| 309 | case ErrorCode::kDownloadMetadataSignatureMissingError: |
| 310 | case ErrorCode::kPayloadMismatchedType: |
| 311 | case ErrorCode::kUnsupportedMajorPayloadVersion: |
| 312 | case ErrorCode::kUnsupportedMinorPayloadVersion: |
Sen Jiang | 8e768e9 | 2017-06-28 17:13:19 -0700 | [diff] [blame] | 313 | case ErrorCode::kPayloadTimestampError: |
Sen Jiang | 57f9180 | 2017-11-14 17:42:13 -0800 | [diff] [blame] | 314 | case ErrorCode::kVerityCalculationError: |
Jae Hoon Kim | 694eeb0 | 2020-06-01 14:24:08 -0700 | [diff] [blame] | 315 | ExcludeCurrentPayload(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 316 | IncrementUrlIndex(); |
| 317 | break; |
| 318 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 319 | // Errors which seem to be just transient network/communication related |
| 320 | // failures and do not indicate any inherent problem with the URL itself. |
| 321 | // So, we should keep the current URL but just increment the |
| 322 | // failure count to give it more chances. This way, while we maximize our |
| 323 | // chances of downloading from the URLs that appear earlier in the |
| 324 | // response (because download from a local server URL that appears earlier |
| 325 | // in a response is preferable than downloading from the next URL which |
| 326 | // could be a internet URL and thus could be more expensive). |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 327 | |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 328 | case ErrorCode::kError: |
| 329 | case ErrorCode::kDownloadTransferError: |
| 330 | case ErrorCode::kDownloadWriteError: |
| 331 | case ErrorCode::kDownloadStateInitializationError: |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 332 | case ErrorCode::kOmahaErrorInHTTPResponse: // Aggregate for HTTP errors. |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 333 | IncrementFailureCount(); |
| 334 | break; |
| 335 | |
| 336 | // Errors which are not specific to a URL and hence shouldn't result in |
| 337 | // the URL being penalized. This can happen in two cases: |
| 338 | // 1. We haven't started downloading anything: These errors don't cost us |
| 339 | // anything in terms of actual payload bytes, so we should just do the |
| 340 | // regular retries at the next update check. |
| 341 | // 2. We have successfully downloaded the payload: In this case, the |
| 342 | // payload attempt number would have been incremented and would take care |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 343 | // of the backoff at the next update check. |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 344 | // In either case, there's no need to update URL index or failure count. |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 345 | case ErrorCode::kOmahaRequestError: |
| 346 | case ErrorCode::kOmahaResponseHandlerError: |
| 347 | case ErrorCode::kPostinstallRunnerError: |
| 348 | case ErrorCode::kFilesystemCopierError: |
| 349 | case ErrorCode::kInstallDeviceOpenError: |
| 350 | case ErrorCode::kKernelDeviceOpenError: |
| 351 | case ErrorCode::kDownloadNewPartitionInfoError: |
| 352 | case ErrorCode::kNewRootfsVerificationError: |
| 353 | case ErrorCode::kNewKernelVerificationError: |
| 354 | case ErrorCode::kPostinstallBootedFromFirmwareB: |
| 355 | case ErrorCode::kPostinstallFirmwareRONotUpdatable: |
| 356 | case ErrorCode::kOmahaRequestEmptyResponseError: |
| 357 | case ErrorCode::kOmahaRequestXMLParseError: |
| 358 | case ErrorCode::kOmahaResponseInvalid: |
| 359 | case ErrorCode::kOmahaUpdateIgnoredPerPolicy: |
| 360 | case ErrorCode::kOmahaUpdateDeferredPerPolicy: |
Kevin Cernekee | 2494e28 | 2016-03-29 18:03:53 -0700 | [diff] [blame] | 361 | case ErrorCode::kNonCriticalUpdateInOOBE: |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 362 | case ErrorCode::kOmahaUpdateDeferredForBackoff: |
| 363 | case ErrorCode::kPostinstallPowerwashError: |
| 364 | case ErrorCode::kUpdateCanceledByChannelChange: |
David Zeuthen | f3e2801 | 2014-08-26 18:23:52 -0400 | [diff] [blame] | 365 | case ErrorCode::kOmahaRequestXMLHasEntityDecl: |
Allie Wood | eb9e6d8 | 2015-04-17 13:55:30 -0700 | [diff] [blame] | 366 | case ErrorCode::kFilesystemVerifierError: |
Alex Deymo | 1f19dcc | 2016-02-03 09:22:17 -0800 | [diff] [blame] | 367 | case ErrorCode::kUserCanceled: |
Weidong Guo | 421ff33 | 2017-04-17 10:08:38 -0700 | [diff] [blame] | 368 | case ErrorCode::kOmahaUpdateIgnoredOverCellular: |
Sen Jiang | fe52282 | 2017-10-31 15:14:11 -0700 | [diff] [blame] | 369 | case ErrorCode::kUpdatedButNotActive: |
Sen Jiang | 89e24c1 | 2018-03-22 18:05:44 -0700 | [diff] [blame] | 370 | case ErrorCode::kNoUpdate: |
Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 371 | case ErrorCode::kRollbackNotPossible: |
Amin Hassani | 80f4d4c | 2018-05-16 13:34:00 -0700 | [diff] [blame] | 372 | case ErrorCode::kFirstActiveOmahaPingSentPersistenceError: |
Amin Hassani | d3d8421 | 2019-08-17 00:27:44 -0700 | [diff] [blame] | 373 | case ErrorCode::kInternalLibCurlError: |
| 374 | case ErrorCode::kUnresolvedHostError: |
| 375 | case ErrorCode::kUnresolvedHostRecovered: |
Jae Hoon Kim | 3e69b4c | 2020-06-16 09:23:39 -0700 | [diff] [blame^] | 376 | case ErrorCode::kPackageExcludedFromUpdate: |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 377 | LOG(INFO) << "Not incrementing URL index or failure count for this error"; |
| 378 | break; |
| 379 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 380 | case ErrorCode::kSuccess: // success code |
| 381 | case ErrorCode::kUmaReportedMax: // not an error code |
| 382 | case ErrorCode::kOmahaRequestHTTPResponseBase: // aggregated already |
| 383 | case ErrorCode::kDevModeFlag: // not an error code |
| 384 | case ErrorCode::kResumedFlag: // not an error code |
| 385 | case ErrorCode::kTestImageFlag: // not an error code |
| 386 | case ErrorCode::kTestOmahaUrlFlag: // not an error code |
| 387 | case ErrorCode::kSpecialFlags: // not an error code |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 388 | // These shouldn't happen. Enumerating these explicitly here so that we |
| 389 | // can let the compiler warn about new error codes that are added to |
| 390 | // action_processor.h but not added here. |
| 391 | LOG(WARNING) << "Unexpected error code for UpdateFailed"; |
| 392 | break; |
| 393 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 394 | // Note: Not adding a default here so as to let the compiler warn us of |
| 395 | // any new enums that were added in the .h but not listed in this switch. |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 396 | } |
| 397 | } |
| 398 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 399 | bool PayloadState::ShouldBackoffDownload() { |
| 400 | if (response_.disable_payload_backoff) { |
| 401 | LOG(INFO) << "Payload backoff logic is disabled. " |
| 402 | "Can proceed with the download"; |
| 403 | return false; |
| 404 | } |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 405 | if (GetUsingP2PForDownloading() && !GetP2PUrl().empty()) { |
Chris Sosa | 20f005c | 2013-09-05 13:53:08 -0700 | [diff] [blame] | 406 | LOG(INFO) << "Payload backoff logic is disabled because download " |
| 407 | << "will happen from local peer (via p2p)."; |
| 408 | return false; |
| 409 | } |
| 410 | if (system_state_->request_params()->interactive()) { |
| 411 | LOG(INFO) << "Payload backoff disabled for interactive update checks."; |
| 412 | return false; |
| 413 | } |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 414 | for (const auto& package : response_.packages) { |
| 415 | if (package.is_delta) { |
| 416 | // If delta payloads fail, we want to fallback quickly to full payloads as |
| 417 | // they are more likely to succeed. Exponential backoffs would greatly |
| 418 | // slow down the fallback to full payloads. So we don't backoff for delta |
| 419 | // payloads. |
| 420 | LOG(INFO) << "No backoffs for delta payloads. " |
| 421 | << "Can proceed with the download"; |
| 422 | return false; |
| 423 | } |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 424 | } |
| 425 | |
Amin Hassani | ffb6d80 | 2018-03-30 11:43:57 -0700 | [diff] [blame] | 426 | if (!system_state_->hardware()->IsOfficialBuild() && |
| 427 | !prefs_->Exists(kPrefsNoIgnoreBackoff)) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 428 | // Backoffs are needed only for official builds. We do not want any delays |
Amin Hassani | ffb6d80 | 2018-03-30 11:43:57 -0700 | [diff] [blame] | 429 | // or update failures due to backoffs during testing or development. Unless |
| 430 | // the |kPrefsNoIgnoreBackoff| is manually set. |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 431 | LOG(INFO) << "No backoffs for test/dev images. " |
| 432 | << "Can proceed with the download"; |
| 433 | return false; |
| 434 | } |
| 435 | |
| 436 | if (backoff_expiry_time_.is_null()) { |
| 437 | LOG(INFO) << "No backoff expiry time has been set. " |
| 438 | << "Can proceed with the download"; |
| 439 | return false; |
| 440 | } |
| 441 | |
| 442 | if (backoff_expiry_time_ < Time::Now()) { |
| 443 | LOG(INFO) << "The backoff expiry time (" |
| 444 | << utils::ToString(backoff_expiry_time_) |
| 445 | << ") has elapsed. Can proceed with the download"; |
| 446 | return false; |
| 447 | } |
| 448 | |
| 449 | LOG(INFO) << "Cannot proceed with downloads as we need to backoff until " |
| 450 | << utils::ToString(backoff_expiry_time_); |
| 451 | return true; |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 452 | } |
| 453 | |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 454 | void PayloadState::Rollback() { |
| 455 | SetRollbackVersion(system_state_->request_params()->app_version()); |
David Zeuthen | afed4a1 | 2014-04-09 15:28:44 -0700 | [diff] [blame] | 456 | AttemptStarted(AttemptType::kRollback); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 457 | } |
| 458 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 459 | void PayloadState::IncrementPayloadAttemptNumber() { |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 460 | // Update the payload attempt number for both payload types: full and delta. |
| 461 | SetPayloadAttemptNumber(GetPayloadAttemptNumber() + 1); |
| 462 | } |
| 463 | |
| 464 | void PayloadState::IncrementFullPayloadAttemptNumber() { |
| 465 | // Update the payload attempt number for full payloads and the backoff time. |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 466 | if (response_.packages[payload_index_].is_delta) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 467 | LOG(INFO) << "Not incrementing payload attempt number for delta payloads"; |
| 468 | return; |
| 469 | } |
| 470 | |
Alex Deymo | 29b51d9 | 2013-07-09 15:26:24 -0700 | [diff] [blame] | 471 | LOG(INFO) << "Incrementing the full payload attempt number"; |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 472 | SetFullPayloadAttemptNumber(GetFullPayloadAttemptNumber() + 1); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 473 | UpdateBackoffExpiryTime(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 474 | } |
| 475 | |
| 476 | void PayloadState::IncrementUrlIndex() { |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 477 | size_t next_url_index = url_index_ + 1; |
Jae Hoon Kim | a3210e6 | 2020-05-07 11:32:44 -0700 | [diff] [blame] | 478 | size_t max_url_size = candidate_urls_[payload_index_].size(); |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 479 | if (next_url_index < max_url_size) { |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 480 | LOG(INFO) << "Incrementing the URL index for next attempt"; |
| 481 | SetUrlIndex(next_url_index); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 482 | } else { |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 483 | LOG(INFO) << "Resetting the current URL index (" << url_index_ << ") to " |
| 484 | << "0 as we only have " << max_url_size << " candidate URL(s)"; |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 485 | SetUrlIndex(0); |
Alex Deymo | 29b51d9 | 2013-07-09 15:26:24 -0700 | [diff] [blame] | 486 | IncrementPayloadAttemptNumber(); |
| 487 | IncrementFullPayloadAttemptNumber(); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 488 | } |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 489 | |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 490 | // If we have multiple URLs, record that we just switched to another one |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 491 | if (max_url_size > 1) |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 492 | SetUrlSwitchCount(url_switch_count_ + 1); |
| 493 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 494 | // Whenever we update the URL index, we should also clear the URL failure |
| 495 | // count so we can start over fresh for the new URL. |
| 496 | SetUrlFailureCount(0); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 497 | } |
| 498 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 499 | void PayloadState::IncrementFailureCount() { |
| 500 | uint32_t next_url_failure_count = GetUrlFailureCount() + 1; |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 501 | if (next_url_failure_count < response_.max_failure_count_per_url) { |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 502 | LOG(INFO) << "Incrementing the URL failure count"; |
| 503 | SetUrlFailureCount(next_url_failure_count); |
| 504 | } else { |
| 505 | LOG(INFO) << "Reached max number of failures for Url" << GetUrlIndex() |
| 506 | << ". Trying next available URL"; |
Jae Hoon Kim | 694eeb0 | 2020-06-01 14:24:08 -0700 | [diff] [blame] | 507 | ExcludeCurrentPayload(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 508 | IncrementUrlIndex(); |
| 509 | } |
| 510 | } |
| 511 | |
Jae Hoon Kim | 694eeb0 | 2020-06-01 14:24:08 -0700 | [diff] [blame] | 512 | void PayloadState::ExcludeCurrentPayload() { |
| 513 | const auto& package = response_.packages[payload_index_]; |
| 514 | if (!package.can_exclude) { |
| 515 | LOG(INFO) << "Not excluding as marked non-excludable for package hash=" |
| 516 | << package.hash; |
| 517 | return; |
| 518 | } |
| 519 | auto exclusion_name = utils::GetExclusionName(GetCurrentUrl()); |
| 520 | if (!excluder_->Exclude(exclusion_name)) |
| 521 | LOG(WARNING) << "Failed to exclude " |
| 522 | << " Package Hash=" << package.hash |
| 523 | << " CurrentUrl=" << GetCurrentUrl(); |
| 524 | else |
| 525 | LOG(INFO) << "Excluded " |
| 526 | << " Package Hash=" << package.hash |
| 527 | << " CurrentUrl=" << GetCurrentUrl(); |
| 528 | } |
| 529 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 530 | void PayloadState::UpdateBackoffExpiryTime() { |
| 531 | if (response_.disable_payload_backoff) { |
| 532 | LOG(INFO) << "Resetting backoff expiry time as payload backoff is disabled"; |
| 533 | SetBackoffExpiryTime(Time()); |
| 534 | return; |
| 535 | } |
| 536 | |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 537 | if (GetFullPayloadAttemptNumber() == 0) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 538 | SetBackoffExpiryTime(Time()); |
| 539 | return; |
| 540 | } |
| 541 | |
| 542 | // Since we're doing left-shift below, make sure we don't shift more |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 543 | // than this. E.g. if int is 4-bytes, don't left-shift more than 30 bits, |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 544 | // since we don't expect value of kMaxBackoffDays to be more than 100 anyway. |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 545 | int num_days = 1; // the value to be shifted. |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 546 | const int kMaxShifts = (sizeof(num_days) * 8) - 2; |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 547 | |
| 548 | // Normal backoff days is 2 raised to (payload_attempt_number - 1). |
| 549 | // E.g. if payload_attempt_number is over 30, limit power to 30. |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 550 | int power = min(GetFullPayloadAttemptNumber() - 1, kMaxShifts); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 551 | |
| 552 | // The number of days is the minimum of 2 raised to (payload_attempt_number |
| 553 | // - 1) or kMaxBackoffDays. |
| 554 | num_days = min(num_days << power, kMaxBackoffDays); |
| 555 | |
| 556 | // We don't want all retries to happen exactly at the same time when |
| 557 | // retrying after backoff. So add some random minutes to fuzz. |
| 558 | int fuzz_minutes = utils::FuzzInt(0, kMaxBackoffFuzzMinutes); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 559 | TimeDelta next_backoff_interval = |
| 560 | TimeDelta::FromDays(num_days) + TimeDelta::FromMinutes(fuzz_minutes); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 561 | LOG(INFO) << "Incrementing the backoff expiry time by " |
| 562 | << utils::FormatTimeDelta(next_backoff_interval); |
| 563 | SetBackoffExpiryTime(Time::Now() + next_backoff_interval); |
| 564 | } |
| 565 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 566 | void PayloadState::UpdateCurrentDownloadSource() { |
| 567 | current_download_source_ = kNumDownloadSources; |
| 568 | |
David Zeuthen | bb8bdc7 | 2013-09-03 13:43:48 -0700 | [diff] [blame] | 569 | if (using_p2p_for_downloading_) { |
| 570 | current_download_source_ = kDownloadSourceHttpPeer; |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 571 | } else if (payload_index_ < candidate_urls_.size() && |
| 572 | candidate_urls_[payload_index_].size() != 0) { |
| 573 | const string& current_url = candidate_urls_[payload_index_][GetUrlIndex()]; |
| 574 | if (base::StartsWith( |
| 575 | current_url, "https://", base::CompareCase::INSENSITIVE_ASCII)) { |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 576 | current_download_source_ = kDownloadSourceHttpsServer; |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 577 | } else if (base::StartsWith(current_url, |
| 578 | "http://", |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 579 | base::CompareCase::INSENSITIVE_ASCII)) { |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 580 | current_download_source_ = kDownloadSourceHttpServer; |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 581 | } |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 582 | } |
| 583 | |
| 584 | LOG(INFO) << "Current download source: " |
| 585 | << utils::ToString(current_download_source_); |
| 586 | } |
| 587 | |
| 588 | void PayloadState::UpdateBytesDownloaded(size_t count) { |
| 589 | SetCurrentBytesDownloaded( |
| 590 | current_download_source_, |
| 591 | GetCurrentBytesDownloaded(current_download_source_) + count, |
| 592 | false); |
| 593 | SetTotalBytesDownloaded( |
| 594 | current_download_source_, |
| 595 | GetTotalBytesDownloaded(current_download_source_) + count, |
| 596 | false); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 597 | |
| 598 | attempt_num_bytes_downloaded_ += count; |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 599 | } |
| 600 | |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 601 | PayloadType PayloadState::CalculatePayloadType() { |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 602 | for (const auto& package : response_.packages) { |
| 603 | if (package.is_delta) { |
| 604 | return kPayloadTypeDelta; |
| 605 | } |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 606 | } |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 607 | OmahaRequestParams* params = system_state_->request_params(); |
| 608 | if (params->delta_okay()) { |
| 609 | return kPayloadTypeFull; |
| 610 | } |
| 611 | // Full payload, delta was not allowed by request. |
| 612 | return kPayloadTypeForcedFull; |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 613 | } |
| 614 | |
| 615 | // TODO(zeuthen): Currently we don't report the UpdateEngine.Attempt.* |
| 616 | // metrics if the attempt ends abnormally, e.g. if the update_engine |
| 617 | // process crashes or the device is rebooted. See |
| 618 | // http://crbug.com/357676 |
| 619 | void PayloadState::CollectAndReportAttemptMetrics(ErrorCode code) { |
| 620 | int attempt_number = GetPayloadAttemptNumber(); |
| 621 | |
| 622 | PayloadType payload_type = CalculatePayloadType(); |
| 623 | |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 624 | int64_t payload_size = GetPayloadSize(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 625 | |
| 626 | int64_t payload_bytes_downloaded = attempt_num_bytes_downloaded_; |
| 627 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 628 | ClockInterface* clock = system_state_->clock(); |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 629 | TimeDelta duration = clock->GetBootTime() - attempt_start_time_boot_; |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 630 | TimeDelta duration_uptime = |
| 631 | clock->GetMonotonicTime() - attempt_start_time_monotonic_; |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 632 | |
| 633 | int64_t payload_download_speed_bps = 0; |
| 634 | int64_t usec = duration_uptime.InMicroseconds(); |
| 635 | if (usec > 0) { |
| 636 | double sec = static_cast<double>(usec) / Time::kMicrosecondsPerSecond; |
| 637 | double bps = static_cast<double>(payload_bytes_downloaded) / sec; |
| 638 | payload_download_speed_bps = static_cast<int64_t>(bps); |
| 639 | } |
| 640 | |
| 641 | DownloadSource download_source = current_download_source_; |
| 642 | |
| 643 | metrics::DownloadErrorCode payload_download_error_code = |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 644 | metrics::DownloadErrorCode::kUnset; |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 645 | ErrorCode internal_error_code = ErrorCode::kSuccess; |
Alex Deymo | 38429cf | 2015-11-11 18:27:22 -0800 | [diff] [blame] | 646 | metrics::AttemptResult attempt_result = metrics_utils::GetAttemptResult(code); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 647 | |
| 648 | // Add additional detail to AttemptResult |
| 649 | switch (attempt_result) { |
| 650 | case metrics::AttemptResult::kPayloadDownloadError: |
Alex Deymo | 38429cf | 2015-11-11 18:27:22 -0800 | [diff] [blame] | 651 | payload_download_error_code = metrics_utils::GetDownloadErrorCode(code); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 652 | break; |
| 653 | |
| 654 | case metrics::AttemptResult::kInternalError: |
| 655 | internal_error_code = code; |
| 656 | break; |
| 657 | |
| 658 | // Explicit fall-through for cases where we do not have additional |
| 659 | // detail. We avoid the default keyword to force people adding new |
| 660 | // AttemptResult values to visit this code and examine whether |
| 661 | // additional detail is needed. |
| 662 | case metrics::AttemptResult::kUpdateSucceeded: |
| 663 | case metrics::AttemptResult::kMetadataMalformed: |
| 664 | case metrics::AttemptResult::kOperationMalformed: |
| 665 | case metrics::AttemptResult::kOperationExecutionError: |
| 666 | case metrics::AttemptResult::kMetadataVerificationFailed: |
| 667 | case metrics::AttemptResult::kPayloadVerificationFailed: |
| 668 | case metrics::AttemptResult::kVerificationFailed: |
| 669 | case metrics::AttemptResult::kPostInstallFailed: |
| 670 | case metrics::AttemptResult::kAbnormalTermination: |
Alex Deymo | 1f19dcc | 2016-02-03 09:22:17 -0800 | [diff] [blame] | 671 | case metrics::AttemptResult::kUpdateCanceled: |
Sen Jiang | fe52282 | 2017-10-31 15:14:11 -0700 | [diff] [blame] | 672 | case metrics::AttemptResult::kUpdateSucceededNotActive: |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 673 | case metrics::AttemptResult::kNumConstants: |
| 674 | case metrics::AttemptResult::kUnset: |
| 675 | break; |
| 676 | } |
| 677 | |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 678 | system_state_->metrics_reporter()->ReportUpdateAttemptMetrics( |
| 679 | system_state_, |
| 680 | attempt_number, |
| 681 | payload_type, |
| 682 | duration, |
| 683 | duration_uptime, |
| 684 | payload_size, |
Tianjie Xu | 1f93d09 | 2017-10-09 12:13:29 -0700 | [diff] [blame] | 685 | attempt_result, |
| 686 | internal_error_code); |
| 687 | |
| 688 | system_state_->metrics_reporter()->ReportUpdateAttemptDownloadMetrics( |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 689 | payload_bytes_downloaded, |
| 690 | payload_download_speed_bps, |
| 691 | download_source, |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 692 | payload_download_error_code, |
| 693 | attempt_connection_type_); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 694 | } |
| 695 | |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 696 | void PayloadState::PersistAttemptMetrics() { |
| 697 | // TODO(zeuthen): For now we only persist whether an attempt was in |
| 698 | // progress and not values/metrics related to the attempt. This |
| 699 | // means that when this happens, of all the UpdateEngine.Attempt.* |
| 700 | // metrics, only UpdateEngine.Attempt.Result is reported (with the |
| 701 | // value |kAbnormalTermination|). In the future we might want to |
| 702 | // persist more data so we can report other metrics in the |
| 703 | // UpdateEngine.Attempt.* namespace when this happens. |
| 704 | prefs_->SetBoolean(kPrefsAttemptInProgress, true); |
| 705 | } |
| 706 | |
| 707 | void PayloadState::ClearPersistedAttemptMetrics() { |
| 708 | prefs_->Delete(kPrefsAttemptInProgress); |
| 709 | } |
| 710 | |
| 711 | void PayloadState::ReportAndClearPersistedAttemptMetrics() { |
| 712 | bool attempt_in_progress = false; |
| 713 | if (!prefs_->GetBoolean(kPrefsAttemptInProgress, &attempt_in_progress)) |
| 714 | return; |
| 715 | if (!attempt_in_progress) |
| 716 | return; |
| 717 | |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 718 | system_state_->metrics_reporter() |
| 719 | ->ReportAbnormallyTerminatedUpdateAttemptMetrics(); |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 720 | |
| 721 | ClearPersistedAttemptMetrics(); |
| 722 | } |
| 723 | |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 724 | void PayloadState::CollectAndReportSuccessfulUpdateMetrics() { |
Jay Srinivasan | dbd9ea2 | 2013-04-22 17:45:19 -0700 | [diff] [blame] | 725 | string metric; |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 726 | |
| 727 | // Report metrics collected from all known download sources to UMA. |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 728 | int64_t total_bytes_by_source[kNumDownloadSources]; |
| 729 | int64_t successful_bytes = 0; |
| 730 | int64_t total_bytes = 0; |
| 731 | int64_t successful_mbs = 0; |
| 732 | int64_t total_mbs = 0; |
| 733 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 734 | for (int i = 0; i < kNumDownloadSources; i++) { |
| 735 | DownloadSource source = static_cast<DownloadSource>(i); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 736 | int64_t bytes; |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 737 | |
David Zeuthen | 4484860 | 2013-06-24 13:32:14 -0700 | [diff] [blame] | 738 | // Only consider this download source (and send byte counts) as |
| 739 | // having been used if we downloaded a non-trivial amount of bytes |
| 740 | // (e.g. at least 1 MiB) that contributed to the final success of |
| 741 | // the update. Otherwise we're going to end up with a lot of |
| 742 | // zero-byte events in the histogram. |
Jay Srinivasan | dbd9ea2 | 2013-04-22 17:45:19 -0700 | [diff] [blame] | 743 | |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 744 | bytes = GetCurrentBytesDownloaded(source); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 745 | successful_bytes += bytes; |
| 746 | successful_mbs += bytes / kNumBytesInOneMiB; |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 747 | SetCurrentBytesDownloaded(source, 0, true); |
| 748 | |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 749 | bytes = GetTotalBytesDownloaded(source); |
| 750 | total_bytes_by_source[i] = bytes; |
| 751 | total_bytes += bytes; |
| 752 | total_mbs += bytes / kNumBytesInOneMiB; |
| 753 | SetTotalBytesDownloaded(source, 0, true); |
| 754 | } |
| 755 | |
| 756 | int download_overhead_percentage = 0; |
| 757 | if (successful_bytes > 0) { |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 758 | download_overhead_percentage = |
| 759 | (total_bytes - successful_bytes) * 100ULL / successful_bytes; |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 760 | } |
| 761 | |
| 762 | int url_switch_count = static_cast<int>(url_switch_count_); |
| 763 | |
| 764 | int reboot_count = GetNumReboots(); |
| 765 | |
| 766 | SetNumReboots(0); |
| 767 | |
| 768 | TimeDelta duration = GetUpdateDuration(); |
Sen Jiang | 8712e96 | 2018-05-08 12:12:28 -0700 | [diff] [blame] | 769 | TimeDelta duration_uptime = GetUpdateDurationUptime(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 770 | |
| 771 | prefs_->Delete(kPrefsUpdateTimestampStart); |
| 772 | prefs_->Delete(kPrefsUpdateDurationUptime); |
| 773 | |
| 774 | PayloadType payload_type = CalculatePayloadType(); |
| 775 | |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 776 | int64_t payload_size = GetPayloadSize(); |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 777 | |
| 778 | int attempt_count = GetPayloadAttemptNumber(); |
| 779 | |
| 780 | int updates_abandoned_count = num_responses_seen_ - 1; |
| 781 | |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 782 | system_state_->metrics_reporter()->ReportSuccessfulUpdateMetrics( |
| 783 | attempt_count, |
| 784 | updates_abandoned_count, |
| 785 | payload_type, |
| 786 | payload_size, |
| 787 | total_bytes_by_source, |
| 788 | download_overhead_percentage, |
| 789 | duration, |
Sen Jiang | 8712e96 | 2018-05-08 12:12:28 -0700 | [diff] [blame] | 790 | duration_uptime, |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 791 | reboot_count, |
| 792 | url_switch_count); |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 793 | } |
| 794 | |
| 795 | void PayloadState::UpdateNumReboots() { |
| 796 | // We only update the reboot count when the system has been detected to have |
| 797 | // been rebooted. |
| 798 | if (!system_state_->system_rebooted()) { |
| 799 | return; |
| 800 | } |
| 801 | |
| 802 | SetNumReboots(GetNumReboots() + 1); |
| 803 | } |
| 804 | |
| 805 | void PayloadState::SetNumReboots(uint32_t num_reboots) { |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 806 | num_reboots_ = num_reboots; |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 807 | metrics_utils::SetNumReboots(num_reboots, prefs_); |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 808 | } |
| 809 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 810 | void PayloadState::ResetPersistedState() { |
| 811 | SetPayloadAttemptNumber(0); |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 812 | SetFullPayloadAttemptNumber(0); |
Sen Jiang | 97eba34 | 2017-05-22 14:34:11 -0700 | [diff] [blame] | 813 | SetPayloadIndex(0); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 814 | SetUrlIndex(0); |
| 815 | SetUrlFailureCount(0); |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 816 | SetUrlSwitchCount(0); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 817 | UpdateBackoffExpiryTime(); // This will reset the backoff expiry time. |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 818 | SetUpdateTimestampStart(system_state_->clock()->GetWallclockTime()); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 819 | SetUpdateTimestampEnd(Time()); // Set to null time |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 820 | SetUpdateDurationUptime(TimeDelta::FromSeconds(0)); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 821 | ResetDownloadSourcesOnNewUpdate(); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 822 | ResetRollbackVersion(); |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 823 | SetP2PNumAttempts(0); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 824 | SetP2PFirstAttemptTimestamp(Time()); // Set to null time |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 825 | SetScatteringWaitPeriod(TimeDelta()); |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 826 | SetStagingWaitPeriod(TimeDelta()); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 827 | } |
| 828 | |
| 829 | void PayloadState::ResetRollbackVersion() { |
| 830 | CHECK(powerwash_safe_prefs_); |
| 831 | rollback_version_ = ""; |
| 832 | powerwash_safe_prefs_->Delete(kPrefsRollbackVersion); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 833 | } |
| 834 | |
| 835 | void PayloadState::ResetDownloadSourcesOnNewUpdate() { |
| 836 | for (int i = 0; i < kNumDownloadSources; i++) { |
| 837 | DownloadSource source = static_cast<DownloadSource>(i); |
| 838 | SetCurrentBytesDownloaded(source, 0, true); |
| 839 | // Note: Not resetting the TotalBytesDownloaded as we want that metric |
| 840 | // to count the bytes downloaded across various update attempts until |
| 841 | // we have successfully applied the update. |
| 842 | } |
| 843 | } |
| 844 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 845 | string PayloadState::CalculateResponseSignature() { |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 846 | string response_sign; |
| 847 | for (size_t i = 0; i < response_.packages.size(); i++) { |
| 848 | const auto& package = response_.packages[i]; |
| 849 | response_sign += base::StringPrintf( |
| 850 | "Payload %zu:\n" |
| 851 | " Size = %ju\n" |
| 852 | " Sha256 Hash = %s\n" |
| 853 | " Metadata Size = %ju\n" |
| 854 | " Metadata Signature = %s\n" |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 855 | " Is Delta = %d\n" |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 856 | " NumURLs = %zu\n", |
| 857 | i, |
| 858 | static_cast<uintmax_t>(package.size), |
| 859 | package.hash.c_str(), |
| 860 | static_cast<uintmax_t>(package.metadata_size), |
| 861 | package.metadata_signature.c_str(), |
Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 862 | package.is_delta, |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 863 | candidate_urls_[i].size()); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 864 | |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 865 | for (size_t j = 0; j < candidate_urls_[i].size(); j++) |
| 866 | response_sign += base::StringPrintf( |
| 867 | " Candidate Url%zu = %s\n", j, candidate_urls_[i][j].c_str()); |
| 868 | } |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 869 | |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 870 | response_sign += base::StringPrintf( |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 871 | "Max Failure Count Per Url = %d\n" |
| 872 | "Disable Payload Backoff = %d\n", |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 873 | response_.max_failure_count_per_url, |
| 874 | response_.disable_payload_backoff); |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 875 | return response_sign; |
| 876 | } |
| 877 | |
| 878 | void PayloadState::LoadResponseSignature() { |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 879 | CHECK(prefs_); |
| 880 | string stored_value; |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 881 | if (prefs_->Exists(kPrefsCurrentResponseSignature) && |
| 882 | prefs_->GetString(kPrefsCurrentResponseSignature, &stored_value)) { |
| 883 | SetResponseSignature(stored_value); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 884 | } |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 885 | } |
| 886 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 887 | void PayloadState::SetResponseSignature(const string& response_signature) { |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 888 | CHECK(prefs_); |
| 889 | response_signature_ = response_signature; |
| 890 | LOG(INFO) << "Current Response Signature = \n" << response_signature_; |
| 891 | prefs_->SetString(kPrefsCurrentResponseSignature, response_signature_); |
| 892 | } |
| 893 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 894 | void PayloadState::LoadPayloadAttemptNumber() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 895 | SetPayloadAttemptNumber( |
| 896 | GetPersistedValue(kPrefsPayloadAttemptNumber, prefs_)); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 897 | } |
| 898 | |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 899 | void PayloadState::LoadFullPayloadAttemptNumber() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 900 | SetFullPayloadAttemptNumber( |
| 901 | GetPersistedValue(kPrefsFullPayloadAttemptNumber, prefs_)); |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 902 | } |
| 903 | |
| 904 | void PayloadState::SetPayloadAttemptNumber(int payload_attempt_number) { |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 905 | payload_attempt_number_ = payload_attempt_number; |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 906 | metrics_utils::SetPayloadAttemptNumber(payload_attempt_number, prefs_); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 907 | } |
| 908 | |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 909 | void PayloadState::SetFullPayloadAttemptNumber( |
| 910 | int full_payload_attempt_number) { |
| 911 | CHECK(prefs_); |
| 912 | full_payload_attempt_number_ = full_payload_attempt_number; |
| 913 | LOG(INFO) << "Full Payload Attempt Number = " << full_payload_attempt_number_; |
| 914 | prefs_->SetInt64(kPrefsFullPayloadAttemptNumber, |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 915 | full_payload_attempt_number_); |
Alex Deymo | 820cc70 | 2013-06-28 15:43:46 -0700 | [diff] [blame] | 916 | } |
| 917 | |
Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 918 | void PayloadState::SetPayloadIndex(size_t payload_index) { |
| 919 | CHECK(prefs_); |
| 920 | payload_index_ = payload_index; |
| 921 | LOG(INFO) << "Payload Index = " << payload_index_; |
| 922 | prefs_->SetInt64(kPrefsUpdateStatePayloadIndex, payload_index_); |
| 923 | } |
| 924 | |
| 925 | bool PayloadState::NextPayload() { |
| 926 | if (payload_index_ + 1 >= candidate_urls_.size()) |
| 927 | return false; |
Jae Hoon Kim | a3210e6 | 2020-05-07 11:32:44 -0700 | [diff] [blame] | 928 | SetUrlIndex(0); |
Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 929 | SetPayloadIndex(payload_index_ + 1); |
| 930 | return true; |
| 931 | } |
| 932 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 933 | void PayloadState::LoadUrlIndex() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 934 | SetUrlIndex(GetPersistedValue(kPrefsCurrentUrlIndex, prefs_)); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 935 | } |
| 936 | |
| 937 | void PayloadState::SetUrlIndex(uint32_t url_index) { |
| 938 | CHECK(prefs_); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 939 | url_index_ = url_index; |
| 940 | LOG(INFO) << "Current URL Index = " << url_index_; |
| 941 | prefs_->SetInt64(kPrefsCurrentUrlIndex, url_index_); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 942 | |
| 943 | // Also update the download source, which is purely dependent on the |
| 944 | // current URL index alone. |
| 945 | UpdateCurrentDownloadSource(); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 946 | } |
| 947 | |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 948 | void PayloadState::LoadScatteringWaitPeriod() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 949 | SetScatteringWaitPeriod(TimeDelta::FromSeconds( |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 950 | GetPersistedValue(kPrefsWallClockScatteringWaitPeriod, prefs_))); |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 951 | } |
| 952 | |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 953 | void PayloadState::SetScatteringWaitPeriod(TimeDelta wait_period) { |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 954 | CHECK(prefs_); |
| 955 | scattering_wait_period_ = wait_period; |
| 956 | LOG(INFO) << "Scattering Wait Period (seconds) = " |
| 957 | << scattering_wait_period_.InSeconds(); |
| 958 | if (scattering_wait_period_.InSeconds() > 0) { |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 959 | prefs_->SetInt64(kPrefsWallClockScatteringWaitPeriod, |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 960 | scattering_wait_period_.InSeconds()); |
| 961 | } else { |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 962 | prefs_->Delete(kPrefsWallClockScatteringWaitPeriod); |
| 963 | } |
| 964 | } |
| 965 | |
| 966 | void PayloadState::LoadStagingWaitPeriod() { |
| 967 | SetStagingWaitPeriod(TimeDelta::FromSeconds( |
| 968 | GetPersistedValue(kPrefsWallClockStagingWaitPeriod, prefs_))); |
| 969 | } |
| 970 | |
| 971 | void PayloadState::SetStagingWaitPeriod(TimeDelta wait_period) { |
| 972 | CHECK(prefs_); |
| 973 | staging_wait_period_ = wait_period; |
| 974 | LOG(INFO) << "Staging Wait Period (days) =" << staging_wait_period_.InDays(); |
| 975 | if (staging_wait_period_.InSeconds() > 0) { |
| 976 | prefs_->SetInt64(kPrefsWallClockStagingWaitPeriod, |
| 977 | staging_wait_period_.InSeconds()); |
| 978 | } else { |
| 979 | prefs_->Delete(kPrefsWallClockStagingWaitPeriod); |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 980 | } |
| 981 | } |
| 982 | |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 983 | void PayloadState::LoadUrlSwitchCount() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 984 | SetUrlSwitchCount(GetPersistedValue(kPrefsUrlSwitchCount, prefs_)); |
David Zeuthen | cc6f996 | 2013-04-18 11:57:24 -0700 | [diff] [blame] | 985 | } |
| 986 | |
| 987 | void PayloadState::SetUrlSwitchCount(uint32_t url_switch_count) { |
| 988 | CHECK(prefs_); |
| 989 | url_switch_count_ = url_switch_count; |
| 990 | LOG(INFO) << "URL Switch Count = " << url_switch_count_; |
| 991 | prefs_->SetInt64(kPrefsUrlSwitchCount, url_switch_count_); |
| 992 | } |
| 993 | |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 994 | void PayloadState::LoadUrlFailureCount() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 995 | SetUrlFailureCount(GetPersistedValue(kPrefsCurrentUrlFailureCount, prefs_)); |
Jay Srinivasan | 2b5a0f0 | 2012-12-19 17:25:56 -0800 | [diff] [blame] | 996 | } |
| 997 | |
| 998 | void PayloadState::SetUrlFailureCount(uint32_t url_failure_count) { |
| 999 | CHECK(prefs_); |
| 1000 | url_failure_count_ = url_failure_count; |
| 1001 | LOG(INFO) << "Current URL (Url" << GetUrlIndex() |
| 1002 | << ")'s Failure Count = " << url_failure_count_; |
| 1003 | prefs_->SetInt64(kPrefsCurrentUrlFailureCount, url_failure_count_); |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 1004 | } |
| 1005 | |
Jay Srinivasan | 0826288 | 2012-12-28 19:29:43 -0800 | [diff] [blame] | 1006 | void PayloadState::LoadBackoffExpiryTime() { |
| 1007 | CHECK(prefs_); |
| 1008 | int64_t stored_value; |
| 1009 | if (!prefs_->Exists(kPrefsBackoffExpiryTime)) |
| 1010 | return; |
| 1011 | |
| 1012 | if (!prefs_->GetInt64(kPrefsBackoffExpiryTime, &stored_value)) |
| 1013 | return; |
| 1014 | |
| 1015 | Time stored_time = Time::FromInternalValue(stored_value); |
| 1016 | if (stored_time > Time::Now() + TimeDelta::FromDays(kMaxBackoffDays)) { |
| 1017 | LOG(ERROR) << "Invalid backoff expiry time (" |
| 1018 | << utils::ToString(stored_time) |
| 1019 | << ") in persisted state. Resetting."; |
| 1020 | stored_time = Time(); |
| 1021 | } |
| 1022 | SetBackoffExpiryTime(stored_time); |
| 1023 | } |
| 1024 | |
| 1025 | void PayloadState::SetBackoffExpiryTime(const Time& new_time) { |
| 1026 | CHECK(prefs_); |
| 1027 | backoff_expiry_time_ = new_time; |
| 1028 | LOG(INFO) << "Backoff Expiry Time = " |
| 1029 | << utils::ToString(backoff_expiry_time_); |
| 1030 | prefs_->SetInt64(kPrefsBackoffExpiryTime, |
| 1031 | backoff_expiry_time_.ToInternalValue()); |
| 1032 | } |
| 1033 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1034 | TimeDelta PayloadState::GetUpdateDuration() { |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 1035 | Time end_time = update_timestamp_end_.is_null() |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1036 | ? system_state_->clock()->GetWallclockTime() |
| 1037 | : update_timestamp_end_; |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1038 | return end_time - update_timestamp_start_; |
| 1039 | } |
| 1040 | |
| 1041 | void PayloadState::LoadUpdateTimestampStart() { |
| 1042 | int64_t stored_value; |
| 1043 | Time stored_time; |
| 1044 | |
| 1045 | CHECK(prefs_); |
| 1046 | |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 1047 | Time now = system_state_->clock()->GetWallclockTime(); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1048 | |
| 1049 | if (!prefs_->Exists(kPrefsUpdateTimestampStart)) { |
| 1050 | // The preference missing is not unexpected - in that case, just |
| 1051 | // use the current time as start time |
| 1052 | stored_time = now; |
| 1053 | } else if (!prefs_->GetInt64(kPrefsUpdateTimestampStart, &stored_value)) { |
| 1054 | LOG(ERROR) << "Invalid UpdateTimestampStart value. Resetting."; |
| 1055 | stored_time = now; |
| 1056 | } else { |
| 1057 | stored_time = Time::FromInternalValue(stored_value); |
| 1058 | } |
| 1059 | |
| 1060 | // Sanity check: If the time read from disk is in the future |
| 1061 | // (modulo some slack to account for possible NTP drift |
| 1062 | // adjustments), something is fishy and we should report and |
| 1063 | // reset. |
| 1064 | TimeDelta duration_according_to_stored_time = now - stored_time; |
| 1065 | if (duration_according_to_stored_time < -kDurationSlack) { |
| 1066 | LOG(ERROR) << "The UpdateTimestampStart value (" |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1067 | << utils::ToString(stored_time) << ") in persisted state is " |
David Zeuthen | 674c318 | 2013-04-18 14:05:20 -0700 | [diff] [blame] | 1068 | << utils::FormatTimeDelta(duration_according_to_stored_time) |
| 1069 | << " in the future. Resetting."; |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1070 | stored_time = now; |
| 1071 | } |
| 1072 | |
| 1073 | SetUpdateTimestampStart(stored_time); |
| 1074 | } |
| 1075 | |
| 1076 | void PayloadState::SetUpdateTimestampStart(const Time& value) { |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1077 | update_timestamp_start_ = value; |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1078 | metrics_utils::SetUpdateTimestampStart(value, prefs_); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1079 | } |
| 1080 | |
| 1081 | void PayloadState::SetUpdateTimestampEnd(const Time& value) { |
| 1082 | update_timestamp_end_ = value; |
| 1083 | LOG(INFO) << "Update Timestamp End = " |
| 1084 | << utils::ToString(update_timestamp_end_); |
| 1085 | } |
| 1086 | |
| 1087 | TimeDelta PayloadState::GetUpdateDurationUptime() { |
| 1088 | return update_duration_uptime_; |
| 1089 | } |
| 1090 | |
| 1091 | void PayloadState::LoadUpdateDurationUptime() { |
| 1092 | int64_t stored_value; |
| 1093 | TimeDelta stored_delta; |
| 1094 | |
| 1095 | CHECK(prefs_); |
| 1096 | |
| 1097 | if (!prefs_->Exists(kPrefsUpdateDurationUptime)) { |
| 1098 | // The preference missing is not unexpected - in that case, just |
| 1099 | // we'll use zero as the delta |
| 1100 | } else if (!prefs_->GetInt64(kPrefsUpdateDurationUptime, &stored_value)) { |
| 1101 | LOG(ERROR) << "Invalid UpdateDurationUptime value. Resetting."; |
| 1102 | stored_delta = TimeDelta::FromSeconds(0); |
| 1103 | } else { |
| 1104 | stored_delta = TimeDelta::FromInternalValue(stored_value); |
| 1105 | } |
| 1106 | |
| 1107 | // Sanity-check: Uptime can never be greater than the wall-clock |
| 1108 | // difference (modulo some slack). If it is, report and reset |
| 1109 | // to the wall-clock difference. |
| 1110 | TimeDelta diff = GetUpdateDuration() - stored_delta; |
| 1111 | if (diff < -kDurationSlack) { |
| 1112 | LOG(ERROR) << "The UpdateDurationUptime value (" |
David Zeuthen | 674c318 | 2013-04-18 14:05:20 -0700 | [diff] [blame] | 1113 | << utils::FormatTimeDelta(stored_delta) |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1114 | << ") in persisted state is " << utils::FormatTimeDelta(diff) |
David Zeuthen | 674c318 | 2013-04-18 14:05:20 -0700 | [diff] [blame] | 1115 | << " larger than the wall-clock delta. Resetting."; |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1116 | stored_delta = update_duration_current_; |
| 1117 | } |
| 1118 | |
| 1119 | SetUpdateDurationUptime(stored_delta); |
| 1120 | } |
| 1121 | |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 1122 | void PayloadState::LoadNumReboots() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1123 | SetNumReboots(GetPersistedValue(kPrefsNumReboots, prefs_)); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 1124 | } |
| 1125 | |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 1126 | void PayloadState::LoadRollbackHappened() { |
| 1127 | CHECK(powerwash_safe_prefs_); |
| 1128 | bool rollback_happened = false; |
| 1129 | powerwash_safe_prefs_->GetBoolean(kPrefsRollbackHappened, &rollback_happened); |
| 1130 | SetRollbackHappened(rollback_happened); |
| 1131 | } |
| 1132 | |
| 1133 | void PayloadState::SetRollbackHappened(bool rollback_happened) { |
| 1134 | CHECK(powerwash_safe_prefs_); |
| 1135 | LOG(INFO) << "Setting rollback-happened to " << rollback_happened << "."; |
| 1136 | rollback_happened_ = rollback_happened; |
| 1137 | if (rollback_happened) { |
| 1138 | powerwash_safe_prefs_->SetBoolean(kPrefsRollbackHappened, |
| 1139 | rollback_happened); |
| 1140 | } else { |
| 1141 | powerwash_safe_prefs_->Delete(kPrefsRollbackHappened); |
| 1142 | } |
| 1143 | } |
| 1144 | |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 1145 | void PayloadState::LoadRollbackVersion() { |
Chris Sosa | b3dcdb3 | 2013-09-04 15:22:12 -0700 | [diff] [blame] | 1146 | CHECK(powerwash_safe_prefs_); |
| 1147 | string rollback_version; |
| 1148 | if (powerwash_safe_prefs_->GetString(kPrefsRollbackVersion, |
| 1149 | &rollback_version)) { |
| 1150 | SetRollbackVersion(rollback_version); |
| 1151 | } |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 1152 | } |
| 1153 | |
| 1154 | void PayloadState::SetRollbackVersion(const string& rollback_version) { |
| 1155 | CHECK(powerwash_safe_prefs_); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1156 | LOG(INFO) << "Blacklisting version " << rollback_version; |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 1157 | rollback_version_ = rollback_version; |
| 1158 | powerwash_safe_prefs_->SetString(kPrefsRollbackVersion, rollback_version); |
Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 1159 | } |
| 1160 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1161 | void PayloadState::SetUpdateDurationUptimeExtended(const TimeDelta& value, |
| 1162 | const Time& timestamp, |
| 1163 | bool use_logging) { |
| 1164 | CHECK(prefs_); |
| 1165 | update_duration_uptime_ = value; |
| 1166 | update_duration_uptime_timestamp_ = timestamp; |
| 1167 | prefs_->SetInt64(kPrefsUpdateDurationUptime, |
| 1168 | update_duration_uptime_.ToInternalValue()); |
| 1169 | if (use_logging) { |
| 1170 | LOG(INFO) << "Update Duration Uptime = " |
David Zeuthen | 674c318 | 2013-04-18 14:05:20 -0700 | [diff] [blame] | 1171 | << utils::FormatTimeDelta(update_duration_uptime_); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1172 | } |
| 1173 | } |
| 1174 | |
| 1175 | void PayloadState::SetUpdateDurationUptime(const TimeDelta& value) { |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 1176 | Time now = system_state_->clock()->GetMonotonicTime(); |
| 1177 | SetUpdateDurationUptimeExtended(value, now, true); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1178 | } |
| 1179 | |
| 1180 | void PayloadState::CalculateUpdateDurationUptime() { |
David Zeuthen | f413fe5 | 2013-04-22 14:04:39 -0700 | [diff] [blame] | 1181 | Time now = system_state_->clock()->GetMonotonicTime(); |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1182 | TimeDelta uptime_since_last_update = now - update_duration_uptime_timestamp_; |
Colin Howes | 0e452c9 | 2018-11-02 13:18:44 -0700 | [diff] [blame] | 1183 | |
| 1184 | if (uptime_since_last_update > TimeDelta::FromSeconds(kUptimeResolution)) { |
| 1185 | TimeDelta new_uptime = update_duration_uptime_ + uptime_since_last_update; |
| 1186 | // We're frequently called so avoid logging this write |
| 1187 | SetUpdateDurationUptimeExtended(new_uptime, now, false); |
| 1188 | } |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 1189 | } |
| 1190 | |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1191 | string PayloadState::GetPrefsKey(const string& prefix, DownloadSource source) { |
| 1192 | return prefix + "-from-" + utils::ToString(source); |
| 1193 | } |
| 1194 | |
| 1195 | void PayloadState::LoadCurrentBytesDownloaded(DownloadSource source) { |
| 1196 | string key = GetPrefsKey(kPrefsCurrentBytesDownloaded, source); |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1197 | SetCurrentBytesDownloaded(source, GetPersistedValue(key, prefs_), true); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1198 | } |
| 1199 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1200 | void PayloadState::SetCurrentBytesDownloaded(DownloadSource source, |
| 1201 | uint64_t current_bytes_downloaded, |
| 1202 | bool log) { |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1203 | CHECK(prefs_); |
| 1204 | |
| 1205 | if (source >= kNumDownloadSources) |
| 1206 | return; |
| 1207 | |
| 1208 | // Update the in-memory value. |
| 1209 | current_bytes_downloaded_[source] = current_bytes_downloaded; |
| 1210 | |
| 1211 | string prefs_key = GetPrefsKey(kPrefsCurrentBytesDownloaded, source); |
| 1212 | prefs_->SetInt64(prefs_key, current_bytes_downloaded); |
| 1213 | LOG_IF(INFO, log) << "Current bytes downloaded for " |
| 1214 | << utils::ToString(source) << " = " |
| 1215 | << GetCurrentBytesDownloaded(source); |
| 1216 | } |
| 1217 | |
| 1218 | void PayloadState::LoadTotalBytesDownloaded(DownloadSource source) { |
| 1219 | string key = GetPrefsKey(kPrefsTotalBytesDownloaded, source); |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1220 | SetTotalBytesDownloaded(source, GetPersistedValue(key, prefs_), true); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1221 | } |
| 1222 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1223 | void PayloadState::SetTotalBytesDownloaded(DownloadSource source, |
| 1224 | uint64_t total_bytes_downloaded, |
| 1225 | bool log) { |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1226 | CHECK(prefs_); |
| 1227 | |
| 1228 | if (source >= kNumDownloadSources) |
| 1229 | return; |
| 1230 | |
| 1231 | // Update the in-memory value. |
| 1232 | total_bytes_downloaded_[source] = total_bytes_downloaded; |
| 1233 | |
| 1234 | // Persist. |
| 1235 | string prefs_key = GetPrefsKey(kPrefsTotalBytesDownloaded, source); |
| 1236 | prefs_->SetInt64(prefs_key, total_bytes_downloaded); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1237 | LOG_IF(INFO, log) << "Total bytes downloaded for " << utils::ToString(source) |
| 1238 | << " = " << GetTotalBytesDownloaded(source); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1239 | } |
| 1240 | |
David Zeuthen | a573d6f | 2013-06-14 16:13:36 -0700 | [diff] [blame] | 1241 | void PayloadState::LoadNumResponsesSeen() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1242 | SetNumResponsesSeen(GetPersistedValue(kPrefsNumResponsesSeen, prefs_)); |
David Zeuthen | a573d6f | 2013-06-14 16:13:36 -0700 | [diff] [blame] | 1243 | } |
| 1244 | |
| 1245 | void PayloadState::SetNumResponsesSeen(int num_responses_seen) { |
| 1246 | CHECK(prefs_); |
| 1247 | num_responses_seen_ = num_responses_seen; |
| 1248 | LOG(INFO) << "Num Responses Seen = " << num_responses_seen_; |
| 1249 | prefs_->SetInt64(kPrefsNumResponsesSeen, num_responses_seen_); |
| 1250 | } |
| 1251 | |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1252 | void PayloadState::ComputeCandidateUrls() { |
Chris Sosa | f7d8004 | 2013-08-22 16:45:17 -0700 | [diff] [blame] | 1253 | bool http_url_ok = true; |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1254 | |
J. Richard Barnette | 056b0ab | 2013-10-29 15:24:56 -0700 | [diff] [blame] | 1255 | if (system_state_->hardware()->IsOfficialBuild()) { |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1256 | const policy::DevicePolicy* policy = system_state_->device_policy(); |
Chris Sosa | f7d8004 | 2013-08-22 16:45:17 -0700 | [diff] [blame] | 1257 | if (policy && policy->GetHttpDownloadsEnabled(&http_url_ok) && !http_url_ok) |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1258 | LOG(INFO) << "Downloads via HTTP Url are not enabled by device policy"; |
| 1259 | } else { |
| 1260 | LOG(INFO) << "Allowing HTTP downloads for unofficial builds"; |
| 1261 | http_url_ok = true; |
| 1262 | } |
| 1263 | |
| 1264 | candidate_urls_.clear(); |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 1265 | for (const auto& package : response_.packages) { |
| 1266 | candidate_urls_.emplace_back(); |
| 1267 | for (const string& candidate_url : package.payload_urls) { |
| 1268 | if (base::StartsWith( |
| 1269 | candidate_url, "http://", base::CompareCase::INSENSITIVE_ASCII) && |
| 1270 | !http_url_ok) { |
| 1271 | continue; |
| 1272 | } |
| 1273 | candidate_urls_.back().push_back(candidate_url); |
| 1274 | LOG(INFO) << "Candidate Url" << (candidate_urls_.back().size() - 1) |
| 1275 | << ": " << candidate_url; |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 1276 | } |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 1277 | LOG(INFO) << "Found " << candidate_urls_.back().size() << " candidate URLs " |
| 1278 | << "out of " << package.payload_urls.size() |
| 1279 | << " URLs supplied in package " << candidate_urls_.size() - 1; |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1280 | } |
Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 1281 | } |
| 1282 | |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 1283 | void PayloadState::UpdateEngineStarted() { |
David Zeuthen | 4e1d149 | 2014-04-25 13:12:27 -0700 | [diff] [blame] | 1284 | // Flush previous state from abnormal attempt failure, if any. |
| 1285 | ReportAndClearPersistedAttemptMetrics(); |
| 1286 | |
Alex Deymo | 569c424 | 2013-07-24 12:01:01 -0700 | [diff] [blame] | 1287 | // Avoid the UpdateEngineStarted actions if this is not the first time we |
| 1288 | // run the update engine since reboot. |
| 1289 | if (!system_state_->system_rebooted()) |
| 1290 | return; |
| 1291 | |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1292 | // Report time_to_reboot if we booted into a new update. |
| 1293 | metrics_utils::LoadAndReportTimeToReboot( |
| 1294 | system_state_->metrics_reporter(), prefs_, system_state_->clock()); |
| 1295 | prefs_->Delete(kPrefsSystemUpdatedMarker); |
| 1296 | |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1297 | // Check if it is needed to send metrics about a failed reboot into a new |
| 1298 | // version. |
| 1299 | ReportFailedBootIfNeeded(); |
| 1300 | } |
| 1301 | |
| 1302 | void PayloadState::ReportFailedBootIfNeeded() { |
| 1303 | // If the kPrefsTargetVersionInstalledFrom is present, a successfully applied |
| 1304 | // payload was marked as ready immediately before the last reboot, and we |
| 1305 | // need to check if such payload successfully rebooted or not. |
| 1306 | if (prefs_->Exists(kPrefsTargetVersionInstalledFrom)) { |
Alex Vakulenko | 4f5b144 | 2014-02-21 12:19:44 -0800 | [diff] [blame] | 1307 | int64_t installed_from = 0; |
| 1308 | if (!prefs_->GetInt64(kPrefsTargetVersionInstalledFrom, &installed_from)) { |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1309 | LOG(ERROR) << "Error reading TargetVersionInstalledFrom on reboot."; |
| 1310 | return; |
| 1311 | } |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 1312 | // Old Chrome OS devices will write 2 or 4 in this setting, with the |
| 1313 | // partition number. We are now using slot numbers (0 or 1) instead, so |
| 1314 | // the following comparison will not match if we are comparing an old |
| 1315 | // partition number against a new slot number, which is the correct outcome |
| 1316 | // since we successfully booted the new update in that case. If the boot |
| 1317 | // failed, we will read this value from the same version, so it will always |
| 1318 | // be compatible. |
| 1319 | if (installed_from == system_state_->boot_control()->GetCurrentSlot()) { |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1320 | // A reboot was pending, but the chromebook is again in the same |
| 1321 | // BootDevice where the update was installed from. |
| 1322 | int64_t target_attempt; |
| 1323 | if (!prefs_->GetInt64(kPrefsTargetVersionAttempt, &target_attempt)) { |
| 1324 | LOG(ERROR) << "Error reading TargetVersionAttempt when " |
| 1325 | "TargetVersionInstalledFrom was present."; |
| 1326 | target_attempt = 1; |
| 1327 | } |
| 1328 | |
| 1329 | // Report the UMA metric of the current boot failure. |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 1330 | system_state_->metrics_reporter()->ReportFailedUpdateCount( |
| 1331 | target_attempt); |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1332 | } else { |
| 1333 | prefs_->Delete(kPrefsTargetVersionAttempt); |
| 1334 | prefs_->Delete(kPrefsTargetVersionUniqueId); |
| 1335 | } |
| 1336 | prefs_->Delete(kPrefsTargetVersionInstalledFrom); |
| 1337 | } |
| 1338 | } |
| 1339 | |
| 1340 | void PayloadState::ExpectRebootInNewVersion(const string& target_version_uid) { |
| 1341 | // Expect to boot into the new partition in the next reboot setting the |
| 1342 | // TargetVersion* flags in the Prefs. |
| 1343 | string stored_target_version_uid; |
| 1344 | string target_version_id; |
| 1345 | string target_partition; |
| 1346 | int64_t target_attempt; |
| 1347 | |
| 1348 | if (prefs_->Exists(kPrefsTargetVersionUniqueId) && |
| 1349 | prefs_->GetString(kPrefsTargetVersionUniqueId, |
| 1350 | &stored_target_version_uid) && |
| 1351 | stored_target_version_uid == target_version_uid) { |
| 1352 | if (!prefs_->GetInt64(kPrefsTargetVersionAttempt, &target_attempt)) |
| 1353 | target_attempt = 0; |
| 1354 | } else { |
| 1355 | prefs_->SetString(kPrefsTargetVersionUniqueId, target_version_uid); |
| 1356 | target_attempt = 0; |
| 1357 | } |
| 1358 | prefs_->SetInt64(kPrefsTargetVersionAttempt, target_attempt + 1); |
| 1359 | |
Alex Vakulenko | 4f5b144 | 2014-02-21 12:19:44 -0800 | [diff] [blame] | 1360 | prefs_->SetInt64(kPrefsTargetVersionInstalledFrom, |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 1361 | system_state_->boot_control()->GetCurrentSlot()); |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1362 | } |
| 1363 | |
| 1364 | void PayloadState::ResetUpdateStatus() { |
| 1365 | // Remove the TargetVersionInstalledFrom pref so that if the machine is |
| 1366 | // rebooted the next boot is not flagged as failed to rebooted into the |
| 1367 | // new applied payload. |
| 1368 | prefs_->Delete(kPrefsTargetVersionInstalledFrom); |
| 1369 | |
| 1370 | // Also decrement the attempt number if it exists. |
| 1371 | int64_t target_attempt; |
| 1372 | if (prefs_->GetInt64(kPrefsTargetVersionAttempt, &target_attempt)) |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 1373 | prefs_->SetInt64(kPrefsTargetVersionAttempt, target_attempt - 1); |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 1374 | } |
| 1375 | |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 1376 | int PayloadState::GetP2PNumAttempts() { |
| 1377 | return p2p_num_attempts_; |
| 1378 | } |
| 1379 | |
| 1380 | void PayloadState::SetP2PNumAttempts(int value) { |
| 1381 | p2p_num_attempts_ = value; |
| 1382 | LOG(INFO) << "p2p Num Attempts = " << p2p_num_attempts_; |
| 1383 | CHECK(prefs_); |
| 1384 | prefs_->SetInt64(kPrefsP2PNumAttempts, value); |
| 1385 | } |
| 1386 | |
| 1387 | void PayloadState::LoadP2PNumAttempts() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1388 | SetP2PNumAttempts(GetPersistedValue(kPrefsP2PNumAttempts, prefs_)); |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 1389 | } |
| 1390 | |
| 1391 | Time PayloadState::GetP2PFirstAttemptTimestamp() { |
| 1392 | return p2p_first_attempt_timestamp_; |
| 1393 | } |
| 1394 | |
| 1395 | void PayloadState::SetP2PFirstAttemptTimestamp(const Time& time) { |
| 1396 | p2p_first_attempt_timestamp_ = time; |
| 1397 | LOG(INFO) << "p2p First Attempt Timestamp = " |
| 1398 | << utils::ToString(p2p_first_attempt_timestamp_); |
| 1399 | CHECK(prefs_); |
| 1400 | int64_t stored_value = time.ToInternalValue(); |
| 1401 | prefs_->SetInt64(kPrefsP2PFirstAttemptTimestamp, stored_value); |
| 1402 | } |
| 1403 | |
| 1404 | void PayloadState::LoadP2PFirstAttemptTimestamp() { |
Tianjie Xu | 90aaa10 | 2017-10-10 17:39:03 -0700 | [diff] [blame] | 1405 | int64_t stored_value = |
| 1406 | GetPersistedValue(kPrefsP2PFirstAttemptTimestamp, prefs_); |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 1407 | Time stored_time = Time::FromInternalValue(stored_value); |
| 1408 | SetP2PFirstAttemptTimestamp(stored_time); |
| 1409 | } |
| 1410 | |
| 1411 | void PayloadState::P2PNewAttempt() { |
| 1412 | CHECK(prefs_); |
| 1413 | // Set timestamp, if it hasn't been set already |
| 1414 | if (p2p_first_attempt_timestamp_.is_null()) { |
| 1415 | SetP2PFirstAttemptTimestamp(system_state_->clock()->GetWallclockTime()); |
| 1416 | } |
| 1417 | // Increase number of attempts |
| 1418 | SetP2PNumAttempts(GetP2PNumAttempts() + 1); |
| 1419 | } |
| 1420 | |
| 1421 | bool PayloadState::P2PAttemptAllowed() { |
| 1422 | if (p2p_num_attempts_ > kMaxP2PAttempts) { |
| 1423 | LOG(INFO) << "Number of p2p attempts is " << p2p_num_attempts_ |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1424 | << " which is greater than " << kMaxP2PAttempts |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 1425 | << " - disallowing p2p."; |
| 1426 | return false; |
| 1427 | } |
| 1428 | |
| 1429 | if (!p2p_first_attempt_timestamp_.is_null()) { |
| 1430 | Time now = system_state_->clock()->GetWallclockTime(); |
| 1431 | TimeDelta time_spent_attempting_p2p = now - p2p_first_attempt_timestamp_; |
| 1432 | if (time_spent_attempting_p2p.InSeconds() < 0) { |
| 1433 | LOG(ERROR) << "Time spent attempting p2p is negative" |
| 1434 | << " - disallowing p2p."; |
| 1435 | return false; |
| 1436 | } |
| 1437 | if (time_spent_attempting_p2p.InSeconds() > kMaxP2PAttemptTimeSeconds) { |
| 1438 | LOG(INFO) << "Time spent attempting p2p is " |
| 1439 | << utils::FormatTimeDelta(time_spent_attempting_p2p) |
| 1440 | << " which is greater than " |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 1441 | << utils::FormatTimeDelta( |
| 1442 | TimeDelta::FromSeconds(kMaxP2PAttemptTimeSeconds)) |
David Zeuthen | dcba809 | 2013-08-06 12:16:35 -0700 | [diff] [blame] | 1443 | << " - disallowing p2p."; |
| 1444 | return false; |
| 1445 | } |
| 1446 | } |
| 1447 | |
| 1448 | return true; |
| 1449 | } |
| 1450 | |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 1451 | int64_t PayloadState::GetPayloadSize() { |
| 1452 | int64_t payload_size = 0; |
| 1453 | for (const auto& package : response_.packages) |
| 1454 | payload_size += package.size; |
| 1455 | return payload_size; |
| 1456 | } |
| 1457 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 1458 | } // namespace chromeos_update_engine |