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

Issue 651262 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jan 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 537368



Sign in to add a comment

bvt-cq:p2p_ServeFiles fails with llvm builds on arm

Project Member Reported by rahulchaudhry@chromium.org, Sep 29 2016

Issue description

For example, see this buildbot run: https://uberchromegw.corp.google.com/i/chromeos/builders/arm-llvm-toolchain/builds/15/steps/HWTest%20%5Bbvt-cq%5D/logs/stdio

The p2p_ServeFiles test fails with:
p2p_ServeFiles                                                               [ FAILED ]
p2p_ServeFiles                                                                 ERROR: Timed out waiting for condition: tcpdump creates the dump file.
p2p_ServeFiles                                                                 retry_count: 2

This happens only on the buildbots building with llvm, and only for arm/arm64 builds.

 

Comment 1 by laszio@chromium.org, Sep 30 2016

Owner: laszio@chromium.org

Comment 2 by laszio@chromium.org, Sep 30 2016

/usr/local/sbin/tcpdump crashed when accessing tls stuffs:

#0  0xb6febb42 in __tls_get_addr (ti=0xb6ffaf74 <__open_2@got.plt>) at dl-tls.c:795
Labels: llvm-runtime
Labels: -Pri-2 Pri-1
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 1 2016

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

commit 365df969175c21feac615af00169dfe330999987
Author: Ting-Yuan Huang <laszio@chromium.org>
Date: Sat Oct 01 00:25:27 2016

sys-libs/libcap-ng: always use gcc to build libcap-ng

clang's having trouble building this library with -mthumb.
It crashed in __tls_get_addr() in some tests.

BUG= chromium:651262 
TEST=test_that p2p_ServeFiles

Change-Id: I0019732b54204e07860becb2b813c152c452c9cd
Reviewed-on: https://chromium-review.googlesource.com/391164
Commit-Ready: Ting-Yuan Huang <laszio@chromium.org>
Tested-by: Ting-Yuan Huang <laszio@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@chromium.org>

[add] https://crrev.com/365df969175c21feac615af00169dfe330999987/chromeos/config/env/sys-libs/libcap-ng

Owner: yunlian@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 11 2016

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

commit 47ab62e0e3a56d55dd857759d0618703d7d49c8d
Author: Yunlian Jiang <yunlian@google.com>
Date: Fri Oct 07 22:53:24 2016

llvm: cherry pick an upstream patch.

This cherry picks an upstream patch to fix a runtime
error on a test.

BUG= chromium:651262 
TEST=build libcap-ng with llvm and the test passes.

Change-Id: Ia2b0b010264c2748803296b60a7e48bc8f4f1a83
Reviewed-on: https://chromium-review.googlesource.com/395510
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>

[add] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/files/cherry/7c4f7f326e3913a48168269198196efc9058b4b1.patch
[rename] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/llvm-3.9_pre265926-r12.ebuild

Project Member

Comment 8 by bugdroid1@chromium.org, Oct 11 2016

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

commit 47ab62e0e3a56d55dd857759d0618703d7d49c8d
Author: Yunlian Jiang <yunlian@google.com>
Date: Fri Oct 07 22:53:24 2016

llvm: cherry pick an upstream patch.

This cherry picks an upstream patch to fix a runtime
error on a test.

BUG= chromium:651262 
TEST=build libcap-ng with llvm and the test passes.

Change-Id: Ia2b0b010264c2748803296b60a7e48bc8f4f1a83
Reviewed-on: https://chromium-review.googlesource.com/395510
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>

[add] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/files/cherry/7c4f7f326e3913a48168269198196efc9058b4b1.patch
[rename] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/llvm-3.9_pre265926-r12.ebuild

Project Member

Comment 9 by bugdroid1@chromium.org, Oct 11 2016

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

commit 47ab62e0e3a56d55dd857759d0618703d7d49c8d
Author: Yunlian Jiang <yunlian@google.com>
Date: Fri Oct 07 22:53:24 2016

llvm: cherry pick an upstream patch.

This cherry picks an upstream patch to fix a runtime
error on a test.

BUG= chromium:651262 
TEST=build libcap-ng with llvm and the test passes.

Change-Id: Ia2b0b010264c2748803296b60a7e48bc8f4f1a83
Reviewed-on: https://chromium-review.googlesource.com/395510
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>

[add] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/files/cherry/7c4f7f326e3913a48168269198196efc9058b4b1.patch
[rename] https://crrev.com/47ab62e0e3a56d55dd857759d0618703d7d49c8d/sys-devel/llvm/llvm-3.9_pre265926-r12.ebuild

can we close this?
need to remove the cros_use_gcc
Project Member

Comment 12 by bugdroid1@chromium.org, Nov 30 2016

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

commit 568f8e99f8baf980de042d988853bae3c05999f5
Author: Yunlian Jiang <yunlian@chromium.org>
Date: Mon Nov 28 21:58:07 2016

Revert "sys-libs/libcap-ng: always use gcc to build libcap-ng"

This reverts commit 365df969175c21feac615af00169dfe330999987.

The fix on compiler side was checked in, so there is no need to keep this workaround.

BUG= chromium:651262 
TEST=test_that p2p_ServeFiles

Change-Id: I9d1772e366a1e110fa99c785df3e254fbb803d71
Reviewed-on: https://chromium-review.googlesource.com/414214
Commit-Ready: Douglas Anderson <dianders@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[delete] https://crrev.com/ad15162288b5aa25acd66bfa8b63907c0f04125d/chromeos/config/env/sys-libs/libcap-ng

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)

Sign in to add a comment