[Mesa-dev] [PATCH] intel: Allocate hiz in intel_renderbuffer_move_to_temp()

[Paul Berry]

On 5 April 2013 15:28, Chad Versace <chad.versace at linux.intel.com> wrote:

> When moving the renderbuffer to a new miptree, we neglected to allocate
> the hiz buffer for the new miptree. Oops.
>
> Fixes all Piglit depthstencil-render-miplevels tests from crash to pass on
> Sandybridge.
>
> CC: Paul Berry <stereotype441 at gmail.com>
> CC: Eric Anholt <eric at anholt.net>
> Signed-off-by: Chad Versace <chad.versace at linux.intel.com>
>

I haven't had a chance to review this yet, but:

Candidate for the 9.1 stable release branch?


> ---
>  src/mesa/drivers/dri/intel/intel_fbo.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c
> b/src/mesa/drivers/dri/intel/intel_fbo.c
> index b91d6e0..2977568 100644
> --- a/src/mesa/drivers/dri/intel/intel_fbo.c
> +++ b/src/mesa/drivers/dri/intel/intel_fbo.c
> @@ -1010,6 +1010,10 @@ intel_renderbuffer_move_to_temp(struct
> intel_context *intel,
>                                   irb->mt->num_samples,
>                                   false /* force_y_tiling */);
>
> +   if (intel->vtbl.is_hiz_depth_format(intel, new_mt->format)) {
> +      intel_miptree_alloc_hiz(intel, new_mt, irb->mt->num_samples);
> +   }
> +
>     intel_miptree_copy_teximage(intel, intel_image, new_mt, invalidate);
>
>     intel_miptree_reference(&irb->mt, intel_image->mt);
> --
> 1.8.1.4
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130405/ea9ce7cb/attachment.html>