From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 015614038A for ; Thu, 18 Dec 2025 13:26:24 +0000 (UTC) Authentication-Results: ffbox; dkim=fail (body hash mismatch (got b'ClHqP+cOfS/J4KsTScZ1jCnX3KKonR9epi4VmcOd4k4=', expected b'nl2zFZCciAwmszcqq1Q941+1N7HxZGNSv/BnXyX5/pA=')) header.d=ffmpeg.org header.i=@ffmpeg.org header.a=rsa-sha256 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1766064356; h=mime-version : to : date : message-id : reply-to : subject : list-id : list-archive : list-archive : list-help : list-owner : list-post : list-subscribe : list-unsubscribe : from : cc : content-type : content-transfer-encoding : from; bh=ClHqP+cOfS/J4KsTScZ1jCnX3KKonR9epi4VmcOd4k4=; b=nSPLmt4EtLNC4AGsssjWgospmv2e+VWSmC1G+sH23yEP83aV4t7yFCtJytixh8NSC/WsY Ms1KBFMSLbZ76Bd8UADYBL3ocmC5P7xpdIP1PFBk4cPa5yjtJYbDbavzXqUpefPX4SNlE7U 5OfwLoypVFcUuDXeB9NIQ6JSan0HCCoJ6q3eGDHvbT9JF2Zc62h/x5rlCOfT32XZA+RwTyF 40IaVXDiGyNiQ/Z0uaKGTyftfxOUXjsCQQVaRj4RVcsbONmGKYQLPOd2EaOIeHJ2ZOBeHF3 wwu+imeJlHDAAn1P4rA9GbgixtnlzS/rtTiXJsmKZfapLnW/6yeZCscgaw8w== Received: from [172.20.0.2] (unknown [172.19.0.4]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id E8604690970; Thu, 18 Dec 2025 15:25:56 +0200 (EET) ARC-Seal: i=1; cv=none; a=rsa-sha256; d=ffmpeg.org; s=arc; t=1766064348; b=Ksh6BMn4yaGXBAHoAi2n8YuEVc2qkeNF2qKbViL7C2uaMrEOaU9Bs2rUHrexfjI2Bj0X0 +1DlPoVu/+q5v9bWU+3x+GX62kK/sLiInRl4A3WLAXlJGAXN7M6ewapOtMtNNbqmATjfesk 4t6BdctRRNENobnQVDnwHw9GZfLs1VWVw5cf3PdzM3p8XQn/l1UA0Xf3h1gar2wiCblRU9w mumz3+gvhijrTy9xG8AZX4obe/nqqpkPDZ03kPUIfPtQ6WthL3cLL6ercfsbxCC0FSsF5xI 0pFG1Dhgm2Ba+Fdv3duAB1ZWKUn6z90KSIzDTZG6HmNFc1t4ByF/aN3B/QDA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=ffmpeg.org; s=arc; t=1766064348; h=from : sender : reply-to : subject : date : message-id : to : cc : mime-version : content-type : content-transfer-encoding : content-id : content-description : resent-date : resent-from : resent-sender : resent-to : resent-cc : resent-message-id : in-reply-to : references : list-id : list-help : list-unsubscribe : list-subscribe : list-post : list-owner : list-archive; bh=qBV9xFSEMR1bPre54fABAOpzljnGMKSDH6Mh6vl5+dA=; b=buvB2bQ58USuMaXUuvKRuEGyeUrGOoIuf6B/Wu+/unvdyWx1915a+xNehIDL2OHxjK6cB 03qxt/TtlB438ow31Hn6mJK6SrZRaL7gdRlbz6wuXk3SPV7Hswj+clQK0SL0W9eK/fJ/Oba 6qnzcUVjVmQJcI3GnsRwPZDOxBu1x5AqklCkNGcIihgT6BMrK73tUhiO4oa2x593j4vdY83 VQ0zrsESap9Uk8BxY3XEXBYrWFjfEGeS/cWbOhrXmaVfbStUzfNPn2qtveIdpUhKqY9vkFU tWvud2Z1qROLIC9bmKxiKLmw7iFQVYvmk+fFLHvWYb7VejdF+MJc2/yjfEzQ== ARC-Authentication-Results: i=1; ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none; dmarc=pass header.from=ffmpeg.org policy.dmarc=quarantine Authentication-Results: ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none (Message is not ARC signed); dmarc=pass (Used From Domain Record) header.from=ffmpeg.org policy.dmarc=quarantine DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1766064340; h=content-type : mime-version : content-transfer-encoding : from : to : reply-to : subject : date : from; bh=nl2zFZCciAwmszcqq1Q941+1N7HxZGNSv/BnXyX5/pA=; b=bMSBS50K3xEg5LIjOOm8xre/BmJp/i/upJA7AEO1nOCnaIi4jRHNo8dpj2B73eaYMPH/N WdzBKU+aZ/AMZ2At3JkeDWIuVEedZCG9tUX1i7rjrFO9AFDKb8jB+cC/4EI+hL9otapHvLw ISjPjhraVw03TNn/Kckoa6FGcSiTtjATTUzlMUJ8LE2p552sLevNL3L19VNvOnyu1AofvBv QkGggL3hw5fgI6BmiHgUodz5chPHg4f8T/2h1z3AVicWIPb/wCcO/UKZmVtbF1aQqonLFr9 rkPEe4D3DYPNGc8/TZUxOFC9bmfONWv3cJxbfTBfaZfHRnfE296CvRCfFeHg== Received: from 55ca25703178 (code.ffmpeg.org [188.245.149.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 09670690954 for ; Thu, 18 Dec 2025 15:25:40 +0200 (EET) MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Date: Thu, 18 Dec 2025 13:25:39 -0000 Message-ID: <176606434018.60.17657347763031990738@2cb04c0e5124> Message-ID-Hash: B7E6PCMSKFN53E4DCRKOZYALY3SV3ZNV X-Message-ID-Hash: B7E6PCMSKFN53E4DCRKOZYALY3SV3ZNV X-MailFrom: code@ffmpeg.org X-Mailman-Rule-Hits: nonmember-moderation X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; header-match-ffmpeg-devel.ffmpeg.org-0; header-match-ffmpeg-devel.ffmpeg.org-1; header-match-ffmpeg-devel.ffmpeg.org-2; header-match-ffmpeg-devel.ffmpeg.org-3; emergency; member-moderation X-Mailman-Version: 3.3.10 Precedence: list Reply-To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATCH] avformat/rawvideodec: add stride option to skip line padding (PR #21233) List-Id: FFmpeg development discussions and patches Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: Zhao Zhili via ffmpeg-devel Cc: Zhao Zhili Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Archived-At: List-Archive: List-Post: PR #21233 opened by Zhao Zhili (quink) URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21233 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21233.patch Some tools like v4l2-ctl dump data without skip padding. If the padding size is not an integer multiple of the number of pixel bytes, we cannot handle it by using a larger width, e.g.,, for RGB24 image with 32 bytes padding in each line. >>From ac104802ad391f752ccf908c8f7bd9032acbdc03 Mon Sep 17 00:00:00 2001 From: Zhao Zhili Date: Thu, 18 Dec 2025 16:23:11 +0800 Subject: [PATCH 1/2] avformat/rawvideodec: use AV_OPT_TYPE_PIXEL_FMT instead of parse from string manually --- libavformat/rawvideodec.c | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/libavformat/rawvideodec.c b/libavformat/rawvideodec.c index 0d0b8876e6..1fde1124a5 100644 --- a/libavformat/rawvideodec.c +++ b/libavformat/rawvideodec.c @@ -32,7 +32,7 @@ typedef struct RawVideoDemuxerContext { const AVClass *class; /**< Class for private options. */ int width, height; /**< Integers describing video size, set by a private option. */ - char *pixel_format; /**< Set by a private option. */ + enum AVPixelFormat pix_fmt; AVRational framerate; /**< AVRational describing framerate, set by a private option. */ } RawVideoDemuxerContext; @@ -42,7 +42,7 @@ typedef struct RawVideoDemuxerContext { static int rawvideo_read_header(AVFormatContext *ctx) { RawVideoDemuxerContext *s = ctx->priv_data; - enum AVPixelFormat pix_fmt; + enum AVPixelFormat pix_fmt = s->pix_fmt; AVStream *st; int packet_size; int ret; @@ -54,16 +54,6 @@ static int rawvideo_read_header(AVFormatContext *ctx) st->codecpar->codec_type = AVMEDIA_TYPE_VIDEO; st->codecpar->codec_id = ffifmt(ctx->iformat)->raw_codec_id; - - if ((ffifmt(ctx->iformat)->raw_codec_id != AV_CODEC_ID_V210) && - (ffifmt(ctx->iformat)->raw_codec_id != AV_CODEC_ID_V210X)) { - if ((pix_fmt = av_get_pix_fmt(s->pixel_format)) == AV_PIX_FMT_NONE) { - av_log(ctx, AV_LOG_ERROR, "No such pixel format: %s.\n", - s->pixel_format); - return AVERROR(EINVAL); - } - } - avpriv_set_pts_info(st, 64, s->framerate.den, s->framerate.num); ret = av_image_check_size(s->width, s->height, 0, ctx); @@ -92,7 +82,7 @@ static int rawvideo_read_header(AVFormatContext *ctx) st->codecpar->codec_id = AV_CODEC_ID_RAWVIDEO; } else { av_log(ctx, AV_LOG_ERROR, "unsupported format: %s for bitpacked.\n", - s->pixel_format); + desc->name); return AVERROR(EINVAL); } st->codecpar->codec_tag = tag; @@ -137,7 +127,7 @@ static int rawvideo_read_packet(AVFormatContext *s, AVPacket *pkt) #define DEC AV_OPT_FLAG_DECODING_PARAM static const AVOption rawvideo_options[] = { /* pixel_format is not used by the v210 demuxers. */ - { "pixel_format", "set pixel format", OFFSET(pixel_format), AV_OPT_TYPE_STRING, {.str = "yuv420p"}, 0, 0, DEC }, + { "pixel_format", "set pixel format", OFFSET(pix_fmt), AV_OPT_TYPE_PIXEL_FMT, {.i64 = AV_PIX_FMT_YUV420P}, AV_PIX_FMT_YUV420P, INT_MAX, DEC }, { "video_size", "set frame size", OFFSET(width), AV_OPT_TYPE_IMAGE_SIZE, {.str = NULL}, 0, 0, DEC }, { "framerate", "set frame rate", OFFSET(framerate), AV_OPT_TYPE_VIDEO_RATE, {.str = "25"}, 0, INT_MAX, DEC }, { NULL }, -- 2.49.1 >>From 1320bc39f2001bebffcd6ac4fb1bc7c73d00f9b9 Mon Sep 17 00:00:00 2001 From: Zhao Zhili Date: Thu, 18 Dec 2025 20:33:19 +0800 Subject: [PATCH 2/2] avformat/rawvideodec: add stride option to skip line padding Some tools like v4l2-ctl dump data without skip padding. If the padding size is not an integer multiple of the number of pixel bytes, we cannot handle it by using a larger width, e.g.,, for RGB24 image with 32 bytes padding in each line. --- libavformat/rawvideodec.c | 84 ++++++++++++++++++++++++++++--- libavformat/version.h | 2 +- tests/fate/demux.mak | 6 +++ tests/ref/fate/rawvideo-rgb-demux | 7 +++ tests/ref/fate/rawvideo-yuv-demux | 7 +++ 5 files changed, 99 insertions(+), 7 deletions(-) create mode 100644 tests/ref/fate/rawvideo-rgb-demux create mode 100644 tests/ref/fate/rawvideo-yuv-demux diff --git a/libavformat/rawvideodec.c b/libavformat/rawvideodec.c index 1fde1124a5..62f1f8aff9 100644 --- a/libavformat/rawvideodec.c +++ b/libavformat/rawvideodec.c @@ -21,6 +21,8 @@ #include "config_components.h" +#include + #include "libavutil/imgutils.h" #include "libavutil/parseutils.h" #include "libavutil/pixdesc.h" @@ -34,6 +36,16 @@ typedef struct RawVideoDemuxerContext { int width, height; /**< Integers describing video size, set by a private option. */ enum AVPixelFormat pix_fmt; AVRational framerate; /**< AVRational describing framerate, set by a private option. */ + + bool has_padding; + /* We could derive linesize[1 to N] from linesize[0] for multiplane formats, + * but having users explicitly specify linesize for each plane can reduce + * unexpected results and support more use cases. + */ + int *linesize; + unsigned nb_linesize; + // linesize without padding + int raw_bytes[4]; } RawVideoDemuxerContext; // v210 frame width is padded to multiples of 48 @@ -63,6 +75,31 @@ static int rawvideo_read_header(AVFormatContext *ctx) st->codecpar->width = s->width; st->codecpar->height = s->height; + if (s->nb_linesize) { + int n = av_pix_fmt_count_planes(pix_fmt); + if (s->nb_linesize != n) { + av_log(ctx, AV_LOG_ERROR, "Invalid number of stride %u, " + "pixel format has %d plane\n", + s->nb_linesize, n); + return AVERROR(EINVAL); + } + + ret = av_image_fill_linesizes(s->raw_bytes, pix_fmt, s->width); + if (ret < 0) + return ret; + + for (int i = 0; i < n; i++) { + if (s->linesize[i] < s->raw_bytes[i]) { + av_log(ctx, AV_LOG_ERROR, "Invalid stride %u of plane %d, " + "minimum required size is %d for width %d\n", + s->linesize[i], i, s->raw_bytes[i], s->width); + return AVERROR(EINVAL); + } + if (s->linesize[i] > s->raw_bytes[i]) + s->has_padding = true; + } + } + if (ffifmt(ctx->iformat)->raw_codec_id == AV_CODEC_ID_BITPACKED) { unsigned int pgroup; /* size of the pixel group in bytes */ unsigned int xinc; @@ -110,24 +147,59 @@ static int rawvideo_read_header(AVFormatContext *ctx) } -static int rawvideo_read_packet(AVFormatContext *s, AVPacket *pkt) +static int rawvideo_read_packet(AVFormatContext *ctx, AVPacket *pkt) { int ret; + RawVideoDemuxerContext *s = ctx->priv_data; - ret = av_get_packet(s->pb, pkt, s->packet_size); - pkt->pts = pkt->dts = pkt->pos / s->packet_size; + if (!s->has_padding) { + ret = av_get_packet(ctx->pb, pkt, ctx->packet_size); + if (ret < 0) + return ret; + pkt->pts = pkt->dts = pkt->pos / ctx->packet_size; - pkt->stream_index = 0; + return 0; + } + + ret = av_new_packet(pkt, ctx->packet_size); if (ret < 0) return ret; + + pkt->pos = avio_tell(ctx->pb); + if (pkt->pos < 0) + return pkt->pos; + + const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(s->pix_fmt); + uint8_t *p = pkt->data; + for (int i = 0; i < s->nb_linesize; i++) { + int shift = (i == 1 || i == 2) ? desc->log2_chroma_h : 0; + int h = (s->height + (1 << shift) - 1) >> shift; + int skip_bytes = s->linesize[i] - s->raw_bytes[i]; + + for (int j = 0; j < h; j++) { + ret = avio_read(ctx->pb, p, s->raw_bytes[i]); + if (ret != s->raw_bytes[i]) + return ret < 0 ? ret : AVERROR_INVALIDDATA; + + p += s->raw_bytes[i]; + avio_skip(ctx->pb, skip_bytes); + } + } + + pkt->pts = pkt->dts = pkt->pos / (p - pkt->data); + return 0; } #define OFFSET(x) offsetof(RawVideoDemuxerContext, x) #define DEC AV_OPT_FLAG_DECODING_PARAM static const AVOption rawvideo_options[] = { + // Only supported by rawvideo demuxer + {"stride", "frame line size in bytes", OFFSET(linesize), AV_OPT_TYPE_INT | AV_OPT_TYPE_FLAG_ARRAY, {.arr = NULL}, 0, INT_MAX, DEC }, +#define BITPACKED_OPTION_OFFSET 1 // skip stride option /* pixel_format is not used by the v210 demuxers. */ { "pixel_format", "set pixel format", OFFSET(pix_fmt), AV_OPT_TYPE_PIXEL_FMT, {.i64 = AV_PIX_FMT_YUV420P}, AV_PIX_FMT_YUV420P, INT_MAX, DEC }, +#define V210_OPTION_OFFSET 2 // skip stride and pixel_format option { "video_size", "set frame size", OFFSET(width), AV_OPT_TYPE_IMAGE_SIZE, {.str = NULL}, 0, 0, DEC }, { "framerate", "set frame rate", OFFSET(framerate), AV_OPT_TYPE_VIDEO_RATE, {.str = "25"}, 0, INT_MAX, DEC }, { NULL }, @@ -155,7 +227,7 @@ const FFInputFormat ff_rawvideo_demuxer = { static const AVClass bitpacked_demuxer_class = { .class_name = "bitpacked demuxer", .item_name = av_default_item_name, - .option = rawvideo_options, + .option = &rawvideo_options[BITPACKED_OPTION_OFFSET], .version = LIBAVUTIL_VERSION_INT, }; @@ -176,7 +248,7 @@ const FFInputFormat ff_bitpacked_demuxer = { static const AVClass v210_demuxer_class = { .class_name = "v210(x) demuxer", .item_name = av_default_item_name, - .option = rawvideo_options + 1, + .option = &rawvideo_options[V210_OPTION_OFFSET], .version = LIBAVUTIL_VERSION_INT, }; diff --git a/libavformat/version.h b/libavformat/version.h index 35d796d318..392211f6b0 100644 --- a/libavformat/version.h +++ b/libavformat/version.h @@ -32,7 +32,7 @@ #include "version_major.h" #define LIBAVFORMAT_VERSION_MINOR 8 -#define LIBAVFORMAT_VERSION_MICRO 100 +#define LIBAVFORMAT_VERSION_MICRO 101 #define LIBAVFORMAT_VERSION_INT AV_VERSION_INT(LIBAVFORMAT_VERSION_MAJOR, \ LIBAVFORMAT_VERSION_MINOR, \ diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index ad1046a453..e904647410 100644 --- a/tests/fate/demux.mak +++ b/tests/fate/demux.mak @@ -136,6 +136,12 @@ fate-pva-demux: CMD = framecrc -idct simple -i $(TARGET_SAMPLES)/pva/PVA_test-pa FATE_SAMPLES_DEMUX-$(call CRC, QCP) += fate-qcp-demux fate-qcp-demux: CMD = crc -i $(TARGET_SAMPLES)/qcp/0036580847.QCP -c:a copy +FATE_SAMPLES_DEMUX-$(call FRAMECRC, RAWVIDEO, RAWVIDEO) += fate-rawvideo-rgb-demux +fate-rawvideo-rgb-demux: CMD = framecrc -f rawvideo -pixel_format rgb24 -video_size 160x500 -stride 512 -i $(TARGET_SAMPLES)/hevc/two_first_slice.mp4 + +FATE_SAMPLES_DEMUX-$(call FRAMECRC, RAWVIDEO, RAWVIDEO) += fate-rawvideo-yuv-demux +fate-rawvideo-yuv-demux: CMD = framecrc -f rawvideo -pixel_format yuv420p -video_size 250x500 -stride 256,128,128 -i $(TARGET_SAMPLES)/hevc/two_first_slice.mp4 + FATE_SAMPLES_DEMUX-$(call FRAMECRC, R3D, JPEG2000 PCM_S32BE) += fate-redcode-demux fate-redcode-demux: CMD = framecrc -i $(TARGET_SAMPLES)/r3d/4MB-sample.r3d -c:v copy -c:a copy diff --git a/tests/ref/fate/rawvideo-rgb-demux b/tests/ref/fate/rawvideo-rgb-demux new file mode 100644 index 0000000000..5231b84a59 --- /dev/null +++ b/tests/ref/fate/rawvideo-rgb-demux @@ -0,0 +1,7 @@ +#tb 0: 1/25 +#media_type 0: video +#codec_id 0: rawvideo +#dimensions 0: 160x500 +#sar 0: 0/1 +0, 0, 0, 1, 240000, 0x184c16a4 +0, 1, 1, 1, 240000, 0xb28b58b2 diff --git a/tests/ref/fate/rawvideo-yuv-demux b/tests/ref/fate/rawvideo-yuv-demux new file mode 100644 index 0000000000..bba00718d1 --- /dev/null +++ b/tests/ref/fate/rawvideo-yuv-demux @@ -0,0 +1,7 @@ +#tb 0: 1/25 +#media_type 0: video +#codec_id 0: rawvideo +#dimensions 0: 250x500 +#sar 0: 0/1 +0, 0, 0, 1, 187500, 0x16e17c12 +0, 1, 1, 1, 187500, 0xd36f0de2 -- 2.49.1 _______________________________________________ ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org