From patchwork Thu Feb 4 19:09:22 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: James Almer X-Patchwork-Id: 25438 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id D732344AD4B for ; Thu, 4 Feb 2021 21:18:08 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C338A68AA98; Thu, 4 Feb 2021 21:18:08 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-qk1-f180.google.com (mail-qk1-f180.google.com [209.85.222.180]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8B5EE68A5A8 for ; Thu, 4 Feb 2021 21:18:06 +0200 (EET) Received: by mail-qk1-f180.google.com with SMTP id d85so4470229qkg.5 for ; Thu, 04 Feb 2021 11:18:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references:mime-version :content-transfer-encoding; bh=StZgGre1j42Pd64kKJyhq+7Jz+vPRm1znQc0H8dGnA0=; b=XzO5/eOhqvXOLhBVs3XevwXppgUEQQle01mMKdcIc7PDK+YLtzMPs/bqdcCdJVaSxP tGihAvq2u+kWscm+EZdOxOW/OFOZ0cQcNwPooSl8FqGOIpK20LU9KUwrpETaw4EGWtw4 CHtfDwjFxRptsfAdaFcN2XB5UVYykGFgROfoCzkkC1XhzK27j1sW/EFavt5DqARsYGEN QhSjiRejJ0uOQ1Lqht7wqsRq+URmi/cfDBI/PWVaO13HXzbNahEGXeRnZsE7nPM6LIu/ /tK7rPMUPnHABF4vVqudAvpHrH2HluAL4CsJpH3VPkcab8IhIqy2lD61MZFC0X8Szs1T gPBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=StZgGre1j42Pd64kKJyhq+7Jz+vPRm1znQc0H8dGnA0=; b=hquseLVX7RGhPyZzPTcugRM/vrM8K4f3PWVN9jRpP2WLEMykiX1UyUqlnGq+D2UQcf brbqU0oAWh9OpVazFIllEmhtXCYXM5eNyFbgh6mSOzboJNBrsFNdNbU5OROwsFtcVzzb En72piuXGPY4IPQBd1CFBQg+HXylm86nMZbzwB03ZzLUwKik/F7n7ac72Z3Y8pX5/1vn VH9olWP6PsSoh1v7nuEZduHR5k6axwhpMAJvzwpi3im5mTOIOyXTVVf48+nK5JL3UjRV wbpl+wtzxCR6VTgDC0F3yct96cTv5F0MYlEnmoPQo6vdu1j1TaAtLI7R0TACAMc3AiqR AY4A== X-Gm-Message-State: AOAM533IXSlXGWFTx6BTYTeGXtvX/b+s5M5FWu+ZIDSxXmnMDN1M4Rki kfiSuDK+YnsQXGJNQm08sZ236O+ZQJg= X-Google-Smtp-Source: ABdhPJwBaNRf/HM+apOgf6Wd/iqTgEGMAgeNrJhqPXtp/ZNgaTR+OUUUhrSyu3hbvQJhjnK8DAvKgA== X-Received: by 2002:ac8:7b86:: with SMTP id p6mr1018552qtu.23.1612465844990; Thu, 04 Feb 2021 11:10:44 -0800 (PST) Received: from localhost.localdomain ([181.23.64.183]) by smtp.gmail.com with ESMTPSA id t14sm5889291qkt.50.2021.02.04.11.10.43 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Feb 2021 11:10:44 -0800 (PST) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Thu, 4 Feb 2021 16:09:22 -0300 Message-Id: <20210204191005.48190-8-jamrial@gmail.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20210204191005.48190-1-jamrial@gmail.com> References: <20210204191005.48190-1-jamrial@gmail.com> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 07/50] avcodec/pthread_frame: use av_packet_alloc() to allocate packets X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Signed-off-by: James Almer --- libavcodec/pthread_frame.c | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 4429a4d59c..7bcb9a7bcc 100644 --- a/libavcodec/pthread_frame.c +++ b/libavcodec/pthread_frame.c @@ -81,7 +81,7 @@ typedef struct PerThreadContext { AVCodecContext *avctx; ///< Context used to decode packets passed to this thread. - AVPacket avpkt; ///< Input packet (for decoding) or output (for encoding). + AVPacket *avpkt; ///< Input packet (for decoding) or output (for encoding). AVFrame *frame; ///< Output frame (for decoding) or input (for encoding). int got_frame; ///< The output of got_picture_ptr from the last avcodec_decode_video() call. @@ -208,7 +208,7 @@ FF_ENABLE_DEPRECATION_WARNINGS av_frame_unref(p->frame); p->got_frame = 0; - p->result = codec->decode(avctx, p->frame, &p->got_frame, &p->avpkt); + p->result = codec->decode(avctx, p->frame, &p->got_frame, p->avpkt); if ((p->result < 0 || !p->got_frame) && p->frame->buf[0]) { if (avctx->codec->caps_internal & FF_CODEC_CAP_ALLOCATE_PROGRESS) @@ -438,8 +438,8 @@ static int submit_packet(PerThreadContext *p, AVCodecContext *user_avctx, } } - av_packet_unref(&p->avpkt); - ret = av_packet_ref(&p->avpkt, avpkt); + av_packet_unref(p->avpkt); + ret = av_packet_ref(p->avpkt, avpkt); if (ret < 0) { pthread_mutex_unlock(&p->mutex); av_log(p->avctx, AV_LOG_ERROR, "av_packet_ref() failed in submit_packet()\n"); @@ -550,7 +550,7 @@ int ff_thread_decode_frame(AVCodecContext *avctx, av_frame_move_ref(picture, p->frame); *got_picture_ptr = p->got_frame; - picture->pkt_dts = p->avpkt.dts; + picture->pkt_dts = p->avpkt->dts; err = p->result; /* @@ -725,7 +725,7 @@ void ff_frame_thread_free(AVCodecContext *avctx, int thread_count) pthread_cond_destroy(&p->input_cond); pthread_cond_destroy(&p->progress_cond); pthread_cond_destroy(&p->output_cond); - av_packet_unref(&p->avpkt); + av_packet_free(&p->avpkt); #if FF_API_THREAD_SAFE_CALLBACKS for (int j = 0; j < p->released_buffers_allocated; j++) @@ -822,6 +822,12 @@ int ff_frame_thread_init(AVCodecContext *avctx) err = AVERROR(ENOMEM); goto error; } + p->avpkt = av_packet_alloc(); + if (!p->avpkt) { + av_freep(©); + err = AVERROR(ENOMEM); + goto error; + } p->parent = fctx; p->avctx = copy; @@ -841,7 +847,7 @@ int ff_frame_thread_init(AVCodecContext *avctx) } *copy->internal = *src->internal; copy->internal->thread_ctx = p; - copy->internal->last_pkt_props = &p->avpkt; + copy->internal->last_pkt_props = p->avpkt; copy->delay = avctx->delay;