From fadc9eaf97c0916d5680c1d3b47e80532ede22c1 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Wed, 4 Jul 2012 10:52:36 -0700 Subject: [PATCH] intel: Add a comment explaining why we early return on matching BO names. Reviewed-by: Chad Versace Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/intel/intel_context.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c index 2494ac7..378859c 100644 --- a/src/mesa/drivers/dri/intel/intel_context.c +++ b/src/mesa/drivers/dri/intel/intel_context.c @@ -893,6 +893,10 @@ intel_process_dri2_buffer(struct intel_context *intel, if (!rb) return; + /* We try to avoid closing and reopening the same BO name, because the first + * use of a mapping of the buffer involves a bunch of page faulting which is + * moderately expensive. + */ if (rb->mt && rb->mt->region && rb->mt->region->name == buffer->name) -- 2.7.4