Closed
Bug 1469496
Opened 6 years ago
Closed 6 years ago
Clear webrender resources when video memory is purged.
Categories
(Core :: Graphics: WebRender, enhancement, P2)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: nical, Assigned: nical)
References
Details
Attachments
(1 file)
6.99 KB,
patch
|
sotaro
:
review+
sotaro
:
feedback+
|
Details | Diff | Splinter Review |
See the corresponding webrender issue (which will be closed since all of the remaining work is on the gecko side: https://github.com/servo/webrender/issues/2792
We just need to check whether gpu-side memory was purged through extensions like NV_robustness_video_memory_purge and use WebRender's ClearCaches API.
Assignee | ||
Comment 1•6 years ago
|
||
Here is a naive attempt at handling the memory purges. Sotaro what do you think? I don't know if it is worth centralizing the device reset and memory purge in one place since the current support we have talks directly to the d3d device, but you know this stuff a lot better than I do.
I haven't properly tested this yet.
Assignee: nobody → nical.bugzilla
Attachment #8986210 -
Flags: feedback?(sotaro.ikeda.g)
Comment 2•6 years ago
|
||
Comment on attachment 8986210 [details] [diff] [review]
Handle video memory purge with WebRender
Review of attachment 8986210 [details] [diff] [review]:
-----------------------------------------------------------------
The patch looks good :) The device reset requests to re-create all compositor sessions and updating d3d devices on windows. But memory purge does not request them. Then it seems simpler just to keep them as separate. By the way, it is nice if we could test memory purge like "Trigger Device Reset" in about:support.
Attachment #8986210 -
Flags: feedback?(sotaro.ikeda.g) → feedback+
Updated•6 years ago
|
Blocks: stage-wr-next
Priority: -- → P2
Assignee | ||
Comment 3•6 years ago
|
||
Comment on attachment 8986210 [details] [diff] [review]
Handle video memory purge with WebRender
I kind of forgot about this one. Let's land it.
Attachment #8986210 -
Flags: review?(sotaro.ikeda.g)
Updated•6 years ago
|
Attachment #8986210 -
Flags: review?(sotaro.ikeda.g) → review+
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/402e30935941
Handle video memory purge with WebRender. r=sotaro
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•