From 67b44a8423b9bf616387309761831caf3aa629c8 Mon Sep 17 00:00:00 2001 From: Topi Pohjolainen Date: Sun, 15 Jan 2017 11:35:55 +0200 Subject: i965/miptree: Drop BO_ALLOC_FOR_RENDER in intel_miptree_alloc_mcs() because buffers get unconditionally initialised by cpu writing. Reviewed-by: Jason Ekstrand Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index c08f14314bc..42fbed34b51 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c @@ -1678,7 +1678,12 @@ intel_miptree_alloc_mcs(struct brw_context *brw, assert(isl_surf_get_mcs_surf(&brw->isl_dev, &temp_main_surf, &temp_mcs_surf)); - const uint32_t alloc_flags = BO_ALLOC_FOR_RENDER; + /* Buffer needs to be initialised requiring the buffer to be immediately + * mapped to cpu space for writing. Therefore do not use the gpu access + * flag which can cause an unnecessary delay if the backing pages happened + * to be just used by the GPU. + */ + const uint32_t alloc_flags = 0; mt->mcs_buf = intel_alloc_aux_buffer(brw, "mcs-miptree", &temp_mcs_surf, alloc_flags, mt); if (!mt->mcs_buf) { -- cgit v1.2.3