Chromium Code Reviews
Help | Chromium Project | Sign in
(44)

Issue 11095006: Blacklist video and flash acceleration on NVIDIA Linux drivers. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 6 months ago by ccameron1
Modified:
1 year, 6 months ago
CC:
chromium-reviews_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, Vangelis Kokkevis
Visibility:
Public.

Description

Blacklist video and flash acceleration on NVIDIA Linux drivers.

If you create a context when you are low memory, this context may get
into a bad state where geomerty is corrupted and the screen takes seconds
to paint. The context remains in this state until it is destroyed, even
if more memory comes available.

This disabled accelerated compositing to limit the exposure to this bug.

I plan to integrate this into M23. We shouldn't re-enable accelerated
compositing until we have a more robust workaround, or NVIDIA fixes the
context creation bug.

BUG=145600
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=161150

Patch Set 1 #

Total comments: 2

Patch Set 2 : Disable just video decode and flash #

Patch Set 3 : Increment revision #

Patch Set 4 : Disable video acceleration as well #

Total comments: 1

Patch Set 5 : Incorporate review feedback #

Patch Set 6 : Fix comma syntax #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -1 line) Lint Patch
M content/browser/gpu/software_rendering_list.json View 1 2 3 4 5 2 chunks +20 lines, -1 line 0 comments ? errors Download
Commit:

Messages

Total messages: 19
ccameron1
1 year, 6 months ago #1
Zhenyao Mo
https://codereview.chromium.org/11095006/diff/1/content/browser/gpu/software_rendering_list.json File content/browser/gpu/software_rendering_list.json (right): https://codereview.chromium.org/11095006/diff/1/content/browser/gpu/software_rendering_list.json#newcode77 content/browser/gpu/software_rendering_list.json:77: "version": "3.7", You need to update blacklist version for ...
1 year, 6 months ago #2
Ken Russell
Sorry, I can not approve this change. It is far too heavy a hammer. Chrome's ...
1 year, 6 months ago #3
ccameron1
How about we disable accelerated compositing for just Flapper and video, and leave it on ...
1 year, 6 months ago #4
Ken Russell
On 2012/10/08 23:09:10, ccameron1 wrote: > How about we disable accelerated compositing for just Flapper ...
1 year, 6 months ago #5
ccameron1
On 2012/10/08 23:13:00, kbr wrote: > On 2012/10/08 23:09:10, ccameron1 wrote: > > How about ...
1 year, 6 months ago #6
vangelis
On 2012/10/08 23:19:09, ccameron1 wrote: > On 2012/10/08 23:13:00, kbr wrote: > > On 2012/10/08 ...
1 year, 6 months ago #7
Ken Russell
On 2012/10/08 23:39:08, vangelis wrote: > On 2012/10/08 23:19:09, ccameron1 wrote: > > On 2012/10/08 ...
1 year, 6 months ago #8
ccameron1
I'll disable accelerated video decode in M23. I'd feel most comfortable with disabling compositing for ...
1 year, 6 months ago #9
ccameron1
I've updated the patch blacklist accelerated video decode, flash_3d, and flash_stage3d, but I'm going to ...
1 year, 6 months ago #10
ccameron1
I've updated the patch to disable triggering compositing for all video and flash. This will ...
1 year, 6 months ago #11
Ken Russell
LGTM as long as this has been well tested and confirmed to only reduce performance ...
1 year, 6 months ago #12
Ken Russell
Also, please update the synopsis of this CL to accurately reflect what is being done, ...
1 year, 6 months ago #13
jbauman
https://codereview.chromium.org/11095006/diff/3003/content/browser/gpu/software_rendering_list.json File content/browser/gpu/software_rendering_list.json (right): https://codereview.chromium.org/11095006/diff/3003/content/browser/gpu/software_rendering_list.json#newcode856 content/browser/gpu/software_rendering_list.json:856: "video" This should be "accelerated_video", I believe.
1 year, 6 months ago #14
ccameron1
Fixed the label, syntax, and updated the description. I've updated the description, and I've tested ...
1 year, 6 months ago #15
jbauman
lgtm
1 year, 6 months ago #16
ccameron1
Thanks!!
1 year, 6 months ago #17
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/11095006/12002
1 year, 6 months ago #18
I haz the power (commit-bot)
1 year, 6 months ago #19
Change committed as 161150
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6