Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Zhao Zhili <quinkblack@foxmail.com>
To: ffmpeg-devel@ffmpeg.org
Cc: Zhao Zhili <zhilizhao@tencent.com>
Subject: [FFmpeg-devel] [PATCH v2 3/4] avcodec/hevcdec: add nuh_layer_id option
Date: Fri,  6 Jan 2023 23:52:29 +0800
Message-ID: <tencent_5F1F4DA9CFDF5521A752B8C9161A4D23A007@qq.com> (raw)
In-Reply-To: <20230106155230.487282-1-quinkblack@foxmail.com>

From: Zhao Zhili <zhilizhao@tencent.com>

It can be used to decode selected independent non-base layer. One
use case is alpha layer decoding.

Partially fix #7965.

Signed-off-by: Zhao Zhili <zhilizhao@tencent.com>
---
 libavcodec/hevcdec.c | 56 +++++++++++++++++++++++++++++++++++++++++++-
 libavcodec/version.h |  2 +-
 2 files changed, 56 insertions(+), 2 deletions(-)

diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
index 567e8d81d4..0b3d4e84d4 100644
--- a/libavcodec/hevcdec.c
+++ b/libavcodec/hevcdec.c
@@ -2956,6 +2956,35 @@ static int hevc_frame_end(HEVCContext *s)
     return 0;
 }
 
+static int check_layer(HEVCContext *s)
+{
+    const HEVCVPS *vps;
+
+    if (!s->nuh_layer_id)
+        return 0;
+
+    for (int i = 0; i < FF_ARRAY_ELEMS(s->ps.vps_list); i++) {
+        if (!s->ps.vps_list[i])
+            continue;
+
+        vps = (const HEVCVPS *)s->ps.vps_list[i]->data;
+        if (vps->vps_max_layers == 1) {
+            av_log(s->avctx, AV_LOG_ERROR,
+                    "Specified nuh_layer_id is %d but vps_max_layers is 1.\n",
+                    s->nuh_layer_id);
+            // Keep going for now.
+        } else if (s->nuh_layer_id > vps->vps_max_layer_id) {
+            av_log(s->avctx, AV_LOG_ERROR,
+                    "Specified nuh_layer_id is %d but vps_max_layer_id is %d.\n",
+                    s->nuh_layer_id, vps->vps_max_layer_id);
+            // It's a known issue that Apple videotoolbox encoder doesn't set
+            // vps_max_layer_id correctly. Keep going on purpose.
+        }
+    }
+
+    return 0;
+}
+
 static int decode_nal_unit(HEVCContext *s, const H2645NAL *nal)
 {
     HEVCLocalContext *lc = s->HEVClc;
@@ -2977,6 +3006,9 @@ static int decode_nal_unit(HEVCContext *s, const H2645NAL *nal)
                 goto fail;
         }
         ret = ff_hevc_decode_nal_vps(gb, s->avctx, &s->ps);
+        if (ret < 0)
+            goto fail;
+        ret = check_layer(s);
         if (ret < 0)
             goto fail;
         break;
@@ -3216,7 +3248,8 @@ static int decode_nal_units(HEVCContext *s, const uint8_t *buf, int length)
 
         if (s->avctx->skip_frame >= AVDISCARD_ALL ||
             (s->avctx->skip_frame >= AVDISCARD_NONREF
-            && ff_hevc_nal_is_nonref(nal->type)) || nal->nuh_layer_id > 0)
+            && ff_hevc_nal_is_nonref(nal->type)) ||
+            nal->nuh_layer_id != s->nuh_layer_id)
             continue;
 
         ret = decode_nal_unit(s, nal);
@@ -3315,6 +3348,10 @@ static int hevc_decode_extradata(HEVCContext *s, uint8_t *buf, int length, int f
     if (ret < 0)
         return ret;
 
+    ret = check_layer(s);
+    if (ret < 0)
+        return ret;
+
     /* export stream parameters from the first SPS */
     for (i = 0; i < FF_ARRAY_ELEMS(s->ps.sps_list); i++) {
         if (first && s->ps.sps_list[i]) {
@@ -3637,6 +3674,21 @@ static av_cold int hevc_decode_init(AVCodecContext *avctx)
     HEVCContext *s = avctx->priv_data;
     int ret;
 
+    if (s->nuh_layer_id > 0) {
+        if (avctx->strict_std_compliance > FF_COMPLIANCE_EXPERIMENTAL) {
+            av_log(avctx, AV_LOG_ERROR,
+                    "Decoding selected nuh_layer_id is under development. "
+                    "Use -strict experimental to use it anyway.\n");
+            return AVERROR(EINVAL);
+        }
+
+        if (avctx->hwaccel) {
+            av_log(avctx, AV_LOG_ERROR,
+                    "Decoding selected nuh_layer_id doesn't work with hwaccel.\n");
+            return AVERROR(EINVAL);
+        }
+    }
+
     if (avctx->active_thread_type & FF_THREAD_SLICE) {
         s->threads_number = avctx->thread_count;
         ret = ff_slice_thread_init_progress(avctx);
@@ -3691,6 +3743,8 @@ static const AVOption options[] = {
         AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, PAR },
     { "strict-displaywin", "stricly apply default display window size", OFFSET(apply_defdispwin),
         AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, PAR },
+    { "nuh_layer_id", "Select which nuh_layer_id to decode (only works with INBL)", OFFSET(nuh_layer_id),
+        AV_OPT_TYPE_INT,  {.i64 = 0}, 0, 62, PAR },
     { NULL },
 };
 
diff --git a/libavcodec/version.h b/libavcodec/version.h
index 6b8a1dbb79..15f7c3fe3d 100644
--- a/libavcodec/version.h
+++ b/libavcodec/version.h
@@ -30,7 +30,7 @@
 #include "version_major.h"
 
 #define LIBAVCODEC_VERSION_MINOR  56
-#define LIBAVCODEC_VERSION_MICRO 100
+#define LIBAVCODEC_VERSION_MICRO 101
 
 #define LIBAVCODEC_VERSION_INT  AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
                                                LIBAVCODEC_VERSION_MINOR, \
-- 
2.25.1

_______________________________________________
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".

  parent reply	other threads:[~2023-01-06  7:53 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20230106155230.487282-1-quinkblack@foxmail.com>
2023-01-06 15:52 ` [FFmpeg-devel] [PATCH v2 1/4] avcodec/hevc_parse: keep nal->nuh_layer_id > 0 Zhao Zhili
2023-01-06 15:52 ` [FFmpeg-devel] [PATCH v2 2/4] avcodec/cbs_h2645: " Zhao Zhili
2023-01-06 15:52 ` Zhao Zhili [this message]
2023-01-06 15:52 ` [FFmpeg-devel] [PATCH v2 4/4] avcodec: add hevc_extract_layer bsf Zhao Zhili

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=tencent_5F1F4DA9CFDF5521A752B8C9161A4D23A007@qq.com \
    --to=quinkblack@foxmail.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    --cc=zhilizhao@tencent.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git