[Mesa-dev] [PATCH 3/3] i965/miptree: Use the new simple alloc_tiled for CCS buffers

[Manolova, Plamena]

Looks good to me :) The series is:

Reviewed-by: Plamena Manolova <plamena.manolova at intel.com>

On Wed, Jun 14, 2017 at 2:19 AM, Jason Ekstrand <jason at jlekstrand.net>
wrote:

> ---
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> index f24b0b5..7991f81 100644
> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> @@ -1684,13 +1684,8 @@ intel_miptree_alloc_non_msrt_mcs(struct
> brw_context *brw,
>     const uint32_t alloc_flags =
>        is_lossless_compressed ? 0 : BO_ALLOC_FOR_RENDER;
>
> -   /* ISL has stricter set of alignment rules then the drm allocator.
> -    * Therefore one can pass the ISL dimensions in terms of bytes instead
> of
> -    * trying to recalculate based on different format block sizes.
> -    */
> -   buf->bo = brw_bo_alloc_tiled_2d(brw->bufmgr, "ccs-miptree",
> -                                   buf->pitch, buf->size / buf->pitch,
> -                                   1, I915_TILING_Y, &buf->pitch,
> alloc_flags);
> +   buf->bo = brw_bo_alloc_tiled(brw->bufmgr, "ccs-miptree", buf->size,
> +                                I915_TILING_Y, buf->pitch, alloc_flags);
>     if (!buf->bo) {
>        free(buf);
>        free(aux_state);
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170614/e01b3495/attachment.html>