[Libva] [PATCH] decoder: h264: don't allocate bottom DMV buffer on Broadwell.

[Gwenole Beauchesne]

Broadwell now uses a unique DMV buffer, irrespective of any field
coding mode. So, the dmv_bottom buffer is obsolete and doesn't need
to be allocated at all.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
---
 src/gen8_mfd.c | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/gen8_mfd.c b/src/gen8_mfd.c
index 425d094..065302d 100644
--- 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
-- 
1.8.3.2