shithub: libvpx

Download patch

ref: 4934e52e437686dae73fa5549e6ef1c7213d251d
parent: 5cc8df5bcfe49fbfca21ee57401c7807c048751b
author: Marco Paniconi <marpan@google.com>
date: Fri Apr 6 01:41:56 EDT 2018

Revert "vp9-svc: Fix to first superframe when inter_layer is off."

This reverts commit 5cc8df5bcfe49fbfca21ee57401c7807c048751b.

Reason for revert: <INSERT REASONING HERE>
We need to do this on all key frames in the stream (not just the first one). Will make another cleaner change for this.

Original change's description:
> vp9-svc: Fix to first superframe when inter_layer is off.
> 
> When the application selects the setting INTER_LAYER_PRED_OFF
> each spatial stream should be decodeable separately.
> For this we need to force key frames on all spatial layers
> on the first superframe.
> 
> In order to maintain the quality at the beginning of the stream
> the active_worst for spatial layer of the second superframe is set
> to the last_QP of the correspondng spatial layer of the first superframe.
> Also make sure nonrd_keyframe is set for non-base spatial layers.
> 
> Change only affects SVC mode wit number_spatial_layers > 1 and
> svc->disable_inter_layer_pred == INTER_LAYER_PRED_OFF.
> And only affects first and second frame of sequence.
> 
> Change-Id: I8ee9a0873ab1d3a02515774571f719617771ad41

TBR=marpan@google.com,builds@webmproject.org,jianj@google.com

Change-Id: If73d9f3932224fc6751e773763adf7e8ee67d17f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

--- a/vp9/encoder/vp9_ratectrl.c
+++ b/vp9/encoder/vp9_ratectrl.c
@@ -702,14 +702,6 @@
   int active_worst_quality;
   int ambient_qp;
   unsigned int num_frames_weight_key = 5 * cpi->svc.number_temporal_layers;
-  // For SVC: when inter layer prediction is off, on the second superframe
-  // (superframe = 1) use the QP from the previous superframe. This needed to
-  // maintain quality since we can't temporally predict from the very first
-  // superframe.
-  if (cpi->use_svc && cpi->svc.current_superframe == 1 &&
-      cpi->svc.number_spatial_layers > 1 &&
-      cpi->svc.disable_inter_layer_pred == INTER_LAYER_PRED_OFF)
-    return rc->last_q[KEY_FRAME];
   if (cm->frame_type == KEY_FRAME || rc->reset_high_source_sad)
     return rc->worst_quality;
   // For ambient_qp we use minimum of avg_frame_qindex[KEY_FRAME/INTER_FRAME]
@@ -1735,13 +1727,8 @@
       LAYER_IDS_TO_IDX(cpi->svc.spatial_layer_id, cpi->svc.temporal_layer_id,
                        cpi->svc.number_temporal_layers);
   // Periodic key frames is based on the super-frame counter
-  // (svc.current_superframe), also only base spatial layer is key frame unless
-  // svc.disable_inter_layer_pred = INTER_LAYER_PRED_OFF.
-  int first_frame_iskey =
-      (cpi->svc.disable_inter_layer_pred == INTER_LAYER_PRED_OFF)
-          ? (cpi->svc.current_superframe == 0)
-          : (cm->current_video_frame == 0);
-  if (first_frame_iskey || (cpi->frame_flags & FRAMEFLAGS_KEY) ||
+  // (svc.current_superframe), also only base spatial layer is key frame.
+  if ((cm->current_video_frame == 0) || (cpi->frame_flags & FRAMEFLAGS_KEY) ||
       (cpi->oxcf.auto_key &&
        (cpi->svc.current_superframe % cpi->oxcf.key_freq == 0) &&
        cpi->svc.spatial_layer_id == 0)) {
@@ -1751,7 +1738,7 @@
       cpi->svc.layer_context[layer].is_key_frame = 1;
       cpi->ref_frame_flags &= (~VP9_LAST_FLAG & ~VP9_GOLD_FLAG & ~VP9_ALT_FLAG);
     } else if (is_one_pass_cbr_svc(cpi)) {
-      if (!first_frame_iskey) vp9_svc_reset_key_frame(cpi);
+      if (cm->current_video_frame > 0) vp9_svc_reset_key_frame(cpi);
       layer = LAYER_IDS_TO_IDX(cpi->svc.spatial_layer_id,
                                cpi->svc.temporal_layer_id,
                                cpi->svc.number_temporal_layers);
@@ -1763,18 +1750,6 @@
     }
   } else {
     cm->frame_type = INTER_FRAME;
-    // Special case for first superframe when inter_layer prediction is off.
-    // Force key frame on base spatial layer for second superframe.
-    // TODO(marpan): This condition may be removed, depending on #spatial_layer
-    // and scaling factor, but keep it for now for simplicity.
-    if (cpi->svc.disable_inter_layer_pred == INTER_LAYER_PRED_OFF &&
-        cpi->svc.current_superframe == 1 && cpi->svc.spatial_layer_id == 0 &&
-        cpi->svc.number_spatial_layers > 1) {
-      vp9_svc_reset_key_frame(cpi);
-      cm->frame_type = KEY_FRAME;
-      cpi->svc.layer_context[layer].is_key_frame = 1;
-      target = calc_iframe_target_size_one_pass_cbr(cpi);
-    }
     if (is_two_pass_svc(cpi)) {
       LAYER_CONTEXT *lc = &cpi->svc.layer_context[layer];
       if (cpi->svc.spatial_layer_id == 0) {
--- a/vp9/encoder/vp9_speed_features.c
+++ b/vp9/encoder/vp9_speed_features.c
@@ -531,9 +531,6 @@
       sf->limit_newmv_early_exit = 1;
       if (!cpi->use_svc) sf->bias_golden = 1;
     }
-    // Keep nonrd_keyframe = 1 for non-base spatial layers to prevent large
-    // encoding time.
-    if (cpi->use_svc && cpi->svc.spatial_layer_id > 0) sf->nonrd_keyframe = 1;
   }
 
   if (speed >= 6) {