Closed
Bug 737686
Opened 13 years ago
Closed 13 years ago
Avoid using glTexSubImage on PowerVR
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: jrmuizel, Assigned: jrmuizel)
References
Details
Attachments
(1 file)
|
1.66 KB,
patch
|
gw280
:
review+
|
Details | Diff | Splinter Review |
glTexSubImage causes a readback on PowerVR hardware if the texture has already been uploaded. Disabling CanUploadSubTextures() brings the median upload speed from 26ms to 10ms.
It's possible this may cause bug 727688 again. We'll need to watch for it.
Attachment #607765 -
Flags: review?(gwright)
|
||
Comment 1•13 years ago
|
||
Comment on attachment 607765 [details] [diff] [review]
Disable glTexSubImage2D on PowerVR
Review of attachment 607765 [details] [diff] [review]:
-----------------------------------------------------------------
Lgtm
Attachment #607765 -
Flags: review?(gwright) → review+
|
|
||
Comment 2•13 years ago
|
||
Comment on attachment 607765 [details] [diff] [review]
Disable glTexSubImage2D on PowerVR
Review of attachment 607765 [details] [diff] [review]:
-----------------------------------------------------------------
Lgtm
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → jmuizelaar
|
|
Assignee | |
Comment 3•13 years ago
|
||
|
||
Comment 4•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
|
||
Comment 5•13 years ago
|
||
This is causing Bug 737437 on PowerVR devices.
|
|
||
Comment 6•13 years ago
|
||
are we running out of memory?
|
|
Assignee | |
Updated•13 years ago
|
Blocks: checkerboarding
You need to log in
before you can comment on or make changes to this bug.
Description
•