mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-01-02 21:48:55 +00:00
Code Issues 1 Projects Releases Wiki Activity

libs: encoder: h265: set no P frame automatically.

Browse source 
The double reference lists may be required by drivers and there should
be no P frames in the of stream. The old way of converting P frames to
B frames is by setting `low-delay-b` property, which is unconvenient
and has bad user experience, since most of the users do not know when
to set this property, and if it is not set correctly, the encoding
pipeline fails or even hangs on some platforms. VA driver now provides
a attribute to query whether both reference lists must be un-NULL for
a profile/entrypoint pair.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/284>
This commit is contained in:
He Junyan 2020-07-01 14:50:51 +08:00 committed by Víctor Manuel Jáquez Leal
parent ee7677211a
commit a73e6b7dac
1 changed files with 49 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

51
gst-libs/gst/vaapi/gstvaapiencoder_h265.c
View file

@ -2323,7 +2323,46 @@ ensure_profile_tier_level (GstVaapiEncoderH265 * encoder)
return GST_VAAPI_ENCODER_STATUS_SUCCESS;
}
static void
static gboolean
check_ref_list (GstVaapiEncoderH265 * encoder)
{
#if VA_CHECK_VERSION(1,9,0)
/* Some driver require both r0 and r1 list are non NULL, i.e. no p frame
in the stream. The traditional P frame can be converted to B frame with
forward dependency only. The new B frame has only forward reference in
both r0 and r1 list, which conforms to H265 spec. This can get some gain
because there are 2 MVs for each frame and can generate better motion
estimation. */
GstVaapiEncoder *const base_encoder = GST_VAAPI_ENCODER (encoder);
guint value = 0;
VAProfile va_profile = gst_vaapi_profile_get_va_profile (encoder->profile);
VAEntrypoint va_entrypoint =
gst_vaapi_entrypoint_get_va_entrypoint (encoder->entrypoint);
encoder->no_p_frame = FALSE;
if (gst_vaapi_get_config_attribute (base_encoder->display, va_profile,
va_entrypoint, VAConfigAttribPredictionDirection, &value)) {
gboolean double_ref_list =
((value & VA_PREDICTION_DIRECTION_BI_NOT_EMPTY) != 0);
if (double_ref_list) {
GST_INFO ("driver does not support P frame, we need to convert P"
" frame to forward dependency B frame.");
encoder->no_p_frame = double_ref_list;
}
}
if (encoder->no_p_frame == TRUE && base_encoder->max_num_ref_frames_1 < 1) {
GST_WARNING ("P frame should be converted to forward dependent B,"
" but reference list 1 is disabled here. Should be an invalid"
" setting or a driver error.");
return FALSE;
}
#endif
return TRUE;
}
static GstVaapiEncoderStatus
reset_properties (GstVaapiEncoderH265 * encoder)
{
GstVaapiEncoder *const base_encoder = GST_VAAPI_ENCODER_CAST (encoder);
@ -2350,6 +2389,9 @@ reset_properties (GstVaapiEncoderH265 * encoder)
gst_vaapi_encoder_ensure_max_num_ref_frames (base_encoder, encoder->profile,
encoder->entrypoint);
if (!check_ref_list (encoder))
return GST_VAAPI_ENCODER_STATUS_ERROR_UNKNOWN;
if (base_encoder->max_num_ref_frames_1 < 1 && encoder->num_bframes > 0) {
GST_WARNING ("Disabling b-frame since the driver doesn't support it");
encoder->num_bframes = 0;
@ -2394,6 +2436,8 @@ reset_properties (GstVaapiEncoderH265 * encoder)
reorder_pool = &encoder->reorder_pool;
reorder_pool->frame_index = 0;
return GST_VAAPI_ENCODER_STATUS_SUCCESS;
}
static void
@ -3151,7 +3195,10 @@ gst_vaapi_encoder_h265_reconfigure (GstVaapiEncoder * base_encoder)
encoder->ctu_height = (encoder->luma_height + 31) / 32;
}
reset_properties (encoder);
status = reset_properties (encoder);
if (status != GST_VAAPI_ENCODER_STATUS_SUCCESS)
return status;
status = ensure_tile (encoder);
if (status != GST_VAAPI_ENCODER_STATUS_SUCCESS)
return status;