Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH 00/42] New API for reference counting and ThreadFrames
@ 2023-09-19 19:38 Andreas Rheinhardt
  2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 01/42] tests/fate-run: Ensure that THREADS=random is actually random Andreas Rheinhardt
                   ` (48 more replies)
  0 siblings, 49 replies; 106+ messages in thread
From: Andreas Rheinhardt @ 2023-09-19 19:38 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

This patchset initially grew out of my dissatisfaction with the fact
that the AVBuffer API contains implicit allocations in av_buffer_ref()
that need to be checked. Later this combined with my dissatisfaction
with the current ThreadFrame API (which contains an implicit
av_frame_ref()) which forbids decoders to set any AVFrame field
of a frame that is used for syncing after having called
ff_thread_finish_setup(). This leads to a race in the vp9-encparams
test that is fixed in this patchset.

Andreas Rheinhardt (42):
  tests/fate-run: Ensure that THREADS=random is actually random
  avcodec/refstruct: Add simple API for refcounted objects
  avcodec/get_buffer: Use RefStruct API for FramePool
  avcodec/h264_ps: Use RefStruct API for SPS/PPS
  avcodec/hevc_ps: Use RefStruct API for parameter sets
  avcodec/vp8: Use RefStruct API for seg_map
  avcodec/wavpack: Use RefStruct API for DSD context
  avcodec/dovi_rpu: Use RefStruct API for Vdr data
  avcodec/refstruct: Allow checking for exclusive ownership
  avcodec/cbs: Use RefStruct-API for unit content
  avcodec/cbs_sei: Use RefStruct API for SEI messages
  avcodec/decode: Use RefStruct API for hwaccel_picture_private
  avcodec/vulkan_decode: Use RefStruct API for shared_ref
  avcodec/hevcdec: Use RefStruct API for RefPicListTap buffer
  avcodec/pthread_frame: Use RefStruct API for ThreadFrame.progress
  avcodec/nvdec: Use RefStruct API for decoder_ref
  avcodec/refstruct: Add RefStruct pool API
  avcodec/h264dec: Use RefStruct-pool API instead of AVBufferPool API
  avcodec/hevcdec: Use RefStruct-pool API instead of AVBufferPool API
  avcodec/nvdec: Use RefStruct-pool API for decoder pool
  avcodec/refstruct: Allow to always return zeroed pool entries
  avcodec/vp9: Use RefStruct-pool API for extradata
  avcodec/vaapi_encode: Use RefStruct pool API, stop abusing AVBuffer
    API
  avcodec/refstruct: Allow to share pools
  avcodec/vp9: Join extradata buffer pools
  avcodec/refstruct: Allow to use a dynamic opaque
  avcodec/pthread_frame: Add new progress API
  avcodec/mimic: Switch to ProgressFrames
  avcodec/vp3: Switch to ProgressFrames
  avcodec/vp9: Switch to ProgressFrames
  avcodec/vp9: Fix race when attaching side-data for show-existing frame
  avcodec/vp9: Reduce wait times
  avcodec/vp9: Simplify replacing VP9Frame
  avcodec/vp9: Replace atomic_store() by atomic_init()
  avcodec/threadprogress: Add new API for frame-threaded progress
  avcodec/wavpack: Use ThreadProgress API
  avcodec/vp8: Convert to ProgressFrame API
  avcodec/codec_internal: Remove FF_CODEC_CAP_ALLOCATE_PROGRESS
  avcodec/hevcdec: Move collocated_ref to HEVCContext
  avcodec/hevcdec: Switch to ProgressFrames
  avcodec/pngdec: Switch to ProgressFrames
  avcodec/ffv1dec: Switch to ProgressFrames

 doc/multithreading.txt                |   8 +-
 libavcodec/Makefile                   |   1 +
 libavcodec/av1dec.c                   |  64 +---
 libavcodec/av1dec.h                   |  14 +-
 libavcodec/avcodec.c                  |   4 +-
 libavcodec/cbs.c                      | 103 +++----
 libavcodec/cbs.h                      |  12 +-
 libavcodec/cbs_av1.c                  |  32 +-
 libavcodec/cbs_av1.h                  |   3 +-
 libavcodec/cbs_h264.h                 |   6 +-
 libavcodec/cbs_h2645.c                | 113 +++----
 libavcodec/cbs_h265.h                 |   9 +-
 libavcodec/cbs_h266.h                 |  11 +-
 libavcodec/cbs_h266_syntax_template.c |  10 +-
 libavcodec/cbs_internal.h             |   7 +-
 libavcodec/cbs_sei.c                  |  61 ++--
 libavcodec/cbs_sei.h                  |  20 +-
 libavcodec/cbs_sei_syntax_template.c  |   5 +
 libavcodec/codec_internal.h           |   7 +-
 libavcodec/decode.c                   | 146 +++++++--
 libavcodec/decode.h                   |  10 +-
 libavcodec/dovi_rpu.c                 |  54 ++--
 libavcodec/dovi_rpu.h                 |   4 +-
 libavcodec/dxva2_vp9.c                |   4 +-
 libavcodec/ffv1.c                     |   1 -
 libavcodec/ffv1.h                     |   4 +-
 libavcodec/ffv1dec.c                  |  82 +++---
 libavcodec/get_buffer.c               |  44 +--
 libavcodec/h264_mb.c                  |   4 +-
 libavcodec/h264_parser.c              |   9 +-
 libavcodec/h264_picture.c             |  91 ++----
 libavcodec/h264_ps.c                  |  57 ++--
 libavcodec/h264_ps.h                  |  13 +-
 libavcodec/h264_refs.c                |   2 +-
 libavcodec/h264_sei.c                 |   2 +-
 libavcodec/h264_slice.c               |  88 +++---
 libavcodec/h264dec.c                  |  21 +-
 libavcodec/h264dec.h                  |  25 +-
 libavcodec/hevc_filter.c              |   8 +-
 libavcodec/hevc_mvs.c                 |   8 +-
 libavcodec/hevc_parser.c              |   8 +-
 libavcodec/hevc_ps.c                  |  80 +++--
 libavcodec/hevc_ps.h                  |  10 +-
 libavcodec/hevc_refs.c                |  64 ++--
 libavcodec/hevc_sei.c                 |   5 +-
 libavcodec/hevcdec.c                  | 113 +++----
 libavcodec/hevcdec.h                  |  22 +-
 libavcodec/hwaccel_internal.h         |   3 +-
 libavcodec/internal.h                 |   4 +-
 libavcodec/mediacodecdec.c            |  10 +-
 libavcodec/mimic.c                    |  60 ++--
 libavcodec/mpeg4videodec.c            |   3 +-
 libavcodec/mpegpicture.c              |  18 +-
 libavcodec/mpegpicture.h              |   1 -
 libavcodec/nvdec.c                    |  97 +++---
 libavcodec/nvdec.h                    |  10 +-
 libavcodec/pngdec.c                   |  67 ++---
 libavcodec/progressframe.h            | 134 +++++++++
 libavcodec/progressframe_internal.h   |  32 ++
 libavcodec/pthread_frame.c            | 127 ++++++--
 libavcodec/refstruct.c                | 407 ++++++++++++++++++++++++++
 libavcodec/refstruct.h                | 360 +++++++++++++++++++++++
 libavcodec/rv30.c                     |   1 -
 libavcodec/rv40.c                     |   1 -
 libavcodec/tests/avcodec.c            |  10 +-
 libavcodec/threadframe.h              |   4 +-
 libavcodec/threadprogress.h           |  37 +++
 libavcodec/utils.c                    |  14 +-
 libavcodec/vaapi_encode.c             |  60 ++--
 libavcodec/vaapi_encode.h             |   5 +-
 libavcodec/videotoolbox.c             |   4 +-
 libavcodec/vp3.c                      | 148 ++++------
 libavcodec/vp8.c                      | 127 +++-----
 libavcodec/vp8.h                      |  11 +-
 libavcodec/vp9.c                      | 195 +++++-------
 libavcodec/vp9_mc_template.c          |   2 +-
 libavcodec/vp9block.c                 |   5 +-
 libavcodec/vp9dec.h                   |   7 +-
 libavcodec/vp9lpf.c                   |   1 +
 libavcodec/vp9mvs.c                   |   4 +-
 libavcodec/vp9recon.c                 |  19 +-
 libavcodec/vp9shared.h                |  14 +-
 libavcodec/vulkan_av1.c               |  11 +-
 libavcodec/vulkan_decode.c            |  49 ++--
 libavcodec/vulkan_decode.h            |   2 +-
 libavcodec/vulkan_h264.c              |  15 +-
 libavcodec/vulkan_hevc.c              |  27 +-
 libavcodec/wavpack.c                  | 125 ++++----
 libavcodec/webp.c                     |   3 +-
 tests/fate-run.sh                     |   2 +-
 90 files changed, 2165 insertions(+), 1540 deletions(-)
 create mode 100644 libavcodec/progressframe.h
 create mode 100644 libavcodec/progressframe_internal.h
 create mode 100644 libavcodec/refstruct.c
 create mode 100644 libavcodec/refstruct.h
 create mode 100644 libavcodec/threadprogress.h

-- 
2.34.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 106+ messages in thread

end of thread, other threads:[~2023-11-09  9:56 UTC | newest]

Thread overview: 106+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-19 19:38 [FFmpeg-devel] [PATCH 00/42] New API for reference counting and ThreadFrames Andreas Rheinhardt
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 01/42] tests/fate-run: Ensure that THREADS=random is actually random Andreas Rheinhardt
2023-09-25 20:01   ` Andreas Rheinhardt
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 02/42] avcodec/refstruct: Add simple API for refcounted objects Andreas Rheinhardt
2023-09-21 19:58   ` Nicolas George
2023-09-21 23:07     ` Andreas Rheinhardt
2023-10-06 18:24   ` Andreas Rheinhardt
2023-10-06 19:43     ` Nicolas George
2023-10-06 20:20       ` Andreas Rheinhardt
2023-10-06 20:37         ` Nicolas George
2023-10-06 20:50           ` Andreas Rheinhardt
2023-10-06 21:22             ` Nicolas George
2023-10-07 21:03   ` James Almer
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 03/42] avcodec/get_buffer: Use RefStruct API for FramePool Andreas Rheinhardt
2023-09-28 12:36   ` Anton Khirnov
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 04/42] avcodec/h264_ps: Use RefStruct API for SPS/PPS Andreas Rheinhardt
2023-09-28 13:03   ` Anton Khirnov
2023-09-28 15:49     ` Andreas Rheinhardt
2023-10-02  9:39       ` Anton Khirnov
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 05/42] avcodec/hevc_ps: Use RefStruct API for parameter sets Andreas Rheinhardt
2023-09-28 13:13   ` Anton Khirnov
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 06/42] avcodec/vp8: Use RefStruct API for seg_map Andreas Rheinhardt
2023-10-02  9:44   ` Anton Khirnov
2023-10-02 10:04     ` Andreas Rheinhardt
2023-10-02 10:14       ` Anton Khirnov
2023-09-19 19:56 ` [FFmpeg-devel] [PATCH 07/42] avcodec/wavpack: Use RefStruct API for DSD context Andreas Rheinhardt
2023-10-02  9:46   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 08/42] avcodec/dovi_rpu: Use RefStruct API for Vdr data Andreas Rheinhardt
2023-10-02  9:51   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 09/42] avcodec/refstruct: Allow checking for exclusive ownership Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 10/42] avcodec/cbs: Use RefStruct-API for unit content Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 11/42] avcodec/cbs_sei: Use RefStruct API for SEI messages Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 12/42] avcodec/decode: Use RefStruct API for hwaccel_picture_private Andreas Rheinhardt
2023-10-02 10:39   ` Anton Khirnov
2023-10-02 12:30   ` Lynne
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 13/42] avcodec/vulkan_decode: Use RefStruct API for shared_ref Andreas Rheinhardt
2023-10-02 12:31   ` Lynne
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 14/42] avcodec/hevcdec: Use RefStruct API for RefPicListTap buffer Andreas Rheinhardt
2023-10-02 10:47   ` Anton Khirnov
2023-10-02 11:07     ` Andreas Rheinhardt
2023-10-04  8:10       ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 15/42] avcodec/pthread_frame: Use RefStruct API for ThreadFrame.progress Andreas Rheinhardt
2023-10-02 11:01   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 16/42] avcodec/nvdec: Use RefStruct API for decoder_ref Andreas Rheinhardt
2023-10-02 10:58   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 17/42] avcodec/refstruct: Add RefStruct pool API Andreas Rheinhardt
2023-09-20 19:58   ` Michael Niedermayer
2023-09-21  0:28     ` Andreas Rheinhardt
2023-10-04  8:39   ` Anton Khirnov
2023-10-04 11:09     ` Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 18/42] avcodec/h264dec: Use RefStruct-pool API instead of AVBufferPool API Andreas Rheinhardt
2023-10-04 14:07   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 19/42] avcodec/hevcdec: " Andreas Rheinhardt
2023-10-04 14:12   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 20/42] avcodec/nvdec: Use RefStruct-pool API for decoder pool Andreas Rheinhardt
2023-10-04 14:28   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 21/42] avcodec/refstruct: Allow to always return zeroed pool entries Andreas Rheinhardt
2023-10-12 12:45   ` Anton Khirnov
2023-10-12 13:25     ` Andreas Rheinhardt
2023-10-12 13:56       ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 22/42] avcodec/vp9: Use RefStruct-pool API for extradata Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 23/42] avcodec/vaapi_encode: Use RefStruct pool API, stop abusing AVBuffer API Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 24/42] avcodec/refstruct: Allow to share pools Andreas Rheinhardt
2023-10-12 13:04   ` Anton Khirnov
2023-10-12 13:51     ` Andreas Rheinhardt
2023-10-12 14:04       ` Anton Khirnov
2023-10-12 14:10         ` Andreas Rheinhardt
2023-10-12 17:09         ` Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 25/42] avcodec/vp9: Join extradata buffer pools Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 26/42] avcodec/refstruct: Allow to use a dynamic opaque Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 27/42] avcodec/pthread_frame: Add new progress API Andreas Rheinhardt
2023-10-21 10:34   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 28/42] avcodec/mimic: Switch to ProgressFrames Andreas Rheinhardt
2023-10-21 10:38   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 29/42] avcodec/vp3: " Andreas Rheinhardt
2023-10-21 10:48   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 30/42] avcodec/vp9: " Andreas Rheinhardt
2023-10-21 11:04   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 31/42] avcodec/vp9: Fix race when attaching side-data for show-existing frame Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 32/42] avcodec/vp9: Reduce wait times Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 33/42] avcodec/vp9: Simplify replacing VP9Frame Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 34/42] avcodec/vp9: Replace atomic_store() by atomic_init() Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 35/42] avcodec/threadprogress: Add new API for frame-threaded progress Andreas Rheinhardt
2023-09-20 19:44   ` Michael Niedermayer
2023-09-21  0:28     ` Andreas Rheinhardt
2023-10-25 13:25   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 36/42] avcodec/wavpack: Use ThreadProgress API Andreas Rheinhardt
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 37/42] avcodec/vp8: Convert to ProgressFrame API Andreas Rheinhardt
2023-10-25 13:35   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 38/42] avcodec/codec_internal: Remove FF_CODEC_CAP_ALLOCATE_PROGRESS Andreas Rheinhardt
2023-10-25 13:38   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 39/42] avcodec/hevcdec: Move collocated_ref to HEVCContext Andreas Rheinhardt
2023-10-25 13:42   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 40/42] avcodec/hevcdec: Switch to ProgressFrames Andreas Rheinhardt
2023-11-09  9:50   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 41/42] avcodec/pngdec: " Andreas Rheinhardt
2023-11-09  9:52   ` Anton Khirnov
2023-09-19 19:57 ` [FFmpeg-devel] [PATCH 42/42] avcodec/ffv1dec: " Andreas Rheinhardt
2023-11-09  9:56   ` Anton Khirnov
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 43/49] avcodec/qsv: Use RefStruct API for memory id (mids) array Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 44/49] avcodec/rkmppdec: Fix double-free on error Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 45/49] avcodec/rkmppdec: Check av_buffer_ref() Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 46/49] avcodec/rkmppdec: Use RefStruct API for references to decoder itself Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 47/49] avcodec/rkmppdec: Allocate AVDRMFrameDescriptor and frame ctx jointly Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 48/49] avcodec/v4l2_m2m: Remove redundant av_frame_unref() Andreas Rheinhardt
2023-10-02 18:13 ` [FFmpeg-devel] [PATCH 49/49] avcodec/v4l2_(m2m|buffers): Use RefStruct API for context references Andreas Rheinhardt

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git