diff options
| author | carpalis <jerome.borsboom@carpalis.nl> | 2017-10-23 11:02:35 +0200 |
|---|---|---|
| committer | Xiang, Haihao <haihao.xiang@intel.com> | 2018-01-05 14:57:31 +0800 |
| commit | 0c2ce9e79bcc917b4469faf85b4f18f6936d5940 (patch) | |
| tree | e65193fbb6ad9df9ac1cc19261ff235d200a562a /src/gen8_mfd.c | |
| parent | 1c848c89d53fcaf834e875fdb47779fd3cce071e (diff) | |
| download | libva-intel-driver-0c2ce9e79bcc917b4469faf85b4f18f6936d5940.tar.gz | |
genX_mfd: no need for loop filtering for skipped pictures
Skipped pictures are equal to their reference frame and, therefore,
should not pass the loop filter. Although the loop filter was already
a null operation as all transform coefficients are zero for skipped
pictures, this may save some cyles in the decoder hardware.
Signed-off-by: Jerome Borsboom <jerome.borsboom@carpalis.nl>
Diffstat (limited to 'src/gen8_mfd.c')
| -rw-r--r-- | src/gen8_mfd.c | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/src/gen8_mfd.c b/src/gen8_mfd.c index 5dcccb7c..d5c76a18 100644 --- a/src/gen8_mfd.c +++ b/src/gen8_mfd.c @@ -1346,12 +1346,18 @@ gen8_mfd_vc1_decode_init(VADriverContextP ctx, dri_bo_unreference(gen7_mfd_context->post_deblocking_output.bo); gen7_mfd_context->post_deblocking_output.bo = obj_surface->bo; dri_bo_reference(gen7_mfd_context->post_deblocking_output.bo); - gen7_mfd_context->post_deblocking_output.valid = pic_param->entrypoint_fields.bits.loopfilter; dri_bo_unreference(gen7_mfd_context->pre_deblocking_output.bo); gen7_mfd_context->pre_deblocking_output.bo = obj_surface->bo; dri_bo_reference(gen7_mfd_context->pre_deblocking_output.bo); - gen7_mfd_context->pre_deblocking_output.valid = !pic_param->entrypoint_fields.bits.loopfilter; + + if (picture_type == GEN7_VC1_SKIPPED_PICTURE) { + gen7_mfd_context->post_deblocking_output.valid = 0; + gen7_mfd_context->pre_deblocking_output.valid = 1; + } else { + gen7_mfd_context->post_deblocking_output.valid = pic_param->entrypoint_fields.bits.loopfilter; + gen7_mfd_context->pre_deblocking_output.valid = !pic_param->entrypoint_fields.bits.loopfilter; + } dri_bo_unreference(gen7_mfd_context->intra_row_store_scratch_buffer.bo); bo = dri_bo_alloc(i965->intel.bufmgr, @@ -1473,6 +1479,7 @@ gen8_mfd_vc1_pic_state(VADriverContextP ctx, int profile; int overlap = 0; int interpolation_mode = 0; + int loopfilter = 0; assert(decode_state->pic_param && decode_state->pic_param->buffer); pic_param = (VAPictureParameterBufferVC1 *)decode_state->pic_param->buffer; @@ -1558,8 +1565,10 @@ gen8_mfd_vc1_pic_state(VADriverContextP ctx, if (picture_type == GEN7_VC1_SKIPPED_PICTURE) ptype = GEN7_VC1_P_PICTURE; - else + else { ptype = pic_param->picture_fields.bits.picture_type; + loopfilter = pic_param->entrypoint_fields.bits.loopfilter; + } if (picture_type == GEN7_VC1_I_PICTURE || picture_type == GEN7_VC1_BI_PICTURE) /* I picture */ trans_ac_y = pic_param->transform_fields.bits.transform_ac_codingset_idx2; @@ -1665,7 +1674,7 @@ gen8_mfd_vc1_pic_state(VADriverContextP ctx, interpolation_mode << 8 | 0 << 7 | /* FIXME: scale up or down ??? */ pic_param->range_reduction_frame << 6 | - pic_param->entrypoint_fields.bits.loopfilter << 5 | + loopfilter << 5 | overlap << 4 | !pic_param->picture_fields.bits.is_first_field << 3 | (pic_param->sequence_fields.bits.profile == 3) << 0); |