On Sat, Oct 8, 2011 at 1:44 AM, Jose Fonseca <jfons...@vmware.com> wrote: > > > ----- Original Message ----- >> NOTE: This is a candidate for the stable branches. >> --- >> src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c | 7 +++++++ >> 1 files changed, 7 insertions(+), 0 deletions(-) >> >> diff --git a/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c >> b/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c >> index 58721c0..0e6896a 100644 >> --- a/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c >> +++ b/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c >> @@ -324,6 +324,13 @@ pb_cache_manager_create_buffer(struct pb_manager >> *_mgr, >> return NULL; >> >> buf->buffer = mgr->provider->create_buffer(mgr->provider, size, >> desc); > > > This looks OK, but shouldn't we try flushing the cache first (i.e., invoke > pb_cache_manager_flush) before flushing the inner buffer manager?
That's what the patch does. mgr->base.flush is equal to pb_cache_manager_flush, but the function is declared later in the code, so I called it this way. Marek _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev