From: Topi Pohjolainen Date: Sun, 15 Jan 2017 09:35:55 +0000 (+0200) Subject: i965/miptree: Drop BO_ALLOC_FOR_RENDER in intel_miptree_alloc_mcs() X-Git-Tag: upstream/18.1.0~8634 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=67b44a8423b9bf616387309761831caf3aa629c8;p=platform%2Fupstream%2Fmesa.git 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 --- diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index c08f143..42fbed3 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) {