blob: 3a205de1078cbd92a817ea38b9ae6911577ff44a [file] [log] [blame]
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001# Copyright (c) 2011 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
Tom Wai-Hong Tamfda76e22012-08-08 17:19:10 +08005import ctypes
Vic Yangb4e3e742012-06-02 13:17:38 +08006import fdpexpect
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08007import logging
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +08008import os
Vic Yangb4e3e742012-06-02 13:17:38 +08009import pexpect
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080010import re
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +080011import sys
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +080012import tempfile
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080013import time
14import xmlrpclib
15
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +080016from autotest_lib.client.bin import utils
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080017from autotest_lib.client.common_lib import error
Vic Yangebd6de62012-06-26 14:25:57 +080018from autotest_lib.server.cros.faft_client_attribute import FAFTClientAttribute
Tom Wai-Hong Tam22b77302011-11-03 13:03:48 +080019from autotest_lib.server.cros.servo_test import ServoTest
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +080020from autotest_lib.site_utils import lab_test
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080021
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +080022dirname = os.path.dirname(sys.modules[__name__].__file__)
23autotest_dir = os.path.abspath(os.path.join(dirname, "..", ".."))
24cros_dir = os.path.join(autotest_dir, "..", "..", "..", "..")
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080025
26class FAFTSequence(ServoTest):
27 """
28 The base class of Fully Automated Firmware Test Sequence.
29
30 Many firmware tests require several reboot cycles and verify the resulted
31 system states. To do that, an Autotest test case should detailly handle
32 every action on each step. It makes the test case hard to read and many
33 duplicated code. The base class FAFTSequence is to solve this problem.
34
35 The actions of one reboot cycle is defined in a dict, namely FAFT_STEP.
36 There are four functions in the FAFT_STEP dict:
37 state_checker: a function to check the current is valid or not,
38 returning True if valid, otherwise, False to break the whole
39 test sequence.
40 userspace_action: a function to describe the action ran in userspace.
Tom Wai-Hong Tamb21b6b42012-07-26 10:46:30 +080041 reboot_action: a function to do reboot, default: sync_and_warm_reboot.
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080042 firmware_action: a function to describe the action ran after reboot.
43
Tom Wai-Hong Tam7c17ff22011-10-26 09:44:09 +080044 And configurations:
45 install_deps_after_boot: if True, install the Autotest dependency after
46 boot; otherwise, do nothing. It is for the cases of recovery mode
47 test. The test boots a USB/SD image instead of an internal image.
48 The previous installed Autotest dependency on the internal image
49 is lost. So need to install it again.
50
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080051 The default FAFT_STEP checks nothing in state_checker and does nothing in
Tom Wai-Hong Tamf1e34972011-11-02 17:07:04 +080052 userspace_action and firmware_action. Its reboot_action is a hardware
53 reboot. You can change the default FAFT_STEP by calling
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080054 self.register_faft_template(FAFT_STEP).
55
56 A FAFT test case consists of several FAFT_STEP's, namely FAFT_SEQUENCE.
57 FAFT_SEQUENCE is an array of FAFT_STEP's. Any missing fields on FAFT_STEP
58 fall back to default.
59
60 In the run_once(), it should register and run FAFT_SEQUENCE like:
61 def run_once(self):
62 self.register_faft_sequence(FAFT_SEQUENCE)
63 self.run_faft_sequnce()
64
65 Note that in the last step, we only run state_checker. The
66 userspace_action, reboot_action, and firmware_action are not executed.
67
68 Attributes:
69 _faft_template: The default FAFT_STEP of each step. The actions would
70 be over-written if the registered FAFT_SEQUENCE is valid.
71 _faft_sequence: The registered FAFT_SEQUENCE.
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +080072 _customized_ctrl_d_key_command: The customized Ctrl-D key command
73 instead of sending key via servo board.
74 _customized_enter_key_command: The customized Enter key command instead
75 of sending key via servo board.
Tom Wai-Hong Tam9e61e662012-08-01 15:10:07 +080076 _customized_space_key_command: The customized Space key command instead
77 of sending key via servo board.
Tom Wai-Hong Tamac943172012-08-01 10:38:39 +080078 _customized_rec_reboot_command: The customized recovery reboot command
79 instead of sending key combination of Power + Esc + F3 for
80 triggering recovery reboot.
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +080081 _install_image_path: The path of Chrome OS test image to be installed.
Tom Wai-Hong Tam1a3ff742012-01-11 16:36:46 +080082 _firmware_update: Boolean. True if firmware update needed after
83 installing the image.
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +080084 """
85 version = 1
86
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +080087
88 # Mapping of partition number of kernel and rootfs.
89 KERNEL_MAP = {'a':'2', 'b':'4', '2':'2', '4':'4', '3':'2', '5':'4'}
90 ROOTFS_MAP = {'a':'3', 'b':'5', '2':'3', '4':'5', '3':'3', '5':'5'}
91 OTHER_KERNEL_MAP = {'a':'4', 'b':'2', '2':'4', '4':'2', '3':'4', '5':'2'}
92 OTHER_ROOTFS_MAP = {'a':'5', 'b':'3', '2':'5', '4':'3', '3':'5', '5':'3'}
93
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +080094 # Delay between power-on and firmware screen.
Tom Wai-Hong Tam66af37b2012-08-01 10:48:42 +080095 FIRMWARE_SCREEN_DELAY = 10
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +080096 # Delay between passing firmware screen and text mode warning screen.
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +080097 TEXT_SCREEN_DELAY = 20
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +080098 # Delay of loading the USB kernel.
Tom Wai-Hong Tam07278c22012-02-08 16:53:00 +080099 USB_LOAD_DELAY = 10
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +0800100 # Delay between USB plug-out and plug-in.
Tom Wai-Hong Tam9ca742a2011-12-05 15:48:57 +0800101 USB_PLUG_DELAY = 10
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +0800102 # Delay after running the 'sync' command.
Tom Wai-Hong Tam6a863ba2011-12-08 10:13:28 +0800103 SYNC_DELAY = 5
Vic Yang59cac9c2012-05-21 15:28:42 +0800104 # Delay for waiting client to return before EC reboot
105 EC_REBOOT_DELAY = 1
106 # Delay between EC reboot and pressing power button
107 POWER_BTN_DELAY = 0.5
Vic Yangf86728a2012-07-30 10:44:07 +0800108 # Delay of EC software sync hash calculating time
109 SOFTWARE_SYNC_DELAY = 6
Vic Yanga7250662012-08-31 04:00:08 +0800110 # Delay between EC boot and ChromeEC console functional
111 EC_BOOT_DELAY = 0.5
112 # Duration of holding cold_reset to reset device
113 COLD_RESET_DELAY = 0.1
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800114
Tom Wai-Hong Tam51ef2e12012-07-27 15:04:12 +0800115 # The developer screen timeouts fit our spec.
116 DEV_SCREEN_TIMEOUT = 30
117
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800118 CHROMEOS_MAGIC = "CHROMEOS"
119 CORRUPTED_MAGIC = "CORRUPTD"
120
Tom Wai-Hong Tamf954d172011-12-08 17:14:15 +0800121 # Recovery reason codes, copied from:
122 # vboot_reference/firmware/lib/vboot_nvstorage.h
123 # vboot_reference/firmware/lib/vboot_struct.h
124 RECOVERY_REASON = {
125 # Recovery not requested
126 'NOT_REQUESTED': '0', # 0x00
127 # Recovery requested from legacy utility
128 'LEGACY': '1', # 0x01
129 # User manually requested recovery via recovery button
130 'RO_MANUAL': '2', # 0x02
131 # RW firmware failed signature check
132 'RO_INVALID_RW': '3', # 0x03
133 # S3 resume failed
134 'RO_S3_RESUME': '4', # 0x04
135 # TPM error in read-only firmware
136 'RO_TPM_ERROR': '5', # 0x05
137 # Shared data error in read-only firmware
138 'RO_SHARED_DATA': '6', # 0x06
139 # Test error from S3Resume()
140 'RO_TEST_S3': '7', # 0x07
141 # Test error from LoadFirmwareSetup()
142 'RO_TEST_LFS': '8', # 0x08
143 # Test error from LoadFirmware()
144 'RO_TEST_LF': '9', # 0x09
145 # RW firmware failed signature check
146 'RW_NOT_DONE': '16', # 0x10
147 'RW_DEV_MISMATCH': '17', # 0x11
148 'RW_REC_MISMATCH': '18', # 0x12
149 'RW_VERIFY_KEYBLOCK': '19', # 0x13
150 'RW_KEY_ROLLBACK': '20', # 0x14
151 'RW_DATA_KEY_PARSE': '21', # 0x15
152 'RW_VERIFY_PREAMBLE': '22', # 0x16
153 'RW_FW_ROLLBACK': '23', # 0x17
154 'RW_HEADER_VALID': '24', # 0x18
155 'RW_GET_FW_BODY': '25', # 0x19
156 'RW_HASH_WRONG_SIZE': '26', # 0x1A
157 'RW_VERIFY_BODY': '27', # 0x1B
158 'RW_VALID': '28', # 0x1C
159 # Read-only normal path requested by firmware preamble, but
160 # unsupported by firmware.
161 'RW_NO_RO_NORMAL': '29', # 0x1D
162 # Firmware boot failure outside of verified boot
163 'RO_FIRMWARE': '32', # 0x20
164 # Recovery mode TPM initialization requires a system reboot.
165 # The system was already in recovery mode for some other reason
166 # when this happened.
167 'RO_TPM_REBOOT': '33', # 0x21
168 # Unspecified/unknown error in read-only firmware
169 'RO_UNSPECIFIED': '63', # 0x3F
170 # User manually requested recovery by pressing a key at developer
171 # warning screen.
172 'RW_DEV_SCREEN': '65', # 0x41
173 # No OS kernel detected
174 'RW_NO_OS': '66', # 0x42
175 # OS kernel failed signature check
176 'RW_INVALID_OS': '67', # 0x43
177 # TPM error in rewritable firmware
178 'RW_TPM_ERROR': '68', # 0x44
179 # RW firmware in dev mode, but dev switch is off.
180 'RW_DEV_MISMATCH': '69', # 0x45
181 # Shared data error in rewritable firmware
182 'RW_SHARED_DATA': '70', # 0x46
183 # Test error from LoadKernel()
184 'RW_TEST_LK': '71', # 0x47
185 # No bootable disk found
186 'RW_NO_DISK': '72', # 0x48
187 # Unspecified/unknown error in rewritable firmware
188 'RW_UNSPECIFIED': '127', # 0x7F
189 # DM-verity error
190 'KE_DM_VERITY': '129', # 0x81
191 # Unspecified/unknown error in kernel
192 'KE_UNSPECIFIED': '191', # 0xBF
193 # Recovery mode test from user-mode
194 'US_TEST': '193', # 0xC1
195 # Unspecified/unknown error in user-mode
196 'US_UNSPECIFIED': '255', # 0xFF
197 }
198
Tom Wai-Hong Tam1e40fa12012-07-25 16:22:24 +0800199 # GBB flags
200 GBB_FLAG_DEV_SCREEN_SHORT_DELAY = 0x00000001
201 GBB_FLAG_LOAD_OPTION_ROMS = 0x00000002
202 GBB_FLAG_ENABLE_ALTERNATE_OS = 0x00000004
203 GBB_FLAG_FORCE_DEV_SWITCH_ON = 0x00000008
204 GBB_FLAG_FORCE_DEV_BOOT_USB = 0x00000010
205 GBB_FLAG_DISABLE_FW_ROLLBACK_CHECK = 0x00000020
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800206 GBB_FLAG_ENTER_TRIGGERS_TONORM = 0x00000040
Tom Wai-Hong Tam1e40fa12012-07-25 16:22:24 +0800207
Tom Wai-Hong Tam39b93b92012-09-04 16:56:05 +0800208 # VbSharedData flags
209 # Copied from vboot_reference/firmware/include/vboot_struct.h
210 VDAT_FLAG_FWB_TRIED = 0x00000001
211 VDAT_FLAG_KERNEL_KEY_VERIFIED = 0x00000002
212 VDAT_FLAG_LF_DEV_SWITCH_ON = 0x00000004
213 VDAT_FLAG_LF_USE_RO_NORMAL = 0x00000008
214 VDAT_FLAG_BOOT_DEV_SWITCH_ON = 0x00000010
215 VDAT_FLAG_BOOT_REC_SWITCH_ON = 0x00000020
216 VDAT_FLAG_BOOT_FIRMWARE_WP_ENABLED = 0x00000040
217 VDAT_FLAG_BOOT_S3_RESUME = 0x00000100
218 VDAT_FLAG_BOOT_RO_NORMAL_SUPPORT = 0x00000200
219 VDAT_FLAG_HONOR_VIRT_DEV_SWITCH = 0x00000400
220 VDAT_FLAG_EC_SOFTWARE_SYNC = 0x00000800
221 VDAT_FLAG_EC_SLOW_UPDATE = 0x00001000
222
Tom Wai-Hong Tame3c72432012-09-05 10:48:31 +0800223 # Firmware preamble flags
224 PREAMBLE_USE_RO_NORMAL = 0x00000001
225
Tom Wai-Hong Tam109f63c2011-12-08 14:58:27 +0800226 _faft_template = {}
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800227 _faft_sequence = ()
228
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800229 _customized_ctrl_d_key_command = None
230 _customized_enter_key_command = None
Tom Wai-Hong Tam9e61e662012-08-01 15:10:07 +0800231 _customized_space_key_command = None
Tom Wai-Hong Tamac943172012-08-01 10:38:39 +0800232 _customized_rec_reboot_command = None
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800233 _install_image_path = None
Tom Wai-Hong Tam1a3ff742012-01-11 16:36:46 +0800234 _firmware_update = False
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800235
236
237 def initialize(self, host, cmdline_args, use_pyauto=False, use_faft=False):
238 # Parse arguments from command line
239 args = {}
240 for arg in cmdline_args:
241 match = re.search("^(\w+)=(.+)", arg)
242 if match:
243 args[match.group(1)] = match.group(2)
244
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800245 # Keep the arguments which will be used later.
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800246 if 'ctrl_d_cmd' in args:
247 self._customized_ctrl_d_key_command = args['ctrl_d_cmd']
248 logging.info('Customized Ctrl-D key command: %s' %
249 self._customized_ctrl_d_key_command)
250 if 'enter_cmd' in args:
251 self._customized_enter_key_command = args['enter_cmd']
252 logging.info('Customized Enter key command: %s' %
253 self._customized_enter_key_command)
Tom Wai-Hong Tam9e61e662012-08-01 15:10:07 +0800254 if 'space_cmd' in args:
255 self._customized_space_key_command = args['space_cmd']
256 logging.info('Customized Space key command: %s' %
257 self._customized_space_key_command)
Tom Wai-Hong Tamac943172012-08-01 10:38:39 +0800258 if 'rec_reboot_cmd' in args:
259 self._customized_rec_reboot_command = args['rec_reboot_cmd']
260 logging.info('Customized recovery reboot command: %s' %
261 self._customized_rec_reboot_command)
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800262 if 'image' in args:
263 self._install_image_path = args['image']
264 logging.info('Install Chrome OS test image path: %s' %
265 self._install_image_path)
Tom Wai-Hong Tam1a3ff742012-01-11 16:36:46 +0800266 if 'firmware_update' in args and args['firmware_update'].lower() \
267 not in ('0', 'false', 'no'):
268 if self._install_image_path:
269 self._firmware_update = True
270 logging.info('Also update firmware after installing.')
271 else:
272 logging.warning('Firmware update will not not performed '
273 'since no image is specified.')
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800274
275 super(FAFTSequence, self).initialize(host, cmdline_args, use_pyauto,
276 use_faft)
Vic Yangebd6de62012-06-26 14:25:57 +0800277 if use_faft:
278 self.client_attr = FAFTClientAttribute(
279 self.faft_client.get_platform_name())
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800280
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800281
282 def setup(self):
283 """Autotest setup function."""
284 super(FAFTSequence, self).setup()
285 if not self._remote_infos['faft']['used']:
286 raise error.TestError('The use_faft flag should be enabled.')
287 self.register_faft_template({
288 'state_checker': (None),
289 'userspace_action': (None),
Tom Wai-Hong Tamb21b6b42012-07-26 10:46:30 +0800290 'reboot_action': (self.sync_and_warm_reboot),
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800291 'firmware_action': (None)
292 })
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800293 self.clear_set_gbb_flags(self.GBB_FLAG_FORCE_DEV_SWITCH_ON |
294 self.GBB_FLAG_DEV_SCREEN_SHORT_DELAY,
295 self.GBB_FLAG_ENTER_TRIGGERS_TONORM)
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800296 if self._install_image_path:
Tom Wai-Hong Tam1a3ff742012-01-11 16:36:46 +0800297 self.install_test_image(self._install_image_path,
298 self._firmware_update)
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800299
300
301 def cleanup(self):
302 """Autotest cleanup function."""
303 self._faft_sequence = ()
Tom Wai-Hong Tam109f63c2011-12-08 14:58:27 +0800304 self._faft_template = {}
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800305 super(FAFTSequence, self).cleanup()
306
307
Tom Wai-Hong Tam91f49822011-12-28 15:44:15 +0800308 def assert_test_image_in_usb_disk(self, usb_dev=None):
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +0800309 """Assert an USB disk plugged-in on servo and a test image inside.
310
Tom Wai-Hong Tam91f49822011-12-28 15:44:15 +0800311 Args:
312 usb_dev: A string of USB stick path on the host, like '/dev/sdc'.
313 If None, it is detected automatically.
314
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +0800315 Raises:
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +0800316 error.TestError: if USB disk not detected or not a test image.
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +0800317 """
Tom Wai-Hong Tam91f49822011-12-28 15:44:15 +0800318 if usb_dev:
319 assert self.servo.get('usb_mux_sel1') == 'servo_sees_usbkey'
320 else:
Vadim Bendeburycacf29f2012-07-30 17:49:11 -0700321 self.servo.enable_usb_hub(host=True)
Tom Wai-Hong Tam91f49822011-12-28 15:44:15 +0800322 usb_dev = self.servo.probe_host_usb_dev()
323 if not usb_dev:
324 raise error.TestError(
325 'An USB disk should be plugged in the servo board.')
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +0800326
327 tmp_dir = tempfile.mkdtemp()
Tom Wai-Hong Tamb0e80852011-12-07 16:15:06 +0800328 utils.system('sudo mount -r -t ext2 %s3 %s' % (usb_dev, tmp_dir))
Tom Wai-Hong Tame77459e2011-11-03 17:19:46 +0800329 code = utils.system(
330 'grep -qE "(Test Build|testimage-channel)" %s/etc/lsb-release' %
331 tmp_dir, ignore_status=True)
Tom Wai-Hong Tam76c75072011-10-25 18:00:12 +0800332 utils.system('sudo umount %s' % tmp_dir)
333 os.removedirs(tmp_dir)
334 if code != 0:
335 raise error.TestError(
336 'The image in the USB disk should be a test image.')
337
338
Simran Basi741b5d42012-05-18 11:27:15 -0700339 def install_test_image(self, image_path=None, firmware_update=False):
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800340 """Install the test image specied by the path onto the USB and DUT disk.
341
342 The method first copies the image to USB disk and reboots into it via
Mike Truty49153d82012-08-21 22:27:30 -0500343 recovery mode. Then runs 'chromeos-install' (and possible
344 chromeos-firmwareupdate') to install it to DUT disk.
345
346 Sample command line:
347
348 run_remote_tests.sh --servo --board=daisy --remote=w.x.y.z \
349 --args="image=/tmp/chromiumos_test_image.bin firmware_update=True" \
350 server/site_tests/firmware_XXXX/control
351
352 This test requires an automated recovery to occur while simulating
353 inserting and removing the usb key from the servo. To allow this the
354 following hardware setup is required:
355 1. servo2 board connected via servoflex.
356 2. USB key inserted in the servo2.
357 3. servo2 connected to the dut via dut_hub_in in the usb 2.0 slot.
358 4. network connected via usb dongle in the dut in usb 3.0 slot.
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800359
360 Args:
361 image_path: Path on the host to the test image.
Tom Wai-Hong Tam1a3ff742012-01-11 16:36:46 +0800362 firmware_update: Also update the firmware after installing.
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800363 """
364 build_ver, build_hash = lab_test.VerifyImageAndGetId(cros_dir,
365 image_path)
366 logging.info('Processing build: %s %s' % (build_ver, build_hash))
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800367
Mike Truty49153d82012-08-21 22:27:30 -0500368 # Reuse the servo method that uses the servo USB key to install
369 # the test image.
370 self.servo.image_to_servo_usb(image_path)
371
372 # DUT is powered off while imaging servo USB.
373 # Now turn it on.
374 self.servo.power_short_press()
375 self.wait_for_client()
376 self.servo.set('usb_mux_sel1', 'dut_sees_usbkey')
377
378 install_cmd = 'chromeos-install --yes'
379 if firmware_update:
380 install_cmd += ' && chromeos-firmwareupdate --mode recovery'
381
382 self.register_faft_sequence((
383 { # Step 1, request recovery boot
384 'state_checker': (self.crossystem_checker, {
385 'mainfw_type': ('developer', 'normal'),
386 }),
387 'userspace_action': self.faft_client.request_recovery_boot,
388 'firmware_action': self.wait_fw_screen_and_plug_usb,
389 'install_deps_after_boot': True,
390 },
391 { # Step 2, expected recovery boot
392 'state_checker': (self.crossystem_checker, {
393 'mainfw_type': 'recovery',
394 'recovery_reason' : self.RECOVERY_REASON['US_TEST'],
395 }),
396 'userspace_action': (self.faft_client.run_shell_command,
397 install_cmd),
398 'reboot_action': self.cold_reboot,
399 'install_deps_after_boot': True,
400 },
401 { # Step 3, expected normal or developer boot (not recovery)
402 'state_checker': (self.crossystem_checker, {
403 'mainfw_type': ('developer', 'normal')
404 }),
405 },
406 ))
407 self.run_faft_sequence()
408 # 'Unplug' any USB keys in the servo from the dut.
409 self.servo.disable_usb_hub()
Tom Wai-Hong Tam40fd9472012-01-09 17:11:02 +0800410
411
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800412 def clear_set_gbb_flags(self, clear_mask, set_mask):
413 """Clear and set the GBB flags in the current flashrom.
Tom Wai-Hong Tam15ce5812012-07-26 14:14:18 +0800414
415 Args:
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800416 clear_mask: A mask of flags to be cleared.
417 set_mask: A mask of flags to be set.
Tom Wai-Hong Tam15ce5812012-07-26 14:14:18 +0800418 """
419 gbb_flags = self.faft_client.get_gbb_flags()
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800420 new_flags = gbb_flags & ctypes.c_uint32(~clear_mask).value | set_mask
421
422 if (gbb_flags != new_flags):
423 logging.info('Change the GBB flags from 0x%x to 0x%x.' %
424 (gbb_flags, new_flags))
Tom Wai-Hong Tam15ce5812012-07-26 14:14:18 +0800425 self.faft_client.run_shell_command(
Tom Wai-Hong Tamfda76e22012-08-08 17:19:10 +0800426 '/usr/share/vboot/bin/set_gbb_flags.sh 0x%x' % new_flags)
Tom Wai-Hong Tamc1c4deb2012-07-26 14:28:11 +0800427 self.faft_client.reload_firmware()
Tom Wai-Hong Tama2481922012-08-08 17:24:42 +0800428 # If changing FORCE_DEV_SWITCH_ON flag, reboot to get a clear state
Tom Wai-Hong Tamfa3142e2012-08-16 11:53:58 +0800429 if ((gbb_flags ^ new_flags) & self.GBB_FLAG_FORCE_DEV_SWITCH_ON):
Tom Wai-Hong Tama2481922012-08-08 17:24:42 +0800430 self.run_faft_step({
431 'firmware_action': self.wait_fw_screen_and_ctrl_d,
432 })
Tom Wai-Hong Tam15ce5812012-07-26 14:14:18 +0800433
434
Vic Yangb4e3e742012-06-02 13:17:38 +0800435 def _open_uart_pty(self):
436 """Open UART pty and spawn pexpect object.
437
438 Returns:
439 Tuple (fd, child): fd is the file descriptor of opened UART pty, and
440 child is a fdpexpect object tied to it.
441 """
442 fd = os.open(self.servo.get("uart1_pty"), os.O_RDWR | os.O_NONBLOCK)
443 child = fdpexpect.fdspawn(fd)
444 return (fd, child)
445
446
447 def _flush_uart_pty(self, child):
448 """Flush UART output to prevent previous pending message interferring.
449
450 Args:
451 child: The fdpexpect object tied to UART pty.
452 """
453 child.sendline("")
454 while True:
455 try:
456 child.expect(".", timeout=0.01)
457 except pexpect.TIMEOUT:
458 break
459
460
461 def _uart_send(self, child, line):
462 """Flush and send command through UART.
463
464 Args:
465 child: The pexpect object tied to UART pty.
466 line: String to send through UART.
467
468 Raises:
469 error.TestFail: Raised when writing to UART fails.
470 """
471 logging.info("Sending UART command: %s" % line)
472 self._flush_uart_pty(child)
473 if child.sendline(line) != len(line) + 1:
474 raise error.TestFail("Failed to send UART command.")
475
476
477 def send_uart_command(self, command):
478 """Send command through UART.
479
480 This function open UART pty when called, and then command is sent
481 through UART.
482
483 Args:
484 command: The command string to send.
485
486 Raises:
487 error.TestFail: Raised when writing to UART fails.
488 """
489 (fd, child) = self._open_uart_pty()
490 try:
491 self._uart_send(child, command)
492 finally:
493 os.close(fd)
494
495
496 def send_uart_command_get_output(self, command, regex_list, timeout=1):
497 """Send command through UART and wait for response.
498
499 This function waits for response message matching regular expressions.
500
501 Args:
502 command: The command sent.
503 regex_list: List of regular expressions used to match response message.
504 Note, list must be ordered.
505
506 Returns:
507 List of match objects of response message.
508
509 Raises:
510 error.TestFail: If timed out waiting for EC response.
511 """
512 if not isinstance(regex_list, list):
513 regex_list = [regex_list]
514 result_list = []
515 (fd, child) = self._open_uart_pty()
516 try:
517 self._uart_send(child, command)
518 for regex in regex_list:
519 child.expect(regex, timeout=timeout)
520 result_list.append(child.match)
521 except pexpect.TIMEOUT:
522 raise error.TestFail("Timeout waiting for UART response.")
523 finally:
524 os.close(fd)
525 return result_list
526
527
Vic Yang4d72cb62012-07-24 11:51:09 +0800528 def check_ec_capability(self, required_cap=[]):
529 """Check if current platform has required EC capabilities.
530
531 Args:
532 required_cap: A list containing required EC capabilities. Pass in
533 None to only check for presence of Chrome EC.
534
535 Returns:
536 True if requirements are met. Otherwise, False.
537 """
538 if not self.client_attr.chrome_ec:
539 logging.warn('Requires Chrome EC to run this test.')
540 return False
541
542 for cap in required_cap:
543 if cap not in self.client_attr.ec_capability:
544 logging.warn('Requires EC capability "%s" to run this test.' %
545 cap)
546 return False
547
548 return True
549
550
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +0800551 def _parse_crossystem_output(self, lines):
552 """Parse the crossystem output into a dict.
553
554 Args:
555 lines: The list of crossystem output strings.
556
557 Returns:
558 A dict which contains the crossystem keys/values.
559
560 Raises:
561 error.TestError: If wrong format in crossystem output.
562
563 >>> seq = FAFTSequence()
564 >>> seq._parse_crossystem_output([ \
565 "arch = x86 # Platform architecture", \
566 "cros_debug = 1 # OS should allow debug", \
567 ])
568 {'cros_debug': '1', 'arch': 'x86'}
569 >>> seq._parse_crossystem_output([ \
570 "arch=x86", \
571 ])
572 Traceback (most recent call last):
573 ...
574 TestError: Failed to parse crossystem output: arch=x86
575 >>> seq._parse_crossystem_output([ \
576 "arch = x86 # Platform architecture", \
577 "arch = arm # Platform architecture", \
578 ])
579 Traceback (most recent call last):
580 ...
581 TestError: Duplicated crossystem key: arch
582 """
583 pattern = "^([^ =]*) *= *(.*[^ ]) *# [^#]*$"
584 parsed_list = {}
585 for line in lines:
586 matched = re.match(pattern, line.strip())
587 if not matched:
588 raise error.TestError("Failed to parse crossystem output: %s"
589 % line)
590 (name, value) = (matched.group(1), matched.group(2))
591 if name in parsed_list:
592 raise error.TestError("Duplicated crossystem key: %s" % name)
593 parsed_list[name] = value
594 return parsed_list
595
596
597 def crossystem_checker(self, expected_dict):
598 """Check the crossystem values matched.
599
600 Given an expect_dict which describes the expected crossystem values,
601 this function check the current crossystem values are matched or not.
602
603 Args:
604 expected_dict: A dict which contains the expected values.
605
606 Returns:
607 True if the crossystem value matched; otherwise, False.
608 """
609 lines = self.faft_client.run_shell_command_get_output('crossystem')
610 got_dict = self._parse_crossystem_output(lines)
611 for key in expected_dict:
612 if key not in got_dict:
613 logging.info('Expected key "%s" not in crossystem result' % key)
614 return False
615 if isinstance(expected_dict[key], str):
616 if got_dict[key] != expected_dict[key]:
617 logging.info("Expected '%s' value '%s' but got '%s'" %
618 (key, expected_dict[key], got_dict[key]))
619 return False
620 elif isinstance(expected_dict[key], tuple):
621 # Expected value is a tuple of possible actual values.
622 if got_dict[key] not in expected_dict[key]:
623 logging.info("Expected '%s' values %s but got '%s'" %
624 (key, str(expected_dict[key]), got_dict[key]))
625 return False
626 else:
627 logging.info("The expected_dict is neither a str nor a dict.")
628 return False
629 return True
630
631
Tom Wai-Hong Tam39b93b92012-09-04 16:56:05 +0800632 def vdat_flags_checker(self, mask, value):
633 """Check the flags from VbSharedData matched.
634
635 This function checks the masked flags from VbSharedData using crossystem
636 are matched the given value.
637
638 Args:
639 mask: A bitmask of flags to be matched.
640 value: An expected value.
641
642 Returns:
643 True if the flags matched; otherwise, False.
644 """
645 lines = self.faft_client.run_shell_command_get_output(
646 'crossystem vdat_flags')
647 vdat_flags = int(lines[0], 16)
648 if vdat_flags & mask != value:
649 logging.info("Expected vdat_flags 0x%x mask 0x%x but got 0x%x" %
650 (value, mask, vdat_flags))
651 return False
652 return True
653
654
Tom Wai-Hong Tam3e82e362012-09-05 10:17:55 +0800655 def ro_normal_checker(self, expected_fw=None, twostop=False):
656 """Check the current boot uses RO boot.
657
658 Args:
659 expected_fw: A string of expected firmware, 'A', 'B', or
660 None if don't care.
661 twostop: True to expect a TwoStop boot; False to expect a RO boot.
662
663 Returns:
664 True if the currect boot firmware matched and used RO boot;
665 otherwise, False.
666 """
667 crossystem_dict = {'tried_fwb': '0'}
668 if expected_fw:
669 crossystem_dict['mainfw_act'] = expected_fw.upper()
670 if self.check_ec_capability():
671 crossystem_dict['ecfw_act'] = ('RW' if twostop else 'RO')
672
673 return (self.vdat_flags_checker(
674 self.VDAT_FLAG_LF_USE_RO_NORMAL,
675 0 if twostop else self.VDAT_FLAG_LF_USE_RO_NORMAL) and
676 self.crossystem_checker(crossystem_dict))
677
678
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800679 def root_part_checker(self, expected_part):
680 """Check the partition number of the root device matched.
681
682 Args:
683 expected_part: A string containing the number of the expected root
684 partition.
685
686 Returns:
687 True if the currect root partition number matched; otherwise, False.
688 """
Tom Wai-Hong Tam6a863ba2011-12-08 10:13:28 +0800689 part = self.faft_client.get_root_part()[-1]
690 if self.ROOTFS_MAP[expected_part] != part:
691 logging.info("Expected root part %s but got %s" %
692 (self.ROOTFS_MAP[expected_part], part))
693 return False
694 return True
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800695
696
Vic Yang59cac9c2012-05-21 15:28:42 +0800697 def ec_act_copy_checker(self, expected_copy):
698 """Check the EC running firmware copy matches.
699
700 Args:
701 expected_copy: A string containing 'RO', 'A', or 'B' indicating
702 the expected copy of EC running firmware.
703
704 Returns:
705 True if the current EC running copy matches; otherwise, False.
706 """
707 lines = self.faft_client.run_shell_command_get_output('ectool version')
708 pattern = re.compile("Firmware copy: (.*)")
709 for line in lines:
710 matched = pattern.match(line)
711 if matched and matched.group(1) == expected_copy:
712 return True
713 return False
714
715
Tom Wai-Hong Tam07278c22012-02-08 16:53:00 +0800716 def check_root_part_on_non_recovery(self, part):
717 """Check the partition number of root device and on normal/dev boot.
718
719 Returns:
720 True if the root device matched and on normal/dev boot;
721 otherwise, False.
722 """
723 return self.root_part_checker(part) and \
724 self.crossystem_checker({
725 'mainfw_type': ('normal', 'developer'),
Tom Wai-Hong Tam07278c22012-02-08 16:53:00 +0800726 })
727
728
Tom Wai-Hong Tamf2103be2011-11-10 07:26:56 +0800729 def _join_part(self, dev, part):
730 """Return a concatenated string of device and partition number.
731
732 Args:
733 dev: A string of device, e.g.'/dev/sda'.
734 part: A string of partition number, e.g.'3'.
735
736 Returns:
737 A concatenated string of device and partition number, e.g.'/dev/sda3'.
738
739 >>> seq = FAFTSequence()
740 >>> seq._join_part('/dev/sda', '3')
741 '/dev/sda3'
742 >>> seq._join_part('/dev/mmcblk0', '2')
743 '/dev/mmcblk0p2'
744 """
745 if 'mmcblk' in dev:
746 return dev + 'p' + part
747 else:
748 return dev + part
749
750
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800751 def copy_kernel_and_rootfs(self, from_part, to_part):
752 """Copy kernel and rootfs from from_part to to_part.
753
754 Args:
755 from_part: A string of partition number to be copied from.
Tom Wai-Hong Tamf2103be2011-11-10 07:26:56 +0800756 to_part: A string of partition number to be copied to.
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800757 """
758 root_dev = self.faft_client.get_root_dev()
Tom Wai-Hong Tamf2103be2011-11-10 07:26:56 +0800759 logging.info('Copying kernel from %s to %s. Please wait...' %
760 (from_part, to_part))
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800761 self.faft_client.run_shell_command('dd if=%s of=%s bs=4M' %
Tom Wai-Hong Tamf2103be2011-11-10 07:26:56 +0800762 (self._join_part(root_dev, self.KERNEL_MAP[from_part]),
763 self._join_part(root_dev, self.KERNEL_MAP[to_part])))
764 logging.info('Copying rootfs from %s to %s. Please wait...' %
765 (from_part, to_part))
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800766 self.faft_client.run_shell_command('dd if=%s of=%s bs=4M' %
Tom Wai-Hong Tamf2103be2011-11-10 07:26:56 +0800767 (self._join_part(root_dev, self.ROOTFS_MAP[from_part]),
768 self._join_part(root_dev, self.ROOTFS_MAP[to_part])))
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800769
770
771 def ensure_kernel_boot(self, part):
772 """Ensure the request kernel boot.
773
774 If not, it duplicates the current kernel to the requested kernel
775 and sets the requested higher priority to ensure it boot.
776
777 Args:
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +0800778 part: A string of kernel partition number or 'a'/'b'.
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800779 """
780 if not self.root_part_checker(part):
Tom Wai-Hong Tam622d0ba2012-08-15 16:29:05 +0800781 if self.faft_client.diff_kernel_a_b():
782 self.copy_kernel_and_rootfs(
783 from_part=self.OTHER_KERNEL_MAP[part],
784 to_part=part)
Tom Wai-Hong Tamc7ecfca2011-12-06 11:12:31 +0800785 self.run_faft_step({
786 'userspace_action': (self.reset_and_prioritize_kernel, part),
787 })
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +0800788
789
Vic Yang416f2032012-08-28 10:18:03 +0800790 def set_hardware_write_protect(self, enabled):
Vic Yang2cabf812012-08-28 02:39:04 +0800791 """Set hardware write protect pin.
792
793 Args:
794 enable: True if asserting write protect pin. Otherwise, False.
795 """
796 self.servo.set('fw_wp_vref', self.client_attr.wp_voltage)
797 self.servo.set('fw_wp_en', 'on')
Vic Yang416f2032012-08-28 10:18:03 +0800798 self.servo.set('fw_wp', 'on' if enabled else 'off')
799
800
801 def set_EC_write_protect_and_reboot(self, enabled):
802 """Set EC write protect status and reboot to take effect.
803
804 EC write protect is only activated if both hardware write protect pin
805 is asserted and software write protect flag is set. Also, a reboot is
806 required for write protect to take effect.
807
808 Since the software write protect flag cannot be unset if hardware write
809 protect pin is asserted, we need to deasserted the pin first if we are
810 deactivating write protect. Similarly, a reboot is required before we
811 can modify the software flag.
812
813 This method asserts/deasserts hardware write protect pin first, and
814 set corresponding EC software write protect flag.
815
816 Args:
817 enable: True if activating EC write protect. Otherwise, False.
818 """
819 self.set_hardware_write_protect(enabled)
820 if enabled:
821 # Set write protect flag and reboot to take effect.
822 self.send_uart_command("flashwp enable")
823 self.sync_and_ec_reboot()
824 else:
825 # Reboot after deasserting hardware write protect pin to deactivate
826 # write protect. And then remove software write protect flag.
827 self.sync_and_ec_reboot()
828 self.send_uart_command("flashwp disable")
Vic Yang2cabf812012-08-28 02:39:04 +0800829
830
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800831 def send_ctrl_d_to_dut(self):
832 """Send Ctrl-D key to DUT."""
833 if self._customized_ctrl_d_key_command:
834 logging.info('running the customized Ctrl-D key command')
835 os.system(self._customized_ctrl_d_key_command)
836 else:
837 self.servo.ctrl_d()
838
839
840 def send_enter_to_dut(self):
841 """Send Enter key to DUT."""
842 if self._customized_enter_key_command:
843 logging.info('running the customized Enter key command')
844 os.system(self._customized_enter_key_command)
845 else:
846 self.servo.enter_key()
847
848
Tom Wai-Hong Tam9e61e662012-08-01 15:10:07 +0800849 def send_space_to_dut(self):
850 """Send Space key to DUT."""
851 if self._customized_space_key_command:
852 logging.info('running the customized Space key command')
853 os.system(self._customized_space_key_command)
854 else:
855 # Send the alternative key combinaton of space key to servo.
856 self.servo.ctrl_refresh_key()
857
858
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +0800859 def wait_fw_screen_and_ctrl_d(self):
860 """Wait for firmware warning screen and press Ctrl-D."""
861 time.sleep(self.FIRMWARE_SCREEN_DELAY)
Tom Wai-Hong Tam1db43832011-12-09 10:50:56 +0800862 self.send_ctrl_d_to_dut()
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +0800863
864
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800865 def wait_fw_screen_and_trigger_recovery(self, need_dev_transition=False):
866 """Wait for firmware warning screen and trigger recovery boot."""
867 time.sleep(self.FIRMWARE_SCREEN_DELAY)
868 self.send_enter_to_dut()
869
870 # For Alex/ZGB, there is a dev warning screen in text mode.
871 # Skip it by pressing Ctrl-D.
872 if need_dev_transition:
873 time.sleep(self.TEXT_SCREEN_DELAY)
874 self.send_ctrl_d_to_dut()
875
876
Mike Truty49153d82012-08-21 22:27:30 -0500877 def wait_fw_screen_and_unplug_usb(self):
878 """Wait for firmware warning screen and then unplug the servo USB."""
Tom Wai-Hong Tama79574c2012-02-07 09:29:03 +0800879 time.sleep(self.USB_LOAD_DELAY)
Tom Wai-Hong Tam5d2f4702011-12-06 10:42:31 +0800880 self.servo.set('usb_mux_sel1', 'servo_sees_usbkey')
881 time.sleep(self.USB_PLUG_DELAY)
Mike Truty49153d82012-08-21 22:27:30 -0500882
883
884 def wait_fw_screen_and_plug_usb(self):
885 """Wait for firmware warning screen and then unplug and plug the USB."""
886 self.wait_fw_screen_and_unplug_usb()
Tom Wai-Hong Tam5d2f4702011-12-06 10:42:31 +0800887 self.servo.set('usb_mux_sel1', 'dut_sees_usbkey')
888
889
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800890 def wait_fw_screen_and_press_power(self):
891 """Wait for firmware warning screen and press power button."""
892 time.sleep(self.FIRMWARE_SCREEN_DELAY)
Tom Wai-Hong Tam7317c042012-08-14 11:59:06 +0800893 # While the firmware screen, the power button probing loop sleeps
894 # 0.25 second on every scan. Use the normal delay (1.2 second) for
895 # power press.
896 self.servo.power_normal_press()
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800897
898
Tom Wai-Hong Tam4f5e5922012-07-27 16:23:15 +0800899 def wait_longer_fw_screen_and_press_power(self):
900 """Wait for firmware screen without timeout and press power button."""
901 time.sleep(self.DEV_SCREEN_TIMEOUT)
902 self.wait_fw_screen_and_press_power()
903
904
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800905 def wait_fw_screen_and_close_lid(self):
906 """Wait for firmware warning screen and close lid."""
907 time.sleep(self.FIRMWARE_SCREEN_DELAY)
908 self.servo.lid_close()
909
910
Tom Wai-Hong Tam473cfa72012-07-27 17:16:57 +0800911 def wait_longer_fw_screen_and_close_lid(self):
912 """Wait for firmware screen without timeout and close lid."""
913 time.sleep(self.FIRMWARE_SCREEN_DELAY)
914 self.wait_fw_screen_and_close_lid()
915
916
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +0800917 def setup_tried_fwb(self, tried_fwb):
918 """Setup for fw B tried state.
919
920 It makes sure the system in the requested fw B tried state. If not, it
921 tries to do so.
922
923 Args:
924 tried_fwb: True if requested in tried_fwb=1; False if tried_fwb=0.
925 """
926 if tried_fwb:
927 if not self.crossystem_checker({'tried_fwb': '1'}):
928 logging.info(
929 'Firmware is not booted with tried_fwb. Reboot into it.')
930 self.run_faft_step({
931 'userspace_action': self.faft_client.set_try_fw_b,
932 })
933 else:
934 if not self.crossystem_checker({'tried_fwb': '0'}):
935 logging.info(
936 'Firmware is booted with tried_fwb. Reboot to clear.')
937 self.run_faft_step({})
938
939
Tom Wai-Hong Tama373f802012-07-31 21:16:48 +0800940 def enable_rec_mode_and_reboot(self):
941 """Switch to rec mode and reboot.
942
943 This method emulates the behavior of the old physical recovery switch,
944 i.e. switch ON + reboot + switch OFF, and the new keyboard controlled
945 recovery mode, i.e. just press Power + Esc + Refresh.
946 """
Tom Wai-Hong Tamac943172012-08-01 10:38:39 +0800947 if self._customized_rec_reboot_command:
948 logging.info('running the customized rec reboot command')
949 os.system(self._customized_rec_reboot_command)
Tom Wai-Hong Tamb0b3f412012-08-13 17:17:06 +0800950 elif self.client_attr.chrome_ec:
Vic Yang81273092012-08-21 15:57:09 +0800951 # Cold reset to clear EC_IN_RW signal
Vic Yanga7250662012-08-31 04:00:08 +0800952 self.servo.set('cold_reset', 'on')
953 time.sleep(self.COLD_RESET_DELAY)
954 self.servo.set('cold_reset', 'off')
955 time.sleep(self.EC_BOOT_DELAY)
Vic Yang81273092012-08-21 15:57:09 +0800956 self.send_uart_command("reboot ap-off")
Vic Yang611dd852012-08-02 15:36:31 +0800957 time.sleep(self.EC_BOOT_DELAY)
958 self.send_uart_command("hostevent set 0x4000")
959 self.servo.power_short_press()
Tom Wai-Hong Tamac943172012-08-01 10:38:39 +0800960 else:
961 self.servo.enable_recovery_mode()
962 self.cold_reboot()
963 time.sleep(self.EC_REBOOT_DELAY)
964 self.servo.disable_recovery_mode()
Tom Wai-Hong Tama373f802012-07-31 21:16:48 +0800965
966
Tom Wai-Hong Tam0b9e6d72012-07-31 20:54:06 +0800967 def enable_dev_mode_and_reboot(self):
968 """Switch to developer mode and reboot."""
Vic Yange7553162012-06-20 16:20:47 +0800969 if self.client_attr.keyboard_dev:
970 self.enable_keyboard_dev_mode()
971 else:
972 self.servo.enable_development_mode()
973 self.faft_client.run_shell_command(
974 'chromeos-firmwareupdate --mode todev && reboot')
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +0800975
976
Tom Wai-Hong Tam0b9e6d72012-07-31 20:54:06 +0800977 def enable_normal_mode_and_reboot(self):
978 """Switch to normal mode and reboot."""
Vic Yange7553162012-06-20 16:20:47 +0800979 if self.client_attr.keyboard_dev:
980 self.disable_keyboard_dev_mode()
981 else:
982 self.servo.disable_development_mode()
983 self.faft_client.run_shell_command(
984 'chromeos-firmwareupdate --mode tonormal && reboot')
985
986
987 def wait_fw_screen_and_switch_keyboard_dev_mode(self, dev):
988 """Wait for firmware screen and then switch into or out of dev mode.
989
990 Args:
991 dev: True if switching into dev mode. Otherwise, False.
992 """
993 time.sleep(self.FIRMWARE_SCREEN_DELAY)
994 if dev:
Tom Wai-Hong Tamfe314ac2012-07-25 14:14:17 +0800995 self.send_ctrl_d_to_dut()
Vic Yange7553162012-06-20 16:20:47 +0800996 else:
Tom Wai-Hong Tamfe314ac2012-07-25 14:14:17 +0800997 self.send_enter_to_dut()
Tom Wai-Hong Tam1408f172012-07-31 15:06:21 +0800998 time.sleep(self.FIRMWARE_SCREEN_DELAY)
Tom Wai-Hong Tamfe314ac2012-07-25 14:14:17 +0800999 self.send_enter_to_dut()
Vic Yange7553162012-06-20 16:20:47 +08001000
1001
1002 def enable_keyboard_dev_mode(self):
1003 logging.info("Enabling keyboard controlled developer mode")
Tom Wai-Hong Tamf1a17d72012-07-26 11:39:52 +08001004 # Plug out USB disk for preventing recovery boot without warning
1005 self.servo.set('usb_mux_sel1', 'servo_sees_usbkey')
Vic Yange7553162012-06-20 16:20:47 +08001006 # Rebooting EC with rec mode on. Should power on AP.
Tom Wai-Hong Tama373f802012-07-31 21:16:48 +08001007 self.enable_rec_mode_and_reboot()
Tom Wai-Hong Tam8c54eb82012-08-01 10:31:07 +08001008 self.wait_for_client_offline()
Vic Yange7553162012-06-20 16:20:47 +08001009 self.wait_fw_screen_and_switch_keyboard_dev_mode(dev=True)
Vic Yange7553162012-06-20 16:20:47 +08001010
1011
1012 def disable_keyboard_dev_mode(self):
1013 logging.info("Disabling keyboard controlled developer mode")
Tom Wai-Hong Tamb0b3f412012-08-13 17:17:06 +08001014 if not self.client_attr.chrome_ec:
Vic Yang611dd852012-08-02 15:36:31 +08001015 self.servo.disable_recovery_mode()
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001016 self.cold_reboot()
Tom Wai-Hong Tam8c54eb82012-08-01 10:31:07 +08001017 self.wait_for_client_offline()
Vic Yange7553162012-06-20 16:20:47 +08001018 self.wait_fw_screen_and_switch_keyboard_dev_mode(dev=False)
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001019
1020
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001021 def setup_dev_mode(self, dev_mode):
1022 """Setup for development mode.
1023
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001024 It makes sure the system in the requested normal/dev mode. If not, it
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001025 tries to do so.
1026
1027 Args:
1028 dev_mode: True if requested in dev mode; False if normal mode.
1029 """
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001030 # Change the default firmware_action for dev mode passing the fw screen.
1031 self.register_faft_template({
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001032 'firmware_action': (self.wait_fw_screen_and_ctrl_d if dev_mode
1033 else None),
1034 })
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001035 if dev_mode:
Vic Yange7553162012-06-20 16:20:47 +08001036 if (not self.client_attr.keyboard_dev and
1037 not self.crossystem_checker({'devsw_cur': '1'})):
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001038 logging.info('Dev switch is not on. Now switch it on.')
1039 self.servo.enable_development_mode()
1040 if not self.crossystem_checker({'devsw_boot': '1',
1041 'mainfw_type': 'developer'}):
1042 logging.info('System is not in dev mode. Reboot into it.')
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001043 self.run_faft_step({
Vic Yange7553162012-06-20 16:20:47 +08001044 'userspace_action': None if self.client_attr.keyboard_dev
1045 else (self.faft_client.run_shell_command,
Tom Wai-Hong Tamc7ecfca2011-12-06 11:12:31 +08001046 'chromeos-firmwareupdate --mode todev && reboot'),
Vic Yange7553162012-06-20 16:20:47 +08001047 'reboot_action': self.enable_keyboard_dev_mode if
1048 self.client_attr.keyboard_dev else None,
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001049 })
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001050 else:
Vic Yange7553162012-06-20 16:20:47 +08001051 if (not self.client_attr.keyboard_dev and
1052 not self.crossystem_checker({'devsw_cur': '0'})):
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001053 logging.info('Dev switch is not off. Now switch it off.')
1054 self.servo.disable_development_mode()
1055 if not self.crossystem_checker({'devsw_boot': '0',
1056 'mainfw_type': 'normal'}):
1057 logging.info('System is not in normal mode. Reboot into it.')
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001058 self.run_faft_step({
Vic Yange7553162012-06-20 16:20:47 +08001059 'userspace_action': None if self.client_attr.keyboard_dev
1060 else (self.faft_client.run_shell_command,
Tom Wai-Hong Tamc7ecfca2011-12-06 11:12:31 +08001061 'chromeos-firmwareupdate --mode tonormal && reboot'),
Vic Yange7553162012-06-20 16:20:47 +08001062 'reboot_action': self.disable_keyboard_dev_mode if
1063 self.client_attr.keyboard_dev else None,
Tom Wai-Hong Tamfd590c92011-11-25 11:50:57 +08001064 })
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001065
1066
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +08001067 def setup_kernel(self, part):
1068 """Setup for kernel test.
1069
1070 It makes sure both kernel A and B bootable and the current boot is
1071 the requested kernel part.
1072
1073 Args:
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001074 part: A string of kernel partition number or 'a'/'b'.
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +08001075 """
1076 self.ensure_kernel_boot(part)
Tom Wai-Hong Tam622d0ba2012-08-15 16:29:05 +08001077 if self.faft_client.diff_kernel_a_b():
1078 self.copy_kernel_and_rootfs(from_part=part,
1079 to_part=self.OTHER_KERNEL_MAP[part])
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +08001080 self.reset_and_prioritize_kernel(part)
1081
1082
1083 def reset_and_prioritize_kernel(self, part):
1084 """Make the requested partition highest priority.
1085
1086 This function also reset kerenl A and B to bootable.
1087
1088 Args:
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001089 part: A string of partition number to be prioritized.
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +08001090 """
1091 root_dev = self.faft_client.get_root_dev()
1092 # Reset kernel A and B to bootable.
1093 self.faft_client.run_shell_command('cgpt add -i%s -P1 -S1 -T0 %s' %
1094 (self.KERNEL_MAP['a'], root_dev))
1095 self.faft_client.run_shell_command('cgpt add -i%s -P1 -S1 -T0 %s' %
1096 (self.KERNEL_MAP['b'], root_dev))
1097 # Set kernel part highest priority.
1098 self.faft_client.run_shell_command('cgpt prioritize -i%s %s' %
1099 (self.KERNEL_MAP[part], root_dev))
Tom Wai-Hong Tam6a863ba2011-12-08 10:13:28 +08001100 # Safer to sync and wait until the cgpt status written to the disk.
1101 self.faft_client.run_shell_command('sync')
1102 time.sleep(self.SYNC_DELAY)
Tom Wai-Hong Tamcfda61f2011-11-02 17:41:01 +08001103
1104
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001105 def warm_reboot(self):
1106 """Request a warm reboot.
1107
Tom Wai-Hong Tamb06f0802012-07-31 16:27:50 +08001108 A wrapper for underlying servo warm reset.
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001109 """
Tom Wai-Hong Tamb06f0802012-07-31 16:27:50 +08001110 # Use cold reset if the warm reset is broken.
1111 if self.client_attr.broken_warm_reset:
Gediminas Ramanauskase021e152012-09-04 19:10:59 -07001112 logging.info('broken_warm_reset is True. Cold rebooting instead.')
1113 self.cold_reboot()
Tom Wai-Hong Tamb06f0802012-07-31 16:27:50 +08001114 else:
1115 self.servo.warm_reset()
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001116
1117
1118 def cold_reboot(self):
1119 """Request a cold reboot.
1120
1121 A wrapper for underlying servo cold reset.
1122 """
Tom Wai-Hong Tama276d0a2012-08-22 11:15:17 +08001123 if self.client_attr.platform == 'Parrot':
1124 self.servo.set('pwr_button', 'press')
1125 self.servo.set('cold_reset', 'on')
1126 self.servo.set('cold_reset', 'off')
1127 time.sleep(self.POWER_BTN_DELAY)
1128 self.servo.set('pwr_button', 'release')
1129 elif self.check_ec_capability():
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001130 # We don't use servo.cold_reset() here because software sync is
1131 # not yet finished, and device may or may not come up after cold
1132 # reset. Pressing power button before firmware comes up solves this.
1133 #
1134 # The correct behavior should be (not work now):
1135 # - If rebooting EC with rec mode on, power on AP and it boots
1136 # into recovery mode.
1137 # - If rebooting EC with rec mode off, power on AP for software
1138 # sync. Then AP checks if lid open or not. If lid open, continue;
1139 # otherwise, shut AP down and need servo for a power button
1140 # press.
1141 self.servo.set('cold_reset', 'on')
1142 self.servo.set('cold_reset', 'off')
1143 time.sleep(self.POWER_BTN_DELAY)
1144 self.servo.power_short_press()
1145 else:
1146 self.servo.cold_reset()
1147
1148
Tom Wai-Hong Tamb21b6b42012-07-26 10:46:30 +08001149 def sync_and_warm_reboot(self):
Tom Wai-Hong Tamf1e34972011-11-02 17:07:04 +08001150 """Request the client sync and do a warm reboot.
1151
1152 This is the default reboot action on FAFT.
1153 """
1154 self.faft_client.run_shell_command('sync')
Tom Wai-Hong Tam6a863ba2011-12-08 10:13:28 +08001155 time.sleep(self.SYNC_DELAY)
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001156 self.warm_reboot()
Tom Wai-Hong Tamf1e34972011-11-02 17:07:04 +08001157
1158
Tom Wai-Hong Tamb21b6b42012-07-26 10:46:30 +08001159 def sync_and_cold_reboot(self):
1160 """Request the client sync and do a cold reboot.
1161
1162 This reboot action is used to reset EC for recovery mode.
1163 """
1164 self.faft_client.run_shell_command('sync')
1165 time.sleep(self.SYNC_DELAY)
Tom Wai-Hong Tam7ad99ab2012-07-30 19:30:51 +08001166 self.cold_reboot()
Tom Wai-Hong Tamb21b6b42012-07-26 10:46:30 +08001167
1168
Vic Yangaeb10392012-08-28 09:25:09 +08001169 def sync_and_ec_reboot(self, args=''):
1170 """Request the client sync and do a EC triggered reboot.
1171
1172 Args:
1173 args: Arguments passed to "ectool reboot_ec". Including:
1174 RO: jump to EC RO firmware.
1175 RW: jump to EC RW firmware.
1176 cold: Cold/hard reboot.
1177 """
Vic Yang59cac9c2012-05-21 15:28:42 +08001178 self.faft_client.run_shell_command('sync')
1179 time.sleep(self.SYNC_DELAY)
Vic Yangaeb10392012-08-28 09:25:09 +08001180 # Since EC reboot happens immediately, delay before actual reboot to
1181 # allow FAFT client returning.
1182 self.faft_client.run_shell_command('(sleep %d; ectool reboot_ec %s)&' %
1183 (self.EC_REBOOT_DELAY, args))
Vic Yangf86728a2012-07-30 10:44:07 +08001184 time.sleep(self.EC_REBOOT_DELAY)
1185 self.check_lid_and_power_on()
1186
1187
1188 def check_lid_and_power_on(self):
1189 """
1190 On devices with EC software sync, system powers on after EC reboots if
1191 lid is open. Otherwise, the EC shuts down CPU after about 3 seconds.
1192 This method checks lid switch state and presses power button if
1193 necessary.
1194 """
1195 if self.servo.get("lid_open") == "no":
1196 time.sleep(self.SOFTWARE_SYNC_DELAY)
1197 self.servo.power_short_press()
Vic Yang59cac9c2012-05-21 15:28:42 +08001198
1199
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001200 def _modify_usb_kernel(self, usb_dev, from_magic, to_magic):
1201 """Modify the kernel header magic in USB stick.
1202
1203 The kernel header magic is the first 8-byte of kernel partition.
1204 We modify it to make it fail on kernel verification check.
1205
1206 Args:
1207 usb_dev: A string of USB stick path on the host, like '/dev/sdc'.
1208 from_magic: A string of magic which we change it from.
1209 to_magic: A string of magic which we change it to.
1210
1211 Raises:
1212 error.TestError: if failed to change magic.
1213 """
1214 assert len(from_magic) == 8
1215 assert len(to_magic) == 8
Tom Wai-Hong Tama1d9a0f2011-12-23 09:13:33 +08001216 # USB image only contains one kernel.
1217 kernel_part = self._join_part(usb_dev, self.KERNEL_MAP['a'])
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001218 read_cmd = "sudo dd if=%s bs=8 count=1 2>/dev/null" % kernel_part
1219 current_magic = utils.system_output(read_cmd)
1220 if current_magic == to_magic:
1221 logging.info("The kernel magic is already %s." % current_magic)
1222 return
1223 if current_magic != from_magic:
1224 raise error.TestError("Invalid kernel image on USB: wrong magic.")
1225
1226 logging.info('Modify the kernel magic in USB, from %s to %s.' %
1227 (from_magic, to_magic))
1228 write_cmd = ("echo -n '%s' | sudo dd of=%s oflag=sync conv=notrunc "
1229 " 2>/dev/null" % (to_magic, kernel_part))
1230 utils.system(write_cmd)
1231
1232 if utils.system_output(read_cmd) != to_magic:
1233 raise error.TestError("Failed to write new magic.")
1234
1235
1236 def corrupt_usb_kernel(self, usb_dev):
1237 """Corrupt USB kernel by modifying its magic from CHROMEOS to CORRUPTD.
1238
1239 Args:
1240 usb_dev: A string of USB stick path on the host, like '/dev/sdc'.
1241 """
1242 self._modify_usb_kernel(usb_dev, self.CHROMEOS_MAGIC,
1243 self.CORRUPTED_MAGIC)
1244
1245
1246 def restore_usb_kernel(self, usb_dev):
1247 """Restore USB kernel by modifying its magic from CORRUPTD to CHROMEOS.
1248
1249 Args:
1250 usb_dev: A string of USB stick path on the host, like '/dev/sdc'.
1251 """
1252 self._modify_usb_kernel(usb_dev, self.CORRUPTED_MAGIC,
1253 self.CHROMEOS_MAGIC)
1254
1255
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001256 def _call_action(self, action_tuple):
1257 """Call the action function with/without arguments.
1258
1259 Args:
1260 action_tuple: A function, or a tuple which consisted of a function
1261 and its arguments (if any).
1262
1263 Returns:
1264 The result value of the action function.
1265 """
1266 if isinstance(action_tuple, tuple):
1267 action = action_tuple[0]
1268 args = action_tuple[1:]
1269 if callable(action):
1270 logging.info('calling %s with parameter %s' % (
Tom Wai-Hong Tam2b48cbc2012-08-08 19:58:00 +08001271 str(action), str(args)))
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001272 return action(*args)
1273 else:
1274 logging.info('action is not callable!')
1275 else:
1276 action = action_tuple
1277 if action is not None:
1278 if callable(action):
Tom Wai-Hong Tame8f291a2011-12-08 22:03:53 +08001279 logging.info('calling %s' % str(action))
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001280 return action()
1281 else:
1282 logging.info('action is not callable!')
1283
1284 return None
1285
1286
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001287 def run_shutdown_process(self, shutdown_action, pre_power_action=None,
1288 post_power_action=None):
1289 """Run shutdown_action(), which makes DUT shutdown, and power it on.
1290
1291 Args:
1292 shutdown_action: a function which makes DUT shutdown, like pressing
1293 power key.
1294 pre_power_action: a function which is called before next power on.
1295 post_power_action: a function which is called after next power on.
1296
1297 Raises:
1298 error.TestFail: if the shutdown_action() failed to turn DUT off.
1299 """
1300 self._call_action(shutdown_action)
1301 logging.info('Wait to ensure DUT shut down...')
1302 try:
1303 self.wait_for_client()
1304 raise error.TestFail(
1305 'Should shut the device down after calling %s.' %
1306 str(shutdown_action))
1307 except AssertionError:
1308 logging.info(
1309 'DUT is surely shutdown. We are going to power it on again...')
1310
1311 if pre_power_action:
1312 self._call_action(pre_power_action)
Tom Wai-Hong Tam610262a2012-01-12 14:16:53 +08001313 self.servo.power_short_press()
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001314 if post_power_action:
1315 self._call_action(post_power_action)
1316
1317
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001318 def register_faft_template(self, template):
1319 """Register FAFT template, the default FAFT_STEP of each step.
1320
Tom Wai-Hong Tam109f63c2011-12-08 14:58:27 +08001321 Any missing field falls back to the original faft_template.
1322
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001323 Args:
1324 template: A FAFT_STEP dict.
1325 """
Tom Wai-Hong Tam109f63c2011-12-08 14:58:27 +08001326 self._faft_template.update(template)
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001327
1328
1329 def register_faft_sequence(self, sequence):
1330 """Register FAFT sequence.
1331
1332 Args:
1333 sequence: A FAFT_SEQUENCE array which consisted of FAFT_STEP dicts.
1334 """
1335 self._faft_sequence = sequence
1336
1337
Tom Wai-Hong Tam2c50dff2011-11-11 07:01:01 +08001338 def run_faft_step(self, step, no_reboot=False):
1339 """Run a single FAFT step.
1340
1341 Any missing field falls back to faft_template. An empty step means
1342 running the default faft_template.
1343
1344 Args:
1345 step: A FAFT_STEP dict.
1346 no_reboot: True to prevent running reboot_action and firmware_action.
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001347
1348 Raises:
Tom Wai-Hong Tama9c1a502011-11-10 06:39:26 +08001349 error.TestFail: An error when the test failed.
Tom Wai-Hong Tamd8445dc2011-12-15 09:00:04 +08001350 error.TestError: An error when the given step is not valid.
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001351 """
Tom Wai-Hong Tamd8445dc2011-12-15 09:00:04 +08001352 FAFT_STEP_KEYS = ('state_checker', 'userspace_action', 'reboot_action',
1353 'firmware_action', 'install_deps_after_boot')
1354
Tom Wai-Hong Tam2c50dff2011-11-11 07:01:01 +08001355 test = {}
1356 test.update(self._faft_template)
1357 test.update(step)
1358
Tom Wai-Hong Tamd8445dc2011-12-15 09:00:04 +08001359 for key in test:
1360 if key not in FAFT_STEP_KEYS:
Tom Wai-Hong Tam78709592011-12-19 11:16:50 +08001361 raise error.TestError('Invalid key in FAFT step: %s', key)
Tom Wai-Hong Tamd8445dc2011-12-15 09:00:04 +08001362
Tom Wai-Hong Tam2c50dff2011-11-11 07:01:01 +08001363 if test['state_checker']:
1364 if not self._call_action(test['state_checker']):
1365 raise error.TestFail('State checker failed!')
1366
1367 self._call_action(test['userspace_action'])
1368
1369 # Don't run reboot_action and firmware_action if no_reboot is True.
1370 if not no_reboot:
1371 self._call_action(test['reboot_action'])
1372 self.wait_for_client_offline()
1373 self._call_action(test['firmware_action'])
1374
1375 if 'install_deps_after_boot' in test:
1376 self.wait_for_client(
1377 install_deps=test['install_deps_after_boot'])
1378 else:
1379 self.wait_for_client()
1380
1381
1382 def run_faft_sequence(self):
1383 """Run FAFT sequence which was previously registered."""
Tom Wai-Hong Tama70f0fe2011-09-02 18:28:47 +08001384 sequence = self._faft_sequence
Tom Wai-Hong Tame8f291a2011-12-08 22:03:53 +08001385 index = 1
Tom Wai-Hong Tam2c50dff2011-11-11 07:01:01 +08001386 for step in sequence:
Tom Wai-Hong Tame8f291a2011-12-08 22:03:53 +08001387 logging.info('======== Running FAFT sequence step %d ========' %
1388 index)
Tom Wai-Hong Tam2c50dff2011-11-11 07:01:01 +08001389 # Don't reboot in the last step.
1390 self.run_faft_step(step, no_reboot=(step is sequence[-1]))
Tom Wai-Hong Tame8f291a2011-12-08 22:03:53 +08001391 index += 1