[Mesa-dev] [PATCH V3] radeonsi/uvd: fix planar formats broken since f70f6baaa3bb0f8b280ac2eaea69bb

[Benedikt Schemmer]

From: Benedikt Schemmer <b...@besd.de>
Date: Sat, 30 Sep 2017 18:05:43 +0200
Subject: [PATCH V3] radeonsi/uvd: fix planar formats broken since 
f70f6baaa3bb0f8b280ac2eaea69bb

 V1: Marek Olsak
 V2: remove code duplication and one unnecessary variable, minor whitespace fix
 V3: ensure valid resource format when debugging (is assumed to be valid in 
release) (suggested by Christian König)

---
 src/gallium/drivers/radeonsi/si_uvd.c | 41 +++++++++++++++--------------------
 1 file changed, 17 insertions(+), 24 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_uvd.c 
b/src/gallium/drivers/radeonsi/si_uvd.c
index 4e8250664c..3cb45075c4 100644
--- a/src/gallium/drivers/radeonsi/si_uvd.c
+++ b/src/gallium/drivers/radeonsi/si_uvd.c
@@ -64,30 +64,23 @@ struct pipe_video_buffer *si_video_buffer_create(struct 
pipe_context *pipe,
        template.width = align(tmpl->width, VL_MACROBLOCK_WIDTH);
        template.height = align(tmpl->height / array_size, 
VL_MACROBLOCK_HEIGHT);

-       vl_video_buffer_template(&templ, &template, resource_formats[0], 1, 
array_size, PIPE_USAGE_DEFAULT, 0);
        /* TODO: get tiling working */
-       templ.bind = PIPE_BIND_LINEAR;
-       resources[0] = (struct r600_texture *)
-               pipe->screen->resource_create(pipe->screen, &templ);
-       if (!resources[0])
-               goto error;
-
-       if (resource_formats[1] != PIPE_FORMAT_NONE) {
-               vl_video_buffer_template(&templ, &template, 
resource_formats[1], 1, array_size, PIPE_USAGE_DEFAULT, 1);
-               templ.bind = PIPE_BIND_LINEAR;
-               resources[1] = (struct r600_texture *)
-                       pipe->screen->resource_create(pipe->screen, &templ);
-               if (!resources[1])
-                       goto error;
-       }

-       if (resource_formats[2] != PIPE_FORMAT_NONE) {
-               vl_video_buffer_template(&templ, &template, 
resource_formats[2], 1, array_size, PIPE_USAGE_DEFAULT, 2);
-               templ.bind = PIPE_BIND_LINEAR;
-               resources[2] = (struct r600_texture *)
-                       pipe->screen->resource_create(pipe->screen, &templ);
-               if (!resources[2])
-                       goto error;
+       assert(resource_formats[0] != PIPE_FORMAT_NONE);
+
+       for (i = 0; i < VL_NUM_COMPONENTS; ++i) {
+               if (resource_formats[i] != PIPE_FORMAT_NONE) {
+                       vl_video_buffer_template(&templ, &template,
+                                                resource_formats[i], 1,
+                                                array_size, 
PIPE_USAGE_DEFAULT, i);
+                       /* Set PIPE_BIND_SHARED to avoid reallocation in 
r600_texture_get_handle,
+                        * which can't handle joined surfaces. */
+                       templ.bind = PIPE_BIND_LINEAR | PIPE_BIND_SHARED;
+                       resources[i] = (struct r600_texture *)
+                                       
pipe->screen->resource_create(pipe->screen, &templ);
+                       if (!resources[i])
+                               goto error;
+               }
        }

        for (i = 0; i < VL_NUM_COMPONENTS; ++i) {
@@ -159,8 +152,8 @@ struct pipe_video_codec *si_uvd_create_decoder(struct 
pipe_context *context,
        struct si_context *ctx = (struct si_context *)context;
        bool vcn = (ctx->b.family == CHIP_RAVEN) ? true : false;

-        if (templ->entrypoint == PIPE_VIDEO_ENTRYPOINT_ENCODE)
-                return si_vce_create_encoder(context, templ, ctx->b.ws, 
si_vce_get_buffer);
+       if (templ->entrypoint == PIPE_VIDEO_ENTRYPOINT_ENCODE)
+               return si_vce_create_encoder(context, templ, ctx->b.ws, 
si_vce_get_buffer);

        return (vcn) ?  radeon_create_decoder(context, templ) :
                si_common_uvd_create_decoder(context, templ, si_uvd_set_dtb);
-- 
2.11.0

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev