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)

Other
Linux
defect

Tracking

(Not tracked)

VERIFIED FIXED
Q3 11 - Serrano

People

(Reporter: brbaker, Assigned: brbaker)

References

Details

Attachments

(3 files, 5 obsolete files)

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
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
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
Attached patch Add MIPS build to linux-deep (obsolete) (deleted) — Splinter Review
- 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)
Depends on: 504202
Flags: in-testsuite?
Flags: flashplayer-triage+
Attachment #409913 - Flags: review?(dschaffe) → review+
Assignee: nobody → brbaker
Status: NEW → ASSIGNED
Flags: flashplayer-qrb+
Priority: -- → P3
Target Milestone: --- → flash10.1
Target Milestone: flash10.1 → flash10.2
Attached patch Add MIPS build to linux (obsolete) (deleted) — Splinter Review
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)
Attachment #424989 - Flags: review?(dschaffe) → review+
Attached patch avmfeatures update for MIPS (deleted) — Splinter Review
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)
Depends on: 545113
Comment on attachment 425969 [details] [diff] [review] avmfeatures update for MIPS patch will need to be applied to both tamarin-redux and tamarin-argo
Attachment #409913 - Attachment is obsolete: true
Attached patch rev3. Add MIPS build to linux (obsolete) (deleted) — Splinter Review
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)
Attachment #425971 - Flags: review?(dschaffe) → review+
Attachment #425969 - Flags: review?(rreitmai) → review+
Comment on attachment 425969 [details] [diff] [review] avmfeatures update for MIPS Push patch: tr-argo: 3688:63acd0c5916d tr: 3793:7845c032b4f9
Buildbot patch pushed to TR: changeset: 3809:53eba946c870
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
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 → ---
Depends on: 551165
Attached patch Remove -EL and -mips32r2 (deleted) — Splinter Review
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)
Attachment #439529 - Flags: review?(chris)
I thought only mips32r2 was the problem on the 97405?
(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
Attachment #439529 - Flags: review?(dschaffe) → review+
Depends on: 558813
Comment on attachment 439529 [details] [diff] [review] Remove -EL and -mips32r2 Patch pushed: tr-argo: 4008:715d1785e07b tr: 4529:715d1785e07b
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)
rebased patch
Attachment #440287 - Attachment is obsolete: true
Attachment #440289 - Flags: review?(cpeyer)
Attachment #440287 - Flags: review?(cpeyer)
Attachment #440289 - Flags: review?(cpeyer) → review+
Attachment #439529 - Flags: review?(chris)
- 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)
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)
Attachment #441486 - Flags: review?(cpeyer) → review+
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
Follow up to adding MIPS compilation, add the buildstep to compile the builtins changeset: 4580:eb95ae8fd232
Status: REOPENED → RESOLVED
Closed: 15 years ago14 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: