[Chromium] Ganesh's Texture Cache is too small

• Component: Canvas
• Status: RESOLVED FIXED

[Jeff Timanus]

The current Ganesh texture cache is initialized to cache no more than 50 Mb of texture data.  For some demos this is too constrained.

See parallel tracking issue:  crbug.com/110468

[Jeff Timanus]

Created attachment 123201 [details]
Patch

[Vangelis Kokkevis]

Comment on attachment 123201 [details]
Patch

I would feel more comfortable increasing the cache size if we were able to clear the cache when a tab is hidden like we do for the compositor.  In order to do that we may need to switch to using a shared graphics context per webview instead of renderer process or track how many tabs the renderer is dealing with and flush the cache when they are all hidden.

Alternatively, would it be possible to age textures in the cache so that we only evict ones that haven't been used for a while and use this texture limit as a soft limit that can be exceeded if more space is needed to keep the contents of a single frame?

Longer term we need to have a better strategy for doing global tracking of memory like nduca@ suggests.

[Brian Salomon]

(In reply to comment #2)
> (From update of attachment 123201 [details])
> I would feel more comfortable increasing the cache size if we were able to clear the cache when a tab is hidden like we do for the compositor.  In order to do that we may need to switch to using a shared graphics context per webview instead of renderer process or track how many tabs the renderer is dealing with and flush the cache when they are all hidden.
> 
> Alternatively, would it be possible to age textures in the cache so that we only evict ones that haven't been used for a while and use this texture limit as a soft limit that can be exceeded if more space is needed to keep the contents of a single frame?
> 
> Longer term we need to have a better strategy for doing global tracking of memory like nduca@ suggests.


We could make the cache size be a hard limit but periodically walk the cache and purge anything that hasn't been used in a while even when we are under-budget.

[Jeff Timanus]

Comment on attachment 123201 [details]
Patch

From the discussion we had offline today, I think we should let this patch through.

The plan is to introduce the capability of explicitly purging the Ganesh texture cache when all of the tabs associated with a render process have been backgrounded/hidden.

[Jeff Timanus]

Can a reviewer please have a look at this change, and mark it cq+?

[Stephen White]

Comment on attachment 123201 [details]
Patch

OK.  r=me

[WebKit Review Bot]

Comment on attachment 123201 [details]
Patch

Clearing flags on attachment: 123201

Committed r106164: <http://trac.webkit.org/changeset/106164>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.