[FFmpeg-devel] [PATCH] libavcodec/qsvenc: Change the parameter log to be thread safe

2022-07-08 Thread Wenbin Chen
Dividing one line log into several av_log() call is not thread safe. Now merge these strings into one av_log() call. Signed-off-by: Wenbin Chen --- libavcodec/qsvenc.c | 87 ++--- 1 file changed, 42 insertions(+), 45 deletions(-) diff --git a/libavcodec/q

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Jean-Baptiste Kempf
On Thu, 7 Jul 2022, at 22:47, Michael Niedermayer wrote: > previosuly suggested names i found in my very unorgnaized notes: > Mellin, Gauss, Galois, Viterbi, Darwin, Von Neumann, lorentz, poincaré, > desitter, de broglie Did we not use Lorentz for 5.0? I'm ok for re-using it, though. Else, I woul

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Paul B Mahol
On Fri, Jul 8, 2022 at 10:37 AM Jean-Baptiste Kempf wrote: > On Thu, 7 Jul 2022, at 22:47, Michael Niedermayer wrote: > > previosuly suggested names i found in my very unorgnaized notes: > > Mellin, Gauss, Galois, Viterbi, Darwin, Von Neumann, lorentz, poincaré, > > desitter, de broglie > > Did w

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Timo Rothenpieler
The lavfi changes, more specifically wrapped_avframe, still rely on sizeof(AVFrame) being used outside lavu. We really need to stop spreading its use and find a way to change this behavior. I have tried like two different solutions and both were rejected, so it's not going to be me. wrapped_av

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Timo Rothenpieler
On 08/07/2022 12:17, Timo Rothenpieler wrote: The lavfi changes, more specifically wrapped_avframe, still rely on sizeof(AVFrame) being used outside lavu. We really need to stop spreading its use and find a way to change this behavior. I have tried like two different solutions and both were reje

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread James Almer
On 7/8/2022 7:23 AM, Timo Rothenpieler wrote: On 08/07/2022 12:17, Timo Rothenpieler wrote: The lavfi changes, more specifically wrapped_avframe, still rely on sizeof(AVFrame) being used outside lavu. We really need to stop spreading its use and find a way to change this behavior. I have tri

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Michael Niedermayer
On Fri, Jul 08, 2022 at 10:36:54AM +0200, Jean-Baptiste Kempf wrote: > On Thu, 7 Jul 2022, at 22:47, Michael Niedermayer wrote: > > previosuly suggested names i found in my very unorgnaized notes: > > Mellin, Gauss, Galois, Viterbi, Darwin, Von Neumann, lorentz, poincaré, > > desitter, de broglie

Re: [FFmpeg-devel] FFmpeg 5.1

2022-07-08 Thread Leo Izen
On 7/8/22 09:52, Michael Niedermayer wrote: On Fri, Jul 08, 2022 at 10:36:54AM +0200, Jean-Baptiste Kempf wrote: On Thu, 7 Jul 2022, at 22:47, Michael Niedermayer wrote: previosuly suggested names i found in my very unorgnaized notes: Mellin, Gauss, Galois, Viterbi, Darwin, Von Neumann, lorentz

Re: [FFmpeg-devel] [PATCH 01/35] fftools/ffmpeg_mux: add private muxer context

2022-07-08 Thread Michael Niedermayer
On Fri, Jun 17, 2022 at 12:27:18PM +0200, Anton Khirnov wrote: > The current version of this set can also be found in my tree > git://git.khirnov.net/libav > branch ffmpeg_mt/mux There are really many files changing, its hard to say for sure that all are the same issue, but basically it all seems

[FFmpeg-devel] [PATCH] Allow exporting of RGB and BGR images to CUDA.

2022-07-08 Thread dpeeters
Use the step size when calculating the number of channels to allow for more than two channels per plane. This allows the use of AV_PIX_FMT_0BGR32 when using av_hwframe_transfer_data to transfer data from a Vulkan frame to a CUDA frame. Signed-off-by: David Peeters --- libavutil/hwcontext_vul

[FFmpeg-devel] [PATCH v2 2/8] avutil/hwcontext_d3d11va: fix mixed declaration and code

2022-07-08 Thread Timo Rothenpieler
--- libavutil/hwcontext_d3d11va.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index 904d14bbc8..e5afcb2a9d 100644 --- a/libavutil/hwcontext_d3d11va.c +++ b/libavutil/hwcontext_d3d11va.c @@ -397,6 +397,7 @@ stat

[FFmpeg-devel] [PATCH v2 3/8] avutil/hwcontext_d3d11va: fix texture_infos writes on non-fixed-size pools

2022-07-08 Thread Timo Rothenpieler
--- libavutil/hwcontext_d3d11va.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index e5afcb2a9d..6355bd1e29 100644 --- a/libavutil/hwcontext_d3d11va.c +++ b/libavutil/hwcontext_d3d11va.c @@ -166,6 +1

[FFmpeg-devel] [PATCH v2 4/8] avutil/hwcontext_d3d11va: update hwctx flags from input texture

2022-07-08 Thread Timo Rothenpieler
At least QSV relies on those being set correctly when deriving a hwctx. --- libavutil/hwcontext_d3d11va.c | 4 1 file changed, 4 insertions(+) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index 6355bd1e29..d492489b79 100644 --- a/libavutil/hwcontext_d3d11va.c ++

[FFmpeg-devel] [PATCH v2 1/8] fftools/ffmpeg: make debug_ts print raw filter output

