Issue metadata
Sign in to add a comment
|
Recovery failed in ToT (8977.0.0 / 56.0.2907.0) in all boards |
||||||||||||||||||||||
Issue descriptionChromeOS: 8977.0.0 / 56.0.2907.0 What steps will reproduce the problem? (1) ESC-F3-power button (2) Plug in usb stick with recovery image (3) What is the expected result? Recovery should complete successfully What happens instead? Recovery failed with "An unexpected error has occured....." Regression from 8975.0.0/56.0.2907.0
,
Nov 10 2016
Do you have recovery log?
,
Nov 10 2016
,
Nov 10 2016
,
Nov 10 2016
From recovery logs: ================================== + tee -a /run/frecon/vt1 /run/frecon/vt2 .+ cgpt find -1 -u 586f5ae0-ecd2-794a-95bf-2ac9426ac836 ERROR: Can't read enough: -1, not 512 ERROR: Cannot read primary GPT header WARNING: Primary GPT header is invalid WARNING: Secondary GPT header is invalid WARNING: Primary GPT header is invalid WARNING: Secondary GPT header is invalid flashrom v0.9.4 : 65be03a : Nov 08 2016 10:51:57 UTC on Linux 4.4.21-06015-ge301df2 (aarch64) flashrom v0.9.4 : 65be03a : Nov 08 2016 10:51:57 UTC on Linux 4.4.21-06015-ge301df2 (aarch64) Invalid entry specified: RW_GPT ERROR: Cannot exec flashrom to read from RW_GPT section. ReadNorFlash: Is a directory + kern= + newroot=3 + [ -b 3 ] + sleep 1 + dlogf . + + printftee . -a /run/frecon/vt1 /run/frecon/vt2 .+ cgpt find -1 -u 586f5ae0-ecd2-794a-95bf-2ac9426ac836 ERROR: Can't read enough: -1, not 512 ERROR: Cannot read primary GPT header ===================================
,
Nov 10 2016
Recovery still failed in 8978.0.0/56.0.2915.0. Tested kevin/peppy.
,
Nov 10 2016
Assigning to yunlian@ as you fixed a similar issue: crbug/650227 Please pass along to right owner if not you, thanks
,
Nov 10 2016
,
Nov 10 2016
Is this happening on ARM boards? If not, it's unlikely a toolchain problem and we have better get experts of recovery involved as early as possible.
,
Nov 10 2016
Sorry for the typo... Is this also happening on ALL ARM boards? If YES, it's unlikely a toolchain problem and we have better get experts of recovery involved as early as possible. We only rolled all amd64, veyron_minnie and nyan_big to use clang/llvm. Most arm boards are built by gcc currently.
,
Nov 10 2016
Looks like this happen on ARM (kevin) and Intel (peppy)
,
Nov 10 2016
I will try to create a recover image built by gcc to see whether this is related to toolchain changes.
,
Nov 10 2016
To #11, it should not be related to toolchain. We did not touch arm toolchain recently.
,
Nov 11 2016
,
Nov 11 2016
Issue 664407 has been merged into this issue.
,
Nov 11 2016
,
Nov 11 2016
This is a serious regression. Can we get more eyes on this one? Someone needs to bisect changes from 8975 to 8977.
,
Nov 11 2016
The samus_recovery.log shows it failed here: + chroot /usb /usr/sbin/chromeos-recovery /dev/sdb .: 843: Can't open /tmp/tmp.PLBBO99HQC/usr/sbin/write_gpt.sh + local install_status=127 + [ 127 -ne 0 ] + dlog WARNING!!! Installation of software failed. Displaying hw diagnostics This change is suspicious: https://chromium.googlesource.com/chromiumos/platform2/+/4c5a7cb1c287275d72604ff33b7a5093c00d795f%5E%21/#F0
,
Nov 11 2016
,
Nov 11 2016
if it's recent, prob want to revert: https://chromium-review.googlesource.com/405031
,
Nov 11 2016
Looks good. Reverting.
,
Nov 11 2016
Revert here : https://chromium-review.googlesource.com/#/c/410166/
,
Nov 11 2016
,
Nov 12 2016
I've confirmed using a release trybot recovery image that the revert above is effective. Sending it through the CQ again now.
,
Nov 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/495cfbc122c9f3f14324195ca03a749c38ec54b0 commit 495cfbc122c9f3f14324195ca03a749c38ec54b0 Author: Benson Leung <bleung@chromium.org> Date: Fri Nov 11 18:45:01 2016 Revert "installer: Load GPT helper functions before using DEFAULT_ROOTDEV" This reverts commit 4c5a7cb1c287275d72604ff33b7a5093c00d795f. BUG= chromium:664232 TEST=Recovery works again. Change-Id: I5f278248873bf91491841477380d22146487a5b5 Reviewed-on: https://chromium-review.googlesource.com/410166 Commit-Ready: Benson Leung <bleung@chromium.org> Tested-by: Benson Leung <bleung@chromium.org> Reviewed-by: Benson Leung <bleung@chromium.org> Reviewed-by: Alexis Savery <asavery@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [modify] https://crrev.com/495cfbc122c9f3f14324195ca03a749c38ec54b0/installer/chromeos-install
,
Nov 12 2016
8983.0 and later have the revert. Recovery should be working after that point on all boards again.
,
Nov 14 2016
verified in 8989.0.0 / 56.0.2919.0. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by helenzhang@google.com
, Nov 10 2016