From eb1497e968bd4a0edc1606e8a6f708fab3248828 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Fri, 21 Jul 2017 12:29:30 -0700 Subject: [PATCH] i965/bufmgr: Allocate BO pages outside of the kernel's locking. Suggested by Chris Wilson. v2: Set the write domain to 0 (suggested by Chris). Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/brw_bufmgr.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c b/src/mesa/drivers/dri/i965/brw_bufmgr.c index 46da53d3530..cd85874ea52 100644 --- a/src/mesa/drivers/dri/i965/brw_bufmgr.c +++ b/src/mesa/drivers/dri/i965/brw_bufmgr.c @@ -382,6 +382,19 @@ retry: if (bo_set_tiling_internal(bo, tiling_mode, stride)) goto err_free; + + /* Calling set_domain() will allocate pages for the BO outside of the + * struct mutex lock in the kernel, which is more efficient than waiting + * to create them during the first execbuf that uses the BO. + */ + struct drm_i915_gem_set_domain sd = { + .handle = bo->gem_handle, + .read_domains = I915_GEM_DOMAIN_CPU, + .write_domain = 0, + }; + + if (drmIoctl(bo->bufmgr->fd, DRM_IOCTL_I915_GEM_SET_DOMAIN, &sd) != 0) + goto err_free; } bo->name = name; -- 2.30.2