2,100 instances of "robustness marked as unsupported" emitted from gfx/gl/GLContextFeatures.cpp during linux64 debug testing

RESOLVED FIXED in Firefox 50

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: erahm, Assigned: jgilbert)

Tracking

(Blocks: 1 bug)

Trunk
mozilla50
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

3 years ago
> 2131 WARNING: robustness marked as unsupported: file gfx/gl/GLContextFeatures.cpp, line 896

This warning [1] shows up in the following test suites:

>   1681 - [TC] Linux64 mochitest-gl gl
>    122 - [TC] Linux64 reftest-e10s-2 R-e10s2
>    122 - [TC] Linux64 reftest-3 R3
>     40 - [TC] Linux64 mochitest-plain-e10s-2 2
>     40 - [TC] Linux64 mochitest-2 2
>     17 - [TC] Linux64 mochitest-devtools-chrome-5 dt5
>     15 - [TC] Linux64 mochitest-plain-e10s-3 3
>     15 - [TC] Linux64 mochitest-3 3
>     13 - [TC] Linux64 crashtest e10s C
>     13 - [TC] Linux64 crashtest C
>      7 - [TC] Linux64 mochitest-browser-chrome-7 bc7
>      7 - [TC] Linux64 mochitest-browser-chrome-e10s-3 bc3
>      7 - [TC] Linux64 mochitest-browser-chrome-e10s-7 bc7
>      7 - [TC] Linux64 mochitest-browser-chrome-4 bc4
>      7 - [TC] Linux64 mochitest-chrome-1 c1
>      3 - [TC] Linux64 mochitest-media mda
>      3 - [TC] Linux64 mochitest-media-e10s mda
>      3 - [TC] Linux64 mochitest-browser-chrome-1 bc1
>      3 - [TC] Linux64 firefox-ui-tests functional en-US
>      3 - [TC] Linux64 firefox-ui-tests functional e10s en-US
>      3 - [TC] Linux64 mochitest-browser-chrome-e10s-4 bc4

It shows up in 541 tests. A few of the most prevalent:

>     99 -        dom/canvas/test/webgl-mochitest/test_backbuffer_channels.html
>     16 -        dom/canvas/test/webgl-mochitest/test_webgl_request_mismatch.html
>     14 -        dom/canvas/test/webgl-conformance/_wrappers/test_conformance__context__premultiplyalpha-test.html
>     11 -        dom/canvas/test/webgl-conformance/_wrappers/test_conformance__context__context-attributes-alpha-depth-stencil-antialias.html
>     11 -        dom/canvas/test/test_bug1215072.html
>     11 - [e10s] dom/canvas/test/test_bug1215072.html
>     10 -        dom/canvas/test/webgl-conformance/_wrappers/test_conformance__renderbuffers__framebuffer-object-attachment.html
>      8 -        devtools/client/canvasdebugger/test/browser_profiling-webgl.js
>      7 -        dom/canvas/test/webgl-conformance/_wrappers/test_conformance__context__context-lost-restored.html
>      7 - [e10s] toolkit/modules/tests/browser/browser_Troubleshoot.js

[1] https://hg.mozilla.org/mozilla-central/annotate/6a441b2b2997/gfx/gl/GLContextFeatures.cpp#l896
(Reporter)

Comment 1

3 years ago
This regression showed up on central in this push:
https://hg.mozilla.org/mozilla-central/pushloghtml?changeset=fc15477ce628599519cb0055f52cc195d640dc94

I'm guessing it's:
https://hg.mozilla.org/mozilla-central/rev/ca8fae075ef3

Which is bug 1262265.
Flags: needinfo?(jgilbert)
(Assignee)

Comment 2

3 years ago
Thanks for the heads-up.
(Assignee)

Updated

3 years ago
Assignee: nobody → jgilbert
Flags: needinfo?(jgilbert)
This has also started showing up in the console when starting Nightly on OS X.
(Assignee)

Comment 4

2 years ago
Created attachment 8755082 [details]
MozReview Request: Bug 1267879 - r?jrmuizel - Fix WGL robustness.

From 2fbb818c78f0a8093f2fa061cb04d610badb87d9 Mon Sep 17 00:00:00 2001
---
 gfx/gl/GLContextProviderWGL.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Review commit: https://reviewboard.mozilla.org/r/54384/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54384/
Attachment #8755082 - Flags: review?(jmuizelaar)
Attachment #8755083 - Flags: review?(jmuizelaar)
(Assignee)

Comment 5

2 years ago
Created attachment 8755083 [details]
MozReview Request:  r?jrmuizel - Only check robustness if it's supported.

From 15596597a92b139c93fa319433f72d1d3349ee4a Mon Sep 17 00:00:00 2001
---
 gfx/gl/GLContext.cpp | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

Review commit: https://reviewboard.mozilla.org/r/54386/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54386/
(Assignee)

Updated

2 years ago
Flags: needinfo?(jmuizelaar)
Comment on attachment 8755082 [details]
MozReview Request: Bug 1267879 - r?jrmuizel - Fix WGL robustness.

https://reviewboard.mozilla.org/r/54384/#review56044
Attachment #8755082 - Flags: review?(jmuizelaar) → review+
Comment on attachment 8755083 [details]
MozReview Request:  r?jrmuizel - Only check robustness if it's supported.

https://reviewboard.mozilla.org/r/54386/#review56046
Attachment #8755083 - Flags: review?(jmuizelaar) → review+

Comment 8

2 years ago
Pushed by jgilbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4c2a18cac92b
Fix WGL robustness. - r=jrmuizel
https://hg.mozilla.org/integration/mozilla-inbound/rev/4f3df5eadf59
Only check robustness if it's supported. - r=jrmuizel
(Assignee)

Comment 10

2 years ago
Looks like this is exacerbating a possible mismanagement of context-loss-handler lifetimes.
Flags: needinfo?(jmuizelaar)
(Assignee)

Updated

2 years ago
Depends on: 1280507

Comment 11

2 years ago
Pushed by jgilbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/04d79227660a
Fix WGL robustness. - r=jrmuizel
https://hg.mozilla.org/integration/mozilla-inbound/rev/ebf87300031f
Only check robustness if it's supported. - r=jrmuizel
Backed out for failing talos glterrain on Windows:

https://hg.mozilla.org/integration/mozilla-inbound/rev/cd9485fc83bf4056b6764f215f990fe9b84ba415
https://hg.mozilla.org/integration/mozilla-inbound/rev/807b210b03bda01e9cdda57adcf1422496856a6a

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=ebf87300031f092a06fd997a35ddd3aedc907a4c
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=31004080&repo=mozilla-inbound
15:20:49     INFO -  Running cycle 1/1 for glterrain test...
15:20:49     INFO -  TEST-INFO | started process 644 (C:\slave\test\build\application\firefox\firefox -profile c:\users\cltbld~1.t-w\appdata\local\temp\tmp4nluy0\profile -tp file:\C:\slave\test\build\tests\talos\talos\tests\webgl\glterrain.manifest.develop -tpchrome -tpnoisy -tploadnocache -tpcycles 1 -tppagecycles 25)
15:20:50     INFO -  PROCESS | 644 | [GFX1-]: Invalid size in UpdateRenderTarget Size(124,0), 0
15:21:01     INFO -  PROCESS | 644 | RSS: Main: 194777088
15:21:01     INFO -  PROCESS | 644 |
15:21:02     INFO -  PROCESS | 644 | RUNTIME ASSERT: Uninitialized GL function: fGetGraphicsResetStatus
15:21:04     INFO -  TEST-INFO | 644: exit 1
15:21:04     INFO -  TEST-UNEXPECTED-ERROR | glterrain | Could not find report in browser output: [('tsformat', ('__start_report', '__end_report')), ('tpformat', ('__start_tp_report', '__end_tp_report'))]
15:21:04    ERROR -  Traceback (most recent call last):
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\run_tests.py", line 202, in run_tests
15:21:04     INFO -      talos_results.add(mytest.runTest(browser_config, test))
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\ttest.py", line 70, in runTest
15:21:04     INFO -      return self._runTest(browser_config, test_config, setup)
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\ttest.py", line 232, in _runTest
15:21:04     INFO -      else None)
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\results.py", line 94, in add
15:21:04     INFO -      global_counters=self.global_counters
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\results.py", line 311, in __init__
15:21:04     INFO -      self.parse()
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\results.py", line 338, in parse
15:21:04     INFO -      % self.report_tokens)
15:21:04     INFO -    File "C:\slave\test\build\tests\talos\talos\results.py", line 322, in error
15:21:04     INFO -      raise utils.TalosError(message)
15:21:04     INFO -  TalosError: Could not find report in browser output: [('tsformat', ('__start_report', '__end_report')), ('tpformat', ('__start_tp_report', '__end_tp_report'))]
Flags: needinfo?(jgilbert)

Comment 14

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/bba7d02cdb8c
https://hg.mozilla.org/mozilla-central/rev/b2272d35d5f8
https://hg.mozilla.org/mozilla-central/rev/b0383b457801
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
(Assignee)

Updated

2 years ago
Flags: needinfo?(jgilbert)
You need to log in before you can comment on or make changes to this bug.