Closed
Bug 581213
Opened 14 years ago
Closed 14 years ago
Run reftests with Direct3D 9 accelerated layers on Windows desktop platform
Categories
(Release Engineering :: General, defect, P2)
Tracking
(blocking2.0 beta5+)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta5+ |
People
(Reporter: joe, Assigned: armenzg)
References
Details
Attachments
(2 files, 4 obsolete files)
(deleted),
patch
|
catlee
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
catlee
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
Just like bug 580410 - we need to automatically run our reftests with Direct3D 9 accelerated layers. First step is to make sure that they run through to completion, which I will take on before tossing this to RelEng.
Reporter | ||
Updated•14 years ago
|
Reporter | ||
Comment 1•14 years ago
|
||
Here's the way to run with hardware acceleration:
export MOZ_ACCELERATED=11
make reftests
Everything should 'just work' more or less (of course right now we get reftest
failures).
Assignee: joe → armenzg
Reporter | ||
Updated•14 years ago
|
blocking2.0: --- → beta5+
Assignee | ||
Updated•14 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P3
Assignee | ||
Comment 2•14 years ago
|
||
I should get to this before the end of the week.
My understanding is that this has higher priority than bug 580410 and that is why I will tackle this first.
Priority: P3 → P2
Assignee | ||
Comment 3•14 years ago
|
||
I am going to enable OpenGL as well on the patch for this bug.
Tons of oranges but it seems that my patches are getting the job done.
http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTest/1281723827.1281724773.25470.gz&fulltext=1
I will have to get my hands on a staging environment for several hours (I could only borrow one for a short time) to make sure that all the suites requested run on all platforms.
Assignee | ||
Comment 4•14 years ago
|
||
Assignee | ||
Comment 5•14 years ago
|
||
These patches have run through staging but as I mentioned on my previous comment. I want to make sure it runs for all platforms rather than just Linux64.
This patch also enables a11y for macosx which is wanted on bug 524775 and I willing to test it.
Assignee | ||
Updated•14 years ago
|
Attachment #465766 -
Flags: feedback?(catlee)
Assignee | ||
Updated•14 years ago
|
Attachment #465767 -
Flags: feedback?(catlee)
Comment 6•14 years ago
|
||
Comment on attachment 465767 [details] [diff] [review]
WIP [buildbot-configs] add Direct 3D 9 and OpenGL test suites plus a11y for macosx
Looks sane. I'm not keen on 'OpenGL' - 'opengl' would be more consistent with the rest of our names.
Attachment #465767 -
Flags: feedback?(catlee) → feedback+
Comment 7•14 years ago
|
||
Comment on attachment 465766 [details] [diff] [review]
WIP [buildbotcustom] add Direct 3D 9 and OpenGL suites
Same comments about uppercase names vs. lowercase. I think keeping them all lower case is cleaner.
Attachment #465766 -
Flags: feedback?(catlee) → feedback+
Assignee | ||
Comment 8•14 years ago
|
||
Could you guys please have a look at this Direct 3D run and tell me if it looks alright?
http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTest/1281969809.1281970795.8356.gz
Assignee | ||
Comment 9•14 years ago
|
||
(In reply to comment #8)
> Could you guys please have a look at this Direct 3D run and tell me if it looks
> alright?
> http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTest/1281969809.1281970795.8356.gz
The correct log is:
http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTest/1281973439.1281974524.25694.gz
Assignee | ||
Comment 10•14 years ago
|
||
I have correct the naming.
Attachment #465766 -
Attachment is obsolete: true
Attachment #466397 -
Flags: review?(catlee)
Reporter | ||
Comment 11•14 years ago
|
||
There are curiously few failures in this log, but the failures aren't known so it's probably just fine! Let's go ahead with this.
Assignee | ||
Comment 12•14 years ago
|
||
This patch leaves the a11y suite *not* being run for Mac since it did not work (more info on bug 524775).
(The way of disabling is a little unorthodox. I welcome suggestions)
I chatted with joe and we are going to keep 10.6 without this test suite until they can loan a 10.6 machine from us to see why the test suite did not run correctly (see bug 580410 for more details). Having it enabled would have wasted CPU time since the job would only finish after 330 seconds timeout.
Attachment #465767 -
Attachment is obsolete: true
Attachment #466401 -
Flags: review?(catlee)
Updated•14 years ago
|
Attachment #466397 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 13•14 years ago
|
||
The difference with the previous patch is that we won't enable OpenGL for 10.5 until we figure out the issues with regard the resolution.
This also deals with the unorthodox way of getting read of a11y test suite for Mac.
Attachment #466401 -
Attachment is obsolete: true
Attachment #466777 -
Flags: review?(catlee)
Attachment #466401 -
Flags: review?(catlee)
Assignee | ||
Comment 14•14 years ago
|
||
OK this should address your last comment on IRC with regards of making it a little more general (NOTE we can generalize even more but this does what we need now).
Again, I am only adding new test suites for Fedora and Windows.
I would like to deploy this on tomorrow's downtime if possible.
Attachment #466777 -
Attachment is obsolete: true
Attachment #466826 -
Flags: review?(catlee)
Attachment #466777 -
Flags: review?(catlee)
Updated•14 years ago
|
Attachment #466826 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 15•14 years ago
|
||
Comment on attachment 466397 [details] [diff] [review]
[buildbotcustom] Add steps for direct3d and opengl
http://hg.mozilla.org/build/buildbotcustom/rev/994771f2d07f
Attachment #466397 -
Flags: checked-in+
Assignee | ||
Comment 16•14 years ago
|
||
Comment on attachment 466826 [details] [diff] [review]
[configs] enable OpenGL for Fedora and Direct3D for Windows
http://hg.mozilla.org/build/buildbot-configs/rev/77df5479bcc9
Attachment #466826 -
Flags: checked-in+
Assignee | ||
Comment 17•14 years ago
|
||
This has been deployed.
I have kept the test suite hidden on the Firefox tree until the GFX team brings down the count of permanent oranges to zero. AFAIK it is now on GFX's hand to keep track of these oranges and fix them (bug 586460).
If you find any infrastructure problem please file a separate bug or reopen this one.
Once the test suite is free of permanent oranges please go ahead and un-hide it. By then we should have bug 588454 fixed which currently prevents us from showing on TBPL this test suite.
Ping me on IRC or comment on this bug if you have any following questions.
AFAIK this is RESOLVED FIXED.
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 18•14 years ago
|
||
This is perfect, Armen. Thank you.
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•