[Libva] [PATCH v4 intel-driver 01/11] decoder: h264: don't deallocate surface storage of older frames.

Gwenole Beauchesne gb.devel at gmail.com
Thu Jun 5 16:46:24 PDT 2014


Drop the optimization whereby surfaces that are no longer marked as
reference and that were already displayed are to be destroyed. This
is wrong mainly for two reasons:

1. The surface was displayed... once but it may still be needed for
   subsequent operations like displaying it again, using it for a
   transcode pipeline (encode) for instance, etc.

2. The new set of ReferenceFrames[] correspond to the active set of
   reference frames used for decoding the current slice. In presence
   of Multiview Coding (MVC), that could correspond to the current
   view, in view order index, but the surface may still be needed
   for decoding the next view with the same view_id, while also
   decoding other views with another set of reference frames for them.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
---
 src/i965_decoder_utils.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/src/i965_decoder_utils.c b/src/i965_decoder_utils.c
index ae5fd76..2b4651e 100644
--- a/src/i965_decoder_utils.c
+++ b/src/i965_decoder_utils.c
@@ -451,15 +451,6 @@ intel_update_avc_frame_store_index(VADriverContextP ctx,
 
         /* remove it from the internal DPB */
         if (!found) {
-            struct object_surface *obj_surface = frame_store[i].obj_surface;
-            
-            obj_surface->flags &= ~SURFACE_REFERENCED;
-
-            if ((obj_surface->flags & SURFACE_ALL_MASK) == SURFACE_DISPLAYED) {
-                obj_surface->flags &= ~SURFACE_REF_DIS_MASK;
-                i965_destroy_surface_storage(obj_surface);
-            }
-
             frame_store[i].surface_id = VA_INVALID_ID;
             frame_store[i].frame_store_id = -1;
             frame_store[i].obj_surface = NULL;
-- 
1.9.1



More information about the Libva mailing list