shithub: libvpx

Info  •  Files  •  Log  •  Branches

Update use_prev_frame_mvs flag in decoder.

Added check to see if last frame was all intra.  This will
eliminate two checks in find_mv_refs_idx().  Also, do not
update the frame mvs if the current frame is all intra.

This improved performance on material with frequent
intra-only frames.

Change-Id: I44a4042c3670ab0d38439d565062a0e2a1ba9d1e

--- a/vp9/common/vp9_onyxc_int.h

+++ b/vp9/common/vp9_onyxc_int.h

@@ -162,7 +162,8 @@

   int show_existing_frame;

   // Flag signaling that the frame is encoded using only INTRA modes.

-  int intra_only;

+  uint8_t intra_only;

+  uint8_t last_intra_only;

   int allow_high_precision_mv;

--- a/vp9/decoder/vp9_decodeframe.c

+++ b/vp9/decoder/vp9_decodeframe.c

@@ -1313,6 +1313,7 @@

   size_t sz;

   cm->last_frame_type = cm->frame_type;

+  cm->last_intra_only = cm->intra_only;

   if (vp9_rb_read_literal(rb, 2) != VP9_FRAME_MARKER)

       vpx_internal_error(&cm->error, VPX_CODEC_UNSUP_BITSTREAM,

@@ -1639,8 +1640,9 @@

   cm->use_prev_frame_mvs = !cm->error_resilient_mode &&

                            cm->width == cm->last_width &&

                            cm->height == cm->last_height &&

-                           !cm->intra_only &&

-                           cm->last_show_frame;

+                           !cm->last_intra_only &&

+                           cm->last_show_frame &&

+                           (cm->last_frame_type != KEY_FRAME);

   vp9_setup_block_planes(xd, cm->subsampling_x, cm->subsampling_y);

--- a/vp9/decoder/vp9_decodemv.c

+++ b/vp9/decoder/vp9_decodemv.c

@@ -596,19 +596,20 @@

   MV_REF* frame_mvs = cm->cur_frame->mvs + mi_row * cm->mi_cols + mi_col;

   int w, h;

-  if (frame_is_intra_only(cm))

+  if (frame_is_intra_only(cm)) {

     read_intra_frame_mode_info(cm, xd, mi_row, mi_col, r);

-  else

+  } else {

     read_inter_frame_mode_info(pbi, xd, tile, mi_row, mi_col, r);

-  for (h = 0; h < y_mis; ++h) {

-    MV_REF *const frame_mv = frame_mvs + h * cm->mi_cols;

-    for (w = 0; w < x_mis; ++w) {

-      MV_REF *const mv = frame_mv + w;

-      mv->ref_frame[0] = mi->mbmi.ref_frame[0];

-      mv->ref_frame[1] = mi->mbmi.ref_frame[1];

-      mv->mv[0].as_int = mi->mbmi.mv[0].as_int;

-      mv->mv[1].as_int = mi->mbmi.mv[1].as_int;

+    for (h = 0; h < y_mis; ++h) {

+      MV_REF *const frame_mv = frame_mvs + h * cm->mi_cols;

+      for (w = 0; w < x_mis; ++w) {

+        MV_REF *const mv = frame_mv + w;

+        mv->ref_frame[0] = mi->mbmi.ref_frame[0];

+        mv->ref_frame[1] = mi->mbmi.ref_frame[1];

+        mv->mv[0].as_int = mi->mbmi.mv[0].as_int;

+        mv->mv[1].as_int = mi->mbmi.mv[1].as_int;

+      }

     }

   }

 }