Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1 | # Copyright (c) 2014 The Chromium OS Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | import ast |
| 6 | import ctypes |
| 7 | import logging |
| 8 | import os |
| 9 | import re |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 10 | import time |
| 11 | import uuid |
| 12 | |
| 13 | from autotest_lib.client.bin import utils |
| 14 | from autotest_lib.client.common_lib import error |
| 15 | from autotest_lib.server import autotest |
J. Richard Barnette | cab6be3 | 2014-07-17 13:07:39 -0700 | [diff] [blame] | 16 | from autotest_lib.server import test |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 17 | from autotest_lib.server.cros import vboot_constants as vboot |
| 18 | from autotest_lib.server.cros.faft.config.config import Config as FAFTConfig |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 19 | from autotest_lib.server.cros.faft.rpc_proxy import RPCProxy |
J. Richard Barnette | a57ff84 | 2014-06-05 10:00:31 -0700 | [diff] [blame] | 20 | from autotest_lib.server.cros.faft.utils.faft_checkers import FAFTCheckers |
Tom Wai-Hong Tam | f2de4de | 2015-05-02 02:48:08 +0800 | [diff] [blame] | 21 | from autotest_lib.server.cros.faft.utils.mode_switcher import ModeSwitcher |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 22 | from autotest_lib.server.cros.servo import chrome_ec |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 23 | |
| 24 | |
| 25 | class ConnectionError(Exception): |
| 26 | """Raised on an error of connecting DUT.""" |
| 27 | pass |
| 28 | |
| 29 | |
J. Richard Barnette | cab6be3 | 2014-07-17 13:07:39 -0700 | [diff] [blame] | 30 | class FAFTBase(test.test): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 31 | """The base class of FAFT classes. |
| 32 | |
| 33 | It launches the FAFTClient on DUT, such that the test can access its |
| 34 | firmware functions and interfaces. It also provides some methods to |
| 35 | handle the reboot mechanism, in order to ensure FAFTClient is still |
| 36 | connected after reboot. |
| 37 | """ |
| 38 | def initialize(self, host): |
| 39 | """Create a FAFTClient object and install the dependency.""" |
J. Richard Barnette | cab6be3 | 2014-07-17 13:07:39 -0700 | [diff] [blame] | 40 | self.servo = host.servo |
| 41 | self.servo.initialize_dut() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 42 | self._client = host |
| 43 | self._autotest_client = autotest.Autotest(self._client) |
| 44 | self._autotest_client.install() |
| 45 | self.faft_client = RPCProxy(host) |
Duncan Laurie | 10eb618 | 2014-10-07 15:39:05 -0700 | [diff] [blame] | 46 | self.lockfile = '/var/tmp/faft/lock' |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 47 | |
| 48 | def wait_for_client(self, install_deps=False, timeout=100): |
| 49 | """Wait for the client to come back online. |
| 50 | |
| 51 | New remote processes will be launched if their used flags are enabled. |
| 52 | |
| 53 | @param install_deps: If True, install Autotest dependency when ready. |
| 54 | @param timeout: Time in seconds to wait for the client SSH daemon to |
| 55 | come up. |
| 56 | @raise ConnectionError: Failed to connect DUT. |
| 57 | """ |
| 58 | if not self._client.wait_up(timeout): |
| 59 | raise ConnectionError() |
| 60 | if install_deps: |
| 61 | self._autotest_client.install() |
| 62 | # Check the FAFT client is avaiable. |
| 63 | self.faft_client.system.is_available() |
| 64 | |
| 65 | def wait_for_client_offline(self, timeout=60, orig_boot_id=None): |
| 66 | """Wait for the client to come offline. |
| 67 | |
| 68 | @param timeout: Time in seconds to wait the client to come offline. |
| 69 | @param orig_boot_id: A string containing the original boot id. |
| 70 | @raise ConnectionError: Failed to connect DUT. |
| 71 | """ |
| 72 | # When running against panther, we see that sometimes |
| 73 | # ping_wait_down() does not work correctly. There needs to |
| 74 | # be some investigation to the root cause. |
| 75 | # If we sleep for 120s before running get_boot_id(), it |
| 76 | # does succeed. But if we change this to ping_wait_down() |
| 77 | # there are implications on the wait time when running |
| 78 | # commands at the fw screens. |
| 79 | if not self._client.ping_wait_down(timeout): |
| 80 | if orig_boot_id and self._client.get_boot_id() != orig_boot_id: |
| 81 | logging.warn('Reboot done very quickly.') |
| 82 | return |
| 83 | raise ConnectionError() |
| 84 | |
| 85 | |
| 86 | class FirmwareTest(FAFTBase): |
| 87 | """ |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 88 | Base class that sets up helper objects/functions for firmware tests. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 89 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 90 | TODO: add documentaion as the FAFT rework progresses. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 91 | """ |
| 92 | version = 1 |
| 93 | |
| 94 | # Mapping of partition number of kernel and rootfs. |
| 95 | KERNEL_MAP = {'a':'2', 'b':'4', '2':'2', '4':'4', '3':'2', '5':'4'} |
| 96 | ROOTFS_MAP = {'a':'3', 'b':'5', '2':'3', '4':'5', '3':'3', '5':'5'} |
| 97 | OTHER_KERNEL_MAP = {'a':'4', 'b':'2', '2':'4', '4':'2', '3':'4', '5':'2'} |
| 98 | OTHER_ROOTFS_MAP = {'a':'5', 'b':'3', '2':'5', '4':'3', '3':'5', '5':'3'} |
| 99 | |
| 100 | CHROMEOS_MAGIC = "CHROMEOS" |
| 101 | CORRUPTED_MAGIC = "CORRUPTD" |
| 102 | |
| 103 | _SERVOD_LOG = '/var/log/servod.log' |
| 104 | |
| 105 | _ROOTFS_PARTITION_NUMBER = 3 |
| 106 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 107 | _backup_firmware_sha = () |
| 108 | _backup_kernel_sha = dict() |
| 109 | _backup_cgpt_attr = dict() |
| 110 | _backup_gbb_flags = None |
| 111 | _backup_dev_mode = None |
| 112 | |
| 113 | # Class level variable, keep track the states of one time setup. |
| 114 | # This variable is preserved across tests which inherit this class. |
| 115 | _global_setup_done = { |
| 116 | 'gbb_flags': False, |
| 117 | 'reimage': False, |
| 118 | 'usb_check': False, |
| 119 | } |
| 120 | |
| 121 | @classmethod |
| 122 | def check_setup_done(cls, label): |
| 123 | """Check if the given setup is done. |
| 124 | |
| 125 | @param label: The label of the setup. |
| 126 | """ |
| 127 | return cls._global_setup_done[label] |
| 128 | |
| 129 | @classmethod |
| 130 | def mark_setup_done(cls, label): |
| 131 | """Mark the given setup done. |
| 132 | |
| 133 | @param label: The label of the setup. |
| 134 | """ |
| 135 | cls._global_setup_done[label] = True |
| 136 | |
| 137 | @classmethod |
| 138 | def unmark_setup_done(cls, label): |
| 139 | """Mark the given setup not done. |
| 140 | |
| 141 | @param label: The label of the setup. |
| 142 | """ |
| 143 | cls._global_setup_done[label] = False |
| 144 | |
| 145 | def initialize(self, host, cmdline_args, ec_wp=None): |
| 146 | super(FirmwareTest, self).initialize(host) |
| 147 | self.run_id = str(uuid.uuid4()) |
| 148 | logging.info('FirmwareTest initialize begin (id=%s)', self.run_id) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 149 | # Parse arguments from command line |
| 150 | args = {} |
| 151 | self.power_control = host.POWER_CONTROL_RPM |
| 152 | for arg in cmdline_args: |
| 153 | match = re.search("^(\w+)=(.+)", arg) |
| 154 | if match: |
| 155 | args[match.group(1)] = match.group(2) |
| 156 | if 'power_control' in args: |
| 157 | self.power_control = args['power_control'] |
| 158 | if self.power_control not in host.POWER_CONTROL_VALID_ARGS: |
| 159 | raise error.TestError('Valid values for --args=power_control ' |
| 160 | 'are %s. But you entered wrong argument ' |
| 161 | 'as "%s".' |
| 162 | % (host.POWER_CONTROL_VALID_ARGS, |
| 163 | self.power_control)) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 164 | |
| 165 | self.faft_config = FAFTConfig( |
| 166 | self.faft_client.system.get_platform_name()) |
Tom Wai-Hong Tam | 0cc9a4f | 2015-05-02 05:12:39 +0800 | [diff] [blame] | 167 | self.checkers = FAFTCheckers(self) |
Tom Wai-Hong Tam | f2de4de | 2015-05-02 02:48:08 +0800 | [diff] [blame] | 168 | self.switcher = ModeSwitcher(self) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 169 | |
| 170 | if self.faft_config.chrome_ec: |
| 171 | self.ec = chrome_ec.ChromeEC(self.servo) |
| 172 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 173 | self._setup_uart_capture() |
| 174 | self._setup_servo_log() |
| 175 | self._record_system_info() |
Daisuke Nojiri | 682a6d6 | 2014-11-21 09:59:32 -0800 | [diff] [blame] | 176 | self.fw_vboot2 = self.faft_client.system.get_fw_vboot2() |
| 177 | logging.info('vboot version: %d', 2 if self.fw_vboot2 else 1) |
| 178 | if self.fw_vboot2: |
| 179 | self.faft_client.system.set_fw_try_next('A') |
| 180 | if self.faft_client.system.get_crossystem_value('mainfw_act') == 'B': |
| 181 | logging.info('mainfw_act is B. rebooting to set it A') |
| 182 | self.reboot_warm() |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 183 | self._setup_gbb_flags() |
| 184 | self._stop_service('update-engine') |
Duncan Laurie | 10eb618 | 2014-10-07 15:39:05 -0700 | [diff] [blame] | 185 | self._create_faft_lockfile() |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 186 | self._setup_ec_write_protect(ec_wp) |
Yusuf Mohsinally | 1b7a48b | 2014-05-12 19:25:35 -0700 | [diff] [blame] | 187 | # See chromium:239034 regarding needing this sync. |
Yusuf Mohsinally | 1bacc96 | 2014-08-14 11:37:32 -0700 | [diff] [blame] | 188 | self.blocking_sync() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 189 | logging.info('FirmwareTest initialize done (id=%s)', self.run_id) |
| 190 | |
| 191 | def cleanup(self): |
| 192 | """Autotest cleanup function.""" |
| 193 | # Unset state checker in case it's set by subclass |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 194 | logging.info('FirmwareTest cleaning up (id=%s)', self.run_id) |
| 195 | try: |
| 196 | self.faft_client.system.is_available() |
| 197 | except: |
| 198 | # Remote is not responding. Revive DUT so that subsequent tests |
| 199 | # don't fail. |
| 200 | self._restore_routine_from_timeout() |
Tom Wai-Hong Tam | 0cc9a4f | 2015-05-02 05:12:39 +0800 | [diff] [blame] | 201 | self.switcher.restore_mode() |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 202 | self._restore_ec_write_protect() |
| 203 | self._restore_gbb_flags() |
| 204 | self._start_service('update-engine') |
Duncan Laurie | 10eb618 | 2014-10-07 15:39:05 -0700 | [diff] [blame] | 205 | self._remove_faft_lockfile() |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 206 | self._record_servo_log() |
| 207 | self._record_faft_client_log() |
| 208 | self._cleanup_uart_capture() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 209 | super(FirmwareTest, self).cleanup() |
| 210 | logging.info('FirmwareTest cleanup done (id=%s)', self.run_id) |
| 211 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 212 | def _record_system_info(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 213 | """Record some critical system info to the attr keyval. |
| 214 | |
| 215 | This info is used by generate_test_report and local_dash later. |
| 216 | """ |
| 217 | self.write_attr_keyval({ |
| 218 | 'fw_version': self.faft_client.ec.get_version(), |
| 219 | 'hwid': self.faft_client.system.get_crossystem_value('hwid'), |
| 220 | 'fwid': self.faft_client.system.get_crossystem_value('fwid'), |
| 221 | }) |
| 222 | |
| 223 | def invalidate_firmware_setup(self): |
| 224 | """Invalidate all firmware related setup state. |
| 225 | |
| 226 | This method is called when the firmware is re-flashed. It resets all |
| 227 | firmware related setup states so that the next test setup properly |
| 228 | again. |
| 229 | """ |
| 230 | self.unmark_setup_done('gbb_flags') |
| 231 | |
| 232 | def _retrieve_recovery_reason_from_trap(self): |
| 233 | """Try to retrieve the recovery reason from a trapped recovery screen. |
| 234 | |
| 235 | @return: The recovery_reason, 0 if any error. |
| 236 | """ |
| 237 | recovery_reason = 0 |
| 238 | logging.info('Try to retrieve recovery reason...') |
| 239 | if self.servo.get_usbkey_direction() == 'dut': |
| 240 | self.wait_fw_screen_and_plug_usb() |
| 241 | else: |
| 242 | self.servo.switch_usbkey('dut') |
| 243 | |
| 244 | try: |
| 245 | self.wait_for_client(install_deps=True) |
| 246 | lines = self.faft_client.system.run_shell_command_get_output( |
| 247 | 'crossystem recovery_reason') |
| 248 | recovery_reason = int(lines[0]) |
| 249 | logging.info('Got the recovery reason %d.', recovery_reason) |
| 250 | except ConnectionError: |
| 251 | logging.error('Failed to get the recovery reason due to connection ' |
| 252 | 'error.') |
| 253 | return recovery_reason |
| 254 | |
| 255 | def _reset_client(self): |
| 256 | """Reset client to a workable state. |
| 257 | |
| 258 | This method is called when the client is not responsive. It may be |
| 259 | caused by the following cases: |
| 260 | - halt on a firmware screen without timeout, e.g. REC_INSERT screen; |
| 261 | - corrupted firmware; |
| 262 | - corrutped OS image. |
| 263 | """ |
| 264 | # DUT may halt on a firmware screen. Try cold reboot. |
| 265 | logging.info('Try cold reboot...') |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 266 | self.switcher.mode_aware_reboot(reboot_type='cold', |
| 267 | sync_before_boot=False, |
| 268 | wait_for_dut_up=False) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 269 | self.wait_for_client_offline() |
| 270 | self.wait_dev_screen_and_ctrl_d() |
| 271 | try: |
| 272 | self.wait_for_client() |
| 273 | return |
| 274 | except ConnectionError: |
| 275 | logging.warn('Cold reboot doesn\'t help, still connection error.') |
| 276 | |
| 277 | # DUT may be broken by a corrupted firmware. Restore firmware. |
| 278 | # We assume the recovery boot still works fine. Since the recovery |
| 279 | # code is in RO region and all FAFT tests don't change the RO region |
| 280 | # except GBB. |
| 281 | if self.is_firmware_saved(): |
| 282 | self._ensure_client_in_recovery() |
| 283 | logging.info('Try restore the original firmware...') |
| 284 | if self.is_firmware_changed(): |
| 285 | try: |
| 286 | self.restore_firmware() |
| 287 | return |
| 288 | except ConnectionError: |
| 289 | logging.warn('Restoring firmware doesn\'t help, still ' |
| 290 | 'connection error.') |
| 291 | |
| 292 | # Perhaps it's kernel that's broken. Let's try restoring it. |
| 293 | if self.is_kernel_saved(): |
| 294 | self._ensure_client_in_recovery() |
| 295 | logging.info('Try restore the original kernel...') |
| 296 | if self.is_kernel_changed(): |
| 297 | try: |
| 298 | self.restore_kernel() |
| 299 | return |
| 300 | except ConnectionError: |
| 301 | logging.warn('Restoring kernel doesn\'t help, still ' |
| 302 | 'connection error.') |
| 303 | |
| 304 | # DUT may be broken by a corrupted OS image. Restore OS image. |
| 305 | self._ensure_client_in_recovery() |
| 306 | logging.info('Try restore the OS image...') |
| 307 | self.faft_client.system.run_shell_command('chromeos-install --yes') |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 308 | self.switcher.mode_aware_reboot(wait_for_dut_up=False) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 309 | self.wait_for_client_offline() |
| 310 | self.wait_dev_screen_and_ctrl_d() |
| 311 | try: |
| 312 | self.wait_for_client(install_deps=True) |
| 313 | logging.info('Successfully restore OS image.') |
| 314 | return |
| 315 | except ConnectionError: |
| 316 | logging.warn('Restoring OS image doesn\'t help, still connection ' |
| 317 | 'error.') |
| 318 | |
| 319 | def _ensure_client_in_recovery(self): |
| 320 | """Ensure client in recovery boot; reboot into it if necessary. |
| 321 | |
| 322 | @raise TestError: if failed to boot the USB image. |
| 323 | """ |
| 324 | logging.info('Try boot into USB image...') |
Tom Wai-Hong Tam | da6c6ba | 2015-05-02 05:32:41 +0800 | [diff] [blame] | 325 | self.switcher.reboot_to_mode(to_mode='rec', wait_for_dut_up=False) |
Tom Wai-Hong Tam | f2de4de | 2015-05-02 02:48:08 +0800 | [diff] [blame] | 326 | self.servo.switch_usbkey('host') |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 327 | self.wait_fw_screen_and_plug_usb() |
| 328 | try: |
| 329 | self.wait_for_client(install_deps=True) |
| 330 | except ConnectionError: |
| 331 | raise error.TestError('Failed to boot the USB image.') |
| 332 | |
Yusuf Mohsinally | 64ee3a7 | 2014-06-26 10:24:27 -0700 | [diff] [blame] | 333 | def _restore_routine_from_timeout(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 334 | """A routine to try to restore the system from a timeout error. |
| 335 | |
| 336 | This method is called when FAFT failed to connect DUT after reboot. |
| 337 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 338 | @raise TestFail: This exception is already raised, with a decription |
| 339 | why it failed. |
| 340 | """ |
| 341 | # DUT is disconnected. Capture the UART output for debug. |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 342 | self._record_uart_capture() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 343 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 344 | # TODO(waihong@chromium.org): Implement replugging the Ethernet to |
| 345 | # identify if it is a network flaky. |
| 346 | |
| 347 | recovery_reason = self._retrieve_recovery_reason_from_trap() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 348 | |
| 349 | # Reset client to a workable state. |
| 350 | self._reset_client() |
| 351 | |
| 352 | # Raise the proper TestFail exception. |
Yusuf Mohsinally | 64ee3a7 | 2014-06-26 10:24:27 -0700 | [diff] [blame] | 353 | if recovery_reason: |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 354 | raise error.TestFail('Trapped in the recovery screen (reason: %d) ' |
| 355 | 'and timed out' % recovery_reason) |
| 356 | else: |
| 357 | raise error.TestFail('Timed out waiting for DUT reboot') |
| 358 | |
| 359 | def assert_test_image_in_usb_disk(self, usb_dev=None, install_shim=False): |
| 360 | """Assert an USB disk plugged-in on servo and a test image inside. |
| 361 | |
| 362 | @param usb_dev: A string of USB stick path on the host, like '/dev/sdc'. |
| 363 | If None, it is detected automatically. |
| 364 | @param install_shim: True to verify an install shim instead of a test |
| 365 | image. |
| 366 | @raise TestError: if USB disk not detected or not a test (install shim) |
| 367 | image. |
| 368 | """ |
| 369 | if self.check_setup_done('usb_check'): |
| 370 | return |
| 371 | if usb_dev: |
| 372 | assert self.servo.get_usbkey_direction() == 'host' |
| 373 | else: |
| 374 | self.servo.switch_usbkey('host') |
| 375 | usb_dev = self.servo.probe_host_usb_dev() |
| 376 | if not usb_dev: |
| 377 | raise error.TestError( |
| 378 | 'An USB disk should be plugged in the servo board.') |
| 379 | |
| 380 | rootfs = '%s%s' % (usb_dev, self._ROOTFS_PARTITION_NUMBER) |
| 381 | logging.info('usb dev is %s', usb_dev) |
| 382 | tmpd = self.servo.system_output('mktemp -d -t usbcheck.XXXX') |
| 383 | self.servo.system('mount -o ro %s %s' % (rootfs, tmpd)) |
| 384 | |
Julius Werner | dc535df | 2015-02-26 16:42:38 -0800 | [diff] [blame] | 385 | try: |
| 386 | if install_shim: |
| 387 | dir_list = self.servo.system_output('ls -a %s' % |
| 388 | os.path.join(tmpd, 'root')) |
| 389 | if '.factory_installer' not in dir_list: |
| 390 | raise error.TestError( |
| 391 | 'USB stick in servo is not a factory install shim') |
| 392 | else: |
| 393 | usb_lsb = self.servo.system_output('cat %s' % |
| 394 | os.path.join(tmpd, 'etc/lsb-release')) |
| 395 | logging.debug('Dumping lsb-release on USB stick:\n%s', usb_lsb) |
| 396 | dut_lsb = '\n'.join(self.faft_client.system. |
| 397 | run_shell_command_get_output('cat /etc/lsb-release')) |
| 398 | logging.debug('Dumping lsb-release on DUT:\n%s', dut_lsb) |
Julius Werner | 2a26faf | 2015-03-03 14:34:34 -0800 | [diff] [blame] | 399 | if not re.search(r'RELEASE_DESCRIPTION=.*(T|t)est', usb_lsb): |
Julius Werner | dc535df | 2015-02-26 16:42:38 -0800 | [diff] [blame] | 400 | raise error.TestError('USB stick in servo is no test image') |
| 401 | usb_board = re.search(r'BOARD=(.*)', usb_lsb).group(1) |
| 402 | dut_board = re.search(r'BOARD=(.*)', dut_lsb).group(1) |
| 403 | if usb_board != dut_board: |
| 404 | raise error.TestError('USB stick in servo contains a %s ' |
| 405 | 'image, but DUT is a %s' % (usb_board, dut_board)) |
| 406 | finally: |
| 407 | for cmd in ('umount %s' % rootfs, 'sync', 'rm -rf %s' % tmpd): |
| 408 | self.servo.system(cmd) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 409 | |
| 410 | self.mark_setup_done('usb_check') |
| 411 | |
| 412 | def setup_usbkey(self, usbkey, host=None, install_shim=False): |
| 413 | """Setup the USB disk for the test. |
| 414 | |
| 415 | It checks the setup of USB disk and a valid ChromeOS test image inside. |
| 416 | It also muxes the USB disk to either the host or DUT by request. |
| 417 | |
| 418 | @param usbkey: True if the USB disk is required for the test, False if |
| 419 | not required. |
| 420 | @param host: Optional, True to mux the USB disk to host, False to mux it |
| 421 | to DUT, default to do nothing. |
| 422 | @param install_shim: True to verify an install shim instead of a test |
| 423 | image. |
| 424 | """ |
| 425 | if usbkey: |
| 426 | self.assert_test_image_in_usb_disk(install_shim=install_shim) |
| 427 | elif host is None: |
| 428 | # USB disk is not required for the test. Better to mux it to host. |
| 429 | host = True |
| 430 | |
| 431 | if host is True: |
| 432 | self.servo.switch_usbkey('host') |
| 433 | elif host is False: |
| 434 | self.servo.switch_usbkey('dut') |
| 435 | |
| 436 | def get_usbdisk_path_on_dut(self): |
| 437 | """Get the path of the USB disk device plugged-in the servo on DUT. |
| 438 | |
| 439 | Returns: |
| 440 | A string representing USB disk path, like '/dev/sdb', or None if |
| 441 | no USB disk is found. |
| 442 | """ |
| 443 | cmd = 'ls -d /dev/s*[a-z]' |
| 444 | original_value = self.servo.get_usbkey_direction() |
| 445 | |
| 446 | # Make the dut unable to see the USB disk. |
| 447 | self.servo.switch_usbkey('off') |
| 448 | no_usb_set = set( |
| 449 | self.faft_client.system.run_shell_command_get_output(cmd)) |
| 450 | |
| 451 | # Make the dut able to see the USB disk. |
| 452 | self.servo.switch_usbkey('dut') |
| 453 | time.sleep(self.faft_config.between_usb_plug) |
| 454 | has_usb_set = set( |
| 455 | self.faft_client.system.run_shell_command_get_output(cmd)) |
| 456 | |
| 457 | # Back to its original value. |
| 458 | if original_value != self.servo.get_usbkey_direction(): |
| 459 | self.servo.switch_usbkey(original_value) |
| 460 | |
| 461 | diff_set = has_usb_set - no_usb_set |
| 462 | if len(diff_set) == 1: |
| 463 | return diff_set.pop() |
| 464 | else: |
| 465 | return None |
| 466 | |
Duncan Laurie | 10eb618 | 2014-10-07 15:39:05 -0700 | [diff] [blame] | 467 | def _create_faft_lockfile(self): |
| 468 | """Creates the FAFT lockfile.""" |
| 469 | logging.info('Creating FAFT lockfile...') |
| 470 | command = 'touch %s' % (self.lockfile) |
| 471 | self.faft_client.system.run_shell_command(command) |
| 472 | |
| 473 | def _remove_faft_lockfile(self): |
| 474 | """Removes the FAFT lockfile.""" |
| 475 | logging.info('Removing FAFT lockfile...') |
| 476 | command = 'rm -f %s' % (self.lockfile) |
| 477 | self.faft_client.system.run_shell_command(command) |
| 478 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 479 | def _stop_service(self, service): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 480 | """Stops a upstart service on the client. |
| 481 | |
| 482 | @param service: The name of the upstart service. |
| 483 | """ |
| 484 | logging.info('Stopping %s...', service) |
| 485 | command = 'status %s | grep stop || stop %s' % (service, service) |
| 486 | self.faft_client.system.run_shell_command(command) |
| 487 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 488 | def _start_service(self, service): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 489 | """Starts a upstart service on the client. |
| 490 | |
| 491 | @param service: The name of the upstart service. |
| 492 | """ |
| 493 | logging.info('Starting %s...', service) |
| 494 | command = 'status %s | grep start || start %s' % (service, service) |
| 495 | self.faft_client.system.run_shell_command(command) |
| 496 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 497 | def _write_gbb_flags(self, new_flags): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 498 | """Write the GBB flags to the current firmware. |
| 499 | |
| 500 | @param new_flags: The flags to write. |
| 501 | """ |
| 502 | gbb_flags = self.faft_client.bios.get_gbb_flags() |
| 503 | if gbb_flags == new_flags: |
| 504 | return |
| 505 | logging.info('Changing GBB flags from 0x%x to 0x%x.', |
| 506 | gbb_flags, new_flags) |
| 507 | self.faft_client.system.run_shell_command( |
| 508 | '/usr/share/vboot/bin/set_gbb_flags.sh 0x%x' % new_flags) |
| 509 | self.faft_client.bios.reload() |
| 510 | # If changing FORCE_DEV_SWITCH_ON flag, reboot to get a clear state |
| 511 | if ((gbb_flags ^ new_flags) & vboot.GBB_FLAG_FORCE_DEV_SWITCH_ON): |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 512 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 513 | |
| 514 | def clear_set_gbb_flags(self, clear_mask, set_mask): |
| 515 | """Clear and set the GBB flags in the current flashrom. |
| 516 | |
| 517 | @param clear_mask: A mask of flags to be cleared. |
| 518 | @param set_mask: A mask of flags to be set. |
| 519 | """ |
| 520 | gbb_flags = self.faft_client.bios.get_gbb_flags() |
| 521 | new_flags = gbb_flags & ctypes.c_uint32(~clear_mask).value | set_mask |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 522 | self._write_gbb_flags(new_flags) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 523 | |
| 524 | def check_ec_capability(self, required_cap=None, suppress_warning=False): |
| 525 | """Check if current platform has required EC capabilities. |
| 526 | |
| 527 | @param required_cap: A list containing required EC capabilities. Pass in |
| 528 | None to only check for presence of Chrome EC. |
| 529 | @param suppress_warning: True to suppress any warning messages. |
| 530 | @return: True if requirements are met. Otherwise, False. |
| 531 | """ |
| 532 | if not self.faft_config.chrome_ec: |
| 533 | if not suppress_warning: |
| 534 | logging.warn('Requires Chrome EC to run this test.') |
| 535 | return False |
| 536 | |
| 537 | if not required_cap: |
| 538 | return True |
| 539 | |
| 540 | for cap in required_cap: |
| 541 | if cap not in self.faft_config.ec_capability: |
| 542 | if not suppress_warning: |
| 543 | logging.warn('Requires EC capability "%s" to run this ' |
| 544 | 'test.', cap) |
| 545 | return False |
| 546 | |
| 547 | return True |
| 548 | |
| 549 | def check_root_part_on_non_recovery(self, part): |
| 550 | """Check the partition number of root device and on normal/dev boot. |
| 551 | |
| 552 | @param part: A string of partition number, e.g.'3'. |
| 553 | @return: True if the root device matched and on normal/dev boot; |
| 554 | otherwise, False. |
| 555 | """ |
| 556 | return self.checkers.root_part_checker(part) and \ |
| 557 | self.checkers.crossystem_checker({ |
| 558 | 'mainfw_type': ('normal', 'developer'), |
| 559 | }) |
| 560 | |
| 561 | def _join_part(self, dev, part): |
| 562 | """Return a concatenated string of device and partition number. |
| 563 | |
| 564 | @param dev: A string of device, e.g.'/dev/sda'. |
| 565 | @param part: A string of partition number, e.g.'3'. |
| 566 | @return: A concatenated string of device and partition number, |
| 567 | e.g.'/dev/sda3'. |
| 568 | |
| 569 | >>> seq = FirmwareTest() |
| 570 | >>> seq._join_part('/dev/sda', '3') |
| 571 | '/dev/sda3' |
| 572 | >>> seq._join_part('/dev/mmcblk0', '2') |
| 573 | '/dev/mmcblk0p2' |
| 574 | """ |
| 575 | if 'mmcblk' in dev: |
| 576 | return dev + 'p' + part |
| 577 | else: |
| 578 | return dev + part |
| 579 | |
| 580 | def copy_kernel_and_rootfs(self, from_part, to_part): |
| 581 | """Copy kernel and rootfs from from_part to to_part. |
| 582 | |
| 583 | @param from_part: A string of partition number to be copied from. |
| 584 | @param to_part: A string of partition number to be copied to. |
| 585 | """ |
| 586 | root_dev = self.faft_client.system.get_root_dev() |
| 587 | logging.info('Copying kernel from %s to %s. Please wait...', |
| 588 | from_part, to_part) |
| 589 | self.faft_client.system.run_shell_command('dd if=%s of=%s bs=4M' % |
| 590 | (self._join_part(root_dev, self.KERNEL_MAP[from_part]), |
| 591 | self._join_part(root_dev, self.KERNEL_MAP[to_part]))) |
| 592 | logging.info('Copying rootfs from %s to %s. Please wait...', |
| 593 | from_part, to_part) |
| 594 | self.faft_client.system.run_shell_command('dd if=%s of=%s bs=4M' % |
| 595 | (self._join_part(root_dev, self.ROOTFS_MAP[from_part]), |
| 596 | self._join_part(root_dev, self.ROOTFS_MAP[to_part]))) |
| 597 | |
| 598 | def ensure_kernel_boot(self, part): |
| 599 | """Ensure the request kernel boot. |
| 600 | |
| 601 | If not, it duplicates the current kernel to the requested kernel |
| 602 | and sets the requested higher priority to ensure it boot. |
| 603 | |
| 604 | @param part: A string of kernel partition number or 'a'/'b'. |
| 605 | """ |
| 606 | if not self.checkers.root_part_checker(part): |
| 607 | if self.faft_client.kernel.diff_a_b(): |
| 608 | self.copy_kernel_and_rootfs( |
| 609 | from_part=self.OTHER_KERNEL_MAP[part], |
| 610 | to_part=part) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 611 | self.reset_and_prioritize_kernel(part) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 612 | |
| 613 | def set_hardware_write_protect(self, enable): |
| 614 | """Set hardware write protect pin. |
| 615 | |
| 616 | @param enable: True if asserting write protect pin. Otherwise, False. |
| 617 | """ |
| 618 | self.servo.set('fw_wp_vref', self.faft_config.wp_voltage) |
| 619 | self.servo.set('fw_wp_en', 'on') |
| 620 | self.servo.set('fw_wp', 'on' if enable else 'off') |
| 621 | |
| 622 | def set_ec_write_protect_and_reboot(self, enable): |
| 623 | """Set EC write protect status and reboot to take effect. |
| 624 | |
| 625 | The write protect state is only activated if both hardware write |
| 626 | protect pin is asserted and software write protect flag is set. |
| 627 | This method asserts/deasserts hardware write protect pin first, and |
| 628 | set corresponding EC software write protect flag. |
| 629 | |
| 630 | If the device uses non-Chrome EC, set the software write protect via |
| 631 | flashrom. |
| 632 | |
| 633 | If the device uses Chrome EC, a reboot is required for write protect |
| 634 | to take effect. Since the software write protect flag cannot be unset |
| 635 | if hardware write protect pin is asserted, we need to deasserted the |
| 636 | pin first if we are deactivating write protect. Similarly, a reboot |
| 637 | is required before we can modify the software flag. |
| 638 | |
| 639 | @param enable: True if activating EC write protect. Otherwise, False. |
| 640 | """ |
| 641 | self.set_hardware_write_protect(enable) |
| 642 | if self.faft_config.chrome_ec: |
| 643 | self.set_chrome_ec_write_protect_and_reboot(enable) |
| 644 | else: |
| 645 | self.faft_client.ec.set_write_protect(enable) |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 646 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 647 | |
| 648 | def set_chrome_ec_write_protect_and_reboot(self, enable): |
| 649 | """Set Chrome EC write protect status and reboot to take effect. |
| 650 | |
| 651 | @param enable: True if activating EC write protect. Otherwise, False. |
| 652 | """ |
| 653 | if enable: |
| 654 | # Set write protect flag and reboot to take effect. |
| 655 | self.ec.set_flash_write_protect(enable) |
| 656 | self.sync_and_ec_reboot() |
| 657 | else: |
| 658 | # Reboot after deasserting hardware write protect pin to deactivate |
| 659 | # write protect. And then remove software write protect flag. |
| 660 | self.sync_and_ec_reboot() |
| 661 | self.ec.set_flash_write_protect(enable) |
| 662 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 663 | def _setup_ec_write_protect(self, ec_wp): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 664 | """Setup for EC write-protection. |
| 665 | |
| 666 | It makes sure the EC in the requested write-protection state. If not, it |
| 667 | flips the state. Flipping the write-protection requires DUT reboot. |
| 668 | |
| 669 | @param ec_wp: True to request EC write-protected; False to request EC |
| 670 | not write-protected; None to do nothing. |
| 671 | """ |
| 672 | if ec_wp is None: |
| 673 | self._old_ec_wp = None |
| 674 | return |
| 675 | self._old_ec_wp = self.checkers.crossystem_checker({'wpsw_boot': '1'}) |
| 676 | if ec_wp != self._old_ec_wp: |
| 677 | logging.info('The test required EC is %swrite-protected. Reboot ' |
| 678 | 'and flip the state.', '' if ec_wp else 'not ') |
Yusuf Mohsinally | cae7902 | 2014-05-14 14:44:39 -0700 | [diff] [blame] | 679 | self.do_reboot_action((self.set_ec_write_protect_and_reboot, ec_wp)) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 680 | self.wait_dev_screen_and_ctrl_d() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 681 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 682 | def _restore_ec_write_protect(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 683 | """Restore the original EC write-protection.""" |
| 684 | if (not hasattr(self, '_old_ec_wp')) or (self._old_ec_wp is None): |
| 685 | return |
| 686 | if not self.checkers.crossystem_checker( |
| 687 | {'wpsw_boot': '1' if self._old_ec_wp else '0'}): |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 688 | logging.info('Restore original EC write protection and reboot.') |
Yusuf Mohsinally | cae7902 | 2014-05-14 14:44:39 -0700 | [diff] [blame] | 689 | self.do_reboot_action((self.set_ec_write_protect_and_reboot, |
| 690 | self._old_ec_wp)) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 691 | self.wait_dev_screen_and_ctrl_d() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 692 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 693 | def wait_dev_screen_and_ctrl_d(self): |
| 694 | """Wait for firmware warning screen and press Ctrl-D.""" |
| 695 | time.sleep(self.faft_config.dev_screen) |
Tom Wai-Hong Tam | 408c995 | 2015-04-30 00:37:36 +0800 | [diff] [blame] | 696 | self.servo.ctrl_d() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 697 | |
| 698 | def wait_fw_screen_and_ctrl_d(self): |
| 699 | """Wait for firmware warning screen and press Ctrl-D.""" |
| 700 | time.sleep(self.faft_config.firmware_screen) |
Tom Wai-Hong Tam | 408c995 | 2015-04-30 00:37:36 +0800 | [diff] [blame] | 701 | self.servo.ctrl_d() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 702 | |
| 703 | def wait_fw_screen_and_ctrl_u(self): |
| 704 | """Wait for firmware warning screen and press Ctrl-U.""" |
| 705 | time.sleep(self.faft_config.firmware_screen) |
Tom Wai-Hong Tam | 408c995 | 2015-04-30 00:37:36 +0800 | [diff] [blame] | 706 | self.servo.ctrl_u() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 707 | |
| 708 | def wait_fw_screen_and_trigger_recovery(self, need_dev_transition=False): |
| 709 | """Wait for firmware warning screen and trigger recovery boot. |
| 710 | |
| 711 | @param need_dev_transition: True when needs dev mode transition, only |
| 712 | for Alex/ZGB. |
| 713 | """ |
| 714 | time.sleep(self.faft_config.firmware_screen) |
| 715 | |
| 716 | # Pressing Enter for too long triggers a second key press. |
| 717 | # Let's press it without delay |
Tom Wai-Hong Tam | 408c995 | 2015-04-30 00:37:36 +0800 | [diff] [blame] | 718 | self.servo.enter_key(press_secs=0) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 719 | |
| 720 | # For Alex/ZGB, there is a dev warning screen in text mode. |
| 721 | # Skip it by pressing Ctrl-D. |
| 722 | if need_dev_transition: |
| 723 | time.sleep(self.faft_config.legacy_text_screen) |
Tom Wai-Hong Tam | 408c995 | 2015-04-30 00:37:36 +0800 | [diff] [blame] | 724 | self.servo.ctrl_d() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 725 | |
| 726 | def wait_fw_screen_and_unplug_usb(self): |
| 727 | """Wait for firmware warning screen and then unplug the servo USB.""" |
| 728 | time.sleep(self.faft_config.load_usb) |
| 729 | self.servo.switch_usbkey('host') |
| 730 | time.sleep(self.faft_config.between_usb_plug) |
| 731 | |
| 732 | def wait_fw_screen_and_plug_usb(self): |
| 733 | """Wait for firmware warning screen and then unplug and plug the USB.""" |
| 734 | self.wait_fw_screen_and_unplug_usb() |
| 735 | self.servo.switch_usbkey('dut') |
| 736 | |
| 737 | def wait_fw_screen_and_press_power(self): |
| 738 | """Wait for firmware warning screen and press power button.""" |
| 739 | time.sleep(self.faft_config.firmware_screen) |
| 740 | # While the firmware screen, the power button probing loop sleeps |
| 741 | # 0.25 second on every scan. Use the normal delay (1.2 second) for |
| 742 | # power press. |
| 743 | self.servo.power_normal_press() |
| 744 | |
| 745 | def wait_longer_fw_screen_and_press_power(self): |
| 746 | """Wait for firmware screen without timeout and press power button.""" |
| 747 | time.sleep(self.faft_config.dev_screen_timeout) |
| 748 | self.wait_fw_screen_and_press_power() |
| 749 | |
| 750 | def wait_fw_screen_and_close_lid(self): |
| 751 | """Wait for firmware warning screen and close lid.""" |
| 752 | time.sleep(self.faft_config.firmware_screen) |
| 753 | self.servo.lid_close() |
| 754 | |
| 755 | def wait_longer_fw_screen_and_close_lid(self): |
| 756 | """Wait for firmware screen without timeout and close lid.""" |
| 757 | time.sleep(self.faft_config.firmware_screen) |
| 758 | self.wait_fw_screen_and_close_lid() |
| 759 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 760 | def _setup_uart_capture(self): |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 761 | """Setup the CPU/EC/PD UART capture.""" |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 762 | self.cpu_uart_file = os.path.join(self.resultsdir, 'cpu_uart.txt') |
| 763 | self.servo.set('cpu_uart_capture', 'on') |
| 764 | self.ec_uart_file = None |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 765 | self.usbpd_uart_file = None |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 766 | if self.faft_config.chrome_ec: |
| 767 | try: |
| 768 | self.servo.set('ec_uart_capture', 'on') |
| 769 | self.ec_uart_file = os.path.join(self.resultsdir, 'ec_uart.txt') |
| 770 | except error.TestFail as e: |
| 771 | if 'No control named' in str(e): |
| 772 | logging.warn('The servod is too old that ec_uart_capture ' |
| 773 | 'not supported.') |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 774 | # Log separate PD console if supported |
| 775 | if self.check_ec_capability(['usbpd_uart'], suppress_warning=True): |
| 776 | try: |
| 777 | self.servo.set('usbpd_uart_capture', 'on') |
| 778 | self.usbpd_uart_file = os.path.join(self.resultsdir, |
| 779 | 'usbpd_uart.txt') |
| 780 | except error.TestFail as e: |
| 781 | if 'No control named' in str(e): |
| 782 | logging.warn('The servod is too old that ' |
| 783 | 'usbpd_uart_capture is not supported.') |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 784 | else: |
| 785 | logging.info('Not a Google EC, cannot capture ec console output.') |
| 786 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 787 | def _record_uart_capture(self): |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 788 | """Record the CPU/EC/PD UART output stream to files.""" |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 789 | if self.cpu_uart_file: |
| 790 | with open(self.cpu_uart_file, 'a') as f: |
| 791 | f.write(ast.literal_eval(self.servo.get('cpu_uart_stream'))) |
| 792 | if self.ec_uart_file and self.faft_config.chrome_ec: |
| 793 | with open(self.ec_uart_file, 'a') as f: |
| 794 | f.write(ast.literal_eval(self.servo.get('ec_uart_stream'))) |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 795 | if (self.usbpd_uart_file and self.faft_config.chrome_ec and |
| 796 | self.check_ec_capability(['usbpd_uart'], suppress_warning=True)): |
| 797 | with open(self.usbpd_uart_file, 'a') as f: |
| 798 | f.write(ast.literal_eval(self.servo.get('usbpd_uart_stream'))) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 799 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 800 | def _cleanup_uart_capture(self): |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 801 | """Cleanup the CPU/EC/PD UART capture.""" |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 802 | # Flush the remaining UART output. |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 803 | self._record_uart_capture() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 804 | self.servo.set('cpu_uart_capture', 'off') |
| 805 | if self.ec_uart_file and self.faft_config.chrome_ec: |
| 806 | self.servo.set('ec_uart_capture', 'off') |
Duncan Laurie | af61c1f | 2014-10-07 15:35:18 -0700 | [diff] [blame] | 807 | if (self.usbpd_uart_file and self.faft_config.chrome_ec and |
| 808 | self.check_ec_capability(['usbpd_uart'], suppress_warning=True)): |
| 809 | self.servo.set('usbpd_uart_capture', 'off') |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 810 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 811 | def _fetch_servo_log(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 812 | """Fetch the servo log.""" |
| 813 | cmd = '[ -e %s ] && cat %s || echo NOTFOUND' % ((self._SERVOD_LOG,) * 2) |
| 814 | servo_log = self.servo.system_output(cmd) |
| 815 | return None if servo_log == 'NOTFOUND' else servo_log |
| 816 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 817 | def _setup_servo_log(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 818 | """Setup the servo log capturing.""" |
| 819 | self.servo_log_original_len = -1 |
| 820 | if self.servo.is_localhost(): |
| 821 | # No servo log recorded when servod runs locally. |
| 822 | return |
| 823 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 824 | servo_log = self._fetch_servo_log() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 825 | if servo_log: |
| 826 | self.servo_log_original_len = len(servo_log) |
| 827 | else: |
| 828 | logging.warn('Servo log file not found.') |
| 829 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 830 | def _record_servo_log(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 831 | """Record the servo log to the results directory.""" |
| 832 | if self.servo_log_original_len != -1: |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 833 | servo_log = self._fetch_servo_log() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 834 | servo_log_file = os.path.join(self.resultsdir, 'servod.log') |
| 835 | with open(servo_log_file, 'a') as f: |
| 836 | f.write(servo_log[self.servo_log_original_len:]) |
| 837 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 838 | def _record_faft_client_log(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 839 | """Record the faft client log to the results directory.""" |
| 840 | client_log = self.faft_client.system.dump_log(True) |
| 841 | client_log_file = os.path.join(self.resultsdir, 'faft_client.log') |
| 842 | with open(client_log_file, 'w') as f: |
| 843 | f.write(client_log) |
| 844 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 845 | def _setup_gbb_flags(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 846 | """Setup the GBB flags for FAFT test.""" |
| 847 | if self.faft_config.gbb_version < 1.1: |
| 848 | logging.info('Skip modifying GBB on versions older than 1.1.') |
| 849 | return |
| 850 | |
| 851 | if self.check_setup_done('gbb_flags'): |
| 852 | return |
| 853 | |
| 854 | self._backup_gbb_flags = self.faft_client.bios.get_gbb_flags() |
| 855 | |
| 856 | logging.info('Set proper GBB flags for test.') |
| 857 | self.clear_set_gbb_flags(vboot.GBB_FLAG_DEV_SCREEN_SHORT_DELAY | |
| 858 | vboot.GBB_FLAG_FORCE_DEV_SWITCH_ON | |
| 859 | vboot.GBB_FLAG_FORCE_DEV_BOOT_USB | |
| 860 | vboot.GBB_FLAG_DISABLE_FW_ROLLBACK_CHECK, |
| 861 | vboot.GBB_FLAG_ENTER_TRIGGERS_TONORM | |
| 862 | vboot.GBB_FLAG_FAFT_KEY_OVERIDE) |
| 863 | self.mark_setup_done('gbb_flags') |
| 864 | |
| 865 | def drop_backup_gbb_flags(self): |
| 866 | """Drops the backup GBB flags. |
| 867 | |
| 868 | This can be used when a test intends to permanently change GBB flags. |
| 869 | """ |
| 870 | self._backup_gbb_flags = None |
| 871 | |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 872 | def _restore_gbb_flags(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 873 | """Restore GBB flags to their original state.""" |
| 874 | if not self._backup_gbb_flags: |
| 875 | return |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 876 | self._write_gbb_flags(self._backup_gbb_flags) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 877 | self.unmark_setup_done('gbb_flags') |
| 878 | |
| 879 | def setup_tried_fwb(self, tried_fwb): |
| 880 | """Setup for fw B tried state. |
| 881 | |
| 882 | It makes sure the system in the requested fw B tried state. If not, it |
| 883 | tries to do so. |
| 884 | |
| 885 | @param tried_fwb: True if requested in tried_fwb=1; |
| 886 | False if tried_fwb=0. |
| 887 | """ |
| 888 | if tried_fwb: |
| 889 | if not self.checkers.crossystem_checker({'tried_fwb': '1'}): |
| 890 | logging.info( |
| 891 | 'Firmware is not booted with tried_fwb. Reboot into it.') |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 892 | self.faft_client.system.set_try_fw_b() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 893 | else: |
| 894 | if not self.checkers.crossystem_checker({'tried_fwb': '0'}): |
| 895 | logging.info( |
| 896 | 'Firmware is booted with tried_fwb. Reboot to clear.') |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 897 | |
| 898 | def power_on(self): |
| 899 | """Switch DUT AC power on.""" |
| 900 | self._client.power_on(self.power_control) |
| 901 | |
| 902 | def power_off(self): |
| 903 | """Switch DUT AC power off.""" |
| 904 | self._client.power_off(self.power_control) |
| 905 | |
| 906 | def power_cycle(self): |
| 907 | """Power cycle DUT AC power.""" |
| 908 | self._client.power_cycle(self.power_control) |
| 909 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 910 | def setup_rw_boot(self, section='a'): |
| 911 | """Make sure firmware is in RW-boot mode. |
| 912 | |
| 913 | If the given firmware section is in RO-boot mode, turn off the RO-boot |
| 914 | flag and reboot DUT into RW-boot mode. |
| 915 | |
| 916 | @param section: A firmware section, either 'a' or 'b'. |
| 917 | """ |
| 918 | flags = self.faft_client.bios.get_preamble_flags(section) |
| 919 | if flags & vboot.PREAMBLE_USE_RO_NORMAL: |
| 920 | flags = flags ^ vboot.PREAMBLE_USE_RO_NORMAL |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 921 | self.faft_client.bios.set_preamble_flags(section, flags) |
| 922 | self.reboot_warm() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 923 | |
| 924 | def setup_kernel(self, part): |
| 925 | """Setup for kernel test. |
| 926 | |
| 927 | It makes sure both kernel A and B bootable and the current boot is |
| 928 | the requested kernel part. |
| 929 | |
| 930 | @param part: A string of kernel partition number or 'a'/'b'. |
| 931 | """ |
| 932 | self.ensure_kernel_boot(part) |
| 933 | logging.info('Checking the integrity of kernel B and rootfs B...') |
| 934 | if (self.faft_client.kernel.diff_a_b() or |
| 935 | not self.faft_client.rootfs.verify_rootfs('B')): |
| 936 | logging.info('Copying kernel and rootfs from A to B...') |
| 937 | self.copy_kernel_and_rootfs(from_part=part, |
| 938 | to_part=self.OTHER_KERNEL_MAP[part]) |
| 939 | self.reset_and_prioritize_kernel(part) |
| 940 | |
| 941 | def reset_and_prioritize_kernel(self, part): |
| 942 | """Make the requested partition highest priority. |
| 943 | |
| 944 | This function also reset kerenl A and B to bootable. |
| 945 | |
| 946 | @param part: A string of partition number to be prioritized. |
| 947 | """ |
| 948 | root_dev = self.faft_client.system.get_root_dev() |
| 949 | # Reset kernel A and B to bootable. |
| 950 | self.faft_client.system.run_shell_command( |
| 951 | 'cgpt add -i%s -P1 -S1 -T0 %s' % (self.KERNEL_MAP['a'], root_dev)) |
| 952 | self.faft_client.system.run_shell_command( |
| 953 | 'cgpt add -i%s -P1 -S1 -T0 %s' % (self.KERNEL_MAP['b'], root_dev)) |
| 954 | # Set kernel part highest priority. |
| 955 | self.faft_client.system.run_shell_command('cgpt prioritize -i%s %s' % |
| 956 | (self.KERNEL_MAP[part], root_dev)) |
| 957 | |
Yusuf Mohsinally | 1bacc96 | 2014-08-14 11:37:32 -0700 | [diff] [blame] | 958 | def blocking_sync(self): |
| 959 | """Run a blocking sync command.""" |
| 960 | # The double calls to sync fakes a blocking call |
| 961 | # since the first call returns before the flush |
| 962 | # is complete, but the second will wait for the |
| 963 | # first to finish. |
| 964 | self.faft_client.system.run_shell_command('sync') |
| 965 | self.faft_client.system.run_shell_command('sync') |
| 966 | |
Ryan Lin | 5bee610 | 2014-09-16 13:17:02 -0700 | [diff] [blame] | 967 | # sync only sends SYNCHRONIZE_CACHE but doesn't |
Steve Fung | b575242 | 2015-01-09 16:45:32 -0800 | [diff] [blame] | 968 | # check the status. For mmc devices, use `mmc |
| 969 | # status get` command to send an empty command to |
| 970 | # wait for the disk to be available again. For |
| 971 | # other devices, hdparm sends TUR to check if |
Ryan Lin | 5bee610 | 2014-09-16 13:17:02 -0700 | [diff] [blame] | 972 | # a device is ready for transfer operation. |
| 973 | root_dev = self.faft_client.system.get_root_dev() |
Steve Fung | b575242 | 2015-01-09 16:45:32 -0800 | [diff] [blame] | 974 | if 'mmcblk' in root_dev: |
| 975 | self.faft_client.system.run_shell_command('mmc status get %s' % |
| 976 | root_dev) |
| 977 | else: |
| 978 | self.faft_client.system.run_shell_command('hdparm -f %s' % root_dev) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 979 | |
| 980 | ################################################ |
| 981 | # Reboot APIs |
| 982 | |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 983 | def reboot_warm(self, sync_before_boot=True, |
| 984 | wait_for_dut_up=True, ctrl_d=False): |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 985 | """ |
| 986 | Perform a warm reboot. |
| 987 | |
| 988 | This is the highest level function that most users will need. |
| 989 | It performs a sync, triggers a reboot and waits for kernel to boot. |
| 990 | |
| 991 | @param sync_before_boot: bool, sync to disk before booting. |
| 992 | @param wait_for_dut_up: bool, wait for dut to boot before returning. |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 993 | @param ctrl_d: bool, press ctrl-D at dev screen. |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 994 | """ |
| 995 | if sync_before_boot: |
Yusuf Mohsinally | 1bacc96 | 2014-08-14 11:37:32 -0700 | [diff] [blame] | 996 | self.blocking_sync() |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 997 | self.reboot_warm_trigger() |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 998 | if ctrl_d: |
| 999 | self.wait_dev_screen_and_ctrl_d() |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1000 | if wait_for_dut_up: |
| 1001 | self.wait_for_client_offline() |
| 1002 | self.wait_for_kernel_up() |
| 1003 | |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 1004 | def reboot_cold(self, sync_before_boot=True, |
| 1005 | wait_for_dut_up=True, ctrl_d=False): |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1006 | """ |
| 1007 | Perform a cold reboot. |
| 1008 | |
| 1009 | This is the highest level function that most users will need. |
| 1010 | It performs a sync, triggers a reboot and waits for kernel to boot. |
| 1011 | |
| 1012 | @param sync_before_boot: bool, sync to disk before booting. |
| 1013 | @param wait_for_dut_up: bool, wait for dut to boot before returning. |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 1014 | @param ctrl_d: bool, press ctrl-D at dev screen. |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1015 | """ |
| 1016 | if sync_before_boot: |
Yusuf Mohsinally | 1bacc96 | 2014-08-14 11:37:32 -0700 | [diff] [blame] | 1017 | self.blocking_sync() |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1018 | self.reboot_cold_trigger() |
Vic Yang | 9887e6f | 2014-06-03 11:11:30 -0700 | [diff] [blame] | 1019 | if ctrl_d: |
| 1020 | self.wait_dev_screen_and_ctrl_d() |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1021 | if wait_for_dut_up: |
| 1022 | self.wait_for_client_offline() |
| 1023 | self.wait_for_kernel_up() |
| 1024 | |
| 1025 | def do_reboot_action(self, func): |
| 1026 | """ |
| 1027 | Helper function that wraps the reboot function so that we check if the |
| 1028 | DUT went down. |
| 1029 | |
| 1030 | @param func: function to trigger the reboot. |
| 1031 | """ |
| 1032 | logging.info("-[FAFT]-[ start do_reboot_action ]----------") |
| 1033 | boot_id = self.get_bootid() |
| 1034 | self._call_action(func) |
| 1035 | self.wait_for_client_offline(orig_boot_id=boot_id) |
| 1036 | logging.info("-[FAFT]-[ end do_reboot_action ]------------") |
| 1037 | |
| 1038 | def wait_for_kernel_up(self, install_deps=False): |
| 1039 | """ |
| 1040 | Helper function that waits for the device to boot up to kernel. |
| 1041 | |
| 1042 | @param install_deps: bool, install deps after boot. |
| 1043 | """ |
| 1044 | logging.info("-[FAFT]-[ start wait_for_kernel_up ]---") |
Duncan Laurie | b3abc43 | 2014-10-07 15:48:15 -0700 | [diff] [blame] | 1045 | # Wait for the system to respond to ping before attempting ssh |
| 1046 | if not self._client.ping_wait_up(90): |
| 1047 | logging.warning("-[FAFT]-[ system did not respond to ping ]") |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1048 | try: |
Danny Chan | 1437ad5 | 2014-06-30 13:57:12 -0700 | [diff] [blame] | 1049 | logging.info("Installing deps after boot : %s", install_deps) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1050 | self.wait_for_client(install_deps=install_deps) |
| 1051 | # Stop update-engine as it may change firmware/kernel. |
Yusuf Mohsinally | 8b377eb | 2014-05-12 18:50:03 -0700 | [diff] [blame] | 1052 | self._stop_service('update-engine') |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1053 | except ConnectionError: |
| 1054 | logging.error('wait_for_client() timed out.') |
Yusuf Mohsinally | 64ee3a7 | 2014-06-26 10:24:27 -0700 | [diff] [blame] | 1055 | self._restore_routine_from_timeout() |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1056 | logging.info("-[FAFT]-[ end wait_for_kernel_up ]-----") |
| 1057 | |
| 1058 | def reboot_warm_trigger(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1059 | """Request a warm reboot. |
| 1060 | |
| 1061 | A wrapper for underlying servo warm reset. |
| 1062 | """ |
Tom Wai-Hong Tam | b43a9a5 | 2015-05-05 03:05:02 +0800 | [diff] [blame] | 1063 | self.servo.get_power_state_controller().warm_reset() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1064 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1065 | def reboot_cold_trigger(self): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1066 | """Request a cold reboot. |
| 1067 | |
| 1068 | A wrapper for underlying servo cold reset. |
| 1069 | """ |
J. Richard Barnette | 4b6af0d | 2014-06-05 09:57:20 -0700 | [diff] [blame] | 1070 | self.servo.get_power_state_controller().reset() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1071 | |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1072 | def sync_and_ec_reboot(self, flags=''): |
| 1073 | """Request the client sync and do a EC triggered reboot. |
| 1074 | |
| 1075 | @param flags: Optional, a space-separated string of flags passed to EC |
| 1076 | reboot command, including: |
| 1077 | default: EC soft reboot; |
| 1078 | 'hard': EC cold/hard reboot. |
| 1079 | """ |
Yusuf Mohsinally | 1bacc96 | 2014-08-14 11:37:32 -0700 | [diff] [blame] | 1080 | self.blocking_sync() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1081 | self.ec.reboot(flags) |
| 1082 | time.sleep(self.faft_config.ec_boot_to_console) |
| 1083 | self.check_lid_and_power_on() |
| 1084 | |
| 1085 | def reboot_with_factory_install_shim(self): |
| 1086 | """Request reboot with factory install shim to reset TPM. |
| 1087 | |
| 1088 | Factory install shim requires dev mode enabled. So this method switches |
| 1089 | firmware to dev mode first and reboot. The client uses factory install |
| 1090 | shim to reset TPM values. |
| 1091 | """ |
| 1092 | # Unplug USB first to avoid the complicated USB autoboot cases. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1093 | is_dev = self.checkers.crossystem_checker({'devsw_boot': '1'}) |
| 1094 | if not is_dev: |
Tom Wai-Hong Tam | da6c6ba | 2015-05-02 05:32:41 +0800 | [diff] [blame] | 1095 | self.switcher.reboot_to_mode(to_mode='dev', wait_for_dut_up=False) |
Tom Wai-Hong Tam | f2de4de | 2015-05-02 02:48:08 +0800 | [diff] [blame] | 1096 | self.switcher.reboot_to_mode(to_mode='rec') |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1097 | self.wait_fw_screen_and_plug_usb() |
| 1098 | time.sleep(self.faft_config.install_shim_done) |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 1099 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1100 | |
| 1101 | def full_power_off_and_on(self): |
| 1102 | """Shutdown the device by pressing power button and power on again.""" |
Danny Chan | 101b0b2 | 2014-11-06 10:08:54 -0800 | [diff] [blame] | 1103 | boot_id = self.get_bootid() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1104 | # Press power button to trigger Chrome OS normal shutdown process. |
| 1105 | # We use a customized delay since the normal-press 1.2s is not enough. |
| 1106 | self.servo.power_key(self.faft_config.hold_pwr_button) |
Danny Chan | 101b0b2 | 2014-11-06 10:08:54 -0800 | [diff] [blame] | 1107 | # device can take 44-51 seconds to restart, |
| 1108 | # add buffer from the default timeout of 60 seconds. |
| 1109 | self.wait_for_client_offline(timeout=100, orig_boot_id=boot_id) |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1110 | time.sleep(self.faft_config.shutdown) |
| 1111 | # Short press power button to boot DUT again. |
| 1112 | self.servo.power_short_press() |
| 1113 | |
| 1114 | def check_lid_and_power_on(self): |
| 1115 | """ |
| 1116 | On devices with EC software sync, system powers on after EC reboots if |
| 1117 | lid is open. Otherwise, the EC shuts down CPU after about 3 seconds. |
| 1118 | This method checks lid switch state and presses power button if |
| 1119 | necessary. |
| 1120 | """ |
| 1121 | if self.servo.get("lid_open") == "no": |
| 1122 | time.sleep(self.faft_config.software_sync) |
| 1123 | self.servo.power_short_press() |
| 1124 | |
| 1125 | def _modify_usb_kernel(self, usb_dev, from_magic, to_magic): |
| 1126 | """Modify the kernel header magic in USB stick. |
| 1127 | |
| 1128 | The kernel header magic is the first 8-byte of kernel partition. |
| 1129 | We modify it to make it fail on kernel verification check. |
| 1130 | |
| 1131 | @param usb_dev: A string of USB stick path on the host, like '/dev/sdc'. |
| 1132 | @param from_magic: A string of magic which we change it from. |
| 1133 | @param to_magic: A string of magic which we change it to. |
| 1134 | @raise TestError: if failed to change magic. |
| 1135 | """ |
| 1136 | assert len(from_magic) == 8 |
| 1137 | assert len(to_magic) == 8 |
| 1138 | # USB image only contains one kernel. |
| 1139 | kernel_part = self._join_part(usb_dev, self.KERNEL_MAP['a']) |
| 1140 | read_cmd = "sudo dd if=%s bs=8 count=1 2>/dev/null" % kernel_part |
| 1141 | current_magic = self.servo.system_output(read_cmd) |
| 1142 | if current_magic == to_magic: |
| 1143 | logging.info("The kernel magic is already %s.", current_magic) |
| 1144 | return |
| 1145 | if current_magic != from_magic: |
| 1146 | raise error.TestError("Invalid kernel image on USB: wrong magic.") |
| 1147 | |
| 1148 | logging.info('Modify the kernel magic in USB, from %s to %s.', |
| 1149 | from_magic, to_magic) |
| 1150 | write_cmd = ("echo -n '%s' | sudo dd of=%s oflag=sync conv=notrunc " |
| 1151 | " 2>/dev/null" % (to_magic, kernel_part)) |
| 1152 | self.servo.system(write_cmd) |
| 1153 | |
| 1154 | if self.servo.system_output(read_cmd) != to_magic: |
| 1155 | raise error.TestError("Failed to write new magic.") |
| 1156 | |
| 1157 | def corrupt_usb_kernel(self, usb_dev): |
| 1158 | """Corrupt USB kernel by modifying its magic from CHROMEOS to CORRUPTD. |
| 1159 | |
| 1160 | @param usb_dev: A string of USB stick path on the host, like '/dev/sdc'. |
| 1161 | """ |
| 1162 | self._modify_usb_kernel(usb_dev, self.CHROMEOS_MAGIC, |
| 1163 | self.CORRUPTED_MAGIC) |
| 1164 | |
| 1165 | def restore_usb_kernel(self, usb_dev): |
| 1166 | """Restore USB kernel by modifying its magic from CORRUPTD to CHROMEOS. |
| 1167 | |
| 1168 | @param usb_dev: A string of USB stick path on the host, like '/dev/sdc'. |
| 1169 | """ |
| 1170 | self._modify_usb_kernel(usb_dev, self.CORRUPTED_MAGIC, |
| 1171 | self.CHROMEOS_MAGIC) |
| 1172 | |
| 1173 | def _call_action(self, action_tuple, check_status=False): |
| 1174 | """Call the action function with/without arguments. |
| 1175 | |
| 1176 | @param action_tuple: A function, or a tuple (function, args, error_msg), |
| 1177 | in which, args and error_msg are optional. args is |
| 1178 | either a value or a tuple if multiple arguments. |
| 1179 | This can also be a list containing multiple |
| 1180 | function or tuple. In this case, these actions are |
| 1181 | called in sequence. |
| 1182 | @param check_status: Check the return value of action function. If not |
| 1183 | succeed, raises a TestFail exception. |
| 1184 | @return: The result value of the action function. |
| 1185 | @raise TestError: An error when the action function is not callable. |
| 1186 | @raise TestFail: When check_status=True, action function not succeed. |
| 1187 | """ |
| 1188 | if isinstance(action_tuple, list): |
| 1189 | return all([self._call_action(action, check_status=check_status) |
| 1190 | for action in action_tuple]) |
| 1191 | |
| 1192 | action = action_tuple |
| 1193 | args = () |
| 1194 | error_msg = 'Not succeed' |
| 1195 | if isinstance(action_tuple, tuple): |
| 1196 | action = action_tuple[0] |
| 1197 | if len(action_tuple) >= 2: |
| 1198 | args = action_tuple[1] |
| 1199 | if not isinstance(args, tuple): |
| 1200 | args = (args,) |
| 1201 | if len(action_tuple) >= 3: |
| 1202 | error_msg = action_tuple[2] |
| 1203 | |
| 1204 | if action is None: |
| 1205 | return |
| 1206 | |
| 1207 | if not callable(action): |
| 1208 | raise error.TestError('action is not callable!') |
| 1209 | |
| 1210 | info_msg = 'calling %s' % str(action) |
| 1211 | if args: |
| 1212 | info_msg += ' with args %s' % str(args) |
| 1213 | logging.info(info_msg) |
| 1214 | ret = action(*args) |
| 1215 | |
| 1216 | if check_status and not ret: |
| 1217 | raise error.TestFail('%s: %s returning %s' % |
| 1218 | (error_msg, info_msg, str(ret))) |
| 1219 | return ret |
| 1220 | |
| 1221 | def run_shutdown_process(self, shutdown_action, pre_power_action=None, |
| 1222 | post_power_action=None, shutdown_timeout=None): |
| 1223 | """Run shutdown_action(), which makes DUT shutdown, and power it on. |
| 1224 | |
| 1225 | @param shutdown_action: function which makes DUT shutdown, like |
| 1226 | pressing power key. |
| 1227 | @param pre_power_action: function which is called before next power on. |
| 1228 | @param post_power_action: function which is called after next power on. |
| 1229 | @param shutdown_timeout: a timeout to confirm DUT shutdown. |
| 1230 | @raise TestFail: if the shutdown_action() failed to turn DUT off. |
| 1231 | """ |
| 1232 | self._call_action(shutdown_action) |
| 1233 | logging.info('Wait to ensure DUT shut down...') |
| 1234 | try: |
| 1235 | if shutdown_timeout is None: |
| 1236 | shutdown_timeout = self.faft_config.shutdown_timeout |
| 1237 | self.wait_for_client(timeout=shutdown_timeout) |
| 1238 | raise error.TestFail( |
| 1239 | 'Should shut the device down after calling %s.' % |
| 1240 | str(shutdown_action)) |
| 1241 | except ConnectionError: |
| 1242 | logging.info( |
| 1243 | 'DUT is surely shutdown. We are going to power it on again...') |
| 1244 | |
| 1245 | if pre_power_action: |
| 1246 | self._call_action(pre_power_action) |
| 1247 | self.servo.power_short_press() |
| 1248 | if post_power_action: |
| 1249 | self._call_action(post_power_action) |
| 1250 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1251 | def get_bootid(self, retry=3): |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1252 | """ |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1253 | Return the bootid. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1254 | """ |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1255 | boot_id = None |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1256 | while retry: |
| 1257 | try: |
| 1258 | boot_id = self._client.get_boot_id() |
| 1259 | break |
| 1260 | except error.AutoservRunError: |
| 1261 | retry -= 1 |
| 1262 | if retry: |
| 1263 | logging.info('Retry to get boot_id...') |
| 1264 | else: |
| 1265 | logging.warning('Failed to get boot_id.') |
| 1266 | logging.info('boot_id: %s', boot_id) |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1267 | return boot_id |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1268 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1269 | def check_state(self, func): |
| 1270 | """ |
| 1271 | Wrapper around _call_action with check_status set to True. This is a |
| 1272 | helper function to be used by tests and is currently implemented by |
| 1273 | calling _call_action with check_status=True. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1274 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1275 | TODO: This function's arguments need to be made more stringent. And |
| 1276 | its functionality should be moved over to check functions directly in |
| 1277 | the future. |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1278 | |
Yusuf Mohsinally | ab1b5fc | 2014-05-08 12:46:10 -0700 | [diff] [blame] | 1279 | @param func: A function, or a tuple (function, args, error_msg), |
| 1280 | in which, args and error_msg are optional. args is |
| 1281 | either a value or a tuple if multiple arguments. |
| 1282 | This can also be a list containing multiple |
| 1283 | function or tuple. In this case, these actions are |
| 1284 | called in sequence. |
| 1285 | @return: The result value of the action function. |
| 1286 | @raise TestFail: If the function does notsucceed. |
| 1287 | """ |
| 1288 | logging.info("-[FAFT]-[ start stepstate_checker ]----------") |
| 1289 | self._call_action(func, check_status=True) |
| 1290 | logging.info("-[FAFT]-[ end state_checker ]----------------") |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1291 | |
| 1292 | def get_current_firmware_sha(self): |
| 1293 | """Get current firmware sha of body and vblock. |
| 1294 | |
| 1295 | @return: Current firmware sha follows the order ( |
| 1296 | vblock_a_sha, body_a_sha, vblock_b_sha, body_b_sha) |
| 1297 | """ |
| 1298 | current_firmware_sha = (self.faft_client.bios.get_sig_sha('a'), |
| 1299 | self.faft_client.bios.get_body_sha('a'), |
| 1300 | self.faft_client.bios.get_sig_sha('b'), |
| 1301 | self.faft_client.bios.get_body_sha('b')) |
| 1302 | if not all(current_firmware_sha): |
| 1303 | raise error.TestError('Failed to get firmware sha.') |
| 1304 | return current_firmware_sha |
| 1305 | |
| 1306 | def is_firmware_changed(self): |
| 1307 | """Check if the current firmware changed, by comparing its SHA. |
| 1308 | |
| 1309 | @return: True if it is changed, otherwise Flase. |
| 1310 | """ |
| 1311 | # Device may not be rebooted after test. |
| 1312 | self.faft_client.bios.reload() |
| 1313 | |
| 1314 | current_sha = self.get_current_firmware_sha() |
| 1315 | |
| 1316 | if current_sha == self._backup_firmware_sha: |
| 1317 | return False |
| 1318 | else: |
| 1319 | corrupt_VBOOTA = (current_sha[0] != self._backup_firmware_sha[0]) |
| 1320 | corrupt_FVMAIN = (current_sha[1] != self._backup_firmware_sha[1]) |
| 1321 | corrupt_VBOOTB = (current_sha[2] != self._backup_firmware_sha[2]) |
| 1322 | corrupt_FVMAINB = (current_sha[3] != self._backup_firmware_sha[3]) |
| 1323 | logging.info("Firmware changed:") |
| 1324 | logging.info('VBOOTA is changed: %s', corrupt_VBOOTA) |
| 1325 | logging.info('VBOOTB is changed: %s', corrupt_VBOOTB) |
| 1326 | logging.info('FVMAIN is changed: %s', corrupt_FVMAIN) |
| 1327 | logging.info('FVMAINB is changed: %s', corrupt_FVMAINB) |
| 1328 | return True |
| 1329 | |
| 1330 | def backup_firmware(self, suffix='.original'): |
| 1331 | """Backup firmware to file, and then send it to host. |
| 1332 | |
| 1333 | @param suffix: a string appended to backup file name |
| 1334 | """ |
| 1335 | remote_temp_dir = self.faft_client.system.create_temp_dir() |
| 1336 | self.faft_client.bios.dump_whole(os.path.join(remote_temp_dir, 'bios')) |
| 1337 | self._client.get_file(os.path.join(remote_temp_dir, 'bios'), |
| 1338 | os.path.join(self.resultsdir, 'bios' + suffix)) |
| 1339 | |
| 1340 | self._backup_firmware_sha = self.get_current_firmware_sha() |
| 1341 | logging.info('Backup firmware stored in %s with suffix %s', |
| 1342 | self.resultsdir, suffix) |
| 1343 | |
| 1344 | def is_firmware_saved(self): |
| 1345 | """Check if a firmware saved (called backup_firmware before). |
| 1346 | |
| 1347 | @return: True if the firmware is backuped; otherwise False. |
| 1348 | """ |
| 1349 | return self._backup_firmware_sha != () |
| 1350 | |
| 1351 | def clear_saved_firmware(self): |
| 1352 | """Clear the firmware saved by the method backup_firmware.""" |
| 1353 | self._backup_firmware_sha = () |
| 1354 | |
| 1355 | def restore_firmware(self, suffix='.original'): |
| 1356 | """Restore firmware from host in resultsdir. |
| 1357 | |
| 1358 | @param suffix: a string appended to backup file name |
| 1359 | """ |
| 1360 | if not self.is_firmware_changed(): |
| 1361 | return |
| 1362 | |
| 1363 | # Backup current corrupted firmware. |
| 1364 | self.backup_firmware(suffix='.corrupt') |
| 1365 | |
| 1366 | # Restore firmware. |
| 1367 | remote_temp_dir = self.faft_client.system.create_temp_dir() |
| 1368 | self._client.send_file(os.path.join(self.resultsdir, 'bios' + suffix), |
| 1369 | os.path.join(remote_temp_dir, 'bios')) |
| 1370 | |
| 1371 | self.faft_client.bios.write_whole( |
| 1372 | os.path.join(remote_temp_dir, 'bios')) |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 1373 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1374 | logging.info('Successfully restore firmware.') |
| 1375 | |
| 1376 | def setup_firmwareupdate_shellball(self, shellball=None): |
| 1377 | """Deside a shellball to use in firmware update test. |
| 1378 | |
| 1379 | Check if there is a given shellball, and it is a shell script. Then, |
| 1380 | send it to the remote host. Otherwise, use |
| 1381 | /usr/sbin/chromeos-firmwareupdate. |
| 1382 | |
| 1383 | @param shellball: path of a shellball or default to None. |
| 1384 | |
| 1385 | @return: Path of shellball in remote host. If use default shellball, |
| 1386 | reutrn None. |
| 1387 | """ |
| 1388 | updater_path = None |
| 1389 | if shellball: |
| 1390 | # Determine the firmware file is a shellball or a raw binary. |
| 1391 | is_shellball = (utils.system_output("file %s" % shellball).find( |
| 1392 | "shell script") != -1) |
| 1393 | if is_shellball: |
| 1394 | logging.info('Device will update firmware with shellball %s', |
| 1395 | shellball) |
| 1396 | temp_dir = self.faft_client.system.create_temp_dir( |
| 1397 | 'shellball_') |
| 1398 | temp_shellball = os.path.join(temp_dir, 'updater.sh') |
| 1399 | self._client.send_file(shellball, temp_shellball) |
| 1400 | updater_path = temp_shellball |
| 1401 | else: |
| 1402 | raise error.TestFail( |
| 1403 | 'The given shellball is not a shell script.') |
| 1404 | return updater_path |
| 1405 | |
| 1406 | def is_kernel_changed(self): |
| 1407 | """Check if the current kernel is changed, by comparing its SHA1 hash. |
| 1408 | |
| 1409 | @return: True if it is changed; otherwise, False. |
| 1410 | """ |
| 1411 | changed = False |
| 1412 | for p in ('A', 'B'): |
| 1413 | backup_sha = self._backup_kernel_sha.get(p, None) |
| 1414 | current_sha = self.faft_client.kernel.get_sha(p) |
| 1415 | if backup_sha != current_sha: |
| 1416 | changed = True |
| 1417 | logging.info('Kernel %s is changed', p) |
| 1418 | return changed |
| 1419 | |
| 1420 | def backup_kernel(self, suffix='.original'): |
| 1421 | """Backup kernel to files, and the send them to host. |
| 1422 | |
| 1423 | @param suffix: a string appended to backup file name. |
| 1424 | """ |
| 1425 | remote_temp_dir = self.faft_client.system.create_temp_dir() |
| 1426 | for p in ('A', 'B'): |
| 1427 | remote_path = os.path.join(remote_temp_dir, 'kernel_%s' % p) |
| 1428 | self.faft_client.kernel.dump(p, remote_path) |
| 1429 | self._client.get_file( |
| 1430 | remote_path, |
| 1431 | os.path.join(self.resultsdir, 'kernel_%s%s' % (p, suffix))) |
| 1432 | self._backup_kernel_sha[p] = self.faft_client.kernel.get_sha(p) |
| 1433 | logging.info('Backup kernel stored in %s with suffix %s', |
| 1434 | self.resultsdir, suffix) |
| 1435 | |
| 1436 | def is_kernel_saved(self): |
| 1437 | """Check if kernel images are saved (backup_kernel called before). |
| 1438 | |
| 1439 | @return: True if the kernel is saved; otherwise, False. |
| 1440 | """ |
| 1441 | return len(self._backup_kernel_sha) != 0 |
| 1442 | |
| 1443 | def clear_saved_kernel(self): |
| 1444 | """Clear the kernel saved by backup_kernel().""" |
| 1445 | self._backup_kernel_sha = dict() |
| 1446 | |
| 1447 | def restore_kernel(self, suffix='.original'): |
| 1448 | """Restore kernel from host in resultsdir. |
| 1449 | |
| 1450 | @param suffix: a string appended to backup file name. |
| 1451 | """ |
| 1452 | if not self.is_kernel_changed(): |
| 1453 | return |
| 1454 | |
| 1455 | # Backup current corrupted kernel. |
| 1456 | self.backup_kernel(suffix='.corrupt') |
| 1457 | |
| 1458 | # Restore kernel. |
| 1459 | remote_temp_dir = self.faft_client.system.create_temp_dir() |
| 1460 | for p in ('A', 'B'): |
| 1461 | remote_path = os.path.join(remote_temp_dir, 'kernel_%s' % p) |
| 1462 | self._client.send_file( |
| 1463 | os.path.join(self.resultsdir, 'kernel_%s%s' % (p, suffix)), |
| 1464 | remote_path) |
| 1465 | self.faft_client.kernel.write(p, remote_path) |
| 1466 | |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 1467 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1468 | logging.info('Successfully restored kernel.') |
| 1469 | |
| 1470 | def backup_cgpt_attributes(self): |
| 1471 | """Backup CGPT partition table attributes.""" |
| 1472 | self._backup_cgpt_attr = self.faft_client.cgpt.get_attributes() |
| 1473 | |
| 1474 | def restore_cgpt_attributes(self): |
| 1475 | """Restore CGPT partition table attributes.""" |
| 1476 | current_table = self.faft_client.cgpt.get_attributes() |
| 1477 | if current_table == self._backup_cgpt_attr: |
| 1478 | return |
| 1479 | logging.info('CGPT table is changed. Original: %r. Current: %r.', |
| 1480 | self._backup_cgpt_attr, |
| 1481 | current_table) |
| 1482 | self.faft_client.cgpt.set_attributes(self._backup_cgpt_attr) |
| 1483 | |
Tom Wai-Hong Tam | a704f18 | 2015-05-06 06:12:55 +0800 | [diff] [blame^] | 1484 | self.switcher.mode_aware_reboot() |
Yusuf Mohsinally | 05c3c55 | 2014-05-07 23:56:42 -0700 | [diff] [blame] | 1485 | logging.info('Successfully restored CGPT table.') |
Shelley Chen | 3edea98 | 2014-12-30 14:54:21 -0800 | [diff] [blame] | 1486 | |
| 1487 | def try_fwb(self, count=0): |
| 1488 | """set to try booting FWB count # times |
| 1489 | |
| 1490 | Wrapper to set fwb_tries for vboot1 and fw_try_count,fw_try_next for |
| 1491 | vboot2 |
| 1492 | |
| 1493 | @param count: an integer specifying value to program into |
| 1494 | fwb_tries(vb1)/fw_try_next(vb2) |
| 1495 | """ |
| 1496 | if self.fw_vboot2: |
| 1497 | self.faft_client.system.set_fw_try_next('B', count) |
| 1498 | else: |
| 1499 | # vboot1: we need to boot into fwb at least once |
| 1500 | if not count: |
| 1501 | count = count + 1 |
| 1502 | self.faft_client.system.set_try_fw_b(count) |
| 1503 | |