From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 307304356B for ; Thu, 16 Jun 2022 21:41:48 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 2049F68B815; Fri, 17 Jun 2022 00:41:46 +0300 (EEST) Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-oln040092075050.outbound.protection.outlook.com [40.92.75.50]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 0A50868B63D for ; Fri, 17 Jun 2022 00:41:39 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AnBfpEPSaDjdhUxQn2ZcF1VQbvlfs78UgWUUxz8QGeWCl867jFYRhv2JtmxxPi8H8yu6kaec10UD9bJn1uV5R5QSzHJ8B83GOQqJQIQA3p0+ayOeYsPAPhKByPM/VTAO8Zx/17jPhApCsSxsmZV6myJrs0GPS1kLM2aYHeOKB296DiMz++u5u3mOWf7x+nzaHQh0stNNs9U2RZ61rlYYLo4OTLoJWKfxIejSkQFqTu0UxfPf2MGGckNZtSWukoN//1L482P5wCPDm0EttrDCb+PDfL1gXQaJEZ98gYzvC+YJ7269Le6GZBAnApap/zFRxGnsISm4fLL+yjTscFLMxA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=2ut/KdElknhR6nWE93uEsQwFSiWdvig0sLlTqDt/MmQ=; b=J9YcDnB+A33faDWPw7bAOIcz/VYF5SUcxhRD100cAyNR/bvqH/6GuJI9P/lV/DGbQHYaaKwVh3kXfnDxamTgwzfWqncfElxEawOY3DNhnC7xi4zHw+c1SEGVEp8v8D/Ko1NIufIDO309VOf7WRrMHpG2CwkybF1j12dr24aVX02HJu5IAHuFQYH/9DoGaCjskumK6sH/GXrnaxJEeWhk4VJnp9yPPeddeJqMg7Boy8N518WfQeTE1ovUKb1FTed7vSeMkz2qR1NQKpEaRhL+91LgqmNavhez5QLHl+pyagmmzq5QrmgbqAQyQ7dS5e8DqjlctBBuKwy+32aTw8mYsg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2ut/KdElknhR6nWE93uEsQwFSiWdvig0sLlTqDt/MmQ=; b=OKg0eV6St+FH6hZ5vb6p6sjV+L1OlMLBbt9FnwfHJ4NVKJb98Z+2AdDZaTk3UhYpHc5islPZhso3lEzrND9OqcFkPuZE8zzukAe/I7yar+HTMs9nAqsx5QgyzxISpVGXfYMlT17FTsrPbetddYs4JMr7f2S/5IMkcFOT5xHO33WxwcR1MjFn5FveBSpTmifyJmKYHGfEXQMBey9hhWaROa6UNPvPsYrNrnc4lv42wjjd/mUFR0k4upODfJ3TjXoiokeU657CaEP0AFjZ1EIZc9MJ+sNPRzSolPzX6GSP832IOUIIuIzJgyvtSfrE55uTy/kRGCTJmO8t74W5P9xZLA== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by DB9PR01MB8887.eurprd01.prod.exchangelabs.com (2603:10a6:10:371::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5353.16; Thu, 16 Jun 2022 21:41:38 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c%10]) with mapi id 15.20.5332.023; Thu, 16 Jun 2022 21:41:38 +0000 Message-ID: Date: Thu, 16 Jun 2022 23:41:36 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220616195534.5278-1-anton@khirnov.net> <20220616195534.5278-21-anton@khirnov.net> From: Andreas Rheinhardt In-Reply-To: <20220616195534.5278-21-anton@khirnov.net> X-TMN: [cF5//scJs5lUwqOkCuU7CGXaqKOrWghc] X-ClientProxiedBy: ZR0P278CA0020.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:1c::7) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <77a62278-fb94-c5b4-44db-834d992d1a5e@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 9c296b48-e03d-4ef2-0f49-08da4fe0fe1c X-MS-TrafficTypeDiagnostic: DB9PR01MB8887:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 5yjU9hFA7+SIby56iiwCl3dXXd4V5u8zUPpxdLAuIxRzp7fzwQPQaosVKnLxU58yz3QcFoxN0FdSIL/baOmbMwhjNI7fgVsbTwQeLKYUez68dxJi4TCve1ujSHwZN+qKNZJOCrIPaWDWTawWNdGihyKcx6jFQe1BA17zgBkW7xWZVmV95Dy87tmhELSQrn3ncbCtkiFgdS4tCeEL/eUdxuMtKNVexOREUoQODvjsBWQ03aqLo9A2LOH3mylbjqEzQQiuL55M7tdTtMWRumGCFJ3/mF4CuWPY7P2WVIKgHrPUlHepdABRiecuMQX/QmPbMS+1rCBUKlvRjsxKMNUmgdnXCuaVhSthRCpzIhR+aon5tAu6891GenUJHtGjeYMmVZqkkKe57tyNIOkl5bMW9+dsOTl9rw6CD1xpL4Sg3QsQEZKatuenNZHyF9DTopVJwviCdwKZiYvMbyXYpsbF8LQTBfc3JQZopxAIFeUT2dsu8AIgiqbyIfrTsnu98KLYuacdzTY25Q9WfX/U2diTAajMmvkixpYtkVABe9+oy2bu7OMn1jj+4RauYCbWA3ts0CIBUs5mT8dAtjwcuc0V8Q== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aXl0SzNGR2ZMMHdPWUgxK0g4N0k1T2EyZWxmdllkSUZZNWlaTGtkTjZzSm5r?= =?utf-8?B?bktkdFM5WHJEaG91cGhzejBRdzR6SWJSM25KZzFUOFY1MWtuenF0dnFMNVB4?= =?utf-8?B?ajZ4d2NUcUk0LzEvbDZnbjBtUCtFQmpGbTRNZWw0NjV6NHVaV0g0dGNjVCtv?= =?utf-8?B?SW16WmtSRW1Kei9Eemx5SXdJeVh1dThWY01Bem1hTEI3KzA1Ym51c0NBYmla?= =?utf-8?B?S0EzcFQyajMvL1gxYjFJbWlDYVZhbjRxYVMvU3c3VFVxZ2NYZnA0eXd1dGpN?= =?utf-8?B?MmU0dTMvSjJITktRRU0wTy9xY0dNR2t4bkdBU3J2S1VEbEJidXFVZmJKV0hi?= =?utf-8?B?Z2w4c2ZOSXUzK0F4c2FNQ2kzQytOSExTanp0ditvQm1Sa1grbFhUVmY4RGxz?= =?utf-8?B?YTBNSXB2bC9Qc3o5UVpTWDJYd1FXeUUrclMwUVZkZkdkR3E3UnYycndMM0J0?= =?utf-8?B?RmlyakhjMWZLQTRrbEJlOEVqS3FkK2dQSEMrMUxPcUllRUowbEtKYkNrYU16?= =?utf-8?B?djhFQStEVCt3S3ZndEtyQ0w0MU8wRUFlamltZUQ3VThtcWFZSXlrRHZmMTNN?= =?utf-8?B?YlorTmxzd05tbE81eFNRUjVydjArWUcxSld1TDE2bm04UnF4QVU3ZUxqMWRM?= =?utf-8?B?UWpweFk4OERTY2JSdEl0czJmNjAvNDVUZ1FPWnQ5UmM3ZGJ6aHg3cjU2bWZ6?= =?utf-8?B?bjJsdU54V1djdncrcGVuamVoT1hpNXUyMW83S3MyOEtEYjlqL0E1SHEvdE9F?= =?utf-8?B?Q1A5QzlVZzdKQlVyNUw5UXpoRXJuQ3M2cE1tQmtpT1pCRWRLbkRQa0FaQ0V5?= =?utf-8?B?UDJLYmFndjVUb0U5SGZNQmVnZmFvT285QlY2L2g0U0RwUkMxNXJxVFdUYTk1?= =?utf-8?B?VXB3UWZrZ0RLb0xCWFNpMVRlcWs2aEh0MGpEU210ZEdDTDlJeVZSUE1JWms5?= =?utf-8?B?RjBnaUhpRXZ1b3ZnMmF2Rk5xSFZaZDFwMGpJYjMwc0dSaExtVFZpem5scGVo?= =?utf-8?B?dXVQOEZuS2NYNkhMenNKbUJoMVdSZzdYL1ltTnh2cDUyVk5ERkVBNXpkMVpr?= =?utf-8?B?clQ0TzhJMUJBdHpTN2Y5OUhBOS9JK01XUVpaRG14TFJSUG9uKytvcUtNeTY0?= =?utf-8?B?cmdkMzVmbmJucWk0eEIzQllkVW1lbi9RdlZ4M0IzWTkyb2l4eE1ESUxzVzl1?= =?utf-8?B?dWs2UGMvbzBzd1ZwSXh2dlVLSlMyTnBEQ3lkcHIyV25WaVE0UHcxMFh5Tk15?= =?utf-8?B?ZFZhWkc3L2pWdHhKSGhiQ0lBVGI4OGJaRXRxbFVKMWRENkREem9QQUlaaFBV?= =?utf-8?B?a3RhdFMvQmEzMFRpRDFCY2xiUFhFU0J2MGczMFBBbUlLR2lZR1IwMm5ZcFpW?= =?utf-8?B?ZFkxdWc2SzE3dlFrVDlnWDZjQ1p2ajBLa2E2RHV3SmZ2SDNxRkpuQVdsTVpZ?= =?utf-8?B?RXBWZmhKUFU3aFQ5MGVyMUo1aWt0K3E5Q3VGWW42Z2NuNG9GeXhBR3B4UlJt?= =?utf-8?B?K2tFejZCTW1jY2ZqQ0ZQQVo1dU0veFZLNUJwQXh5VWV2RFJqZFprNkJYTzB6?= =?utf-8?B?ZWJOMEtlSUJ6UUhDVHNISzJGejB2Sjd2U2Z3SDBLV25JTmNOWG1tSVZkZ1pP?= =?utf-8?B?a3FVUVdkRmFERWtzc3FPaWV4UmpOMW5Kc2MwUGt6NzJyRkk3V1hRcFJZQWt4?= =?utf-8?B?cEtVVWw5bHh3SlJ1Uk1FVE5RbGF0YzBYZWpPUEl3RUxZUThZWHVVYWFldEM3?= =?utf-8?B?MzVybG5BRXVNczdmVjdKVkx6OHptZTZOUmNsVGhvcVMvSHk0blNEZXlzeVo0?= =?utf-8?B?d2xqWkNPUUdUbjd4TjFTczhCVFNBYlB4eWpvdWV6L2dwYURsb3d0UmRHNERH?= =?utf-8?B?Tm9wQnZYSjlvSVIzSmhydEVKNURKa2tBNThlOVhBNjZtU0pIT3NZT0kyTWNZ?= =?utf-8?B?V1dIek1yYWdVRENoMFRjMXJLcW1mRXloY2RaeEsvY0RERmZZYWl4clVBcGxK?= =?utf-8?B?S2JjQ200QUtnPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9c296b48-e03d-4ef2-0f49-08da4fe0fe1c X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jun 2022 21:41:38.4460 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR01MB8887 Subject: Re: [FFmpeg-devel] [PATCH 21/35] fftools: add an object pool X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Anton Khirnov: > Allows to avoid constantly allocating and freeing objects like AVFrame > or AVPacket. > --- > fftools/objpool.c | 131 ++++++++++++++++++++++++++++++++++++++++++++++ > fftools/objpool.h | 37 +++++++++++++ > 2 files changed, 168 insertions(+) > create mode 100644 fftools/objpool.c > create mode 100644 fftools/objpool.h > > diff --git a/fftools/objpool.c b/fftools/objpool.c > new file mode 100644 > index 0000000000..b1561ecd69 > --- /dev/null > +++ b/fftools/objpool.c > @@ -0,0 +1,131 @@ > +/* > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * FFmpeg is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with FFmpeg; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA > + */ > + > +#include > + > +#include "libavcodec/packet.h" > + > +#include "libavutil/common.h" > +#include "libavutil/error.h" > +#include "libavutil/frame.h" > +#include "libavutil/mem.h" > + > +#include "objpool.h" > + > +struct ObjPool { > + void *pool[32]; > + unsigned int pool_count; > + > + ObjPoolCBAlloc alloc; > + ObjPoolCBReset reset; > + ObjPoolCBFree free; > +}; > + > +ObjPool *objpool_alloc(ObjPoolCBAlloc cb_alloc, ObjPoolCBReset cb_reset, > + ObjPoolCBFree cb_free) > +{ > + ObjPool *op = av_mallocz(sizeof(*op)); > + > + if (!op) > + return NULL; > + > + op->alloc = cb_alloc; > + op->reset = cb_reset; > + op->free = cb_free; > + > + return op; > +} > + > +void objpool_free(ObjPool **pop) > +{ > + ObjPool *op = *pop; > + > + if (!op) > + return; > + > + for (unsigned int i = 0; i < op->pool_count; i++) > + op->free(&op->pool[i]); > + > + av_freep(pop); > +} > + > +int objpool_get(ObjPool *op, void **obj) > +{ > + if (op->pool_count) { > + *obj = op->pool[--op->pool_count]; > + op->pool[op->pool_count] = NULL; > + } else > + *obj = op->alloc(); > + > + return *obj ? 0 : AVERROR(ENOMEM); > +} > + > +void objpool_release(ObjPool *op, void **obj) > +{ > + if (!*obj) > + return; > + > + op->reset(*obj); > + > + if (op->pool_count < FF_ARRAY_ELEMS(op->pool)) > + op->pool[op->pool_count++] = *obj; > + else > + op->free(obj); > + > + *obj = NULL; > +} > + > +static void *alloc_packet(void) > +{ > + return av_packet_alloc(); > +} > +static void *alloc_frame(void) > +{ > + return av_frame_alloc(); > +} > + > +static void reset_packet(void *obj) > +{ > + return av_packet_unref(obj); > +} > +static void reset_frame(void *obj) > +{ > + return av_frame_unref(obj); > +} > + > +static void free_packet(void **obj) > +{ > + AVPacket *pkt = *obj; > + av_packet_free(&pkt); > + *obj = NULL; > +} > +static void free_frame(void **obj) > +{ > + AVFrame *frame = *obj; > + av_frame_free(&frame); > + *obj = NULL; > +} > + > +ObjPool *objpool_alloc_packets(void) > +{ > + return objpool_alloc(alloc_packet, reset_packet, free_packet); > +} > +ObjPool *objpool_alloc_frames(void) > +{ > + return objpool_alloc(alloc_frame, reset_frame, free_frame); > +} > diff --git a/fftools/objpool.h b/fftools/objpool.h > new file mode 100644 > index 0000000000..1b2aea6aca > --- /dev/null > +++ b/fftools/objpool.h > @@ -0,0 +1,37 @@ > +/* > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * FFmpeg is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with FFmpeg; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA > + */ > + > +#ifndef FFTOOLS_OBJPOOL_H > +#define FFTOOLS_OBJPOOL_H > + > +typedef struct ObjPool ObjPool; > + > +typedef void* (*ObjPoolCBAlloc)(void); > +typedef void (*ObjPoolCBReset)(void *); > +typedef void (*ObjPoolCBFree)(void **); > + > +void objpool_free(ObjPool **op); > +ObjPool *objpool_alloc(ObjPoolCBAlloc cb_alloc, ObjPoolCBReset cb_reset, > + ObjPoolCBFree cb_free); > +ObjPool *objpool_alloc_packets(void); > +ObjPool *objpool_alloc_frames(void); > + > +int objpool_get(ObjPool *op, void **obj); > +void objpool_release(ObjPool *op, void **obj); > + > +#endif // FFTOOLS_OBJPOOL_H AVFifos are often used with non-POD elements that need custom init, reset (unref) and free callbacks (in addition to the move callbacks already supported). So why not add it to AVFifo? The only drawback to this that I see is that the pool could not be shared among multiple AVFifos, but apart from that it should support the usecases that you propose and do so in a way that avoids having to drain the fifos manually when freeing it. - Andreas _______________________________________________ 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".