From 8037c0b69c483d94854bd9adc8fb23902077bc13 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Fri, 30 Aug 2013 13:03:52 -0700 Subject: [PATCH] i965: Always allocate validated miptrees from level 0. No change in copies during a piglit run, but it's one less first_level != 0 in our codebase. Reviewed-by: Chad Versace --- src/mesa/drivers/dri/i965/intel_tex_validate.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_tex_validate.c b/src/mesa/drivers/dri/i965/intel_tex_validate.c index 42533bbac73..3ad0feb9265 100644 --- a/src/mesa/drivers/dri/i965/intel_tex_validate.c +++ b/src/mesa/drivers/dri/i965/intel_tex_validate.c @@ -66,7 +66,7 @@ intel_finalize_mipmap_tree(struct brw_context *brw, GLuint unit) int validate_last_level = intelObj->_MaxLevel; /* Skip the loop over images in the common case of no images having - * changed. But if the GL_BASE_LEVEL / GL_MAX_LEVEL change to something we + * changed. But if the GL_BASE_LEVEL or GL_MAX_LEVEL change to something we * haven't looked at, then we do need to look at those new images. */ if (!intelObj->needs_validate && @@ -99,16 +99,15 @@ intel_finalize_mipmap_tree(struct brw_context *brw, GLuint unit) intel_miptree_get_dimensions_for_image(&firstImage->base.Base, &width, &height, &depth); - perf_debug("Creating new %s %dx%dx%d %d..%d miptree to handle finalized " - "texture miptree.\n", + perf_debug("Creating new %s %dx%dx%d %d-level miptree to handle " + "finalized texture miptree.\n", _mesa_get_format_name(firstImage->base.Base.TexFormat), - width, height, depth, - validate_first_level, validate_last_level); + width, height, depth, validate_last_level + 1); intelObj->mt = intel_miptree_create(brw, intelObj->base.Target, firstImage->base.Base.TexFormat, - validate_first_level, + 0, /* first_level */ validate_last_level, width, height, -- 2.30.2