[Intel-gfx] [PATCH] intel: Limit re-use cache size

[Keith Packard]

On Mon, 2008-12-15 at 17:49 -0800, Eric Anholt wrote:
> On Mon, 2008-12-15 at 15:08 -0800, Keith Packard wrote:
> > This limits reuse to 16MB per cache level to try and reduce application
> > memory consumption.
> 
> Is 16MB based on any measurements at all?

Nope. It's random.

> I don't like this patch.  I'd prefer to have us figure out bugs that
> make the cache grow overly large, and free old buffers (which we *still*
> haven't done), instead of having static limits that'll just hide failure
> so that you end up with both poor performance and excessive memory
> consumption.

No argument; I've been using this patch to see what impact it has on
performance, but I'm not strongly in favor of actually using it.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20081215/0ae48f6f/attachment.sig>