Closed Bug 801386 Opened 12 years ago Closed 12 years ago

Please enable PGO on Win64 as bug #795594 is fixed now

Categories

(Release Engineering :: General, defect)

Product:
Release Engineering
Component:
General
Platform:
x86_64
Windows 7
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: Virtual, Assigned: Virtual)

References

Details

(Keywords: nightly-community)

Attachments

(1 file, 1 obsolete file)

[buildbot-configs] enable Win64 PGO v2 (completly undo patch from bug #795748)
1.70 KB, patch
Callek
: review+
roc
: feedback+
philor
: checked-in+
Details | Diff | Splinter Review
Assignee: nobody → BernesB
Comment on attachment 671192 [details] [diff] [review]
[buildbot-configs] enable Win64 PGO

Review of attachment 671192 [details] [diff] [review]:
-----------------------------------------------------------------

First off, thanks for this (first) patch!

The change looks correct in that it does what this bug wants to do (enable PGO on win64 on every tree that has PGO)

The concern I have, and why I'm f?-ing to :roc is that it looks like in order to have successful win64 PGO on aurora we need Bug 795594 approved/landed there. Of which I have no idea on the risk on that bug, so would like him and relman to make that call.

If we can't enable PGO properly on aurora due to risk in that patch, we'll need to tweak what we land to disable it on the 18 train.
Attachment #671192 - Flags: review?(bugspam.Callek)
Attachment #671192 - Flags: review+
Attachment #671192 - Flags: feedback?(roc)
Thank you very much!
Firstly I'm not programmer, but this patch looks very easy to do, so I decided that I will create it to get PGO Win64 builds back and lift some burden from you guys. I only need to modify the patch from bug #795748. The first part of patch was only reverting a made changes and the second part of patch was deleted, because it probably only added info about on which platform we build PGO build, so we won't to get that reverted as I deduce and understand it correctly.
Comment on attachment 671192 [details] [diff] [review]
[buildbot-configs] enable Win64 PGO

Review of attachment 671192 [details] [diff] [review]:
-----------------------------------------------------------------

We may as well get the bug 795594 patch on Aurora.
Attachment #671192 - Flags: feedback?(roc) → feedback+
(In reply to Virtual_ManPL [:Virtual] from comment #3)
> ...  I only need to modify the patch from bug #795748. The
> first part of patch was only reverting a made changes and the second part of
> patch was deleted, because it probably only added info about on which
> platform we build PGO build, so we won't to get that reverted as I deduce
> and understand it correctly.

We should revert that second hunk too, which was
 +BRANCHES['try']['pgo_platforms'] = ('linux', 'linux64', 'win32', 'win64')
in config.py. We'll get that from the default anyway (the first hunk), and leaving the line in is likely to cause confusion later (eg we change the default again, everyone goes 'huh?? why has try got this line here').
Corrected patch as Nick Thomas suggested.
Attachment #671192 - Attachment is obsolete: true
Attachment #671192 - Flags: checked-in?
Attachment #671268 - Flags: review?(bugspam.Callek)
Attachment #671268 - Flags: review?(bugspam.Callek) → review+
check in?
To check all is well I've pushed to try
  https://tbpl.mozilla.org/?tree=Try&rev=4b82c4f065ff
Sometimes the RelEng infra differs developer's machines so this is worth checking prior to deployment.

If that's green and someone verifies the build starts OK then the next step is to land the change in buildbot-configs on the default branch. Then it'll get picked up by the next code deployment to the buildbot masters.
Status: UNCONFIRMED → ASSIGNED
Component: Release Engineering → Release Engineering: Automation (General)
Ever confirmed: true
QA Contact: catlee
LGTM


Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:19.0) Gecko/19.0 Firefox/19.0
buildID: 20121018183205

Source
Built from http://hg.mozilla.org/try/rev/4b82c4f065ff
(In reply to Nick Thomas [:nthomas] from comment #10)
> Builds are in
> http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/nthomas@mozilla.
> com-4b82c4f065ff/try-win64/. Please test the zip/installer out.

Works fine for me.
What's the progress on this right now? It's been a while...
You can push this patch to the 'default' branch of buildbot-configs, and it will get deployed sometime this week.

Please include this bug number and r=Callek in your commit message.
I will be glad if someone with privileges can push this patch ;)
Comment on attachment 671268 [details] [diff] [review]
[buildbot-configs] enable Win64 PGO v2 (completly undo patch from bug #795748)

http://hg.mozilla.org/build/buildbot-configs/rev/b0be95c1e5d1
Attachment #671268 - Flags: checked-in+
In production.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Thank you very much! :)
Status: RESOLVED → VERIFIED
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: