WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 64497
Don't restore WebGL context if it was guilty of a graphics reset
https://bugs.webkit.org/show_bug.cgi?id=64497
Summary
Don't restore WebGL context if it was guilty of a graphics reset
Kenneth Russell
Reported
2011-07-13 18:03:38 PDT
The most significant outstanding problem with WebGL implementations is that of denial of service. Currently, if a WebGL application issues a very expensive draw call, it may cause the GPU to lock up for long periods of time. Windows' TDR and GL_ARB_robustness' "lose context on reset" functionality are the mechanisms on which detection of graphics reset will be built. In order to take a first step toward solving the DoS problem, the WebGLRenderingContext needs to know as precisely as possible why a graphics reset occurred. It seems clear that if the WebGL context was guilty of causing the reset, we definitely should not attempt to restore it once lost. More work on policy will need to follow; for example, a port might want to prompt the user if the reset occurred for unknown reasons, or maybe even if the context was guilty of causing the reset. This bug covers only the exposure of the real graphics reset status and a first pass at shutting down guilty contexts.
Attachments
Patch
(10.19 KB, patch)
2011-07-13 18:27 PDT
,
Kenneth Russell
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from ec2-cr-linux-01
(5.86 MB, application/zip)
2011-07-13 19:44 PDT
,
WebKit Review Bot
no flags
Details
Archive of layout-test-results from ec2-cr-linux-02
(5.77 MB, application/zip)
2011-07-13 20:32 PDT
,
WebKit Review Bot
no flags
Details
Patch
(12.02 KB, patch)
2011-07-14 17:51 PDT
,
Kenneth Russell
senorblanco
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Kenneth Russell
Comment 1
2011-07-13 18:27:32 PDT
Created
attachment 100748
[details]
Patch
WebKit Review Bot
Comment 2
2011-07-13 19:44:31 PDT
Comment on
attachment 100748
[details]
Patch
Attachment 100748
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/9019866
New failing tests: fast/canvas/webgl/context-lost.html
WebKit Review Bot
Comment 3
2011-07-13 19:44:37 PDT
Created
attachment 100759
[details]
Archive of layout-test-results from ec2-cr-linux-01 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-01 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
WebKit Review Bot
Comment 4
2011-07-13 20:32:51 PDT
Comment on
attachment 100748
[details]
Patch
Attachment 100748
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/9050412
New failing tests: fast/canvas/webgl/context-lost.html
WebKit Review Bot
Comment 5
2011-07-13 20:32:58 PDT
Created
attachment 100761
[details]
Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Kenneth Russell
Comment 6
2011-07-14 17:51:17 PDT
Created
attachment 100905
[details]
Patch
Kenneth Russell
Comment 7
2011-07-14 17:52:46 PDT
Revised patch updates test expectations for fast/canvas/webgl/context-lost.html.
Stephen White
Comment 8
2011-07-15 12:19:39 PDT
Comment on
attachment 100905
[details]
Patch Looks good. r=me
Kenneth Russell
Comment 9
2011-07-15 14:23:02 PDT
Committed
r91112
: <
http://trac.webkit.org/changeset/91112
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug