mirrors/FreeRDP
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix libavcodec encoding errors and set tunables.

Browse Source
This commit is contained in:
Sri Ramanujam 2017-08-18 13:19:44 -04:00
parent 7717a42f6c
commit 66c925c9e4
2 changed files with 32 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
libfreerdp/codec/h264_ffmpeg.c
View File

@ -140,13 +140,28 @@ static BOOL libavcodec_create_encoder(H264_CONTEXT* h264)
{ {
1, h264->FrameRate 1, h264->FrameRate
}; };
sys->codecEncoderContext->gop_size = 0;
av_opt_set(sys->codecEncoderContext, "preset", "veryfast", AV_OPT_SEARCH_CHILDREN);
av_opt_set(sys->codecEncoderContext, "tune", "zerolatency", AV_OPT_SEARCH_CHILDREN);
sys->codecEncoderContext->flags |= CODEC_FLAG_LOOP_FILTER;
sys->codecEncoderContext->me_cmp |= 1;
sys->codecEncoderContext->me_subpel_quality = 3;
sys->codecEncoderContext->me_range = 16;
sys->codecEncoderContext->gop_size = 60;
sys->codecEncoderContext->keyint_min = 25;
sys->codecEncoderContext->i_quant_factor = 0.71;
sys->codecEncoderContext->qcompress = 0.6;
sys->codecEncoderContext->qmin = 18;
sys->codecEncoderContext->qmax = 51;
sys->codecEncoderContext->max_qdiff = 4;
sys->codecEncoderContext->max_b_frames = 0; sys->codecEncoderContext->max_b_frames = 0;
sys->codecEncoderContext->refs = 1;
sys->codecEncoderContext->trellis = 0;
sys->codecEncoderContext->thread_count = 2;
sys->codecEncoderContext->level = 31;
sys->codecEncoderContext->b_quant_factor = 0;
sys->codecEncoderContext->pix_fmt = AV_PIX_FMT_YUV420P; sys->codecEncoderContext->pix_fmt = AV_PIX_FMT_YUV420P;
sys->codecEncoderContext->colorspace = AVCOL_SPC_BT709;
av_opt_set(sys->codecEncoderContext->priv_data, "preset", "fast", 0);
av_opt_set(sys->codecEncoderContext->priv_data, "vprofile", "baseline", 0);
av_opt_set(sys->codecEncoderContext->priv_data, "tune", "zerolatency", 0);
if (avcodec_open2(sys->codecEncoderContext, sys->codecEncoder, NULL) < 0) if (avcodec_open2(sys->codecEncoderContext, sys->codecEncoder, NULL) < 0)
goto EXCEPTION; goto EXCEPTION;
@ -231,7 +246,7 @@ static int libavcodec_compress(H264_CONTEXT* h264, BYTE** ppDstData, UINT32* pDs
if (!libavcodec_create_encoder(h264)) if (!libavcodec_create_encoder(h264))
return -1; return -1;
av_free(sys->packet.data); av_packet_unref(&sys->packet);
av_init_packet(&sys->packet); av_init_packet(&sys->packet);
sys->packet.data = NULL; sys->packet.data = NULL;
sys->packet.size = 0; sys->packet.size = 0;
@ -241,6 +256,7 @@ static int libavcodec_compress(H264_CONTEXT* h264, BYTE** ppDstData, UINT32* pDs
#if LIBAVUTIL_VERSION_INT >= AV_VERSION_INT(52, 48, 100) #if LIBAVUTIL_VERSION_INT >= AV_VERSION_INT(52, 48, 100)
sys->videoFrame->colorspace = AVCOL_SPC_BT709; sys->videoFrame->colorspace = AVCOL_SPC_BT709;
#endif #endif
sys->videoFrame->chroma_location = AVCHROMA_LOC_LEFT;
sys->videoFrame->data[0] = h264->pYUVData[0]; sys->videoFrame->data[0] = h264->pYUVData[0];
sys->videoFrame->data[1] = h264->pYUVData[1]; sys->videoFrame->data[1] = h264->pYUVData[1];
sys->videoFrame->data[2] = h264->pYUVData[2]; sys->videoFrame->data[2] = h264->pYUVData[2];
@ -248,7 +264,6 @@ static int libavcodec_compress(H264_CONTEXT* h264, BYTE** ppDstData, UINT32* pDs
sys->videoFrame->linesize[1] = h264->iStride[1]; sys->videoFrame->linesize[1] = h264->iStride[1];
sys->videoFrame->linesize[2] = h264->iStride[2]; sys->videoFrame->linesize[2] = h264->iStride[2];
sys->videoFrame->pts++; sys->videoFrame->pts++;
sys->videoFrame->pict_type = AV_PICTURE_TYPE_I;
/* avcodec_encode_video2 is deprecated with libavcodec 57.48.101 */ /* avcodec_encode_video2 is deprecated with libavcodec 57.48.101 */
#if LIBAVCODEC_VERSION_INT >= AV_VERSION_INT(57, 48, 101) #if LIBAVCODEC_VERSION_INT >= AV_VERSION_INT(57, 48, 101)
status = avcodec_send_frame(sys->codecEncoderContext, sys->videoFrame); status = avcodec_send_frame(sys->codecEncoderContext, sys->videoFrame);
@ -279,7 +294,7 @@ static int libavcodec_compress(H264_CONTEXT* h264, BYTE** ppDstData, UINT32* pDs
&sys->packet, &sys->packet,
sys->videoFrame, &gotFrame); sys->videoFrame, &gotFrame);
} }
while ((status >= 0) && (gotFrame != 0)); while ((status >= 0) && (gotFrame == 0));
#endif #endif
@ -294,7 +309,10 @@ static int libavcodec_compress(H264_CONTEXT* h264, BYTE** ppDstData, UINT32* pDs
*pDstSize = sys->packet.size; *pDstSize = sys->packet.size;
if (!gotFrame) if (!gotFrame)
{
WLog_ERR(TAG, "Did not get frame! (%s [%d])", av_err2str(status), status);
return -2; return -2;
}
return 1; return 1;
} }

6
server/shadow/shadow_client.c
View File

@ -826,7 +826,10 @@ static BOOL shadow_client_send_surface_gfx(rdpShadowClient* client,
if (avc420_compress(encoder->h264, pSrcData, cmd.format, nSrcStep, if (avc420_compress(encoder->h264, pSrcData, cmd.format, nSrcStep,
nWidth, nHeight, &avc444.bitstream[0].data, nWidth, nHeight, &avc444.bitstream[0].data,
&avc444.bitstream[0].length) < 0) &avc444.bitstream[0].length) < 0)
{
WLog_ERR(TAG, "avc420_compress failed for avc444");
return FALSE; return FALSE;
}
regionRect.left = cmd.left; regionRect.left = cmd.left;
regionRect.top = cmd.top; regionRect.top = cmd.top;
@ -868,7 +871,10 @@ static BOOL shadow_client_send_surface_gfx(rdpShadowClient* client,
if (avc420_compress(encoder->h264, pSrcData, cmd.format, nSrcStep, if (avc420_compress(encoder->h264, pSrcData, cmd.format, nSrcStep,
nWidth, nHeight, &avc420.data, &avc420.length) < 0) nWidth, nHeight, &avc420.data, &avc420.length) < 0)
{
WLog_ERR(TAG, "avc420_compress failed");
return FALSE; return FALSE;
}
cmd.codecId = RDPGFX_CODECID_AVC420; cmd.codecId = RDPGFX_CODECID_AVC420;
cmd.extra = (void*)&avc420; cmd.extra = (void*)&avc420;