New issue
Advanced search Search tips

Issue 829598 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocking:
issue 824615



Sign in to add a comment

openssl fails to build on tael

Project Member Reported by smbar...@chromium.org, Apr 5 2018

Issue description

https://logs.chromium.org/v/?s=chromiumos%2Fbb%2Fchromiumos%2Ftael-full%2F6%2F%2B%2Frecipes%2Fsteps%2FBuildPackages%2F0%2Fstdout

openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s: Assembler messages:
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:1631: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2037: IT blocks containing 16-bit Thumb instructions of the following class are deprecated in ARMv8: ADD/SUB sp, sp #imm
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2037: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2189: IT blocks containing 32-bit Thumb instructions are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2189: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2202: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2979: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:3814: IT blocks containing more than one conditional instruction are deprecated in ARMv8
openssl-1.0.2n: /build/tael/tmp/portage/dev-libs/openssl-1.0.2n/temp/bad_dtls_test-9e25bc.s:2378: Error: branch out of range
openssl-1.0.2n: clang-7.0: error: assembler command failed with exit code 1 (use -v to see invocation)
 
Blocking: 824615
Status: WontFix (was: Available)
This seems to have gone away (I also never saw it during local builds) 
tael paladin has been green since build #62: 
https://build.chromium.org/p/chromiumos/builders/tael-paladin

closing, re-open if it comes back
Status: Untriaged (was: WontFix)
re-opening -- tael-full builder sees this problem, but tale-paladin must be getting a binary package from somewhere else.
Owner: sonnyrao@chromium.org
Status: Started (was: Untriaged)
so far I'e figured out that this package builds on arm-generic and on kevin
both of those set more compiler flags than tael does including march, mtune, and mfpu

arm-generic uses these flags:
-march=armv7-a -mtune=cortex-a15 -mfpu=neon -mfloat-abi=hard

kevin uses these flags:
-march=armv8-a+crc -mtune=cortex-a57.cortex-a53 -mfpu=crypto-neon-fp-armv8 -mfloat-abi=hard 

tael has none of those flags

Comment 5 by vapier@chromium.org, Apr 27 2018

the arm-generic settings should be safe for us to use here
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 27 2018

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

commit ba3fd57bbf174e7d5333913e3ff2c526586e8ab3
Author: Sonny Rao <sonnyrao@chromium.org>
Date: Fri Apr 27 10:54:57 2018

tael: set compiler flags to match arm-generic

On tael, openssl fails to compile without setting march to something
reasonable, so this makes tael match arm-generic's compiler flags.

BUG= chromium:829598 
TEST="emerge-tael openssl" succeeds

Change-Id: Ifba6d61078ac22fa12e2996a0e4109b416f32025
Reviewed-on: https://chromium-review.googlesource.com/1031776
Commit-Ready: Sonny Rao <sonnyrao@chromium.org>
Tested-by: Sonny Rao <sonnyrao@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[add] https://crrev.com/ba3fd57bbf174e7d5333913e3ff2c526586e8ab3/overlay-tael/make.conf

Status: Fixed (was: Started)
tael full bulder is green now

Sign in to add a comment