Re: [FFmpeg-devel] [PATCH] avcodec/videotoolboxenc: Fixes non-B-Frame encoding

2021-09-27 Thread Rick Kern
On Sun, Sep 26, 2021 at 8:48 AM NoHalfBits wrote: > Sets vtctx->has_b_frames to 0 if the VideoToolbox compression > session will not emit B-frames (and, in consequence, no valid > DTSs). Required for the handling of invalid DTSs in > 'vtenc_cm_to_avpacket' (line 2018ff) to work correctly and not

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolboxenc: fix pixel buffer memory leak

2021-11-09 Thread Rick Kern
On Tue, Nov 9, 2021 at 5:57 AM Steven Liu wrote: > <13102179...@163.com> 于2021年11月9日周二 下午6:52写道: > > > > From: songyutong > > > > In function vtenc_populate_extradata(), there is a manually created > > pixel buffer that has not been released. So we should use > CVPixelBufferRelease > > to releas

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add myself as a mediacodec and videotoolbox maintainer

2023-11-06 Thread Rick Kern
Philip Langdale, Carl Eugen Hoyos > - videotoolbox* Rick Kern, Aman Gupta > + videotoolbox* Rick Kern, Aman Gupta, Zhao Zhili Sounds good on the videotoolbox files. > > > libavdevice > -- > 2.25.1 > >

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix compilation for IOS < 11.0 and OSX, < 10.13

2019-05-30 Thread Rick Kern
On Fri, May 24, 2019 at 6:34 PM Thilo Borgmann wrote: > Am 22.05.19 um 14:27 schrieb Thilo Borgmann: > > Am 22.05.19 um 01:41 schrieb Jan Ekström: > >> Hi, > >> > >> On Tue, May 14, 2019 at 4:16 PM Thilo Borgmann > wrote: > >>> > >>> $Subject > >>> > >>> Tested compilation only, sanity test actu

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: add hdr10, linear, hlg color transfer function for videotoolboxenc

2019-07-16 Thread Rick Kern
On Wed, Jul 10, 2019 at 5:29 AM Dennis Mungai wrote: > On Wed, Jul 10, 2019, 11:06 Hendrik Leppkes wrote: > > > On Wed, Jul 10, 2019 at 4:23 AM Dennis Mungai wrote: > > > > > > On Wed, Jul 10, 2019, 03:05 Aman Gupta wrote: > > > > > > > On Wed, Jun 26, 2019 at 4:25 AM wrote: > > > > > > > > >

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: make transfer_fnc initialized for unsupport function

2019-08-11 Thread Rick Kern
On Sat, Aug 10, 2019 at 11:02 AM Limin Wang wrote: > > > ping the change! > Tests look good. Pushed. > > On Thu, Jul 18, 2019 at 06:59:35AM +0800, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > The current function will report one error message, but the caller func > > haven't chec

