From: Zhao Zhili <quinkblack@foxmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v7 06/11] avformat: add demuxer and probe support for H266/VVC
Date: Mon, 18 Dec 2023 19:43:15 +0800
Message-ID: <tencent_34AC8E3A41C2ADB700681200B96B3A7DEA0A@qq.com> (raw)
In-Reply-To: <20230321150124.21999-7-thomas.ff@spin-digital.com>
> On Mar 21, 2023, at 23:01, Thomas Siedel <thomas.ff@spin-digital.com> wrote:
>
> Add demuxer to probe raw vvc and parse vvcc byte stream format.
>
> Co-authored-by: Nuo Mi <nuomi2021@gmail.com>
> ---
>
> +
> +static int h266_probe(const AVProbeData *p)
> +{
> + uint32_t code = -1;
> + int sps = 0, pps = 0, irap = 0;
> + int i;
> +
> + for (i = 0; i < p->buf_size - 1; i++) {
> + code = (code << 8) + p->buf[i];
> + if ((code & 0xffffff00) == 0x100) {
> + uint8_t nal2 = p->buf[i + 1];
> + int type = (nal2 & 0xF8) >> 3;
> +
> + if (code & 0x80) // forbidden_zero_bit
> + return 0;
> +
> + if ((nal2 & 0x7) == 0) // nuh_temporal_id_plus1
> + return 0;
> +
> + switch (type) {
> + case VVC_SPS_NUT: sps++; break;
> + case VVC_PPS_NUT: pps++; break;
> + case VVC_IDR_N_LP:
> + case VVC_IDR_W_RADL:
> + case VVC_CRA_NUT:
> + case VVC_GDR_NUT: irap++; break;
> + }
> + }
> + }
> +
> + if (sps && pps && irap)
> + return AVPROBE_SCORE_EXTENSION + 1; // 1 more than .mpg
The probe isn’t reliable enough. It mistreated an TS as vvc,
https://trac.ffmpeg.org/ticket/10731#comment:1
Any idea to reduce false positive?
> + return 0;
> +}
> +
> +FF_DEF_RAWVIDEO_DEMUXER(h266, "raw H.266/VVC video", h266_probe, "h266,266,vvc", AV_CODEC_ID_VVC)
> --
> 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".
_______________________________________________
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".
next prev parent reply other threads:[~2023-12-18 11:43 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-21 15:01 [FFmpeg-devel] [PATCH v7 00/11] Add " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 01/11] avcodec: add enum types " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 02/11] avcodec: add cbs " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 03/11] avcodec: add bitstream parser " Thomas Siedel
2023-06-29 20:35 ` Andreas Rheinhardt
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 04/11] avcodec: add h266_metadata_bsf support " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 05/11] avcodec: add MP4 to annexb " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 06/11] avformat: add demuxer and probe " Thomas Siedel
2023-12-18 11:43 ` Zhao Zhili [this message]
2023-12-18 12:25 ` Nuo Mi
2023-12-19 4:01 ` Zhao Zhili
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 07/11] avformat: add muxer " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 08/11] avcodec: add external decoder libvvdec " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 09/11] avcodec: add external encoder libvvenc " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 10/11] avformat: add ts stream types " Thomas Siedel
2023-03-21 15:01 ` [FFmpeg-devel] [PATCH v7 11/11] avcodec: increase minor version " Thomas Siedel
2023-06-29 17:38 ` [FFmpeg-devel] [PATCH v7 00/11] Add support " James Almer
2023-07-01 11:24 ` Nuo Mi
2023-07-03 15:22 ` Thomas Siedel
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_34AC8E3A41C2ADB700681200B96B3A7DEA0A@qq.com \
--to=quinkblack@foxmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
/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