Closed
Bug 524129
Opened 15 years ago
Closed 14 years ago
Update x-platform build script to target MIPS
Categories
(Tamarin Graveyard :: Build Config, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
Q3 11 - Serrano
People
(Reporter: brbaker, Assigned: brbaker)
References
Details
Attachments
(3 files, 5 obsolete files)
(deleted),
patch
|
rreitmai
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
dschaffe
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
cpeyer
:
review+
|
Details | Diff | Splinter Review |
1) Need to update the x-platform compile script to be able to target the MIPS platform.
2) integrate the building of the avmshell for MIPS into the buildbot system
Assignee | ||
Comment 1•15 years ago
|
||
This compiler is known to work for compiling the shell for MIPS:
mips-linux-gnu-g++ (Sourcery G++ Lite 4.3-154) 4.3.3
http://www.codesourcery.com/sgpp/lite/mips/portal/release824
Assignee | ||
Comment 2•15 years ago
|
||
Updated patches are available via bug 504202 that will get the x-platform build script working for mips-linux builds.
https://bug504202.bugzilla.mozilla.org/attachment.cgi?id=409763
Assignee | ||
Comment 3•15 years ago
|
||
- add cross compilation of the MIPS target to the linux-deep builder. Binaries are posted to asteam but are not run or checked in any way, just making sure that it compiles without error.
Attachment #409913 -
Flags: review?(dschaffe)
Assignee | ||
Updated•15 years ago
|
Flags: in-testsuite?
Flags: flashplayer-triage+
Updated•15 years ago
|
Attachment #409913 -
Flags: review?(dschaffe) → review+
Updated•15 years ago
|
Assignee: nobody → brbaker
Status: NEW → ASSIGNED
Flags: flashplayer-qrb+
Priority: -- → P3
Target Milestone: --- → flash10.1
Assignee | ||
Comment 4•15 years ago
|
||
rebase patch to tr 3721:eda0633ab9ef
- MIPS will only be built in the tamarin-redux builds as it will not be supported in argo.
- NOTE: do not build MIPS in the sandbox for now since we want to make sure that ARGO can be pushed into the sandbox
Attachment #424989 -
Flags: review?(dschaffe)
Updated•15 years ago
|
Attachment #424989 -
Flags: review?(dschaffe) → review+
Assignee | ||
Comment 5•15 years ago
|
||
Need to have AVMPLUS_MIPS as it is needed in nanojit/nanojit.h:57
#elif defined AVMPLUS_MIPS
#define NANOJIT_MIPS
#else
so that it will include the proper MIPS implemenation in nanojit/Native.h:67
#elif defined(NANOJIT_MIPS)
#include "NativeMIPS.h"
#else
Attachment #425969 -
Flags: review?(rreitmai)
Assignee | ||
Comment 6•15 years ago
|
||
Comment on attachment 425969 [details] [diff] [review]
avmfeatures update for MIPS
patch will need to be applied to both tamarin-redux and tamarin-argo
Assignee | ||
Updated•15 years ago
|
Attachment #409913 -
Attachment is obsolete: true
Assignee | ||
Comment 7•15 years ago
|
||
Add MIPS compilation to tr-argo, tr and sandbox.
This patch relies on:
patch: https://bug524129.bugzilla.mozilla.org/attachment.cgi?id=425969
bug: https://bugzilla.mozilla.org/show_bug.cgi?id=545113
Attachment #424989 -
Attachment is obsolete: true
Attachment #425971 -
Flags: review?(dschaffe)
Updated•15 years ago
|
Attachment #425971 -
Flags: review?(dschaffe) → review+
Updated•15 years ago
|
Attachment #425969 -
Flags: review?(rreitmai) → review+
Assignee | ||
Comment 8•15 years ago
|
||
Comment on attachment 425969 [details] [diff] [review]
avmfeatures update for MIPS
Push patch:
tr-argo: 3688:63acd0c5916d
tr: 3793:7845c032b4f9
Comment 9•15 years ago
|
||
Buildbot patch pushed to TR:
changeset: 3809:53eba946c870
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•15 years ago
|
||
Reopened bug since the patch to enable this in buildbot needs to be reverted since MIPS is not currently compiling.
reverted change in tr 3948:d2502cc1b180
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 11•15 years ago
|
||
Remove -EL and -mips32r2 from the the compiler options for mips. This can be set if wanted via CXX etc if somebody wants to. Removing now since the hardware that we are putting into the build system does NOT support using these flags so remove them and make them generic
>$ CXX="mips-linux-gnu-g++ -EL -mips32r2"
Attachment #439529 -
Flags: review?(dschaffe)
Assignee | ||
Updated•15 years ago
|
Attachment #439529 -
Flags: review?(chris)
Comment 12•15 years ago
|
||
I thought only mips32r2 was the problem on the 97405?
Assignee | ||
Comment 13•15 years ago
|
||
(In reply to comment #12)
> I thought only mips32r2 was the problem on the 97405?
No need to use -EL as we are using a little-endian compiler
Updated•15 years ago
|
Attachment #439529 -
Flags: review?(dschaffe) → review+
Assignee | ||
Comment 14•15 years ago
|
||
Comment on attachment 439529 [details] [diff] [review]
Remove -EL and -mips32r2
Patch pushed:
tr-argo: 4008:715d1785e07b
tr: 4529:715d1785e07b
Assignee | ||
Comment 15•14 years ago
|
||
Getting prepared to add MIPS compilation into the build system. Will ONLY add to tamarin-redux builders since the patches will only land in tamarin-redux and not argo.
If the patches do end up in argo then we can added the compilation to the argo builders and the sandbox.
Attachment #425971 -
Attachment is obsolete: true
Attachment #440287 -
Flags: review?(cpeyer)
Assignee | ||
Comment 16•14 years ago
|
||
rebased patch
Attachment #440287 -
Attachment is obsolete: true
Attachment #440289 -
Flags: review?(cpeyer)
Attachment #440287 -
Flags: review?(cpeyer)
Updated•14 years ago
|
Attachment #440289 -
Flags: review?(cpeyer) → review+
Assignee | ||
Updated•14 years ago
|
Attachment #439529 -
Flags: review?(chris)
Assignee | ||
Comment 17•14 years ago
|
||
- SSH scripts for remote calling are in bug# 559038 (Attachment #441485 [details] [diff])
- adds compilation of MIPS to linux host machine
- runs smoke testing of the shell on MIPS machine, uses the set of tests to run as defined in test/runsmokes-arm-android.txt
Attachment #440289 -
Attachment is obsolete: true
Attachment #441486 -
Flags: review?(cpeyer)
Assignee | ||
Comment 18•14 years ago
|
||
Comment on attachment 441486 [details] [diff] [review]
rev5. add mips compilation and smoke testing to build system
- only adds MIPS to tamarin-redux as this is the only branch with the MIPS patches
- does NOT add MIPS to the sandbox builders since they must still work for the argo branch. Can easily add to the sandbox if it is thought that nobody will be running sandbox builds from argo branch other than QE (can take the MIPS buildslave offline and run sandbox to not run into issues)
Updated•14 years ago
|
Attachment #441486 -
Flags: review?(cpeyer) → review+
Assignee | ||
Comment 19•14 years ago
|
||
Comment on attachment 441486 [details] [diff] [review]
rev5. add mips compilation and smoke testing to build system
Patch pushed to tamarin-redux as 4579:9ce007525ae4
Assignee | ||
Comment 20•14 years ago
|
||
Follow up to adding MIPS compilation, add the buildstep to compile the builtins
changeset: 4580:eb95ae8fd232
Status: REOPENED → RESOLVED
Closed: 15 years ago → 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•