New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 664232 link

Starred by 5 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 0
Type: Bug-Regression



Sign in to add a comment

Recovery failed in ToT (8977.0.0 / 56.0.2907.0) in all boards

Project Member Reported by helenzhang@google.com, Nov 10 2016

Issue description

ChromeOS: 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

 
Labels: -Type-Bug Type-Bug-Regression

Comment 2 by ka...@chromium.org, Nov 10 2016

Cc: yunlian@chromium.org gkihumba@chromium.org laszio@chromium.org
Components: OS>Firmware
Labels: -ReleaseBlock-Stable ReleaseBlock-Dev
Do you have recovery log?

Comment 3 by ka...@chromium.org, Nov 10 2016

Cc: dchan@chromium.org

Comment 5 by son...@google.com, 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
===================================
Recovery still failed in 8978.0.0/56.0.2915.0. Tested kevin/peppy. 

Comment 7 by gkihumba@google.com, Nov 10 2016

Owner: yunlian@chromium.org
Status: Assigned (was: Untriaged)
Assigning to yunlian@ as you fixed a similar issue: crbug/650227
Please pass along to right owner if not you, thanks

Comment 8 by dchan@chromium.org, Nov 10 2016

Cc: philipchen@chromium.org dhend...@chromium.org waihong@chromium.org

Comment 9 by laszio@chromium.org, 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.
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.

Comment 11 by dchan@chromium.org, Nov 10 2016

Looks like this happen on ARM (kevin) and Intel (peppy)
I will try to create a recover image built by gcc to see whether this is related to toolchain changes.
Owner: ----
To #11, it should not be related to toolchain. We did not touch arm toolchain recently.
Cc: adurbin@chromium.org

Comment 15 by ka...@chromium.org, Nov 11 2016

Issue 664407 has been merged into this issue.
Owner: yunlian@chromium.org
Cc: bleung@chromium.org
This is a serious regression. Can we get more eyes on this one? Someone needs to bisect changes from 8975 to 8977.
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
Cc: vapier@chromium.org asavery@chromium.org
if it's recent, prob want to revert:
  https://chromium-review.googlesource.com/405031
Labels: -Pri-1 Pri-0
Owner: bleung@chromium.org
Looks good. Reverting.
Status: Started (was: Assigned)
I've confirmed using a release trybot recovery image that the revert above is effective. Sending it through the CQ again now.
Project Member

Comment 25 by bugdroid1@chromium.org, 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

Labels: Week-1645
Status: Fixed (was: Started)
8983.0 and later have the revert. Recovery should be working after that point on all boards again.
Status: Verified (was: Fixed)
verified in 8989.0.0 / 56.0.2919.0. 

Sign in to add a comment