From bff94472a66167ad9ba40ee3c5ad554cda1c8ed8 Mon Sep 17 00:00:00 2001 From: Marco Paniconi Date: Wed, 21 Nov 2018 15:01:04 -0800 Subject: [PATCH] vp9 screen-content: Keep lower step_param for quality layers. Issue with step_param = 2 seems specific to lower layers with different resolution. Change-Id: I26405488ac7691b3e471e98e794d4b1d8098a91d --- vp9/encoder/vp9_speed_features.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/vp9/encoder/vp9_speed_features.c b/vp9/encoder/vp9_speed_features.c index 9ea84f365..fbd24d0d2 100644 --- a/vp9/encoder/vp9_speed_features.c +++ b/vp9/encoder/vp9_speed_features.c @@ -804,8 +804,12 @@ static void set_rt_speed_feature_framesize_independent( cpi->svc.spatial_layer_id == 0 && (cpi->rc.high_num_blocks_with_motion || cpi->svc.last_layer_dropped[0])) { sf->mv.search_method = NSTEP; - // TODO(marpan/jianj): Investigate issue for lower setting of step_param. - sf->mv.fullpel_search_step_param = 4; + sf->mv.fullpel_search_step_param = 2; + // TODO(marpan/jianj): Investigate issue for lower setting of step_param + // for spatial layers (namely on lower layers). + if (cpi->use_svc && cm->width != cpi->oxcf.width && + cm->height != cpi->oxcf.height) + sf->mv.fullpel_search_step_param = 4; } } -- 2.40.0