Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 499650 - www-client/firefox fix for PGO compilation
Summary: www-client/firefox fix for PGO compilation
Status: RESOLVED DUPLICATE of bug 388585
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-01-29 03:43 UTC by Daniel Rammelt
Modified: 2016-08-06 13:11 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch for Makefile.in (Makefile-pgo.patch,263 bytes, patch)
2014-01-29 03:44 UTC, Daniel Rammelt
Details | Diff
Patch for 24.2.0 ebuild (firefox-ebuild-pgo.patch,727 bytes, patch)
2014-01-29 03:45 UTC, Daniel Rammelt
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Rammelt 2014-01-29 03:43:35 UTC
The PGO use flag does not work in current Gentoo ebuilds. I have created 2 patches which resolve this.

Patch one is for the ebuild itself which corrects the pgo call and adds the 2nd patch.
http://pastebin.com/CvKNnd7a

Patch two fixes the missing build option in the Makefile.in 'pgo-profile-run'.
http://pastebin.com/D2fTDBeM

Patches are against www-client/firefox-24.2.0, have tested against version 26 also.

Compiled with use flags as follows.
www-client/firefox-26.0-r1::x-portage  USE="alsa bindist dbus gstreamer jit libnotify minimal pgo startup-notification system-cairo system-icu system-jpeg system-sqlite -custom-cflags -custom-optimization -debug -pulseaudio (-selinux) -wifi"


Reproducible: Always




The PGO script itself profileserver.py uses port 8888 which conflicts with the default port used by BTSync. BTSync must be closed before compilation.
Comment 1 Daniel Rammelt 2014-01-29 03:44:36 UTC
Created attachment 369030 [details, diff]
Patch for Makefile.in
Comment 2 Daniel Rammelt 2014-01-29 03:45:05 UTC
Created attachment 369032 [details, diff]
Patch for 24.2.0 ebuild
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2014-01-29 10:30:51 UTC
USE=pgo is still masked.

*** This bug has been marked as a duplicate of bug 388585 ***
Comment 4 Daniel Rammelt 2014-01-29 17:46:17 UTC
I dont understand, this is not a duplicate of the bugs mentioned. I am proposing a fix not a bug, besides most of those bugs are unrelated to the actual PGO build error that is recieved when building the current versions of firefox.

Can you atleast suggest how these patches get put into firefox?
Comment 5 Ian Stakenvicius (RETIRED) gentoo-dev 2014-01-30 14:57:23 UTC
They don't, because we don't support PGO (which is the reason for the RESO/DUP status and linked bug).  We are not ever going to include PGO patches in the tree for firefox et. al. until/unless this stance changes.

Furthermore, user-contributed patches that do support PGO should go as attachments to the aforementioned bug, to keep them all in one place.

Firefox et. al. ebuilds do have "epatch_user" support, for those that do want to try and use PGO, to make applying supporting patches easier.
Comment 6 microcai 2014-11-01 02:18:05 UTC
check  https://developer.mozilla.org/en-US/docs/Building_with_Profile-Guided_Optimization

the page says "
You must have tests enabled (do not include --disable-tests in your build 
configuration).
"

so, you shoud enable test and pgo both, otherwise , make wooun't be able to find  gpo build target
Comment 7 . 2015-03-09 21:29:14 UTC
(In reply to microcai from comment #6)
> check 
> https://developer.mozilla.org/en-US/docs/Building_with_Profile-
> Guided_Optimization
> 
> the page says "
> You must have tests enabled (do not include --disable-tests in your build 
> configuration).
> "
> 
> so, you shoud enable test and pgo both, otherwise , make wooun't be able to
> find  gpo build target

"test" USE flag isn't needed, I've compiled FF 36.0.1 only by following instructions on http://forums.gentoo.org/viewtopic-p-7490084.html

btw. performance increase is ~5% according to peacekeeper benchmark