[FFmpeg-devel] [PATCH] avcodec/nvenc: migrate to new encode API
Philip Langdale
philipl at overt.org
Wed Sep 6 18:14:12 EEST 2017
On Sat, 2 Sep 2017 21:27:03 +0200
Timo Rothenpieler <timo at rothenpieler.org> wrote:
> Signed-off-by: Timo Rothenpieler <timo at rothenpieler.org>
> ---
> libavcodec/nvenc.c | 65
> ++++++++++++++++++++++++++++++++++++++++---------
> libavcodec/nvenc.h | 6 +++++ libavcodec/nvenc_h264.c | 6 +++++
> libavcodec/nvenc_hevc.c | 4 +++
> 4 files changed, 70 insertions(+), 11 deletions(-)
>
> diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c
> index 744e5e0e01..b54cb1275b 100644
> --- a/libavcodec/nvenc.c
> +++ b/libavcodec/nvenc.c
> @@ -1807,8 +1807,7 @@ static int output_ready(AVCodecContext *avctx,
> int flush) return (nb_ready > 0) && (nb_ready + nb_pending >=
> ctx->async_depth); }
>
> -int ff_nvenc_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
> - const AVFrame *frame, int *got_packet)
> +int ff_nvenc_send_frame(AVCodecContext *avctx, const AVFrame *frame)
> {
> NVENCSTATUS nv_status;
> CUresult cu_res;
> @@ -1823,12 +1822,16 @@ int ff_nvenc_encode_frame(AVCodecContext
> *avctx, AVPacket *pkt, NV_ENC_PIC_PARAMS pic_params = { 0 };
> pic_params.version = NV_ENC_PIC_PARAMS_VER;
>
> + if (!ctx->cu_context || !ctx->nvencoder)
> + return AVERROR(EINVAL);
> +
> + if (ctx->encoder_flushing)
> + return AVERROR_EOF;
> +
> if (frame) {
> inSurf = get_free_frame(ctx);
> - if (!inSurf) {
> - av_log(avctx, AV_LOG_ERROR, "No free surfaces\n");
> - return AVERROR_BUG;
> - }
> + if (!inSurf)
> + return AVERROR(EAGAIN);
>
> cu_res = dl_fn->cuda_dl->cuCtxPushCurrent(ctx->cu_context);
> if (cu_res != CUDA_SUCCESS) {
> @@ -1876,6 +1878,7 @@ int ff_nvenc_encode_frame(AVCodecContext
> *avctx, AVPacket *pkt, nvenc_codec_specific_pic_params(avctx,
> &pic_params); } else {
> pic_params.encodePicFlags = NV_ENC_PIC_FLAG_EOS;
> + ctx->encoder_flushing = 1;
> }
>
> cu_res = dl_fn->cuda_dl->cuCtxPushCurrent(ctx->cu_context);
> @@ -1914,7 +1917,23 @@ int ff_nvenc_encode_frame(AVCodecContext
> *avctx, AVPacket *pkt, }
> }
>
> - if (output_ready(avctx, !frame)) {
> + return 0;
> +}
> +
> +int ff_nvenc_receive_packet(AVCodecContext *avctx, AVPacket *pkt)
> +{
> + CUresult cu_res;
> + CUcontext dummy;
> + NvencSurface *tmpoutsurf;
You can remove tmpoutsurf in ff_nvenc_send_frame
> + int res;
> +
> + NvencContext *ctx = avctx->priv_data;
> + NvencDynLoadFunctions *dl_fn = &ctx->nvenc_dload_funcs;
> +
> + if (!ctx->cu_context || !ctx->nvencoder)
> + return AVERROR(EINVAL);
> +
> + if (output_ready(avctx, ctx->encoder_flushing)) {
> av_fifo_generic_read(ctx->output_surface_ready_queue,
> &tmpoutsurf, sizeof(tmpoutsurf), NULL);
> cu_res = dl_fn->cuda_dl->cuCtxPushCurrent(ctx->cu_context);
> @@ -1935,10 +1954,34 @@ int ff_nvenc_encode_frame(AVCodecContext
> *avctx, AVPacket *pkt, return res;
>
> av_fifo_generic_write(ctx->unused_surface_queue,
> &tmpoutsurf, sizeof(tmpoutsurf), NULL); -
> - *got_packet = 1;
> + } else if (ctx->encoder_flushing) {
> + return AVERROR_EOF;
> } else {
> + return AVERROR(EAGAIN);
> + }
> +
> + return 0;
> +}
> +
> +int ff_nvenc_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
> + const AVFrame *frame, int *got_packet)
> +{
> + NvencContext *ctx = avctx->priv_data;
> + int res;
> +
> + if (!ctx->encoder_flushing) {
> + res = ff_nvenc_send_frame(avctx, frame);
> + if (res < 0)
> + return res;
> + }
> +
> + res = ff_nvenc_receive_packet(avctx, pkt);
> + if (res == AVERROR(EAGAIN) || res == AVERROR_EOF) {
> *got_packet = 0;
> + } else if (res < 0) {
> + return res;
> + } else {
> + *got_packet = 1;
> }
>
> return 0;
> diff --git a/libavcodec/nvenc.h b/libavcodec/nvenc.h
> index 2c682275da..afb93cc22c 100644
> --- a/libavcodec/nvenc.h
> +++ b/libavcodec/nvenc.h
> @@ -116,6 +116,8 @@ typedef struct NvencContext
> AVFifoBuffer *output_surface_ready_queue;
> AVFifoBuffer *timestamp_list;
>
> + int encoder_flushing;
> +
> struct {
> CUdeviceptr ptr;
> NV_ENC_REGISTERED_PTR regptr;
> @@ -169,6 +171,10 @@ int ff_nvenc_encode_init(AVCodecContext *avctx);
>
> int ff_nvenc_encode_close(AVCodecContext *avctx);
>
> +int ff_nvenc_send_frame(AVCodecContext *avctx, const AVFrame *frame);
> +
> +int ff_nvenc_receive_packet(AVCodecContext *avctx, AVPacket *pkt);
> +
> int ff_nvenc_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
> const AVFrame *frame, int *got_packet);
>
> diff --git a/libavcodec/nvenc_h264.c b/libavcodec/nvenc_h264.c
> index 9adbe9f909..c3b4bac749 100644
> --- a/libavcodec/nvenc_h264.c
> +++ b/libavcodec/nvenc_h264.c
> @@ -164,6 +164,8 @@ AVCodec ff_nvenc_encoder = {
> .type = AVMEDIA_TYPE_VIDEO,
> .id = AV_CODEC_ID_H264,
> .init = nvenc_old_init,
> + .send_frame = ff_nvenc_send_frame,
> + .receive_packet = ff_nvenc_receive_packet,
> .encode2 = ff_nvenc_encode_frame,
> .close = ff_nvenc_encode_close,
> .priv_data_size = sizeof(NvencContext),
> @@ -190,6 +192,8 @@ AVCodec ff_nvenc_h264_encoder = {
> .type = AVMEDIA_TYPE_VIDEO,
> .id = AV_CODEC_ID_H264,
> .init = nvenc_old_init,
> + .send_frame = ff_nvenc_send_frame,
> + .receive_packet = ff_nvenc_receive_packet,
> .encode2 = ff_nvenc_encode_frame,
> .close = ff_nvenc_encode_close,
> .priv_data_size = sizeof(NvencContext),
> @@ -216,6 +220,8 @@ AVCodec ff_h264_nvenc_encoder = {
> .type = AVMEDIA_TYPE_VIDEO,
> .id = AV_CODEC_ID_H264,
> .init = ff_nvenc_encode_init,
> + .send_frame = ff_nvenc_send_frame,
> + .receive_packet = ff_nvenc_receive_packet,
> .encode2 = ff_nvenc_encode_frame,
> .close = ff_nvenc_encode_close,
> .priv_data_size = sizeof(NvencContext),
> diff --git a/libavcodec/nvenc_hevc.c b/libavcodec/nvenc_hevc.c
> index 81da9d222d..89e8c3e53a 100644
> --- a/libavcodec/nvenc_hevc.c
> +++ b/libavcodec/nvenc_hevc.c
> @@ -153,6 +153,8 @@ AVCodec ff_nvenc_hevc_encoder = {
> .type = AVMEDIA_TYPE_VIDEO,
> .id = AV_CODEC_ID_HEVC,
> .init = nvenc_old_init,
> + .send_frame = ff_nvenc_send_frame,
> + .receive_packet = ff_nvenc_receive_packet,
> .encode2 = ff_nvenc_encode_frame,
> .close = ff_nvenc_encode_close,
> .priv_data_size = sizeof(NvencContext),
> @@ -178,6 +180,8 @@ AVCodec ff_hevc_nvenc_encoder = {
> .type = AVMEDIA_TYPE_VIDEO,
> .id = AV_CODEC_ID_HEVC,
> .init = ff_nvenc_encode_init,
> + .send_frame = ff_nvenc_send_frame,
> + .receive_packet = ff_nvenc_receive_packet,
> .encode2 = ff_nvenc_encode_frame,
> .close = ff_nvenc_encode_close,
> .priv_data_size = sizeof(NvencContext),
Otherwise looks good.
--phil
More information about the ffmpeg-devel
mailing list