gallium/pipe: add interface update_decoder_target
authorRuijing Dong <ruijing.dong@amd.com>
Tue, 23 May 2023 13:47:59 +0000 (09:47 -0400)
committerMarge Bot <emma+marge@anholt.net>
Tue, 23 May 2023 19:56:24 +0000 (19:56 +0000)
reason:
decoder uses the target buffer address in record
to indentify the reference frames. When target
buffer has changed outside of decoding process,
it has to be updated back to decoder, otherwise
the outdated reference will cause image corruption.

Cc: mesa-stable
Reviewed-by: Boyuan Zhang <Boyuan.Zhang@amd.com>
Reviewed-by: Sil Vilerino <sivileri@microsoft.com>
Signed-off-by: Ruijing Dong <ruijing.dong@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23061>

src/gallium/include/pipe/p_video_codec.h

index c68d8fd..bda37e3 100644 (file)
@@ -133,6 +133,19 @@ struct pipe_video_codec
    int (*get_decoder_fence)(struct pipe_video_codec *codec,
                             struct pipe_fence_handle *fence,
                             uint64_t timeout);
+   /**
+    * Update target buffer address.
+    *
+    * Due to reallocation, target buffer address has changed, and the
+    * changed buffer will need to update to decoder so that when this buffer
+    * used as a reference frame, decoder can obtain its recorded information.
+    * Failed updating this buffer will miss reference frames and
+    * cause image corruption in the sebsequent output.
+    * If no target buffer change, this call is not necessary.
+    */
+   void (*update_decoder_target)(struct pipe_video_codec *codec,
+                                 struct pipe_video_buffer *old,
+                                 struct pipe_video_buffer *updated);
 };
 
 /**