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

  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