Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(989)

Issue 9185015: Compile libvpx code as PIC for all Linux targets (Closed)

Created:
8 years, 11 months ago by Roland McGrath
Modified:
8 years, 11 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Compile libvpx code as PIC for all Linux targets TEXTRELs (i.e. non-PIC code in a DSO or PIE such as chrome) are a bad bad thing on Linux. As of M16 we managed to get rid of all of them (except for libffmpegsumo.so, where it will take a bunch of work). In M17 this regressed, due to the libvpx code. This makes users of Linux distributions such as Fedora very unhappy. It's a simple configuration change to tell libvpx to build PIC on ia32. This gets rid of all TEXTRELs in the chrome build again. BUG= http://code.google.com/p/chromium/issues/detail?id=109318 TEST= none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117274

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -7 lines) Patch
M third_party/libvpx/generate_gypi.sh View 1 2 chunks +3 lines, -3 lines 0 comments Download
M third_party/libvpx/source/config/linux/arm-neon/vpx_config.asm View 1 1 chunk +1 line, -1 line 0 comments Download
M third_party/libvpx/source/config/linux/ia32/vpx_config.h View 1 2 chunks +2 lines, -2 lines 0 comments Download
M third_party/libvpx/source/config/linux/ia32/vpx_config.asm View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Roland McGrath
It's not clear to me whether this alone is sufficient to make sure the change ...
8 years, 11 months ago (2012-01-11 18:53:12 UTC) #1
Roland McGrath
I harmonized all the occurrences of CONFIG_PIC though I think the first patch changed the ...
8 years, 11 months ago (2012-01-11 19:27:23 UTC) #2
jkoleszar
8 years, 11 months ago (2012-01-11 19:44:14 UTC) #3
On 2012/01/11 19:27:23, Roland McGrath wrote:
> I harmonized all the occurrences of CONFIG_PIC though I think the first patch
> changed the only thing that has any practical effect.

LGTM

nit: this patch also has the effect of turning on PIC on ARM, which I suspect is
the right thing to do anyway, but you may want to change the subject of this
patch to indicate 'all targets' or alternately define it for CONFIG_X86 instead
of CONFIG_COMMON.

Powered by Google App Engine
This is Rietveld 408576698