Commit: 60ea472b116a2e245fa8579355c47eb501bfa20a - hardware-intel-intel-driver (git)

hardware/intel/intel-driver

Commit MetaInfo

• Révision: 60ea472b116a2e245fa8579355c47eb501bfa20a (tree)
• l'heure: 2014-06-05 17:27:32
• Auteur: Gwenole Beauchesne <gwenole.beauchesne@inte...>
• Commiter: Gwenole Beauchesne

Message de Log

decoder: h264: don't allocate bottom DMV buffer on Broadwell.

Broadwell now uses a unique DMV buffer, irrespective of any field
coding mode. The dmv_buffer is not used, so it doesn't need to be
allocated at all.

Signed-off-by: Gwenole Beauchesne <[e-mail]>

Change Summary

• modified: src/gen8_mfd.c

Modification

--- a/src/gen8_mfd.c
+++ b/src/gen8_mfd.c
@@ -78,9 +78,8 @@ gen8_mfd_init_avc_surface(VADriverContextP ctx,
         obj_surface->private_data = gen7_avc_surface;
     }
 
-    gen7_avc_surface->dmv_bottom_flag = (pic_param->pic_fields.bits.field_pic_flag &&
-                                         !pic_param->seq_fields.bits.direct_8x8_inference_flag);
-
+    /* DMV buffers now relate to the whole frame, irrespective of
+       field coding modes */
     if (gen7_avc_surface->dmv_top == NULL) {
         gen7_avc_surface->dmv_top = dri_bo_alloc(i965->intel.bufmgr,
                                                  "direct mv w/r buffer",
@@ -88,15 +87,6 @@ gen8_mfd_init_avc_surface(VADriverContextP ctx,
                                                  0x1000);
         assert(gen7_avc_surface->dmv_top);
     }
-
-    if (gen7_avc_surface->dmv_bottom_flag &&
-        gen7_avc_surface->dmv_bottom == NULL) {
-        gen7_avc_surface->dmv_bottom = dri_bo_alloc(i965->intel.bufmgr,
-                                                    "direct mv w/r buffer",
-                                                    width_in_mbs * height_in_mbs * 128,                                                    
-                                                    0x1000);
-        assert(gen7_avc_surface->dmv_bottom);
-    }
 }
 
 static void