[FFmpeg-devel] [PATCH 7/7] avcodec: [loongarch] Optimize pred16x16_plane with LASX.

2021-12-13 Thread Hao Chen
./ffmpeg -i ../1_h264_1080p_30fps_3Mbps.mp4 -f rawvideo -y /dev/null -an before:295 after :296 Change-Id: I281bc739f708d45f91fc3860150944c0b8a6a5ba --- libavcodec/h264pred.c | 2 + libavcodec/h264pred.h | 2 + libavcodec/loongarch/Makefile

[FFmpeg-devel] [PATCH 5/7] avcodec: [loongarch] Optimize h264idct with LASX.

2021-12-13 Thread Hao Chen
From: Lu Wang ./ffmpeg -i ../1_h264_1080p_30fps_3Mbps.mp4 -f rawvideo -y /dev/null -an before:282 after :293 Change-Id: Ia8889935a6359630dd5dbb61263287f1cb24a0a4 --- libavcodec/loongarch/Makefile | 3 +- libavcodec/loongarch/h264dsp_init_loongarch.c | 15 + libavcodec/loongar

[FFmpeg-devel] [PATCH 6/7] avcodec: [loongarch] Optimize h264_deblock with LASX.

2021-12-13 Thread Hao Chen
From: Jin Bo ./ffmpeg -i ../1_h264_1080p_30fps_3Mbps.mp4 -f rawvideo -y /dev/null -an before:293 after :295 Change-Id: I5ff6cba4eaca0c4218c0c97b880ca500e35f9c87 Signed-off-by: Hao Chen --- libavcodec/loongarch/Makefile | 3 +- libavcodec/loongarch/h264_deblock_lasx.c | 1

[FFmpeg-devel] [PATCH 4/7] avcodec: [loongarch] Optimize h264dsp with LASX.

2021-12-13 Thread Hao Chen
From: gxw ./ffmpeg -i ../1_h264_1080p_30fps_3Mbps.mp4 -f rawvideo -y /dev/null -an before:225 after :282 Change-Id: Ibe245827dcdfe8fc1541c6b172483151bfa9e642 --- libavcodec/h264dsp.c |1 + libavcodec/h264dsp.h |2 + libavcodec/loongarch/

[FFmpeg-devel] [PATCH 3/7] avcodec: [loongarch] Optimize h264qpel with LASX.

2021-12-13 Thread Hao Chen
From: Shiyou Yin ./ffmpeg -i ../1_h264_1080p_30fps_3Mbps.mp4 -f rawvideo -y /dev/null -an before:183 after :225 Change-Id: I7c7d2f34cd82ef728aab5ce8f6bfb46dd81f0da4 --- libavcodec/h264qpel.c |2 + libavcodec/h264qpel.h |1 + libavcodec/loo

[FFmpeg-devel] [PATCH 1/7] avutil: [loongarch] Add support for loongarch SIMD.

2021-12-13 Thread Hao Chen
From: Shiyou Yin LSX and LASX is loongarch SIMD extention. They are enabled by default if compiler support it, and can be disabled with '--disable-lsx' '--disable-lasx'. Change-Id: Ie2608ea61dbd9b7fffadbf0ec2348bad6c124476 --- Makefile | 2 +- configure|

[FFmpeg-devel] Optimize H264 decoding for loongarch.

2021-12-13 Thread Hao Chen
___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH] lavc/qsvenc: add tile encoding support for VP9

2021-12-13 Thread Haihao Xiang
Add -tile_rows and -tile_cols options to specify the number of tile rows and columns --- libavcodec/qsvenc.c | 4 libavcodec/qsvenc_vp9.c | 10 ++ 2 files changed, 14 insertions(+) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 26a94cd419..270bdad544 100644 --- a/l

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi

2021-12-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of U. Artie > Eoff > Sent: Tuesday, December 14, 2021 5:08 AM > To: ffmpeg-devel@ffmpeg.org > Cc: U. Artie Eoff > Subject: [FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi > > Current listed maintainers for vaapi plugin

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Anton Khirnov
Quoting Marton Balint (2021-12-13 22:30:25) > > > On Mon, 13 Dec 2021, Jean-Baptiste Kempf wrote: > > > On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: > >> If you know of any major issues which need to be done before the release > >> do them > >> now. If you know of any issues which

Re: [FFmpeg-devel] [PATCH] libavutil/hwcontext_qsv: clean padding when upload qsv frames

2021-12-13 Thread Eoff, Ullysses A
> -Original Message- > From: ffmpeg-devel On Behalf Of Chen, Wenbin > Sent: Monday, December 13, 2021 6:24 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] libavutil/hwcontext_qsv: clean padding > when upload qsv frames > > > On Thu, 2021-12-02 at 15:40 +0800, Wenbi

[FFmpeg-devel] [PATCH] MAINTAINERS: Add Haihao Xiang for vaapi

2021-12-13 Thread U. Artie Eoff
Current listed maintainers for vaapi plugin are not reponsive and/or currently active in the ffmpeg community. Thus, vaapi plugin patches (and qsv plugin) have generally gone ignored or lost in the ether for too long. Remove Gwenole Beauchesne from vaapi maintainer who has not been active since 2

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Daniel > Cantarín > Sent: Tuesday, December 14, 2021 4:05 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Politics > > > > > > As mentioned already, I have an offer to make. It might not be exactly > > what you wa

Re: [FFmpeg-devel] [PATCH] lavc/{av1, h264, h265}_metadata_bsf: fix description of tick_rate

2021-12-13 Thread myp...@gmail.com
On Fri, Aug 27, 2021 at 3:14 PM Zhao Zhili wrote: > > Users may take the description literally which leads to inverted > results. > --- > doc/bitstream_filters.texi | 8 > libavcodec/av1_metadata_bsf.c | 2 +- > libavcodec/h264_metadata_bsf.c | 2 +- > libavcodec/h265_metadata_bsf.c

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Daniel Cantarín
> > As mentioned already, I have an offer to make. It might not be exactly > what you want, but it's all you can get. > > Everybody will need to make up his mind and decide whether the benefits > will outweigh the drawback from one's own point of view - or not. > I don't feel I have a voice her

Re: [FFmpeg-devel] [PATCH] avfilter: add yadif implementation using Metal.framework

2021-12-13 Thread Aman Karmani
On Mon, Dec 13, 2021 at 6:21 PM Lynne wrote: > 14 Dec 2021, 02:47 by ffm...@tmm1.net: > > > From: Aman Karmani > > > > deinterlaces CVPixelBuffers, i.e. AV_PIX_FMT_VIDEOTOOLBOX frames > > > > for example, an interlaced mpeg2 file can be decoded by avcodec, > > uploaded into a CVPixelBuffer, dein

Re: [FFmpeg-devel] [PATCH] libavutil/hwcontext_qsv: clean padding when upload qsv frames

2021-12-13 Thread Chen, Wenbin
> On Thu, 2021-12-02 at 15:40 +0800, Wenbin Chen wrote: > > When we upload a frame that is not padded as MSDK requires, we create a > > new AVFrame to copy data. The frame's padding data is uninitialized so > > it brings run to run problem. For example, If we run the following > > command serveral

Re: [FFmpeg-devel] [PATCH] avfilter: add yadif implementation using Metal.framework

2021-12-13 Thread Lynne
14 Dec 2021, 02:47 by ffm...@tmm1.net: > From: Aman Karmani > > deinterlaces CVPixelBuffers, i.e. AV_PIX_FMT_VIDEOTOOLBOX frames > > for example, an interlaced mpeg2 file can be decoded by avcodec, > uploaded into a CVPixelBuffer, deinterlaced by metal, and then > encoded to h264 by VideoToolbox

Re: [FFmpeg-devel] [PATCH 001/279] Add a new channel layout API

2021-12-13 Thread Lynne
14 Dec 2021, 02:13 by c...@passwd.hu: > > > On Tue, 7 Dec 2021, James Almer wrote: > > I would like to attach some extendable, possibly per-channel metadata to the > channel layout. I'd rather put it into AVChannelLayout, so native layouts > could also have metadata. This must be dynamically all

[FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_dec: dequeue frame if input isn't ready

2021-12-13 Thread Cameron Gutman
The V4L2M2M API operates asynchronously, so multiple packets can be enqueued before getting a batch of frames back. Since it was only possible to receive a frame by submitting another packet, there wasn't a way to drain those excess output frames from when avcodec_receive_frame() returned AVERROR(E

[FFmpeg-devel] [PATCH] avfilter: add yadif implementation using Metal.framework

2021-12-13 Thread Aman Karmani
From: Aman Karmani deinterlaces CVPixelBuffers, i.e. AV_PIX_FMT_VIDEOTOOLBOX frames for example, an interlaced mpeg2 file can be decoded by avcodec, uploaded into a CVPixelBuffer, deinterlaced by metal, and then encoded to h264 by VideoToolbox as follows: ffmpeg \ -init_hw_device

Re: [FFmpeg-devel] [PATCH 001/279] Add a new channel layout API

2021-12-13 Thread James Almer
On 12/13/2021 10:13 PM, Marton Balint wrote: On Tue, 7 Dec 2021, James Almer wrote: From: Anton Khirnov [...] -static const char *get_channel_name(int channel_id) +static const char *get_channel_name(enum AVChannel channel_id) { -    if (channel_id < 0 || channel_id >= FF_ARRAY_ELEMS(

Re: [FFmpeg-devel] [PATCH 001/279] Add a new channel layout API

2021-12-13 Thread Marton Balint
On Tue, 7 Dec 2021, James Almer wrote: From: Anton Khirnov [...] -static const char *get_channel_name(int channel_id) +static const char *get_channel_name(enum AVChannel channel_id) { -if (channel_id < 0 || channel_id >= FF_ARRAY_ELEMS(channel_names)) -return NULL; +if (

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Daniel Cantarín
> > Incidentally, the "fps" filter is probably one of the simplest examples > of a filter that does not involve subtitles which would benefit from a > "heartbeat" mechanism. > > Currently in order to be able to output a frame, the fps filter needs > to have 2 frames buffered (or alternately, one

Re: [FFmpeg-devel] [PATCH 000/279] New channel layout API

2021-12-13 Thread James Almer
On 12/13/2021 8:36 PM, Nicolas George wrote: James Almer (12021-12-13): To achieve this you don't need the same AVChannel value to appear several times in the same layout. You have INT_MAX values available, so just assign one to each of these you mentioned. No need for an abstract value "user

[FFmpeg-devel] [PATCH v24 21/21] doc/APIchanges: update for subtitle filtering changes

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- doc/APIchanges | 23 +++ libavcodec/version.h | 2 +- libavutil/version.h | 2 +- 3 files changed, 25 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 17aa664ca3..1e12225b60 100644 --- a/doc/APIchanges +++

[FFmpeg-devel] [PATCH v24 20/21] avfilter/subscale: Add filter for scaling and/or re-arranging graphical subtitles

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- configure | 1 + doc/filters.texi | 164 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/sf_subscale.c | 883 ++ 5 files changed, 1050 insertions(+) create mode 1

[FFmpeg-devel] [PATCH v24 19/21] avfilter/graphicsub2text: Add new graphicsub2text filter (OCR)

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- configure| 1 + doc/filters.texi | 55 + libavfilter/Makefile | 2 + libavfilter/allfilters.c | 1 + libavfilter/sf_graphicsub2text.c | 354 +++ 5 files changed, 413 ins

[FFmpeg-devel] [PATCH v24 18/21] avfilter/splitcc: Add splitcc filter for closed caption handling

2021-12-13 Thread Soft Works
- splitcc {V -> VS) Extract closed-caption (A53) data from video frames as subtitle Frames ffmpeg -y -loglevel verbose -i "https://streams.videolan.org/streams/ts/CC/NewsStream-608-ac3.ts"; -filter_complex "[0:v]splitcc[vid1],textmod=mode=remove_chars:find='@',[vid1]overlay_textsubs" outpu

[FFmpeg-devel] [PATCH v24 17/21] avfilter/stripstyles: Add stripstyles filter

2021-12-13 Thread Soft Works
- stripstyles {S -> S) Remove all inline styles from subtitle events Signed-off-by: softworkz --- doc/filters.texi | 37 +++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/sf_stripstyles.c | 196 +++ 4 fi

[FFmpeg-devel] [PATCH v24 16/21] avfilter/textmod: Add textmod, censor and show_speaker filters

2021-12-13 Thread Soft Works
s- textmod {S -> S) Modify subtitle text in a number of ways - censor {S -> S) Censor subtitles using a word list - show_speaker {S -> S) Prepend speaker names from ASS subtitles to the visible text lines Signed-off-by: softworkz --- doc/filters.texi | 206 libavfilt

[FFmpeg-devel] [PATCH v24 15/21] avfilter/overlaytextsubs: Add overlaytextsubs and textsubs2video filters

2021-12-13 Thread Soft Works
- overlaytextsubs {VS -> V) Overlay text subtitles onto a video stream. - textsubs2video {S -> V) Converts text subtitles to video frames Signed-off-by: softworkz --- configure| 2 + doc/filters.texi | 113 ++ libavfilter/Makefile |

[FFmpeg-devel] [PATCH v24 14/21] avfilter/avfilter: Fix hardcoded input index

2021-12-13 Thread Soft Works
This fix targets (rare) cases where multiple input pads have a .filter_frame function. ff_request_frame_to_filter needs to call ff_request_frame with the correct input pad instead of the hardcoded first one. Signed-off-by: softworkz --- libavfilter/avfilter.c | 18 +- 1 file chan

[FFmpeg-devel] [PATCH v24 13/21] fftools/ffmpeg: Replace sub2video with subtitle frame filtering and use new frame-based subtitle encoding API

2021-12-13 Thread Soft Works
This commit actually enables subtitle filtering in ffmpeg by sending and receiving subtitle frames to and from a filtergraph. The heartbeat functionality from the previous sub2video implementation is retained and applied to all subtitle frames (bitmap, text, ..). The other part of sub2video funct

[FFmpeg-devel] [PATCH v24 12/21] avfilter/overlaygraphicsubs: Add overlaygraphicsubs and graphicsub2video filters

2021-12-13 Thread Soft Works
- overlaygraphicsubs (VS -> V) Overlay graphic subtitles onto a video stream - graphicsub2video {S -> V) Converts graphic subtitles to video frames (with alpha) Gets auto-inserted for retaining compatibility with sub2video command lines Signed-off-by: softworkz --- doc/filters.texi

[FFmpeg-devel] [PATCH v24 11/21] avfilter/sbuffer: Add sbuffersrc and sbuffersink filters

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- configure| 2 +- libavfilter/allfilters.c | 2 ++ libavfilter/buffersink.c | 54 ++ libavfilter/buffersink.h | 7 libavfilter/buffersrc.c | 72 libavfilter/buffersrc.h | 1

[FFmpeg-devel] [PATCH v24 10/21] avfilter/avfilter: Handle subtitle frames

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- libavfilter/avfilter.c | 8 +--- libavfilter/avfilter.h | 11 +++ libavfilter/avfiltergraph.c | 5 + libavfilter/formats.c | 22 ++ libavfilter/formats.h | 3 +++ libavfilter/internal.h | 18 +++

[FFmpeg-devel] [PATCH v24 09/21] avfilter/subtitles: Add subtitles.c for subtitle frame allocation

2021-12-13 Thread Soft Works
Analog to avfilter/video.c and avfilter/audio.c Signed-off-by: softworkz --- libavfilter/Makefile| 1 + libavfilter/avfilter.c | 4 +++ libavfilter/internal.h | 1 + libavfilter/subtitles.c | 63 + libavfilter/subtitles.h | 44

[FFmpeg-devel] [PATCH v24 08/21] fftools/play, probe: Adjust for subtitle changes

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- fftools/ffplay.c | 102 +- fftools/ffprobe.c | 48 ++ 2 files changed, 78 insertions(+), 72 deletions(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index e7b20be76b..0af32888da 100644 --- a/fftoo

[FFmpeg-devel] [PATCH v24 07/21] avcodec/subtitles: Replace deprecated enum values

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- libavcodec/ass.h | 2 +- libavcodec/assdec.c| 2 +- libavcodec/dvbsubdec.c | 2 +- libavcodec/dvdsubdec.c | 2 +- libavcodec/dvdsubenc.c | 2 +- libavcodec/pgssubdec.c | 2 +- libavcodec/xsubdec.c | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) d

[FFmpeg-devel] [PATCH v24 06/21] avcodec/subtitles: Migrate subtitle encoders to frame-based API and provide a compatibility shim for the legacy api

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- libavcodec/assenc.c| 90 + libavcodec/avcodec.h | 5 +- libavcodec/dvbsubenc.c | 96 +-- libavcodec/dvdsubenc.c | 100 +++-- libavcodec/encode.c| 63

[FFmpeg-devel] [PATCH v24 05/21] avcodec, avutil: Move ass helper functions to avutil as avpriv_ and extend ass dialog parsing

2021-12-13 Thread Soft Works
Also add - hard_space callback (for upcoming fix) - extensible callback (for future extension) Signed-off-by: softworkz --- libavcodec/Makefile | 56 +++ libavcodec/ass.h | 147 ++ libavcodec/assdec.c

[FFmpeg-devel] [PATCH v24 04/21] avfilter/subtitles: Update vf_subtitles to use new decoding api

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- libavfilter/vf_subtitles.c | 54 +- 1 file changed, 42 insertions(+), 12 deletions(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 3fc4eeb63d..66b2082894 100644 --- a/libavfilter/vf_subtitles.c +++ b/l

[FFmpeg-devel] [PATCH v24 03/21] avcodec/subtitles: Introduce new frame-based subtitle decoding API

2021-12-13 Thread Soft Works
- Add avcodec_decode_subtitle3 which takes subtitle frames, serving as compatibility shim to legacy subtitle decoding - Add additional methods for conversion between old and new API Signed-off-by: softworkz --- libavcodec/avcodec.h| 8 +- libavcodec/codec_desc.c | 11 +++ libavcodec/cod

[FFmpeg-devel] [PATCH v24 02/21] avutil/frame: Prepare AVFrame for subtitle handling

2021-12-13 Thread Soft Works
Root commit for adding subtitle filtering capabilities. In detail: - Add type (AVMediaType) field to AVFrame Replaces previous way of distinction which was based on checking width and height to determine whether a frame is audio or video - Add subtitle fields to AVFrame - Add new struct AVSubt

[FFmpeg-devel] [PATCH v24 01/21] avcodec, avutil: Move enum AVSubtitleType to avutil, add new and deprecate old values

2021-12-13 Thread Soft Works
Signed-off-by: softworkz --- libavcodec/avcodec.h | 19 + libavutil/Makefile | 1 + libavutil/subfmt.h | 68 libavutil/version.h | 1 + 4 files changed, 71 insertions(+), 18 deletions(-) create mode 100644 libavutil/subfmt.h diff -

[FFmpeg-devel] [PATCH v24 00/21] Subtitle Filtering

2021-12-13 Thread Soft Works
New in V24 - Fixes bugs as reported by Michal - graphicsub2video: use 1x1 output frame size as long as subtitle size is unknown (0x0) and no size is not explicitly set - decode: set subtitle frame size from decoding context - ffmpeg: re-init graph when subtitle size changes - ffmpeg_filter: alway

Re: [FFmpeg-devel] [PATCH 000/279] New channel layout API

2021-12-13 Thread Nicolas George
James Almer (12021-12-13): > To achieve this you don't need the same AVChannel value to appear several > times in the same layout. You have INT_MAX values available, so just assign > one to each of these you mentioned. No need for an abstract value "user > defined" that would then show up several t

[FFmpeg-devel] [PATCH] mov_read_mvhd: Expose Quicktime poster_time value as metadata TAG.

2021-12-13 Thread Bryce Newman
I need the ability to derive the poster time found in the mvhd, so I can use that value to create a thumbnail from ffmpeg. More details can be found here https://www.mail-archive.com/ffmpeg-user@ffmpeg.org/msg30003.html. Signed-off-by: Bryce Chester Newman mailto:bryce.new...@gettyimages.com>> --

Re: [FFmpeg-devel] [PATCH 000/279] New channel layout API

2021-12-13 Thread James Almer
On 12/12/2021 5:00 PM, Nicolas George wrote: Anton Khirnov (12021-12-12): So what are you proposing? In my view, such higher level information should live at a higher level - e.g. in the side data. You can then have a filter that reads this side data and gets you the group you want. So, wha

Re: [FFmpeg-devel] [PATCH 000/279] New channel layout API

2021-12-13 Thread Marton Balint
On Sun, 12 Dec 2021, Anton Khirnov wrote: Quoting Marton Balint (2021-12-10 01:04:57) On Thu, 9 Dec 2021, Anton Khirnov wrote: Quoting Nicolas George (2021-12-09 11:31:54) Anton Khirnov (12021-12-09): I disagree. Technical limitations that were overcome 10 years ago should not guide ne

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Hendrik Leppkes
On Mon, Dec 13, 2021 at 8:48 PM Anton Khirnov wrote: > > > - The request to wait for a new string handling API to land and be used > > for the relevant functions in this set. This has not seen opposition > > from anyone yet, but it's not very feasible if said new API is not yet > > in a state beyo

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Marton Balint
On Mon, 13 Dec 2021, Jean-Baptiste Kempf wrote: On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: If you know of any major issues which need to be done before the release do them now. If you know of any issues which are release-blocking list them in a reply here please. Maybe the a

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Diederick C. Niehorster
Hi Michael, On Mon, Dec 13, 2021 at 4:26 PM Michael Niedermayer wrote: > > If you know of any major issues which need to be done before the release do > them > now. If you know of any issues which are release-blocking list them in a reply > here please. Not major, but https://ffmpeg.org/piperm

Re: [FFmpeg-devel] [PATCH v2 02/03] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length.

2021-12-13 Thread Marvin Scholz
On 13 Dec 2021, at 21:29, Romain Beauxis wrote: On Dec 13, 2021, at 12:56 PM, Marvin Scholz wrote: On 13 Dec 2021, at 17:39, Romain Beauxis wrote: This is the second patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch fixes the c

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Nicolas George
James Almer (12021-12-13): > I wholly agree. Hence me suggesting using AVBPrint in the meantime. It > certainly beats a uint8_t* + size_t combination, and it's trivial to do. There are a few misunderstanding here. For the per-channel label, I insist on at least a string, but a plain C string, eve

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Nicolas George
Lynne (12021-12-13): > Consensus was reached. A 16-bit opaque identifier, which users can > simply use as an index into their own frame opaque structure, if they > want, and a 16-bit channel identifier, which would result in a 32-bit > structure, which is pretty acceptable. No, consensus was not r

[FFmpeg-devel] [PATCH] lavc/mediacodecdec: set codec profile and level from extradata for H264+HEVC

2021-12-13 Thread sfan5
This value is later passed to MediaCodec and checked at decoder init. Notably decoding of 10-bit streams before this commit would "work" without returning errors but only return garbage output (on most Android devices). From 304d1bd55e72212c1e907922547d40da240b Mon Sep 17 00:00:00 2001 From: s

[FFmpeg-devel] [PATCH] lavf/tls_mbedtls: fix handling of tls_verify=0

2021-12-13 Thread sfan5
If ca_file was set, setting tls_verify=0 would not actually disable verification. From 2677353187c4e3c20b50a3f9aab53130e3ead99b Mon Sep 17 00:00:00 2001 From: sfan5 Date: Mon, 13 Dec 2021 21:35:40 +0100 Subject: [PATCH] lavf/tls_mbedtls: fix handling of tls_verify=0 If ca_file was set, setting t

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Lynne > Sent: Monday, December 13, 2021 8:49 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] Politics > > 13 Dec 2021, 19:26 by softwo...@hotmail.com: > > > > > > >> -Original Message- > >

Re: [FFmpeg-devel] [PATCH v2 02/03] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length.

2021-12-13 Thread Romain Beauxis
> On Dec 13, 2021, at 12:56 PM, Marvin Scholz wrote: > > > > On 13 Dec 2021, at 17:39, Romain Beauxis wrote: > >> This is the second patch of a series of 3 that cleanup and enhance the >> avfoundation implementation for libavdevice. >> >> This patch fixes the concurrency model. Avfoundation

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Anton Khirnov
Quoting James Almer (2021-12-13 21:05:30) > > > On 12/13/2021 5:01 PM, Anton Khirnov wrote: > > Quoting James Almer (2021-12-13 20:51:05) > >> > >> > >> On 12/13/2021 4:47 PM, Anton Khirnov wrote: > >>> Quoting James Almer (2021-12-13 20:04:34) > > > On 12/13/2021 3:05 PM, Jean-Bap

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread James Almer
On 12/13/2021 5:01 PM, Anton Khirnov wrote: Quoting James Almer (2021-12-13 20:51:05) On 12/13/2021 4:47 PM, Anton Khirnov wrote: Quoting James Almer (2021-12-13 20:04:34) On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: If yo

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Anton Khirnov
Quoting James Almer (2021-12-13 20:51:05) > > > On 12/13/2021 4:47 PM, Anton Khirnov wrote: > > Quoting James Almer (2021-12-13 20:04:34) > >> > >> > >> On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: > >>> On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: > If you know of any majo

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Calvin > Walton > Sent: Monday, December 13, 2021 8:32 PM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] Politics > > On Sun, 2021-12-12 at 06:15 +, Soft Works wrote: > > Good Morning, > > > > ye

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Calvin Walton
On Mon, 2021-12-13 at 14:32 -0500, Calvin Walton wrote: > like, for example, I rewrote the "fps" filter to use the activate > callback to handle filling in large timestamp gaps a frame at a time > rather than buffering a ton of frames all at once and causing OOM. But > that only fixed the situation

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread James Almer
On 12/13/2021 4:47 PM, Anton Khirnov wrote: Quoting James Almer (2021-12-13 20:04:34) On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: If you know of any major issues which need to be done before the release do them now. If you kn

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Lynne
13 Dec 2021, 19:26 by softwo...@hotmail.com: > > >> -Original Message- >> From: Jean-Baptiste Kempf >> Sent: Monday, December 13, 2021 7:01 PM >> To: FFmpeg development discussions and patches >> Cc: Soft Works >> Subject: Re: [FFmpeg-devel] Politics >> >> Hello Softworkz >> >> On Sun,

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Anton Khirnov
Quoting James Almer (2021-12-13 20:04:34) > > > On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: > > On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: > >> If you know of any major issues which need to be done before the release > >> do them > >> now. If you know of any issues which are

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Lynne
13 Dec 2021, 20:04 by jamr...@gmail.com: > > > On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: > >> On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: >> >>> If you know of any major issues which need to be done before the release do >>> them >>> now. If you know of any issues which are

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Calvin Walton
On Sun, 2021-12-12 at 06:15 +, Soft Works wrote: > Good Morning, > > yesterday, it happened for the 4th and 5th times that another > developer > called my patchset a “hack”. This might be partially a language problem. Ffrom familiarity with the usage in several primarily-English development

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread James Almer
On 12/13/2021 3:05 PM, Jean-Baptiste Kempf wrote: On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: If you know of any major issues which need to be done before the release do them now. If you know of any issues which are release-blocking list them in a reply here please. Maybe the

Re: [FFmpeg-devel] [PATCH v2 02/03] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length.

2021-12-13 Thread Marvin Scholz
On 13 Dec 2021, at 17:39, Romain Beauxis wrote: This is the second patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch fixes the concurrency model. Avfoundation runs its own producing thread to send produced frames and ffmpeg runs its

Re: [FFmpeg-devel] [PATCH v2 03/03] libavdevice/avfoundation.m: Allow to select devices by digest.

2021-12-13 Thread Romain Beauxis
> On Dec 13, 2021, at 12:25 PM, Marvin Scholz wrote: > > On 13 Dec 2021, at 17:40, Romain Beauxis wrote: > >> This is the third patch of a series of 3 that cleanup and enhance the >> avfoundation implementation for libavdevice. >> >> This patch adds a digest to avfoundation devices, when ava

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Soft Works
> -Original Message- > From: Jean-Baptiste Kempf > Sent: Monday, December 13, 2021 7:01 PM > To: FFmpeg development discussions and patches > Cc: Soft Works > Subject: Re: [FFmpeg-devel] Politics > > Hello Softworkz > > On Sun, 12 Dec 2021, at 07:15, Soft Works wrote: > > yesterday,

Re: [FFmpeg-devel] [PATCH v2 03/03] libavdevice/avfoundation.m: Allow to select devices by digest.

2021-12-13 Thread Marvin Scholz
On 13 Dec 2021, at 17:40, Romain Beauxis wrote: This is the third patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch adds a digest to avfoundation devices, when available. This is needed because device index can change while the machine i

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Jean-Baptiste Kempf
On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: > If you know of any major issues which need to be done before the release do > them > now. If you know of any issues which are release-blocking list them in a reply > here please. Maybe the audio channel layout would be nice to settle bef

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Jean-Baptiste Kempf
On Mon, 13 Dec 2021, at 16:25, Michael Niedermayer wrote: > previous unused suggestions where: > Von Neumann, Lorentz, Poincaré, Desitter, De Broglie, Gauss, Galois, > Viterbi, Darwin I'd love a "Lorentz" release :) -- Jean-Baptiste Kempf - President +33 672 704 734 ___

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Jean-Baptiste Kempf
On Sun, 12 Dec 2021, at 20:42, Nicolas George wrote: >> Now, if there's issues with his approach, that's fine. > > There are many issues, I pointed them at the time and was rudely > ignored. Would you care to share a link to that email? It's been difficult to follow to be honest, with the numer

Re: [FFmpeg-devel] Politics

2021-12-13 Thread Jean-Baptiste Kempf
Hello Softworkz On Sun, 12 Dec 2021, at 07:15, Soft Works wrote: > yesterday, it happened for the 4th and 5th times that another developer > called my patchset a “hack”. > > In none of the 4 cases, anybody was able to give even a single reason. > My assessment is that when a skilled developer is u

[FFmpeg-devel] [PATCH v20 02/20] avutil/frame: Prepare AVFrame\n for subtitle handling

2021-12-13 Thread Daniel Cantarín
the v23_plus set still fails: ./ffmpeg -ss 20 -i dvbsubtest.ts -filter_complex "[0:v][0:s]overlay[v]" -map '[v]' -map 0:a -acodec copy -vcodec mpeg4 -t 5 -bitexact /tmp/file.avi Input #0, mpegts, from 'dvbsubtest.ts': Duration: 00:00:34.64, start: 79677.098467, bitrate: 4842 kb/s Program

[FFmpeg-devel] [PATCH v2 03/03] libavdevice/avfoundation.m: Allow to select devices by digest.

2021-12-13 Thread Romain Beauxis
This is the third patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch adds a digest to avfoundation devices, when available. This is needed because device index can change while the machine is running when devices are plugged or unplugged and

[FFmpeg-devel] [PATCH v2 02/03] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length.

2021-12-13 Thread Romain Beauxis
This is the second patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch fixes the concurrency model. Avfoundation runs its own producing thread to send produced frames and ffmpeg runs its own thread to consume them. The existing implementati

[FFmpeg-devel] [PATCH v2 01/03] libavdevice/avfoundation.m: use AudioConvert, extend supported formats

2021-12-13 Thread Romain Beauxis
This is the first patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. This patch: * Implements support for AudioConverter * Switches to AudioConverter's API to convert unsupported PCM formats (non-interleaved, non-packed) to supported formats * Minimiz

Re: [FFmpeg-devel] 5.0 release

2021-12-13 Thread Michael Niedermayer
On Thu, Nov 25, 2021 at 06:19:32PM +0100, Michael Niedermayer wrote: > Hi all > > just wanted to remind everyone of the plan (suggested by jb) to make the > 5.0 release in december > i dont know if that will work out but please avoid introducing > risky changes until the release branch splits off

[FFmpeg-devel] [PATCH 14/24] ffmpeg: move closing the file into of_write_trailer()

2021-12-13 Thread Anton Khirnov
The current code postpones closing the files until after printing the final report, which accesses the output file size. Deal with this by storing the final file size before closing the file. --- fftools/ffmpeg.c | 13 - fftools/ffmpeg_mux.c | 15 ++- 2 files changed, 1

[FFmpeg-devel] [PATCH 13/24] ffmpeg: add a helper function to access output file size

2021-12-13 Thread Anton Khirnov
Stop accessing muxer internals from outside of ffmpeg_mux. --- fftools/ffmpeg.c | 6 +- fftools/ffmpeg.h | 1 + fftools/ffmpeg_mux.c | 6 ++ 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index c01ff2fa92..b7f26fe288 100644 --- a/

[FFmpeg-devel] [PATCH 23/24] ffmpeg: simplify the use of OutputStream.frame_number

2021-12-13 Thread Anton Khirnov
It features in limiting the number of output frames (-frames option) and currently can be incremented from two places: - for video encoding (not streamcopy), in do_video_out() after each successful avcodec_send_frame() call - for all other cases, in of_submit_packet() Not only is this inconsiste

[FFmpeg-devel] [PATCH 18/24] ffmpeg: move the mux queue into muxer private data

2021-12-13 Thread Anton Khirnov
The muxing queue currently lives in OutputStream, which is a very large struct storing the state for both encoding and muxing. The muxing queue is only used by the code in ffmpeg_mux, so it makes sense to restrict it to that file. This makes the first step towards reducing the scope of OutputStrea

[FFmpeg-devel] [PATCH 21/24] ffmpeg_mux: split of_write_packet()

2021-12-13 Thread Anton Khirnov
It is currently called from two places: - output_packet() in ffmpeg.c, which submits the newly available output packet to the muxer - from of_check_init() in ffmpeg_mux.c after the header has been written, to flush the muxing queue Some packets will thus be processed by this function twice, so

[FFmpeg-devel] [PATCH 16/24] ffmpeg: access output file chapters through a wrapper

2021-12-13 Thread Anton Khirnov
Avoid accessing the muxer context directly, as this will become forbidden in future commits. --- fftools/ffmpeg.c | 15 +-- fftools/ffmpeg.h | 2 ++ fftools/ffmpeg_mux.c | 7 +++ 3 files changed, 18 insertions(+), 6 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ff

[FFmpeg-devel] [PATCH 22/24] ffmpeg: move a comment to a more appropriate place

2021-12-13 Thread Anton Khirnov
--- fftools/ffmpeg.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index bbedf867b4..538601e440 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1064,6 +1064,11 @@ static void do_video_out(OutputFile *of, }

[FFmpeg-devel] [PATCH 12/24] ffmpeg: write the header for stream-less outputs when initializing the muxer

2021-12-13 Thread Anton Khirnov
There is no reason to delay this. --- fftools/ffmpeg.c | 11 --- fftools/ffmpeg_mux.c | 7 +++ 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index f76e5df8d2..c01ff2fa92 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -

[FFmpeg-devel] [PATCH 24/24] ffmpeg: move output file opts into private context

2021-12-13 Thread Anton Khirnov
It is private to the muxer, no reason to access it from outside. --- fftools/ffmpeg.h | 3 +-- fftools/ffmpeg_mux.c | 9 ++--- fftools/ffmpeg_opt.c | 12 ++-- 3 files changed, 13 insertions(+), 11 deletions(-) diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h index 76c8dfa4c8..69

[FFmpeg-devel] [PATCH 15/24] ffmpeg: refactor the code checking for bitexact output

2021-12-13 Thread Anton Khirnov
Figure out earlier whether the output stream/file should be bitexact and store this information in a flag in OutputFile/OutputStream. Stop accessing the muxer in set_encoder_id(), which will become forbidden in future commits. --- fftools/ffmpeg.c | 21 + fftools/ffmpeg.h

[FFmpeg-devel] [PATCH 20/24] ffmpeg_mux: split queuing packets into a separate function

2021-12-13 Thread Anton Khirnov
--- fftools/ffmpeg_mux.c | 72 +++- 1 file changed, 44 insertions(+), 28 deletions(-) diff --git a/fftools/ffmpeg_mux.c b/fftools/ffmpeg_mux.c index 9281e6c94c..d4b674c9e2 100644 --- a/fftools/ffmpeg_mux.c +++ b/fftools/ffmpeg_mux.c @@ -63,12 +63,50 @@ stat

[FFmpeg-devel] [PATCH 19/24] ffmpeg: fix initial muxing queue size

2021-12-13 Thread Anton Khirnov
It stores pointers to AVPacket, not AVPackets themselves. --- fftools/ffmpeg_mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg_mux.c b/fftools/ffmpeg_mux.c index 5a12a1c899..9281e6c94c 100644 --- a/fftools/ffmpeg_mux.c +++ b/fftools/ffmpeg_mux.c @@ -411,7 +41

[FFmpeg-devel] [PATCH 17/24] ffmpeg: do not log to the muxer context

2021-12-13 Thread Anton Khirnov
All other logging goes to NULL context. --- fftools/ffmpeg.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index d69e4119ef..afd442ff4e 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2954,7 +2954,6 @@ static void init_encoder_t

[FFmpeg-devel] [PATCH 10/24] ffmpeg: refactor limiting output file size with -fs

2021-12-13 Thread Anton Khirnov
Move the file size checking code to ffmpeg_mux. Stop calling avio_tell(), which is a potentially heavy operation - use the recently introduced bytes_written instead. --- fftools/ffmpeg.c | 4 +--- fftools/ffmpeg.h | 4 ++-- fftools/ffmpeg_mux.c | 14 +- fftools/ffmpeg_opt.c |

[FFmpeg-devel] [PATCH 09/24] ffmpeg_mux: add private muxer context

2021-12-13 Thread Anton Khirnov
Move header_written into it, which is not (and should not be) used by any code outside of ffmpeg_mux. In the future this context will contain more muxer-private state that should not be visible to other code. --- fftools/ffmpeg.h | 6 -- fftools/ffmpeg_mux.c | 26 ++--

  1   2   >