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

Issue 595406 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Xoogler
Closed: Oct 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue 550886



Sign in to add a comment

Shim layer broke Linux Desktop with uClibc

Project Member Reported by primiano@chromium.org, Mar 16 2016

Issue description

Turns out that after crrev.com/1675143004 and crrev.com/1781573002 the uLibc config of Chrome for Linux fails to link, as the new code assumes either tcmalloc or GlibC. Unfortunately no bot is covering that config.

Need to figure out a plan here, very likely just bypass the shim if GLIBC is not defined

+bratell/mostynb@ who seemed to care about uLibc in 2013 in crrev.com/15405003
 

Comment 1 by vapier@chromium.org, Mar 16 2016

Cc: vapier@chromium.org
Summary: Shim layer broke Linux Desktop with uClibc (was: Shim layer broke Linux Desktop with uLibc)

Comment 2 by most...@opera.com, Mar 16 2016

We're still interested in keeping uClibc for the content API layer and below, does this just affect desktop chromium, or content_shell also?
Owner: primiano@chromium.org
Status: Assigned (was: Untriaged)
Yeah that was not intentional.
While I figure out a fix, a workaround is setting 
use_experimental_allocator_shim=0 in gyp (or false in GN)
that flag will stay there at least until 51

> does this just affect desktop chromium, or content_shell also?
Affects all targets for Linux desktop that depend on base (so virtually all targets on Linux desktop)
Cc: picksi@chromium.org
+picksi: first good bug for our new team member

Comment 5 by picksi@chromium.org, Mar 17 2016

Cc: petrcermak@chromium.org
Adding Petr to give him visibility, both as Grigoriy's "buddy" and expert co-worker.

Comment 6 by picksi@chromium.org, Mar 17 2016

Labels: Hotlist-GoodFirstBug
Cc: kraynov@chromium.org
+cc Grigoriy
Cc: -kraynov@chromium.org primiano@chromium.org
Owner: kraynov@chromium.org
Status: Started (was: Assigned)
Cc: -petrcermak@chromium.org
Cc: -petrcermak@chromium.org
Project Member

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

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/045df297e953e60f953710e4266205729050e1cf

commit 045df297e953e60f953710e4266205729050e1cf
Author: kraynov <kraynov@chromium.org>
Date: Thu Oct 06 14:06:50 2016

Allocator shim explanatory error message.

Error message explicitly tells to disable allocator shim if libc
is unsupported. Currently it's hardly achievable to figure out
target libc implementation on the GN generation stage.

BUG= 595406 

Review-Url: https://codereview.chromium.org/2393413003
Cr-Commit-Position: refs/heads/master@{#423516}

[modify] https://crrev.com/045df297e953e60f953710e4266205729050e1cf/base/allocator/allocator_shim_override_glibc_weak_symbols.h

Status: WontFix (was: Started)
Now it's shows meaningful error message asking to set 'use_experimental_allocator_shim=false' in GN args. There is no efficient way to tell (on gn gen stage) which implementation of libc is used on target.
Project Member

Comment 13 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/045df297e953e60f953710e4266205729050e1cf

commit 045df297e953e60f953710e4266205729050e1cf
Author: kraynov <kraynov@chromium.org>
Date: Thu Oct 06 14:06:50 2016

Allocator shim explanatory error message.

Error message explicitly tells to disable allocator shim if libc
is unsupported. Currently it's hardly achievable to figure out
target libc implementation on the GN generation stage.

BUG= 595406 

Review-Url: https://codereview.chromium.org/2393413003
Cr-Commit-Position: refs/heads/master@{#423516}

[modify] https://crrev.com/045df297e953e60f953710e4266205729050e1cf/base/allocator/allocator_shim_override_glibc_weak_symbols.h

Comment 14 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment