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

From: Zhao Zhili <quinkblack@foxmail.com>
To: ffmpeg-devel@ffmpeg.org
Cc: Rick Kern <kernrj@gmail.com>
Subject: Re: [FFmpeg-devel] [PATCH 0/2] lavc/videotoolbox: use objpool instead of allocating memory each frame
Date: Sun, 20 Nov 2022 22:47:32 +0800
Message-ID: <tencent_D8B8BA965D4D2BA04ECDFDCF93D510271C07@qq.com> (raw)
In-Reply-To: <20221119213516.45148-1-kernrj@gmail.com>

On Sat, 2022-11-19 at 16:35 -0500, Rick Kern wrote:
> 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 Kern (2):
>   fftools/objpool: move objpool to libavutil
>   lavc/videotoolboxenc: use objectpool for encoder output information
> 
>  fftools/Makefile                 |  1 -
>  fftools/ffmpeg_mux.c             |  2 +-
>  fftools/sync_queue.c             |  2 +-
>  fftools/thread_queue.c           |  2 +-
>  fftools/thread_queue.h           |  2 +-
>  libavcodec/videotoolboxenc.c     | 80 ++++++++++++++++++++++++++++
> ----
>  libavutil/Makefile               |  2 +
>  {fftools => libavutil}/objpool.c |  0
>  {fftools => libavutil}/objpool.h |  0
>  9 files changed, 77 insertions(+), 14 deletions(-)
>  rename {fftools => libavutil}/objpool.c (100%)
>  rename {fftools => libavutil}/objpool.h (100%)
> 

libavutil.v specifies that only symbols with av prefix can be visible
outside of libavutil. So it doesn't work without rename those
functions.

Meanwhile I don't know if it's appropriate to add these functions to
libavutil.

_______________________________________________
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".