New issue
Advanced search Search tips

Issue 844002 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 8
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Build-Toolchain



Sign in to add a comment

Build strace with clang causes unknown uint128 type error on ARM64

Project Member Reported by ka...@inf.u-szeged.hu, May 17 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.119 Safari/537.36
Platform: arm64-generic

Steps to reproduce the problem:
1. ./setup_board --board arm64-generic
2. emerge-arm64-generic strace

What is the expected behavior?

What went wrong?
It seems there is no proper define for uint128 targeting aarch64, it odds because it works with (armv7-clang) for arm-generic.

In file included from mpers-m32/siginfo_t.c:38:
In file included from ../../../../../../usr/include/signal.h:306:
In file included from ../../../../../../usr/include/bits/sigcontext.h:27:
../../../../../../usr/include/asm/sigcontext.h:53:2: error: unknown type name '__uint128_t'
        __uint128_t vregs[32];
        ^
In file included from mpers-m32/siginfo_t.c:38:
In file included from ../../../../../../usr/include/signal.h:326:
In file included from ../../../../../../usr/include/sys/ucontext.h:26:
In file included from ../../../../../../usr/include/sys/procfs.h:34:
../../../../../../usr/include/sys/user.h:32:3: error: unknown type name '__uint128_t'
  __uint128_t  vregs[32];
  ^
2 errors generated.
make: *** [Makefile:6311: mpers-m32.stamp] Error 1
rm ioctlsort0 ioctls_all0.h ioctlsort0.o ioctlsort1.o ioctls_all1.h ioctlsort1

Did this work before? N/A 

Chrome version:   Channel: dev
OS Version: 
Flash Version:
 
full_build_log.txt
52.2 KB View Download
Cc: manojgupta@chromium.org
Use of "-m32" seems strange since this is arm64 and not x86_64. You can check if upstream strace has a fix for it.
Components: OS>Kernel
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 14

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

commit 92c59e13ccda919249b8047f1949a4f31abf580f
Author: Manoj Gupta <manojgupta@google.com>
Date: Sat Jul 14 11:58:18 2018

arm64-generic: Make little-endian profile as parent.

Make arm64 little endian profile as parent.
Also remove llvm features since that will be handled
by the arm64/little-endian profile.

BUG= chromium:844002 

Change-Id: I244ce559268cb42c235ea135c05e68506516c679
Reviewed-on: https://chromium-review.googlesource.com/1137700
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/92c59e13ccda919249b8047f1949a4f31abf580f/overlay-arm64-generic/profiles/base/parent

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 15

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

commit 203231be59c82ba2fffcaf94ec1a283d54bdf1fc
Author: Manoj Gupta <manojgupta@google.com>
Date: Sun Jul 15 01:11:49 2018

arm64: Make clang the default compiler.

Use clang as the default compiler for arm64.

CQ-DEPEND=CL:1137700

BUG= chromium:844002 
TEST=emerge-arm64-generic works. Clang is used as compiler.

Change-Id: I5f7f7ff59d07eb37ec26ee9b7759be7b850a80e8
Reviewed-on: https://chromium-review.googlesource.com/1137701
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/203231be59c82ba2fffcaf94ec1a283d54bdf1fc/profiles/default/linux/arm64/10.0/chromeos/little-endian/parent

Components: Tools>ChromeOS-Toolchain
Owner: manojgupta@chromium.org
Status: Started (was: Unconfirmed)
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/793f925824295110b298398b14e2a6eef94d0793

commit 793f925824295110b298398b14e2a6eef94d0793
Author: Manoj Gupta <manojgupta@google.com>
Date: Sat Aug 04 08:59:36 2018

strace: Update to upstream 4.22.

Changes:
KEYWORDS -> "*"

Fixed ebuild to work on arm64 by disabling mpers (CL:1137702).

CQ-DEPEND=CL:1137702

BUG= chromium:844002 
TEST=strace builds on host/kevin/arm64-generic.

Change-Id: I9919f3b511d5fc0ca35dba4d0d3118a4eff9536d
Reviewed-on: https://chromium-review.googlesource.com/1064818
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[rename] https://crrev.com/793f925824295110b298398b14e2a6eef94d0793/dev-util/strace/strace-4.22.ebuild
[modify] https://crrev.com/793f925824295110b298398b14e2a6eef94d0793/dev-util/strace/Manifest

Project Member

Comment 7 by bugdroid1@chromium.org, Aug 4

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

commit b3530055508a84a0e0c5c3afd6f16fe3cce034b0
Author: Manoj Gupta <manojgupta@google.com>
Date: Sat Aug 04 08:59:35 2018

strace: Disable mpers on arm64.

CQ-DEPEND=CL:1064818

BUG= chromium:844002 
TEST=strace builds on host/kevin/arm64-generic

Change-Id: I76bd96f861b2b2edfa87174936f5be82104a1b68
Reviewed-on: https://chromium-review.googlesource.com/1137702
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[add] https://crrev.com/b3530055508a84a0e0c5c3afd6f16fe3cce034b0/chromeos/config/env/dev-util/strace

Status: Verified (was: Started)
Labels: cros-arm64

Sign in to add a comment