Re: [FFmpeg-devel] [PATCH] [PATCH] avcodec: Fix h264_videotoolbox writing b-frames (fix ticket #8353)

2020-03-21 Thread Rick Kern
On Wed, Mar 18, 2020 at 12:32 PM Nomis101 wrote: > Am 29.02.20 um 02:39 schrieb Nomis101: > > From: Damiano Galassi > > > > Signed-off-by: Nomis101 > > --- > > libavcodec/videotoolboxenc.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/

Re: [FFmpeg-devel] [PATCH] avcodec/videotoolboxenc: ignore encoded h264 SEI nalu by default

2021-02-15 Thread Rick Kern
On Mon, Feb 15, 2021 at 7:51 AM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > sharpbai: > > Before macOS 11, the encoded h264 sample contains fixed SEI before > > IDR frame, which the content is below > > > > 06 05 10 b9 ed b9 30 5d 21 4b 71 83 71 2c 10 a3 > > 14 bb 29 80 > > > > The

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Add support for HEVC with Alpha.

2021-03-19 Thread Rick Kern
On Tue, Mar 16, 2021 at 5:59 AM wrote: > Greetings, > > Thank you for your help in advance. > Unfortunately, my patch, which add support the "HEVC with alpha" profile > to the videotoolbox encoder, has not been reviewed for nearly three weeks. > Would it be possible to give me why nobody has revi

Re: [FFmpeg-devel] [PATCH] videotoolboxenc: enable constant quality with -q:v on Apple Silicon Macs and use b-frames für HEVC and H264 and b-pyramid for HEVC.

2021-03-19 Thread Rick Kern
On Fri, Jan 22, 2021 at 3:28 PM wrote: > From: Simone Karin Lehmann > > Signed-off-by: Simone Karin Lehmann > Applied > --- > libavcodec/videotoolboxenc.c | 54 > 1 file changed, 42 insertions(+), 12 deletions(-) > > diff --git a/libavcodec/videotoolboxen

Re: [FFmpeg-devel] [PATCH v1 1/2] avcodec/videotoolboxenc: add H264 Extended profile and level

2019-09-02 Thread Rick Kern
On Mon, Sep 2, 2019 at 9:53 AM Limin Wang wrote: > On Tue, Aug 27, 2019 at 10:56:42AM -0400, Richard Kern wrote: > > > I’ll look at it this weekend. > ping > > It shows error kVTParameterErr when using the software encoder: Error setting profile/level property: -12902 I used the following comma

Re: [FFmpeg-devel] [PATCH] [PATCH] avcodec: Add more kCVImageBufferColorPrimaries to videotoolboxenc

2019-11-17 Thread Rick Kern
On Sun, Nov 17, 2019 at 10:25 AM Nomis101 wrote: > Am 17.11.19 um 01:05 schrieb Richard Kern: > > > >> On Nov 16, 2019, at 7:03 PM, Nomis101 wrote: > >> > >> Thanks for reviewing. Would be nice, if somebody could push to master > then. I can't. > >> > >> > > I’ll test it out tomorrow and push.

[FFmpeg-devel] [PATCH] configure: Fix build on MacOS 10.15

2019-11-17 Thread Rick Kern
The default ./configure produces binaries that segfault on MacOS 10.15. Fixes #8073 using the work-around mentioned in the ticket. --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 31a3744dc6..6e371ae433 100755 --- a/configure +++ b/configure @@ -6878,6

[FFmpeg-devel] [PATCH v2] configure: Work around a start-up crash when building with clang.

2019-11-20 Thread Rick Kern
The current version of clang enables stack checking by default, causing a crash when binaries are run. Fixes #8073 using the -fno-stack-check work-around mentioned in the ticket. --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 8f4f2884cf..97e614f18a 1

Re: [FFmpeg-devel] [PATCH v2] configure: Work around a start-up crash when building with clang.

2019-11-21 Thread Rick Kern
On Wed, Nov 20, 2019 at 11:29 AM Hendrik Leppkes wrote: > On Wed, Nov 20, 2019 at 5:00 PM Timo Rothenpieler > wrote: > > > > On 20.11.2019 13:51, Rick Kern wrote: > > > The current version of clang enables stack checking by default, causing > > > a crash whe

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: set DataRateLimits for hevc

2021-04-25 Thread Rick Kern
On Sun, Apr 25, 2021 at 4:06 AM Zhao Zhili wrote: > From the comment it's not available on old version. It works now > by testing on macOS 11.2.1. There is no document about since when. > So trying to set the configuration and ignore the error for hevc. > --- > libavcodec/videotoolboxenc.c | 10

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: set DataRateLimits for hevc

2021-05-05 Thread Rick Kern
On Mon, Apr 26, 2021 at 3:24 AM "zhilizhao(赵志立)" wrote: > > > > On Apr 25, 2021, at 11:31 PM, Rick Kern wrote: > > > > On Sun, Apr 25, 2021 at 4:06 AM Zhao Zhili > wrote: > > > >> From the comment it's not available on old version.

Re: [FFmpeg-devel] [PATCH 42/46] avcodec/videotoolboxenc: Avoid copying data, allow user-supplied buffers

2021-05-05 Thread Rick Kern
On Thu, Apr 29, 2021 at 8:04 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Here the packet size is known before allocating the packet because > the encoder provides said information (and works with internal buffers > itself), so one use this information to avoid the implicit use

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc.c: Fix the trc support for iec61966-2-1 (sRGB)

2021-07-19 Thread Rick Kern
On Thu, Jul 15, 2021 at 12:47 AM Hao Guan wrote: > --- > libavcodec/videotoolboxenc.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c > index 4eaabed5d8..61eb6e1921 100644 > --- a/libavcodec/videotoolboxenc.c > +++ b/libavc

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add myself as videotoolbox* maintainer

2017-11-13 Thread Rick Kern
Ivo van Poorten vc2* Rostislav Pehlivanov vcr1.cMichael Niedermayer - videotoolboxenc.c Rick Kern + videotoolboxenc.c Rick Kern, Aman Gupta vima.c

[FFmpeg-devel] [PATCH v5] Added VideoToolbox H.264 encoder.

2016-02-15 Thread Rick Kern
Enable with configure --enable-vtenc and encode using -codec:v vtenc_h264. Signed-off-by: Rick Kern --- MAINTAINERS|1 + configure |3 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1269

[FFmpeg-devel] [PATCH v6] Added VideoToolbox H.264 encoder.

2016-02-17 Thread Rick Kern
Autodetected by default. Encode using -codec:v vtenc. Signed-off-by: Rick Kern --- MAINTAINERS|1 + configure |5 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1162

[FFmpeg-devel] [PATCH v7] VideoToolbox H.264 Encoder

2016-02-29 Thread Rick Kern
Autodetected by default. Encode using -codec:v vtenc. Signed-off-by: Rick Kern --- MAINTAINERS|1 + configure | 19 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1320

[FFmpeg-devel] [PATCH v8] VideoToolbox H.264 Encoder

2016-02-29 Thread Rick Kern
Autodetected by default. Encode using -codec:v vtenc. Signed-off-by: Rick Kern --- MAINTAINERS|1 + configure | 19 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1339

[FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-02-29 Thread Rick Kern
Replaced color_range with av_frame_get_color_range(). Rick Kern (1): VideoToolbox H.264 Encoder MAINTAINERS|1 + configure | 19 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1339

[FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-02-29 Thread Rick Kern
Autodetected by default. Encode using -codec:v vtenc. Signed-off-by: Rick Kern --- MAINTAINERS|1 + configure | 19 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c | 1339

[FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread Rick Kern
Uses the --disable-videotoolbox configure switch like the hwaccel, changed encoder filename, changed free() to av_free(). Rick Kern (1): VideoToolbox H.264 Encoder MAINTAINERS |1 + configure| 26 +- libavcodec/Makefile |1 + libavcodec

[FFmpeg-devel] [PATCH v10] VideoToolbox H.264 Encoder

2016-03-02 Thread Rick Kern
Autodetected by default. Encode using -codec:v h264_videotoolbox. Signed-off-by: Rick Kern --- MAINTAINERS |1 + configure| 26 +- libavcodec/Makefile |1 + libavcodec/allcodecs.c |1 + libavcodec/videotoolboxenc.c | 1339

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix crash when closing codec after error

2016-03-20 Thread Rick Kern
Fixes crash in #5352. VTCompressionSessionInvalidate() crashes if the internal encoder hasn't completed, but hasn't experienced an error. The function call isn't needed since the encoder is invalidated when the reference count reaches 0 anyway. Signed-off-by: Rick Kern

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Workaround encoder error

2016-03-21 Thread Rick Kern
Fixes bug #5352: Error when fetching parameter sets. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 38 -- 1 file changed, 24 insertions(+), 14 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 0791146

[FFmpeg-devel] [PATCH v2] lavc/videotoolboxenc: Workaround encoder error

2016-03-24 Thread Rick Kern
CMVideoFormatDescriptionGetH264ParameterSetAtIndex() fails on some hardware/OS versions when retrieving the parameter set count alone. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 43 --- 1 file changed, 32 insertions(+), 11 deletions

[FFmpeg-devel] [PATCH 02/13] lavc/videotoolboxenc: Handle hwaccel format as input

2016-04-09 Thread Rick Kern
Handle AV_PIX_FMT_VIDEOTOOLBOX. This results in better energy usage and faster encoding, especially on iOS. When the buffer comes from the media server, no memcpy's are needed. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file change

[FFmpeg-devel] [PATCH 01/13] lavc/videotoolboxenc: Use shared pixel buffer pool

2016-04-09 Thread Rick Kern
This reduces the chance of a memcpy in the media server. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 224 ++- 1 file changed, 180 insertions(+), 44 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c

[FFmpeg-devel] [PATCH 08/13] lavc/videotoolboxenc: Add realtime encoding property

2016-04-09 Thread Rick Kern
Hint to the encoder that encoding should be done in real-time, even at the expense of quality. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index a3f7e5e

[FFmpeg-devel] [PATCH 03/13] lavc/videotoolboxenc: Remove redundant code

2016-04-09 Thread Rick Kern
Don't require hardware encoding and explicitly fallback to software if it fails. Enabling it without requiring it will use hardware encoding if available. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 20 1 file changed, 20 deletions(-) diff --

[FFmpeg-devel] [PATCH 10/13] lavc/videotoolboxenc: add concatentation properties

2016-04-09 Thread Rick Kern
Add frames_before and frames_after as hints that there will be frames before or after the frames produced in this session. This may help with concatenation issues like bit rate spikes. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 30 ++ 1 file changed

[FFmpeg-devel] [PATCH 07/13] lavc/videotoolboxenc: Handle out-of-memory and fix memory leak

2016-04-09 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 196f00b..a3f7e5e 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c @@ -714,9 +714,12

[FFmpeg-devel] [PATCH 06/13] lavc/videotoolboxenc: Fix DTS

2016-04-09 Thread Rick Kern
Some devices output an invalid DTS when B-frames aren't used. Using PTS for the DTS in this case. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 83

[FFmpeg-devel] [PATCH 05/13] lavc/videotoolboxenc: Fix AVCodecContext.has_b_frames usage.

2016-04-09 Thread Rick Kern
Now set by the encoder, not used as an input parameter. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 6586ca7..8352ba9 100644 --- a

[FFmpeg-devel] [PATCH 12/13] lavc/videotoolboxenc: Support pixel aspect ratio

2016-04-09 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 54 1 file changed, 54 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 0a79ee0..72743b5 100644 --- a/libavcodec/videotoolboxenc.c +++ b

[FFmpeg-devel] [PATCH 00/13] videotoolbox encoder features and fixes

2016-04-09 Thread Rick Kern
Added several encoding parameters (ticket #5357), and fixed some bugs. Rick Kern (13): lavc/videotoolboxenc: Use shared pixel buffer pool lavc/videotoolboxenc: Handle hwaccel format as input lavc/videotoolboxenc: Remove redundant code lavc/videotoolboxenc: Add entropy setting lavc

[FFmpeg-devel] [PATCH 13/13] lavc/videotoolboxenc: Set colorimetry values

2016-04-09 Thread Rick Kern
Signed-off-by: Rick Kern --- configure| 3 + libavcodec/videotoolboxenc.c | 209 +++ 2 files changed, 212 insertions(+) diff --git a/configure b/configure index 94a66d8..10d4bab 100755 --- a/configure +++ b/configure @@ -5425,6

[FFmpeg-devel] [PATCH 04/13] lavc/videotoolboxenc: Add entropy setting

2016-04-09 Thread Rick Kern
Add an entropy setting to choose between CAVLC and CABAC. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 32 1 file changed, 32 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 2bcd3ef..6586ca7 100644

[FFmpeg-devel] [PATCH 09/13] lavc/videotoolboxenc: Add keyframes_only property

2016-04-09 Thread Rick Kern
Outputs only I-frames when set. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 5430479..c9a60b7 100644 --- a/libavcodec/videotoolboxenc.c +++ b

[FFmpeg-devel] [PATCH 11/13] lavc/videotoolboxenc: Support for forced I-frames

2016-04-09 Thread Rick Kern
Setting AVFrame.pic_type to AV_PICTURE_TYPE_I will force an I-frame. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index ff914f7

[FFmpeg-devel] [PATCH v2 03/12] lavc/videotoolboxenc: Require hardware encoding

2016-04-22 Thread Rick Kern
Software encoding can be allowed by setting allow_sw to 1. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index

[FFmpeg-devel] [PATCH v2 00/12] videotoolbox encoder features and fixes

2016-04-22 Thread Rick Kern
Sorry for the delay. The commit for the keyframes_only property was removed - this was already supported with -g 1. Rick Kern (12): lavc/videotoolboxenc: Use shared pixel buffer pool lavc/videotoolboxenc: Handle hwaccel format as input lavc/videotoolboxenc: Require hardware encoding lavc

[FFmpeg-devel] [PATCH v2 01/12] lavc/videotoolboxenc: Use shared pixel buffer pool

2016-04-22 Thread Rick Kern
This reduces the chance of a memcpy in the media server. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 224 ++- 1 file changed, 180 insertions(+), 44 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c

[FFmpeg-devel] [PATCH v2 02/12] lavc/videotoolboxenc: Handle hwaccel format as input

2016-04-22 Thread Rick Kern
Handle AV_PIX_FMT_VIDEOTOOLBOX. This results in better energy usage and faster encoding, especially on iOS. When the buffer comes from the media server, no memcpy's are needed. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file change

[FFmpeg-devel] [PATCH v2 04/12] lavc/videotoolboxenc: Add entropy setting

2016-04-22 Thread Rick Kern
Add an entropy setting to choose between CAVLC and CABAC. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index ec69bb1..07577b6 100644

[FFmpeg-devel] [PATCH v2 08/12] lavc/videotoolboxenc: Add realtime encoding property

2016-04-22 Thread Rick Kern
Hint to the encoder that encoding should be done in real-time, even at the expense of quality. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 073b87a

[FFmpeg-devel] [PATCH v2 09/12] lavc/videotoolboxenc: add concatentation properties

2016-04-22 Thread Rick Kern
Add frames_before and frames_after as hints that there will be frames before or after the frames produced in this session. This may help with concatenation issues like bit rate spikes. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 30 ++ 1 file changed

[FFmpeg-devel] [PATCH v2 05/12] lavc/videotoolboxenc: Fix AVCodecContext.has_b_frames usage.

2016-04-22 Thread Rick Kern
Now set by the encoder, not used as an input parameter. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 07577b6..6a9a2fe 100644 --- a

[FFmpeg-devel] [PATCH v2 10/12] lavc/videotoolboxenc: Support for forced I-frames

2016-04-22 Thread Rick Kern
Setting AVFrame.pic_type to AV_PICTURE_TYPE_I will force an I-frame. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index c630825

[FFmpeg-devel] [PATCH v2 07/12] lavc/videotoolboxenc: Handle out-of-memory and fix memory leak

2016-04-22 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 5594e69..073b87a 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c @@ -728,9 +728,12

[FFmpeg-devel] [PATCH v2 11/12] lavc/videotoolboxenc: Support pixel aspect ratio

2016-04-22 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 63 1 file changed, 63 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 5935d09..768405d 100644 --- a/libavcodec/videotoolboxenc.c +++ b

[FFmpeg-devel] [PATCH v2 12/12] lavc/videotoolboxenc: Set colorimetry values

2016-04-22 Thread Rick Kern
Signed-off-by: Rick Kern --- configure| 5 +- libavcodec/videotoolboxenc.c | 209 +++ 2 files changed, 213 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 97f374b..c484a10 100755 --- a/configure +++ b/configure

[FFmpeg-devel] [PATCH v2 06/12] lavc/videotoolboxenc: Fix DTS

2016-04-22 Thread Rick Kern
Some devices output an invalid DTS when B-frames aren't used. Using PTS for the DTS in this case. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 6a

[FFmpeg-devel] [PATCH v3 00/12] videotoolbox encoder features and fixes

2016-04-27 Thread Rick Kern
Updates from code review: Have allow_sw for both OS X and iOS Changed "entropy" option to "coder" to match libx264 Updated configure for CONFIG_VT_BT2020 macro Bool option usage Formatting Rick Kern (12): lavc/videotoolboxenc: Use shared pixel buffer pool lavc/videotoolbo

[FFmpeg-devel] [PATCH v3 01/12] lavc/videotoolboxenc: Use shared pixel buffer pool

2016-04-27 Thread Rick Kern
This reduces the chance of a memcpy in the media server. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 204 +-- 1 file changed, 160 insertions(+), 44 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c

[FFmpeg-devel] [PATCH v3 03/12] lavc/videotoolboxenc: Require hardware encoding

2016-04-27 Thread Rick Kern
Software encoding can be allowed by setting allow_sw to 1. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 38 +- 1 file changed, 17 insertions(+), 21 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index

[FFmpeg-devel] [PATCH v3 04/12] lavc/videotoolboxenc: Add entropy setting

2016-04-27 Thread Rick Kern
Add an entropy setting to choose between CAVLC and CABAC. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 33 + 1 file changed, 33 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 90b9a96..bd153e7 100644

[FFmpeg-devel] [PATCH v3 09/12] lavc/videotoolboxenc: add concatentation properties

2016-04-27 Thread Rick Kern
Add frames_before and frames_after as hints that there will be frames before or after the frames produced in this session. This may help with concatenation issues like bit rate spikes. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 31 +++ 1 file changed

[FFmpeg-devel] [PATCH v3 10/12] lavc/videotoolboxenc: Support for forced I-frames

2016-04-27 Thread Rick Kern
Setting AVFrame.pic_type to AV_PICTURE_TYPE_I will force an I-frame. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index e155383

[FFmpeg-devel] [PATCH v3 12/12] lavc/videotoolboxenc: Set colorimetry values

2016-04-27 Thread Rick Kern
Signed-off-by: Rick Kern --- configure| 5 +- libavcodec/videotoolboxenc.c | 209 +++ 2 files changed, 213 insertions(+), 1 deletion(-) diff --git a/configure b/configure index ef0b689..43d21fa 100755 --- a/configure +++ b/configure

[FFmpeg-devel] [PATCH v3 11/12] lavc/videotoolboxenc: Support pixel aspect ratio

2016-04-27 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 63 1 file changed, 63 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 5f02756..4a40f35 100644 --- a/libavcodec/videotoolboxenc.c +++ b

[FFmpeg-devel] [PATCH v3 07/12] lavc/videotoolboxenc: Handle out-of-memory and fix memory leak

2016-04-27 Thread Rick Kern
Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 72c7ba8..f53015a 100644 --- a/libavcodec/videotoolboxenc.c +++ b/libavcodec/videotoolboxenc.c @@ -706,9 +706,14

[FFmpeg-devel] [PATCH v3 02/12] lavc/videotoolboxenc: Handle hwaccel format as input

2016-04-27 Thread Rick Kern
Handle AV_PIX_FMT_VIDEOTOOLBOX. This results in better energy usage and faster encoding, especially on iOS. When the buffer comes from the media server, no memcpy's are needed. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 26 +- 1 file change

[FFmpeg-devel] [PATCH v3 06/12] lavc/videotoolboxenc: Fix DTS

2016-04-27 Thread Rick Kern
Some devices output an invalid DTS when B-frames aren't used. Using PTS for the DTS in this case. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 25

[FFmpeg-devel] [PATCH v3 08/12] lavc/videotoolboxenc: Add realtime encoding property

2016-04-27 Thread Rick Kern
Hint to the encoder that encoding should be done in real-time, even at the expense of quality. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index f53015a

[FFmpeg-devel] [PATCH v3 05/12] lavc/videotoolboxenc: Fix AVCodecContext.has_b_frames usage.

2016-04-27 Thread Rick Kern
Now set by the encoder, not used as an input parameter. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index bd153e7..25c9648 100644 --- a

[FFmpeg-devel] [PATCH] lavd/avfoundation: use AVCodecParameters

2016-05-05 Thread Rick Kern
Fixes "Could not find codec parameters for stream" error (#5494) Signed-off-by: Rick Kern --- libavdevice/avfoundation.m | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libavdevice/avfoundation.m b/libavdevice/avfoundation.m ind

[FFmpeg-devel] [PATCH] lavc/videotoolbox: fix H.264 hwaccel init issue

2016-06-01 Thread Rick Kern
Fixes VTDecompressionSessionCreate() error. Signed-off-by: Rick Kern --- libavcodec/videotoolbox.c | 59 --- 1 file changed, 45 insertions(+), 14 deletions(-) diff --git a/libavcodec/videotoolbox.c b/libavcodec/videotoolbox.c index 2f4d531..cadfb23

[FFmpeg-devel] [PATCH] lavc/audiotoolboxenc: fix dropped frames on iOS

2016-06-01 Thread Rick Kern
AudioConverterFillComplexBuffer() doesn't always call its callback. A frame queue is used to prevent skipped audio samples. Signed-off-by: Rick Kern --- libavcodec/audiotoolboxenc.c | 78 +--- 1 file changed, 52 insertions(+), 26 deletions(-) diff

[FFmpeg-devel] [PATCH 1/2] lavc/videotoolboxenc: fix allocated buffer size

2016-06-01 Thread Rick Kern
Allocates a buffer with padding, and with enough room for start codes when length codes are smaller. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 223 ++- 1 file changed, 94 insertions(+), 129 deletions(-) diff --git a/libavcodec

[FFmpeg-devel] [PATCH 2/2] lavc/videotoolboxenc: set extradata when opening codec

2016-06-01 Thread Rick Kern
VideoToolbox doesn't supply parameter sets until the first frame is done encoding. This spins up a temporary encoder and encodes a single frame to get this data. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 326 --- 1 file changed

[FFmpeg-devel] [PATCH] MAINTAINERS: Add myself for videotoolbox.c, remove vda* maintainer

2016-06-16 Thread Rick Kern
Person in MAINTAINERS hasn't responded to a patch on the ML or private email, and doesn't maintain the files according to git. Signed-off-by: Rick Kern --- MAINTAINERS | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 4fe999d..ef23

[FFmpeg-devel] [PATCH 0/2] DeckLink build fixes

2016-06-23 Thread Rick Kern
This fixes two build issues on OS X when --enable-decklink. Rick Kern (2): lavd/decklink: Fix compile issue on OS X lavd/decklink_common: Fix error caused by -Werror=missing-prototypes common.mak | 2 +- configure | 1 + libavdevice

[FFmpeg-devel] [PATCH 2/2] lavd/decklink_common: Fix error caused by -Werror=missing-prototypes

2016-06-23 Thread Rick Kern
decklink_common.cpp includes a .cpp file from the DeckLink API which fails to build because there are non-static functions in the included .cpp file. This temporarily disables the missing-prototypes error so the file can be included. Signed-off-by: Rick Kern --- libavdevice/decklink_common.cpp

[FFmpeg-devel] [PATCH 1/2] lavd/decklink: Fix compile issue on OS X

2016-06-23 Thread Rick Kern
Fixes #4124: Invalid argument '-std=c99' not allowed with 'C++/ObjC++' C++ files fail to compile. This adds '-std=c++11' to CXX_FLAGS to fix. Signed-off-by: Rick Kern --- common.mak | 2 +- configure | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff -

[FFmpeg-devel] [PATCH] lavd/avfoundation: Fix skewed video output

2016-06-24 Thread Rick Kern
Fixes #5654. The linesize can be greater than the minimum required. This copies the frame taking linesize into account. Signed-off-by: Rick Kern --- libavdevice/avfoundation.m | 56 -- 1 file changed, 49 insertions(+), 7 deletions(-) diff --git a

[FFmpeg-devel] [PATCH v2] lavd/decklink_common: Fix error caused by -Werror=missing-prototypes

2016-06-24 Thread Rick Kern
decklink_common.cpp includes a .cpp file from the DeckLink API which fails to build because there are non-static functions in the included .cpp file. This disables the missing-prototypes error so the file can be included. Signed-off-by: Rick Kern --- configure | 3 ++- 1 file changed, 2

[FFmpeg-devel] [PATCH] configure: use c++98 for c++ files

2016-06-26 Thread Rick Kern
Use c++98 standard instead of c++11. Signed-off-by: Rick Kern --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 19aea61..76d2d27 100755 --- a/configure +++ b/configure @@ -4529,7 +4529,7 @@ fi add_cppflags -D_ISOC99_SOURCE

[FFmpeg-devel] [PATCH] configure: remove -std= from c++ compilation

2016-06-26 Thread Rick Kern
d=c99 when the -std option is supported. Signed-off-by: Rick Kern --- common.mak | 2 +- configure | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/common.mak b/common.mak index 3f2096d..c4ab049 100644 --- a/common.mak +++ b/common.mak @@ -39,7 +39,7 @@ CCFLAGS = $(CPP

[FFmpeg-devel] [PATCH v3] lavd/decklink_common: Fix error caused by -Werror=missing-prototypes

2016-06-26 Thread Rick Kern
decklink_common.cpp includes a .cpp file from the DeckLink API which fails to build because there are non-static functions in the included .cpp file. This disables the missing-prototypes error so the file can be included. Signed-off-by: Rick Kern --- libavdevice/Makefile | 2 ++ 1 file changed

Re: [FFmpeg-devel] [PATCH] avcodec/videotoolboxenc: Add CBR option to H264 and HEVC encoder

2022-08-28 Thread Rick Kern
On Fri, Aug 26, 2022 at 7:42 AM Sebastian Beckmann < beckmann.sebast...@outlook.de> wrote: > Adds an option to use constant bitrate instead of average bitrate to the > videotoolbox encoders. This is enabled via -constant_bit_rate true. > macOS 13 is required for this option to work. > > Signed-off

Re: [FFmpeg-devel] [PATCH v3] avcodec/videotoolboxenc: Add CBR option to H264 and HEVC encoder

2022-08-29 Thread Rick Kern
On Sun, Aug 28, 2022 at 4:27 PM Sebastian Beckmann < beckmann.sebast...@outlook.de> wrote: > Adds an option to use constant bitrate instead of average bitrate to the > videotoolbox encoders. This is enabled via -constant_bit_rate true. > macOS 13 is required for this option to work. > > Signed-off

Re: [FFmpeg-devel] [PATCH v3] avcodec/videotoolboxenc: Add CBR option to H264 and HEVC encoder

2022-08-29 Thread Rick Kern
On Mon, Aug 29, 2022 at 9:05 AM Sebastian Beckmann < beckmann.sebast...@outlook.de> wrote: > > > The patch doesn't apply. Can you create a .patch file with git > format-patch > > and send the file as an attachment? Sometimes email clients/providers > > corrupt inline patches. > Thanks, pushed.

[FFmpeg-devel] [PATCH 0/2] lavc/videotoolbox: use objpool instead of allocating memory each frame

2022-11-19 Thread Rick Kern
objpool was only being used in fftools, but it's useful in other components to reduce alloation time and memory fragmentation. This patch set moves objpool to libavutil, so it can be used by lavc/videotoolboxenc for data being passed from input frames to output packets. Rick Ke

[FFmpeg-devel] [PATCH 1/2] fftools/objpool: move objpool to libavutil

2022-11-19 Thread Rick Kern
Moved objpool to libavutil to make it available outside of fftools. Signed-off-by: Rick Kern --- fftools/Makefile | 1 - fftools/ffmpeg_mux.c | 2 +- fftools/sync_queue.c | 2 +- fftools/thread_queue.c | 2 +- fftools/thread_queue.h

[FFmpeg-devel] [PATCH 2/2] lavc/videotoolboxenc: use objectpool for encoder output information

2022-11-19 Thread Rick Kern
Instead of allocating/freeing, use objpool for data passed from an input frame to an output packet. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 80 1 file changed, 71 insertions(+), 9 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b

[FFmpeg-devel] [PATCH v2 0/2] lavc/videotoolbox: use objpool instead of allocating memory each frame

2022-12-03 Thread Rick Kern
Added 'av' prefix to ObjPool and its functions. Rick Kern (2): fftools/objpool: move objpool to libavutil lavc/videotoolboxenc: use objectpool for encoder output information fftools/Makefile | 1 - fftools/ffmpeg_mux.c | 8 ++-- fftools/sy

[FFmpeg-devel] [PATCH v2 1/2] fftools/objpool: move objpool to libavutil

2022-12-03 Thread Rick Kern
Moved objpool to libavutil to make it available outside of fftools. Signed-off-by: Rick Kern --- fftools/Makefile | 1 - fftools/ffmpeg_mux.c | 8 fftools/sync_queue.c | 18 +- fftools/thread_queue.c | 14

[FFmpeg-devel] [PATCH v2 2/2] lavc/videotoolboxenc: use objectpool for encoder output information

2022-12-03 Thread Rick Kern
Instead of allocating/freeing, use objpool for data passed from an input frame to an output packet. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 80 1 file changed, 71 insertions(+), 9 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b

[FFmpeg-devel] [PATCH v3 0/2] lavc/videotoolbox: use objpool instead of allocating memory each frame

2022-12-03 Thread Rick Kern
Update inclusion guards in objpool.h start with AVUTIL_ instead of FFTOOLS_. Rick Kern (2): fftools/objpool: move objpool to libavutil lavc/videotoolboxenc: use objectpool for encoder output information fftools/Makefile | 1 - fftools/ffmpeg_mux.c | 8

[FFmpeg-devel] [PATCH v3 1/2] fftools/objpool: move objpool to libavutil

2022-12-03 Thread Rick Kern
Moved objpool to libavutil to make it available outside of fftools. Signed-off-by: Rick Kern --- fftools/Makefile | 1 - fftools/ffmpeg_mux.c | 8 fftools/sync_queue.c | 18 +- fftools/thread_queue.c | 14

[FFmpeg-devel] [PATCH v3 2/2] lavc/videotoolboxenc: use objectpool for encoder output information

2022-12-03 Thread Rick Kern
Instead of allocating/freeing, use objpool for data passed from an input frame to an output packet. Signed-off-by: Rick Kern --- libavcodec/videotoolboxenc.c | 80 1 file changed, 71 insertions(+), 9 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: validate vt context in the decoder callback

2022-12-04 Thread Rick Kern
On Tue, Nov 29, 2022 at 11:47 AM Alessandro Di Nepi < alessandro.din...@gmail.com> wrote: > Just to add that this fix, once approved, should be cherry-picked to all > the release branches where d7f4ad88a0df3c1339e142957bf2c40cd056b8ce has > been cherry-picked. > Basically, 4.4, 5.0, and 5.1. > > T

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: validate vt context in the decoder callback

2022-12-05 Thread Rick Kern
On Sun, Dec 4, 2022 at 12:51 PM Alessandro Di Nepi < alessandro.din...@gmail.com> wrote: > On 4 Dec 2022, 17:01 +0200, FFmpeg development discussions and patches < > ffmpeg-devel@ffmpeg.org>, wrote: > > When this happens, does it continue happening, or is it transient? My > main > > concern is log

[FFmpeg-devel] [PATCH] VideoToolbox H.264 Encoder

2015-11-13 Thread Rick Kern
This adds support for H.264 encoding using the VideoToolbox API. Signed-off-by: Rick Kern Rick Kern (1): Added VideoToolbox H.264 encoder. MAINTAINERS|1 + configure | 14 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/vtenc.c

  1   2   >