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

Issue 616806 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

ec: unexplained size differences for builds on different machines

Project Member Reported by sha...@chromium.org, Jun 2 2016

Issue description

From email -->

"I'm trying to run build_packages for board nyan_big, and I'm
getting errors out of chromeos-ec.  Here's what looks like the
relevant text:
    chromeos-ec-0.0.1-r3059: /usr/x86_64-pc-linux-gnu/arm-none-eabi/binutils-bin/2.25.51/ld.bfd.real: No room left in the flash

It's been going on for several days now, I think.  I'm
building from ToT, but it's not happening on the canary.

-- jrb"

We'll investigate this. Please provide the following logs / artifacts:

- Full log from 'emerge nyan_big chromeos-ec'
- Full log from 'cd ~/trunk/src/platform/ec; make BOARD=big'
- Compressed contents of ~/trunk/src/platform/ec/build/big (assuming the previous command failed)
 
Logs attached.

The 'make' command succeeded (go figure).

chromeos-base:chromeos-ec-0.0.1-r3059:20160602-174957.log
57.4 KB View Download
make.out
4.7 KB Download
The image size error is being thrown for the '-DSHIFT_CODE_FOR_TEST' image, which isn't built normally with 'make' / 'make buildall -j'. The produced ec_autest_image.bin image was once used for automated testing, but now it is no longer used. I will remove the building of this image from the ebuild and remove related code from EC source.
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/fdc9e1defb0378d9fd0992ccc5a83ad531d1afef

commit fdc9e1defb0378d9fd0992ccc5a83ad531d1afef
Author: Shawn Nematbakhsh <shawnn@chromium.org>
Date: Thu Jun 02 21:38:00 2016

chromeos-ec: Remove production of ec_autest_image.bin image

ec_autest_image.bin was previously used for FAFT testing but is no
longer in use today.

BUG= chromium:616806 
BRANCH=None
TEST=`emerge-nyan_big chromeos-ec`

Change-Id: I37a45e195c879b17587ae11a91a70245d6e567f6
Reviewed-on: https://chromium-review.googlesource.com/349281
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Tom Tam <waihong@google.com>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>

[modify] https://crrev.com/fdc9e1defb0378d9fd0992ccc5a83ad531d1afef/chromeos-base/chromeos-ec/chromeos-ec-9999.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/0e532c6044e068c3929a0407c31893e0070e1196

commit 0e532c6044e068c3929a0407c31893e0070e1196
Author: Shawn Nematbakhsh <shawnn@chromium.org>
Date: Thu Jun 02 21:34:09 2016

cleanup: Remove SHIFT_CODE_FOR_TEST handling

SHIFT_CODE_FOR_TEST images were used for an old FAFT test but is no
longer in use today.

BUG= chromium:616806 
BRANCH=None
TEST=`make buildall -j`
CQ-DEPEND=CL:349281

Change-Id: I1b4e8c2560e2a7ff507cf2275dbbbdabf435866b
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/349272
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Tom Tam <waihong@google.com>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>

[modify] https://crrev.com/0e532c6044e068c3929a0407c31893e0070e1196/core/cortex-m/ec.lds.S
[modify] https://crrev.com/0e532c6044e068c3929a0407c31893e0070e1196/util/getversion.sh
[modify] https://crrev.com/0e532c6044e068c3929a0407c31893e0070e1196/core/cortex-m0/ec.lds.S
[modify] https://crrev.com/0e532c6044e068c3929a0407c31893e0070e1196/core/nds32/ec.lds.S

Status: Verified (was: Untriaged)
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 24 2016

Labels: merge-merged-firmware-glados-7820.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/cdb6ed82e04a6ff81d9aba7d4c9ffc724464744a

commit cdb6ed82e04a6ff81d9aba7d4c9ffc724464744a
Author: Shawn Nematbakhsh <shawnn@chromium.org>
Date: Thu Jun 02 21:38:00 2016

chromeos-ec: Remove production of ec_autest_image.bin image

ec_autest_image.bin was previously used for FAFT testing but is no
longer in use today.

BUG= chromium:616806 
BRANCH=None
TEST=`emerge-nyan_big chromeos-ec`

Change-Id: I37a45e195c879b17587ae11a91a70245d6e567f6
Reviewed-on: https://chromium-review.googlesource.com/349281
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Tom Tam <waihong@google.com>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
(cherry picked from commit fdc9e1defb0378d9fd0992ccc5a83ad531d1afef)
Reviewed-on: https://chromium-review.googlesource.com/402000
Reviewed-by: Shawn N <shawnn@chromium.org>
Commit-Queue: Shawn N <shawnn@chromium.org>

[modify] https://crrev.com/cdb6ed82e04a6ff81d9aba7d4c9ffc724464744a/chromeos-base/chromeos-ec/chromeos-ec-9999.ebuild

Sign in to add a comment