Output linker max virtual size using 'TinderboxPrint' so it will appear in the TBPL summary box

RESOLVED FIXED in mozilla15

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Trunk
mozilla15
All
Windows Server 2003
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Bug 710712 added the peak virtual memory usage of the linker during the final PGO link phase on Windows to each pgo log:
http://mxr.mozilla.org/mozilla-central/source/build/link.py#53

The current output is of the form:
linker max virtual size: 3021553664

It is my understanding that if we make similar to:
TinderboxPrint: linker max virtual size: 3021553664
...it will show up in the TBPL summary box, saving us from having to open each log manually. (I imagine we'll need a shorter description string)
(Assignee)

Updated

5 years ago
Assignee: nobody → bmo
(Assignee)

Updated

5 years ago
Severity: critical → normal
(Assignee)

Comment 1

5 years ago
Created attachment 619923 [details] [diff] [review]
Patch v1

Shortened the string a bit, hopefully that's ok compared to the others used.
Attachment #619923 - Flags: review?(ted.mielczarek)
Attachment #619923 - Flags: feedback?(philringnalda)
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Comment on attachment 619923 [details] [diff] [review]
Patch v1

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

Yeah, I don't care, that was a totally arbitrary string.
Attachment #619923 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 619923 [details] [diff] [review]
Patch v1

We use tiny little strings because on Tinderbox, a long string would widen the table column and thus widen the whole page and require more horizontal scrolling to see the Windows builds. But if you remember something verbose like Talos showing failure to contact the graphserver, tbpl actually has very wide columns for showing TinderboxPrint, so that should be fine (until we're sending the raw number to graphserver, at which point that should instead TinderboxPrint a couple of digits of maxvsize/1024/1024/1024 so we don't have to remember that 2962464768 is 2.75).
Attachment #619923 - Flags: feedback?(philringnalda) → feedback+
https://hg.mozilla.org/integration/mozilla-inbound/rev/e1f1d4f79b2d
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/e1f1d4f79b2d
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.