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

[Paul Berry]

On 5 April 2013 16:51, Paul Berry <stereotype441 at gmail.com> wrote:

> 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?
>

Ok, if this is marked as a candidate for 9.1, then it is:

Reviewed-by: Paul Berry <stereotype441 at gmail.com>


>
>
>> ---
>>  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/20130408/83f1740c/attachment.html>