<div dir="ltr"><div>This and precursor patches </div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> </div><div class="gmail_extra"> <div class="gmail_quote">On Wed, Jul 19, 2017 at 12:43 PM, Topi Pohjolainen <<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">v2 (Jason): Â Â - Don't trigger miptree re-creation in vain later on with ISL Â Â Â based. Core GL uses zero to indicate single sampled while Â Â Â ISL uses one - this would cause intel_miptree_match_image() Â Â Â to always fail. Â Â - Now that native miptree is already using sample number of Â Â Â one, there is no need for MAX2() when converting to ISL. Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> (v1) Reviewed-by:Â Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> (v1) Signed-off-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>> --- Â src/mesa/drivers/dri/i965/brw_blorp.cÂ Â Â Â Â Â | 16 ++++----- Â src/mesa/drivers/dri/i965/brw_context.cÂ Â Â Â Â |Â 2 +- Â src/mesa/drivers/dri/i965/brw_meta_util.cÂ Â Â Â |Â 2 +- Â src/mesa/drivers/dri/i965/brw_tex_layout.cÂ Â Â Â |Â 4 +-- Â src/mesa/drivers/dri/i965/brw_wm.cÂ Â Â Â Â Â Â Â |Â 4 +-- Â src/mesa/drivers/dri/i965/brw_wm_surface_state.c |Â 2 +- Â src/mesa/drivers/dri/i965/intel_blit.cÂ Â Â Â Â Â |Â 4 +-- Â src/mesa/drivers/dri/i965/intel_fbo.cÂ Â Â Â Â Â |Â 4 +-- Â src/mesa/drivers/dri/i965/intel_mipmap_tree.cÂ Â | 46 ++++++++++++------------ Â src/mesa/drivers/dri/i965/intel_mipmap_tree.hÂ Â |Â 6 ---- Â src/mesa/drivers/dri/i965/intel_pixel_copy.cÂ Â Â |Â 2 +- Â 11 files changed, 43 insertions(+), 49 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index 2776872d92..474dfc61c1 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp.c +++ b/src/mesa/drivers/dri/i965/brw_blorp.c @@ -135,7 +135,7 @@ blorp_surf_for_miptree(struct brw_context *brw, Â Â Â Â Â Â Â Â Â Â Â Â struct isl_surf tmp_surfs[1]) Â { Â Â if (mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY) { -Â Â Â const unsigned num_samples =Â mt->num_samples; +Â Â Â const unsigned num_samples = mt->surf.samples; <div><div class="h5">Â Â Â Â for (unsigned i = 0; i < num_layers; i++) { Â Â Â Â Â for (unsigned s = 0; s < num_samples; s++) { Â Â Â Â Â Â Â const unsigned phys_layer = (start_layer + i) * num_samples + s; @@ -275,7 +275,7 @@ swizzle_to_scs(GLenum swizzle) Â * Note: if the src (or dst) is a 2D multisample array texture on Gen7+ using Â * INTEL_MSAA_LAYOUT_UMS or INTEL_MSAA_LAYOUT_CMS, src_layer (dst_layer) is Â * the physical layer holding sample 0.Â So, for example, if - * src_mt->num_samples == 4, then logical layer n corresponds to src_layer == + * src_mt->surf.samples == 4, then logical layer n corresponds to src_layer == Â * 4*n. Â */ Â void @@ -296,9 +296,9 @@ brw_blorp_blit_miptrees(struct brw_context *brw, Â Â DBG("%s from %dx %s mt %p %d %d (%f,%f) (%f,%f)" Â Â Â Â "to %dx %s mt %p %d %d (%f,%f) (%f,%f) (flip %d,%d)\n", Â Â Â Â __func__, -Â Â Â Â src_mt->num_samples, _mesa_get_format_name(src_mt->format), src_mt, +Â Â Â Â src_mt->surf.samples, _mesa_get_format_name(src_mt->format), src_mt, Â Â Â Â src_level, src_layer, src_x0, src_y0, src_x1, src_y1, -Â Â Â Â dst_mt->num_samples, _mesa_get_format_name(dst_mt->format), dst_mt, +Â Â Â Â dst_mt->surf.samples, _mesa_get_format_name(dst_mt->format), dst_mt, Â Â Â Â dst_level, dst_layer, dst_x0, dst_y0, dst_x1, dst_y1, Â Â Â Â mirror_x, mirror_y); @@ -318,7 +318,7 @@ brw_blorp_blit_miptrees(struct brw_context *brw, Â Â Â * R32_FLOAT, so only the contents of the red channel matters. Â Â Â */ Â Â if (brw->gen == 6 && -Â Â Â Â src_mt->num_samples > 1 && dst_mt->num_samples <= 1 && +Â Â Â Â src_mt->surf.samples > 1 && dst_mt->surf.samples <= 1 && Â Â Â Â src_mt->format == dst_mt->format && Â Â Â Â (dst_format == MESA_FORMAT_L_FLOAT32 || Â Â Â Â Â dst_format == MESA_FORMAT_I_FLOAT32)) { @@ -375,9 +375,9 @@ brw_blorp_copy_miptrees(struct brw_context *brw, Â Â DBG("%s from %dx %s mt %p %d %d (%d,%d) %dx%d" Â Â Â Â "to %dx %s mt %p %d %d (%d,%d)\n", Â Â Â Â __func__, -Â Â Â Â src_mt->num_samples, _mesa_get_format_name(src_mt->format), src_mt, +Â Â Â Â src_mt->surf.samples, _mesa_get_format_name(src_mt->format), src_mt, Â Â Â Â src_level, src_layer, src_x, src_y, src_width, src_height, -Â Â Â Â dst_mt->num_samples, _mesa_get_format_name(dst_mt->format), dst_mt, +Â Â Â Â dst_mt->surf.samples, _mesa_get_format_name(dst_mt->format), dst_mt, Â Â Â Â dst_level, dst_layer, dst_x, dst_y); Â Â struct isl_surf tmp_surfs[2]; @@ -564,7 +564,7 @@ brw_blorp_copytexsubimage(struct brw_context *brw, Â Â struct intel_mipmap_tree *dst_mt = intel_image->mt; Â Â /* There is support for only up to eight samples. */ -Â Â if (src_mt->num_samples > 8 || dst_mt->num_samples > 8) +Â Â if (src_mt->surf.samples > 8 || dst_mt->surf.samples > 8) Â Â Â Â return false; Â Â if (_mesa_get_format_base_format(src_rb->Format) != diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c </div></div>index 303c658543..0ef601d342 100644 --- a/src/mesa/drivers/dri/i965/brw_context.c +++ b/src/mesa/drivers/dri/i965/brw_context.c @@ -1218,7 +1218,7 @@ intel_resolve_for_dri2_flush(struct brw_context *brw, Â Â Â Â rb = intel_get_renderbuffer(fb, buffers[i]); Â Â Â Â if (rb == NULL || rb->mt == NULL) Â Â Â Â Â continue; -Â Â Â if (rb->mt->num_samples == 1) { +Â Â Â if (rb->mt->surf.samples == 1) { Â Â Â Â Â assert(rb->mt_layer == 0 && rb->mt_level == 0 && Â Â Â Â Â Â Â Â Â rb->layer_count == 1); Â Â Â Â Â intel_miptree_prepare_access(brw, rb->mt, 0, 1, 0, 1, false, false); diff --git a/src/mesa/drivers/dri/i965/brw_meta_util.c b/src/mesa/drivers/dri/i965/brw_meta_util.c index 7b5a11811b..ea14f44790 100644 --- a/src/mesa/drivers/dri/i965/brw_meta_util.c +++ b/src/mesa/drivers/dri/i965/brw_meta_util.c @@ -298,7 +298,7 @@ brw_is_color_fast_clear_compatible(struct brw_context *brw, Â Â Â * fast clear because it's very likely to be immediately resolved. Â Â Â */ Â Â if (brw->gen >= 9 && -Â Â Â Â mt->num_samples == 1 && +Â Â Â Â mt->surf.samples == 1 && <div><div class="h5">Â Â Â Â ctx->Color.sRGBEnabled && Â Â Â Â _mesa_get_srgb_format_linear(mt->format) != mt->format) Â Â Â Â return false; diff --git a/src/mesa/drivers/dri/i965/brw_tex_layout.c b/src/mesa/drivers/dri/i965/brw_tex_layout.c index 91e94ee4a0..5be73282dc 100644 --- a/src/mesa/drivers/dri/i965/brw_tex_layout.c +++ b/src/mesa/drivers/dri/i965/brw_tex_layout.c @@ -100,7 +100,7 @@ intel_vertical_texture_alignment_unit(struct brw_context *brw, Â Â if (brw->gen >= 8) Â Â Â Â return 4; -Â Â if (mt->num_samples > 1) +Â Â if (mt->surf.samples > 1) Â Â Â Â return 4; Â Â GLenum base_format = _mesa_get_format_base_format(mt->format); @@ -521,7 +521,7 @@ brw_miptree_choose_tiling(struct brw_context *brw, Â Â Â Â return I915_TILING_NONE; Â Â } -Â Â if (mt->num_samples > 1) { +Â Â if (mt->surf.samples > 1) { Â Â Â Â /* From p82 of the Sandy Bridge PRM, dw3[1] of SURFACE_STATE ("Tiled Â Â Â Â * Surface"): Â Â Â Â * diff --git a/src/mesa/drivers/dri/i965/brw_wm.c b/src/mesa/drivers/dri/i965/brw_wm.c index 9e1dcee8fd..c9c4504590 100644 --- a/src/mesa/drivers/dri/i965/brw_wm.c +++ b/src/mesa/drivers/dri/i965/brw_wm.c @@ -398,12 +398,12 @@ brw_populate_sampler_prog_key_data(struct gl_context *ctx, Â Â Â Â Â Â */ Â Â Â Â Â if (intel_tex->mt->aux_usage == ISL_AUX_USAGE_MCS) { Â Â Â Â Â Â Â assert(brw->gen >= 7); -Â Â Â Â Â Â assert(intel_tex->mt->num_samples > 1); +Â Â Â Â Â Â assert(intel_tex->mt->surf.samples > 1); Â Â Â Â Â Â Â assert(intel_tex->mt->mcs_buf); Â Â Â Â Â Â Â assert(intel_tex->mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY); Â Â Â Â Â Â Â key->compressed_multisample_layout_mask |= 1 << s; -Â Â Â Â Â Â if (intel_tex->mt->num_samples >= 16) { +Â Â Â Â Â Â if (intel_tex->mt->surf.samples >= 16) { Â Â Â Â Â Â Â Â assert(brw->gen >= 9); Â Â Â Â Â Â Â Â key->msaa_16 |= 1 << s; Â Â Â Â Â Â Â } diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c index 67dc9d8943..00f0a72b1f 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c @@ -1036,7 +1036,7 @@ gen4_update_renderbuffer_surface(struct brw_context *brw, Â Â surf[3] = (brw_get_surface_tiling_bits(mt->tiling) | Â Â Â Â Â Â Â (mt->pitch - 1) << BRW_SURFACE_PITCH_SHIFT); -Â Â surf[4] = brw_get_surface_num_multisamples(mt->num_samples); +Â Â surf[4] = brw_get_surface_num_multisamples(mt->surf.samples); Â Â assert(brw->has_surface_tile_offset || (tile_x == 0 && tile_y == 0)); Â Â /* Note that the low bits of these fields are missing, so diff --git a/src/mesa/drivers/dri/i965/intel_blit.c b/src/mesa/drivers/dri/i965/intel_blit.c index 2471380a0a..226c4f23b7 100644 --- a/src/mesa/drivers/dri/i965/intel_blit.c +++ b/src/mesa/drivers/dri/i965/intel_blit.c @@ -298,7 +298,7 @@ intel_miptree_blit(struct brw_context *brw, Â Â Â Â Â Â Â Â Â Â GLenum logicop) Â { Â Â /* The blitter doesn't understand multisampling at all. */ -Â Â if (src_mt->num_samples > 1 || dst_mt->num_samples > 1) +Â Â if (src_mt->surf.samples > 1 || dst_mt->surf.samples > 1) Â Â Â Â return false; Â Â /* No sRGB decode or encode is done by the hardware blitter, which is @@ -371,7 +371,7 @@ intel_miptree_copy(struct brw_context *brw, Â Â Â Â Â Â Â Â Â Â uint32_t src_width, uint32_t src_height) Â { Â Â /* The blitter doesn't understand multisampling at all. */ -Â Â if (src_mt->num_samples > 1 || dst_mt->num_samples > 1) +Â Â if (src_mt->surf.samples > 1 || dst_mt->surf.samples > 1) Â Â Â Â return false; Â Â if (src_mt->format == MESA_FORMAT_S_UINT8) diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c b/src/mesa/drivers/dri/i965/intel_fbo.c </div></div>index b7de49f22e..87dc32673c 100644 --- a/src/mesa/drivers/dri/i965/intel_fbo.c +++ b/src/mesa/drivers/dri/i965/intel_fbo.c @@ -533,7 +533,7 @@ intel_renderbuffer_update_wrapper(struct brw_context *brw, Â Â irb->mt_layer = layer; Â Â const unsigned layer_multiplier = -Â Â Â mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY ? mt->num_samples : 1; +Â Â Â mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY ? mt->surf.samples : 1; Â Â if (!layered) { Â Â Â Â irb->layer_count = 1; @@ -970,7 +970,7 @@ intel_renderbuffer_move_to_temp(struct brw_context *brw, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â intel_image->base.Base.TexFormat, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â 0, 0, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â width, height, 1, -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â irb->mt->num_samples, +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â irb->mt->surf.samples, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â layout_flags); Â Â if (!invalidate) diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index 338e965737..418ec3cf06 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c @@ -168,7 +168,7 @@ intel_miptree_supports_ccs(struct brw_context *brw, Â Â Â Â return false; Â Â /* This function applies only to non-multisampled render targets. */ -Â Â if (mt->num_samples > 1) +Â Â if (mt->surf.samples > 1) Â Â Â Â return false; Â Â /* MCS is only supported for color buffers */ @@ -191,7 +191,7 @@ intel_miptree_supports_ccs(struct brw_context *brw, Â Â Â Â Â * accidentally reject a multisampled surface here. We should have Â Â Â Â Â * rejected it earlier by explicitly checking the sample count. Â Â Â Â Â */ -Â Â Â assert(mt->num_samples == 1); +Â Â Â assert(mt->surf.samples == 1); Â Â } Â Â /* Handle the hardware restrictions... @@ -384,7 +384,7 @@ intel_miptree_create_layout(struct brw_context *brw, Â Â mt->supports_fast_clear = false; Â Â mt->aux_state = NULL; Â Â mt->cpp = _mesa_get_format_bytes(format); -Â Â mt->num_samples = num_samples; +Â Â mt->surf.samples = num_samples; Â Â mt->compressed = _mesa_is_format_compressed(format); Â Â mt->surf.msaa_layout = ISL_MSAA_LAYOUT_NONE; Â Â mt->refcount = 1; @@ -604,7 +604,7 @@ intel_miptree_choose_aux_usage(struct brw_context *brw, Â Â assert(mt->aux_usage == ISL_AUX_USAGE_NONE); Â Â const unsigned no_flags = 0; -Â Â if (mt->num_samples > 1 && is_mcs_supported(brw, mt->format, no_flags)) { +Â Â if (mt->surf.samples > 1 && is_mcs_supported(brw, mt->format, no_flags)) { Â Â Â Â assert(mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY); Â Â Â Â mt->aux_usage = ISL_AUX_USAGE_MCS; Â Â } else if (intel_tiling_supports_ccs(brw, mt->tiling) && @@ -1369,8 +1369,8 @@ intel_miptree_match_image(struct intel_mipmap_tree *mt, Â Â } Â Â int level_depth = mt->level[level].depth; -Â Â if (mt->num_samples > 1 && mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY) -Â Â Â level_depth /= mt->num_samples; +Â Â if (mt->surf.samples > 1 && mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY) +Â Â Â level_depth /= mt->surf.samples; Â Â /* Test image dimensions against the base level image adjusted for Â Â Â * minification.Â This will also catch images not present in the @@ -1383,7 +1383,7 @@ intel_miptree_match_image(struct intel_mipmap_tree *mt, Â Â } Â Â /* Core uses sample number of zero to indicate single-sampled. */ -Â Â if (MAX2(image->NumSamples, 1) != mt->num_samples) +Â Â if (MAX2(image->NumSamples, 1) != mt->surf.samples) Â Â Â Â return false; Â Â return true; @@ -2004,8 +2004,8 @@ intel_miptree_alloc_aux(struct brw_context *brw, Â Â case ISL_AUX_USAGE_MCS: Â Â Â Â assert(_mesa_is_format_color_format(mt->format)); -Â Â Â assert(mt->num_samples > 1); -Â Â Â if (!intel_miptree_alloc_mcs(brw, mt, mt->num_samples)) +Â Â Â assert(mt->surf.samples > 1); +Â Â Â if (!intel_miptree_alloc_mcs(brw, mt, mt->surf.samples)) Â Â Â Â Â return false; Â Â Â Â return true; @@ -2017,7 +2017,7 @@ intel_miptree_alloc_aux(struct brw_context *brw, Â Â case ISL_AUX_USAGE_CCS_E: Â Â Â Â assert(_mesa_is_format_color_format(mt->format)); -Â Â Â assert(mt->num_samples == 1); +Â Â Â assert(mt->surf.samples == 1); Â Â Â Â if (!intel_miptree_alloc_ccs(brw, mt)) Â Â Â Â Â return false; Â Â Â Â return true; @@ -2066,7 +2066,7 @@ intel_miptree_sample_with_hiz(struct brw_context *brw, Â Â Â * There is no such blurb for 1D textures, but there is sufficient evidence Â Â Â * that this is broken on SKL+. Â Â Â */ -Â Â return (mt->num_samples == 1 && +Â Â return (mt->surf.samples == 1 && Â Â Â Â Â Â mt->target != GL_TEXTURE_3D && Â Â Â Â Â Â mt->target != GL_TEXTURE_1D /* gen9+ restriction */); Â } @@ -2126,7 +2126,7 @@ intel_miptree_check_color_resolve(const struct brw_context *brw, Â Â Â Â Â Â (level == 0 && mt->first_level == 0 && mt->last_level == 0)); Â Â /* Compression of arrayed msaa surfaces is supported. */ -Â Â if (mt->num_samples > 1) +Â Â if (mt->surf.samples > 1) Â Â Â Â return; Â Â /* Fast color clear is supported for non-msaa arrays only on Gen8+. */ @@ -2466,7 +2466,7 @@ intel_miptree_prepare_access(struct brw_context *brw, Â Â Â Â if (!mt->mcs_buf) Â Â Â Â Â return; -Â Â Â if (mt->num_samples > 1) { +Â Â Â if (mt->surf.samples > 1) { Â Â Â Â Â /* Nothing to do for MSAA */ Â Â Â Â Â assert(aux_supported && fast_clear_supported); Â Â Â Â } else { @@ -2515,7 +2515,7 @@ intel_miptree_finish_write(struct brw_context *brw, Â Â Â Â if (!mt->mcs_buf) Â Â Â Â Â return; -Â Â Â if (mt->num_samples > 1) { +Â Â Â if (mt->surf.samples > 1) { Â Â Â Â Â for (uint32_t a = 0; a < num_layers; a++) { Â Â Â Â Â Â Â intel_miptree_finish_mcs_write(brw, mt, level, start_layer + a, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â written_with_aux); @@ -2547,7 +2547,7 @@ intel_miptree_get_aux_state(const struct intel_mipmap_tree *mt, Â Â if (_mesa_is_format_color_format(mt->format)) { Â Â Â Â assert(mt->mcs_buf != NULL); -Â Â Â assert(mt->num_samples == 1 || +Â Â Â assert(mt->surf.samples == 1 || Â Â Â Â Â Â Â mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY); Â Â } else if (mt->format == MESA_FORMAT_S_UINT8) { Â Â Â Â unreachable("Cannot get aux state for stencil"); @@ -2568,7 +2568,7 @@ intel_miptree_set_aux_state(struct brw_context *brw, Â Â if (_mesa_is_format_color_format(mt->format)) { Â Â Â Â assert(mt->mcs_buf != NULL); -Â Â Â assert(mt->num_samples == 1 || +Â Â Â assert(mt->surf.samples == 1 || Â Â Â Â Â Â Â mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY); Â Â } else if (mt->format == MESA_FORMAT_S_UINT8) { Â Â Â Â unreachable("Cannot get aux state for stencil"); @@ -2623,7 +2623,7 @@ intel_miptree_prepare_texture_slices(struct brw_context *brw, Â { Â Â bool aux_supported, clear_supported; Â Â if (_mesa_is_format_color_format(mt->format)) { -Â Â Â if (mt->num_samples > 1) { +Â Â Â if (mt->surf.samples > 1) { Â Â Â Â Â aux_supported = clear_supported = true; Â Â Â Â } else { Â Â Â Â Â aux_supported = can_texture_with_ccs(brw, mt, view_format); @@ -2690,7 +2690,7 @@ intel_miptree_prepare_render(struct brw_context *brw, Â Â Â * enabled because otherwise the surface state will be programmed with Â Â Â * the linear equivalent format anyway. Â Â Â */ -Â Â if (brw->gen == 9 && srgb_enabled && mt->num_samples == 1 && +Â Â if (brw->gen == 9 && srgb_enabled && mt->surf.samples == 1 && Â Â Â Â _mesa_get_srgb_format_linear(mt->format) != mt->format) { Â Â Â Â /* Lossless compression is not supported for SRGB formats, it @@ -2753,7 +2753,7 @@ intel_miptree_make_shareable(struct brw_context *brw, Â Â Â * reached for multisample buffers. Â Â Â */ Â Â assert(mt->surf.msaa_layout == ISL_MSAA_LAYOUT_NONE || -Â Â Â Â Â mt->num_samples == 1); +Â Â Â Â Â mt->surf.samples == 1); Â Â intel_miptree_prepare_access(brw, mt, 0, INTEL_REMAINING_LEVELS, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â 0, INTEL_REMAINING_LAYERS, false, false); @@ -2931,7 +2931,7 @@ intel_update_r8stencil(struct brw_context *brw, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â src->logical_width0, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â src->logical_height0, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â src->logical_depth0, -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â src->num_samples, +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â src->surf.samples, Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r8stencil_flags); Â Â Â Â assert(mt->r8stencil_mt); Â Â } @@ -3579,7 +3579,7 @@ intel_miptree_map(struct brw_context *brw, Â { Â Â struct intel_miptree_map *map; -Â Â assert(mt->num_samples == 1 || mt->surf.samples == 1); +Â Â assert(mt->surf.samples == 1); Â Â map = intel_miptree_attach_map(mt, level, slice, x, y, w, h, mode); Â Â if (!map){ @@ -3625,7 +3625,7 @@ intel_miptree_unmap(struct brw_context *brw, Â { Â Â struct intel_miptree_map *map = mt->level[level].slice[slice].map; -Â Â assert(mt->num_samples == 1 || mt->surf.samples == 1); +Â Â assert(mt->surf.samples == 1); Â Â if (!map) Â Â Â Â return; @@ -3794,7 +3794,7 @@ intel_miptree_get_isl_surf(struct brw_context *brw, Â Â } Â Â surf->levels = mt->last_level - mt->first_level + 1; -Â Â surf->samples = mt->num_samples; +Â Â surf->samples = mt->surf.samples; <div class="HOEnZb"><div class="h5"> Â Â surf->size = 0; /* TODO */ Â Â surf->alignment = 0; /* TODO */ diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h index 89590d2ba2..fde9a022fd 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h @@ -414,12 +414,6 @@ struct intel_mipmap_tree Â Â /** Bytes per pixel (or bytes per block if compressed) */ Â Â GLuint cpp; -Â Â /** -Â Â * @see RENDER_SURFACE_STATE.NumberOfMultisamples -Â Â * @see 3DSTATE_MULTISAMPLE.NumberOfMultisamples -Â Â */ -Â Â GLuint num_samples; - Â Â bool compressed; Â Â /** diff --git a/src/mesa/drivers/dri/i965/intel_pixel_copy.c b/src/mesa/drivers/dri/i965/intel_pixel_copy.c index 3ffd68c048..3f8df3002f 100644 --- a/src/mesa/drivers/dri/i965/intel_pixel_copy.c +++ b/src/mesa/drivers/dri/i965/intel_pixel_copy.c @@ -101,7 +101,7 @@ do_blit_copypixels(struct gl_context * ctx, Â Â Â Â return false; Â Â } -Â Â if (draw_irb->mt->num_samples > 1 || read_irb->mt->num_samples > 1) { +Â Â if (draw_irb->mt->surf.samples > 1 || read_irb->mt->surf.samples > 1) { Â Â Â Â perf_debug("glCopyPixels() fallback: multisampled buffers\n"); Â Â Â Â return false; Â Â } -- 2.11.0 _______________________________________________ mesa-dev mailing list <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> </div></div></blockquote></div> </div>