From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id C9655412B9 for ; Tue, 19 Apr 2022 17:04:12 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8805A68B378; Tue, 19 Apr 2022 20:04:09 +0300 (EEST) Received: from mail-yw1-f201.google.com (mail-yw1-f201.google.com [209.85.128.201]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5052F68B2DC for ; Tue, 19 Apr 2022 20:04:03 +0300 (EEST) Received: by mail-yw1-f201.google.com with SMTP id 00721157ae682-2f16f3a7c34so66550097b3.17 for ; Tue, 19 Apr 2022 10:04:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=date:in-reply-to:message-id:mime-version:references:subject:from:to :cc; bh=BGgszom/SwDjnW1xeEfmBZ/QUsxDGNw2RHYIQr8CVfU=; b=GHXAALsjKiaSaf05DCSiOVoLRcp7EzSnim2AbQx61GwhJQIfxZJQGFD9FI5SsplOQh yKdCX+GzcIMhEq1ZqVCkivT6kY7VJ8moBxLz3tq7pls4eerBSK+mJz+/vq99rop2T4b/ LWLf5oERq5WfsVf1i0//eSmrCvirexIC9f81VT2IcX1JB3+P82OuXi2FvwNesCMzdkkB ydYHeg8IK8UT3FSXihuYgXSWej1rcU1yPR8qWt6Ii2sXKRCsCepWYc5Txs8lzo05ZGs1 nhMz2ix0oiefyExOj2bgnCfHPo3Y0DVsYe0uRp5fK37qoaTzmY4qFT5X3gj+TaIXEKae 3cmg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=BGgszom/SwDjnW1xeEfmBZ/QUsxDGNw2RHYIQr8CVfU=; b=5iMyaQjq8VUCGZuQUcdbteWFVy1LaUJ0aO21zdo5sbK+Fs6A/8CO4j4UPysF4Nbayj H6zn8dsnqWc4gFkT60rx4vUsgGy8KGJGygDh3mzMONuoi8mEeVCDW1/ZGhFSRE/Ui5Zu uv15MenqkI44idxzgxLl51NkLmNv/kAQz/CJtK5fU3TgFDh1RT8HN1I4dsHdDjihLHfb HFZhtqE6un4woCg/DVVKSESIDYOAg+9rrbZNe79X3ZjRC6Z6Ns0CklHFWUrcLXQ/FghO HxABvrYjBLmYX62u0x2bO6KRJ+MVmlQXyswrMhP9wOc3UpVGGT4e8c508CDZ/bFYAgPZ v+lw== X-Gm-Message-State: AOAM532AObJVcQh7ETu9pHZpw5Lt5AV492k5f5KRPCxN4qj5kMVInlvr 5bgB421NPqal4MT2TsgSrTpNg3iKivY9Z4qSSwa+IoG4kjSiM6XCk0r7VAHCS7Nzjq11/qNj67b /xibhxAuCNPaxuwJJQLOfYBcKwDUvoN+xZ/HO8nNivHGVyQY3hWhZastiZaYZ8jeDAw== X-Google-Smtp-Source: ABdhPJx1ay0HoMaO1jwKc/efZDUlBysxOIxTEYhLa8VA+Fsv9vd14xeUpnWu2Wti8c5vAHXP2Zrns6Ykfp8A X-Received: from bohanli.mtv.corp.google.com ([2620:0:1000:2511:6ff:7206:1c4e:68fe]) (user=bohanli job=sendgmr) by 2002:a25:ed07:0:b0:63f:b34:94c5 with SMTP id k7-20020a25ed07000000b0063f0b3494c5mr15784664ybh.488.1650387841942; Tue, 19 Apr 2022 10:04:01 -0700 (PDT) Date: Tue, 19 Apr 2022 10:03:24 -0700 In-Reply-To: <20220418195942.3112480-1-bohanli@google.com> Message-Id: <20220419170324.3342219-1-bohanli@google.com> Mime-Version: 1.0 References: <20220418195942.3112480-1-bohanli@google.com> X-Mailer: git-send-email 2.36.0.rc0.470.gd361397f0d-goog From: Bohan Li To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH] avcodec/libaomenc: Add unmet target level warning X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 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 Cc: Bohan Li Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: When target levels are set, this patch checks whether they are satisfied by libaom. If not, a warning is shown. Otherwise the output levels are also logged. This patch applies basically the same approach used for libvpx. Signed-off-by: Bohan Li --- libavcodec/libaomenc.c | 64 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 64 insertions(+) diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c index 054903e6e2..402bcb5198 100644 --- a/libavcodec/libaomenc.c +++ b/libavcodec/libaomenc.c @@ -198,6 +198,12 @@ static const char *const ctlidstr[] = { [AV1E_SET_ENABLE_SMOOTH_INTERINTRA] = "AV1E_SET_ENABLE_SMOOTH_INTERINTRA", [AV1E_SET_ENABLE_REF_FRAME_MVS] = "AV1E_SET_ENABLE_REF_FRAME_MVS", #endif +#ifdef AOM_CTRL_AV1E_GET_SEQ_LEVEL_IDX + [AV1E_GET_SEQ_LEVEL_IDX] = "AV1E_GET_SEQ_LEVEL_IDX", +#endif +#ifdef AOM_CTRL_AV1E_GET_TARGET_SEQ_LEVEL_IDX + [AV1E_GET_TARGET_SEQ_LEVEL_IDX] = "AV1E_GET_TARGET_SEQ_LEVEL_IDX", +#endif }; static av_cold void log_encoder_error(AVCodecContext *avctx, const char *desc) @@ -323,10 +329,68 @@ static av_cold int codecctl_int(AVCodecContext *avctx, return 0; } +#if defined(AOM_CTRL_AV1E_GET_SEQ_LEVEL_IDX) && \ + defined(AOM_CTRL_AV1E_GET_TARGET_SEQ_LEVEL_IDX) +static av_cold int codecctl_intp(AVCodecContext *avctx, +#ifdef UENUM1BYTE + aome_enc_control_id id, +#else + enum aome_enc_control_id id, +#endif + int* ptr) +{ + AOMContext *ctx = avctx->priv_data; + char buf[80]; + int width = -30; + int res; + + snprintf(buf, sizeof(buf), "%s:", ctlidstr[id]); + av_log(avctx, AV_LOG_DEBUG, " %*s%d\n", width, buf, *ptr); + + res = aom_codec_control(&ctx->encoder, id, ptr); + if (res != AOM_CODEC_OK) { + snprintf(buf, sizeof(buf), "Failed to set %s codec control", + ctlidstr[id]); + log_encoder_error(avctx, buf); + return AVERROR(EINVAL); + } + + return 0; +} +#endif + static av_cold int aom_free(AVCodecContext *avctx) { AOMContext *ctx = avctx->priv_data; +#if defined(AOM_CTRL_AV1E_GET_SEQ_LEVEL_IDX) && \ + defined(AOM_CTRL_AV1E_GET_TARGET_SEQ_LEVEL_IDX) + if (!(avctx->flags & AV_CODEC_FLAG_PASS1)) { + int levels[32] = { 0 }; + int target_levels[32] = { 0 }; + + if (!codecctl_intp(avctx, AV1E_GET_SEQ_LEVEL_IDX, levels) && + !codecctl_intp(avctx, AV1E_GET_TARGET_SEQ_LEVEL_IDX, + target_levels)) { + for (int i = 0; i < 32; i++) { + if (levels[i] > target_levels[i]) { + // Warn when the target level was not met + av_log(avctx, AV_LOG_WARNING, + "Could not encode to target level %d.%d for " + "operating point %d. The output level is %d.%d.", + 2 + (target_levels[i] >> 2), target_levels[i] & 3, + i, 2 + (levels[i] >> 2), levels[i] & 3); + } else if (target_levels[i] < 31) { + // Log the encoded level if a target level was given + av_log(avctx, AV_LOG_INFO, + "Output level for operating point %d is %d.%d.", + i, 2 + (levels[i] >> 2), levels[i] & 3); + } + } + } + } +#endif + aom_codec_destroy(&ctx->encoder); av_freep(&ctx->twopass_stats.buf); av_freep(&avctx->stats_out); -- 2.36.0.rc0.470.gd361397f0d-goog _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".