From be673e9719593ee22c16e289fe4d154db3fb7805 Mon Sep 17 00:00:00 2001 From: Brian Date: Wed, 22 Aug 2007 13:33:19 -0600 Subject: [PATCH] Sampler state code now compiles and is built. Not tested yet. --- src/mesa/pipe/i915simple/Makefile | 1 + src/mesa/pipe/i915simple/i915_state.h | 1 + src/mesa/pipe/i915simple/i915_state_sampler.c | 185 ++++++++++++------ 3 files changed, 126 insertions(+), 61 deletions(-) diff --git a/src/mesa/pipe/i915simple/Makefile b/src/mesa/pipe/i915simple/Makefile index bb1e6104d81..7e4e3d413e7 100644 --- a/src/mesa/pipe/i915simple/Makefile +++ b/src/mesa/pipe/i915simple/Makefile @@ -19,6 +19,7 @@ DRIVER_SOURCES = \ i915_state_derived.c \ i915_state_emit.c \ i915_state_fragprog.c \ + i915_state_sampler.c \ i915_strings.c \ i915_prim_emit.c \ i915_tex_layout.c \ diff --git a/src/mesa/pipe/i915simple/i915_state.h b/src/mesa/pipe/i915simple/i915_state.h index 794a251ad88..43058451f29 100644 --- a/src/mesa/pipe/i915simple/i915_state.h +++ b/src/mesa/pipe/i915simple/i915_state.h @@ -42,6 +42,7 @@ struct i915_tracked_state { void i915_update_immediate( struct i915_context *i915 ); void i915_update_dynamic( struct i915_context *i915 ); void i915_update_derived( struct i915_context *i915 ); +void i915_update_samplers( struct i915_context *i915 ); void i915_emit_hardware_state( struct i915_context *i915 ); diff --git a/src/mesa/pipe/i915simple/i915_state_sampler.c b/src/mesa/pipe/i915simple/i915_state_sampler.c index 3e44bba5431..591e3684da2 100644 --- a/src/mesa/pipe/i915simple/i915_state_sampler.c +++ b/src/mesa/pipe/i915simple/i915_state_sampler.c @@ -34,6 +34,7 @@ #include "pipe/p_context.h" #include "pipe/p_state.h" +#include "pipe/p_util.h" #include "i915_state_inlines.h" #include "i915_context.h" @@ -96,18 +97,69 @@ static unsigned translate_mip_filter( unsigned filter ) } } +static unsigned translate_compare_func(unsigned func) +{ + switch (func) { + case PIPE_FUNC_NEVER: + case PIPE_FUNC_LESS: + case PIPE_FUNC_EQUAL: + case PIPE_FUNC_LEQUAL: + case PIPE_FUNC_GREATER: + case PIPE_FUNC_NOTEQUAL: + case PIPE_FUNC_GEQUAL: + case PIPE_FUNC_ALWAYS: + return 0; + default: + assert(0); + return 0; + } +} + -/* Recalculate all state from scratch. Perhaps not the most +static uint +bitcount(uint k) +{ + uint count = 0; + while (k) { + if (k & 1) + count++; + k = k >> 1; + } + return count; +} + + +static boolean +is_power_of_two_texture(const struct pipe_mipmap_tree *mt) +{ + if (bitcount(mt->width0) == 1 && + bitcount(mt->height0) == 1 && + bitcount(mt->depth0) == 1) { + return 1; + } + else + return 0; +} + + +/** + * Compute i915 texture sampling state. + * + * Recalculate all state from scratch. Perhaps not the most * efficient, but this has gotten complex enough that we need * something which is understandable and reliable. + * \param state returns the 3 words of compute state */ -static void update_sampler(struct i915_context *i915, +static void update_sampler(struct i915_context *i915, + uint unit, const struct pipe_sampler_state *sampler, - const struct pipe_surface_state *surface, - unsigned surface_index, - unsigned target, - unsigned *state ) + const struct pipe_mipmap_tree *mt, + const struct pipe_surface *surface, + unsigned state[3] ) { + const unsigned ws = sampler->wrap_s; + const unsigned wt = sampler->wrap_t; + const unsigned wr = sampler->wrap_r; /* Need to do this after updating the maps, which call the * intel_finalize_mipmap_tree and hence can update firstLevel: @@ -119,7 +171,7 @@ static void update_sampler(struct i915_context *i915, mipFilt = translate_mip_filter( sampler->min_mip_filter ); - if (tObj->MaxAnisotropy > 1.0) { + if (sampler->max_anisotropy > 1.0) { minFilt = FILTER_ANISOTROPIC; magFilt = FILTER_ANISOTROPIC; } @@ -134,8 +186,8 @@ static void update_sampler(struct i915_context *i915, state[0] |= ((b << SS2_LOD_BIAS_SHIFT) & SS2_LOD_BIAS_MASK); } - if (surface->format == MESA_FORMAT_YCBCR || - surface->format == MESA_FORMAT_YCBCR_REV) + if (surface->format == PIPE_FORMAT_YCBCR || + surface->format == PIPE_FORMAT_YCBCR_REV) state[0] |= SS2_COLORSPACE_CONVERSION; @@ -155,68 +207,79 @@ static void update_sampler(struct i915_context *i915, (magFilt << SS2_MAG_FILTER_SHIFT)); - if (0) - { - unsigned ws = tObj->WrapS; - unsigned wt = tObj->WrapT; - unsigned wr = tObj->WrapR; - - /* 3D textures don't seem to respect the border color. - * Fallback if there's ever a danger that they might refer to - * it. - * - * Effectively this means fallback on 3D clamp or - * clamp_to_border. - * - * XXX: Check if this is true on i945. - * XXX: Check if this bug got fixed in release silicon. - */ - if (target == PIPE_TEXTURE_3D && - (sampler->min_img_filter != PIPE_TEX_FILTER_NEAREST || - sampler->mag_img_filter != PIPE_TEX_FILTER_NEAREST) && - (ws == GL_CLAMP || - wt == GL_CLAMP || - wr == GL_CLAMP || - ws == GL_CLAMP_TO_BORDER || - wt == GL_CLAMP_TO_BORDER || - wr == GL_CLAMP_TO_BORDER)) { - - if (intel->strict_conformance) { - assert(0); -/* sampler->fallback = true; */ - /* TODO */ - } + /* 3D textures don't seem to respect the border color. + * Fallback if there's ever a danger that they might refer to + * it. + * + * Effectively this means fallback on 3D clamp or + * clamp_to_border. + * + * XXX: Check if this is true on i945. + * XXX: Check if this bug got fixed in release silicon. + */ + if (mt->target == PIPE_TEXTURE_3D && + (sampler->min_img_filter != PIPE_TEX_FILTER_NEAREST || + sampler->mag_img_filter != PIPE_TEX_FILTER_NEAREST) && + (ws == PIPE_TEX_WRAP_CLAMP || + wt == PIPE_TEX_WRAP_CLAMP || + wr == PIPE_TEX_WRAP_CLAMP || + ws == PIPE_TEX_WRAP_CLAMP_TO_BORDER || + wt == PIPE_TEX_WRAP_CLAMP_TO_BORDER || + wr == PIPE_TEX_WRAP_CLAMP_TO_BORDER)) { +#if 0 + if (i915->strict_conformance) { + assert(0); + /* sampler->fallback = true; */ + /* TODO */ } +#endif + } + state[1] = + ((translate_wrap_mode(ws) << SS3_TCX_ADDR_MODE_SHIFT) | + (translate_wrap_mode(wt) << SS3_TCY_ADDR_MODE_SHIFT) | + (translate_wrap_mode(wr) << SS3_TCZ_ADDR_MODE_SHIFT) | + (unit << SS3_TEXTUREMAP_INDEX_SHIFT)); - state[1] = - ((translate_wrap_mode(ws) << SS3_TCX_ADDR_MODE_SHIFT) | - (translate_wrap_mode(wt) << SS3_TCY_ADDR_MODE_SHIFT) | - (translate_wrap_mode(wr) << SS3_TCZ_ADDR_MODE_SHIFT) | - (unit << SS3_TEXTUREMAP_INDEX_SHIFT)); - - if (target != TEXTURE_RECT_BIT) - state[1] |= SS3_NORMALIZED_COORDS; + if (is_power_of_two_texture(mt)) { + state[1] |= SS3_NORMALIZED_COORDS; + } - state[2] = INTEL_PACKCOLOR8888(tObj->_BorderChan[0], - tObj->_BorderChan[1], - tObj->_BorderChan[2], - tObj->_BorderChan[3]); + { + ubyte r = float_to_ubyte(sampler->border_color[0]); + ubyte g = float_to_ubyte(sampler->border_color[1]); + ubyte b = float_to_ubyte(sampler->border_color[2]); + ubyte a = float_to_ubyte(sampler->border_color[3]); + state[2] = I915PACKCOLOR8888(r, g, b, a); + } } void i915_update_samplers( struct i915_context *i915 ) { - int i, dirty = 0, nr = 0; - - for (i = 0; i < I915_TEX_UNITS; i++) { - if (i915->sampler[i].enabled) { /* ??? */ - update_sampler( i915, i, - i915->current.sampler[i] ); - nr++; - dirty |= (1<framebuffer.cbufs[0]; + uint unit; + + i915->current.sampler_enable_nr = 0; + i915->current.sampler_enable_flags = 0x0; + + for (unit = 0; unit < I915_TEX_UNITS; unit++) { + /* determine unit enable/disable by looking for a bound mipmap tree */ + /* could also examine the fragment program? */ + if (i915->texture[unit]) { + update_sampler( i915, + unit, + i915->sampler + unit, /* sampler state */ + i915->texture[unit], /* mipmap tree */ + surface, /* cbuffer info */ + i915->current.sampler[unit] /* the result */ + ); + + i915->current.sampler_enable_nr++; + i915->current.sampler_enable_flags |= (1 << unit); } } -} + i915->hardware_dirty |= I915_HW_SAMPLER; +} -- 2.30.2