2022-07-08 Thread Timo Rothenpieler
--- fftools/ffmpeg.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index e7384f052a..a69364c0d4 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -765,7 +765,9 @@ static double adjust_frame_pts_to_encoder_tb(OutputFile

[FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Timo Rothenpieler
Since a bunch small stuff has changed since the last time I sent these one by one, here's the whole collection again. I intend to push the whole lot within the next 48h, so they all make it in before 5.1 gets branched. Timo Rothenpieler (8): fftools/ffmpeg: make debug_ts print raw filter output

[FFmpeg-devel] [PATCH v2 6/8] avdevice/lavfi: output wrapped AVFrames

2022-07-08 Thread Timo Rothenpieler
This avoids an extra copy of potentially quite big video frames. Instead of copying the entire frames data into a rawvideo packet it packs the frame into a wrapped avframe packet and passes it through as-is. Unfortunately, wrapped avframes are set up to be video frames, so the audio frames continue

[FFmpeg-devel] [PATCH v2 5/8] avutil/hwcontext_d3d11va: add BGRA/RGBA10 formats support

2022-07-08 Thread Timo Rothenpieler
Desktop duplication outputs those --- libavutil/hwcontext_d3d11va.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index d492489b79..27c0c80413 100644 --- a/libavutil/hwcontext_d3d11va.c +++ b/libavutil/hwcontext_d3d11va.c @@ -86

[FFmpeg-devel] [PATCH v2 7/8] avdevice/lavfi: pass forward video framerate

2022-07-08 Thread Timo Rothenpieler
--- libavdevice/lavfi.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index 1b282a70cb..246f7dff3b 100644 --- a/libavdevice/lavfi.c +++ b/libavdevice/lavfi.c @@ -287,6 +287,7 @@ av_cold static int lavfi_read_header(AVFormatContext *avctx) f

[FFmpeg-devel] [PATCH v2 8/8] avfilter: add vsrc_ddagrab

2022-07-08 Thread Timo Rothenpieler
--- Changelog | 1 + configure | 7 + doc/filters.texi | 68 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/version.h | 2 +- libavfilter/vsrc_ddagrab.c

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Timo Rothenpieler > Sent: Saturday, July 9, 2022 12:54 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Timo Rothenpieler > Subject: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c > wrapped_avframe and dependent changes > >

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Soft Works > Sent: Saturday, July 9, 2022 1:46 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Timo Rothenpieler > Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, > lavfi.c wrapped_a

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Timo Rothenpieler
On 09.07.2022 01:46, Soft Works wrote: -Original Message- From: ffmpeg-devel On Behalf Of Timo Rothenpieler Sent: Saturday, July 9, 2022 12:54 AM To: ffmpeg-devel@ffmpeg.org Cc: Timo Rothenpieler Subject: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe an

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Timo Rothenpieler > Sent: Saturday, July 9, 2022 1:49 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, > lavfi.c wrapped_avframe and dependent changes > > On 09.07.2022 01:46, S

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Timo Rothenpieler
On 09.07.2022 02:01, Soft Works wrote: I can submit the missing bit of differences as a patch. I thought it was no longer needed. The requirements have also changed over MSDK versions. The 8bit file mapping that I've shown recently exists in fact because an earlier MSDK version was requesting suc

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Timo Rothenpieler > Sent: Saturday, July 9, 2022 2:15 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, > lavfi.c wrapped_avframe and dependent changes > > On 09.07.2022 02:01, S

Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

2022-07-08 Thread Timo Rothenpieler
On 09.07.2022 02:28, Soft Works wrote: -Original Message- From: ffmpeg-devel On Behalf Of Timo Rothenpieler Sent: Saturday, July 9, 2022 2:15 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes

[FFmpeg-devel] [PATCH] Remove unsafe bitwise OR on FFVulkanExtensions enum

2022-07-08 Thread Amir Mazzarella
FFVulkanExtensions enum does not have a value for 0 defined, and bitwise OR on enums is not safe. The function returns uint64_t, so it makes more sense and is safer to do arithmetic in terms of uint64_t Signed-off-by: Amir Mazzarella --- libavutil/vulkan_loader.h | 4 ++-- 1 file changed, 2 in

[FFmpeg-devel] [PATCH] Make implicit void pointer cast explicit

2022-07-08 Thread Amir Mazzarella
Signed-off-by: Amir Mazzarella --- libavutil/vulkan_loader.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/vulkan_loader.h b/libavutil/vulkan_loader.h index 3f1ee6aa46..fa8e5ed171 100644 --- a/libavutil/vulkan_loader.h +++ b/libavutil/vulkan_loader.h @@ -82,7 +82,7

Re: [FFmpeg-devel] [PATCH v2 8/8] avfilter: add vsrc_ddagrab

2022-07-08 Thread Gyan Doshi
On 2022-07-09 04:24 am, Timo Rothenpieler wrote: --- Changelog | 1 + configure | 7 + doc/filters.texi | 68 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/v

Re: [FFmpeg-devel] [PATCH] Add metadatareader filter.

2022-07-08 Thread Raymond Cheng
Thanks, it will probably take me a couple days to incorporate some of the feedback regarding style, but some comments: 1) The intent of metadatareader is to read what the metadata filter wrote to a file, which as you already know, does not make use of any established format. It simply calls vsn

Re: [FFmpeg-devel] [PATCH] Add metadatareader filter.

2022-07-08 Thread Raymond Cheng
I also feel, btw, that there is already precedent for this kind of implied relationship between f_metadata.c and f_metadatareader.c. For example, the demuxer/muxer pairs in libavformat, such as movenc.c and mov.c are complimentary, the latter is implicitly meant to read what the former has writ

[FFmpeg-devel] [PATCH] avcodec/pthread_frame: update the main avctx from the current, ThreadContext

2022-07-08 Thread Steve Lhomme
Patch attached in the email. In some cases, the submit packet can result in configurations changes of the hardware decoders. The previous HW context is then freed and a new one created. That context is supposed to move up to the main context and the other threads. It appears that when more t