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 E5A24475E5 for ; Mon, 12 Feb 2024 00:03:35 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id B7CCA68D1A4; Mon, 12 Feb 2024 02:02:50 +0200 (EET) Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04olkn2045.outbound.protection.outlook.com [40.92.73.45]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7946168D150 for ; Mon, 12 Feb 2024 02:02:48 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WrSeP+/OcIeXhzkw2Mu1a5n7fGOQb10+KWbAOVyV80x5v46rgyJ2ut7fjBcvy5TsgqT/E+w5dWDAeGMvJb+Jm1V8x5brgvqRVkeZsVCh4MfpCQRZbArLVnzJVgrkZyTIOlt8HLfmnIiESXU04OkwSQSptaI8RYNGyVdOxLxOs8RTzu2/wk2jM1i+Um3kBqnKDqbpkOWgLs906MwgDsH7cD4KrUJyb0xq3E3mXvBEhxzHR+Bd9cZcniC9neEVelstemI7W3zPcbIIfuxL5J2+8jIFT0uAlJqcitcf33rY3D3nbVjKjk5jyz+1wvVmG5QB72wEHKLa1NUWZTZlISFvZw== 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=G4eSMTZKSlFXBZicJmWmg3eJh6TbJDGvAKDw3r1CIVQ=; b=EW7E2LFWL6Gyl2Cs5ayqYJUjC4kdF3Uf8o322NWINJX76FR41ln4T6GVsfuYveLb9HPsUqB2MrIi31oNHmz2sEKFOC31Ww3Z4X6gB9riG4+gBVaof9HnzOe0gVsDXF+q5OlThpt8uW2gQKRlwO2q3Vy9q+26DiX9hz+bmgjgUaCScKqLQxIm00UOqUoj5zkmXrDaLvhC0L0qiuY0xP/sLITIontV7f/M7sAbWKtpxpyzWqYuMjx4vJt3llPweUawl/6E24cgacoru7skblr50HgPJbZLVuGuVFNgyMO94QWxoTJu65fgVqfjfm0w4l/guwQC8clXk9DaEVuPJf0Uuw== 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=G4eSMTZKSlFXBZicJmWmg3eJh6TbJDGvAKDw3r1CIVQ=; b=fCpkTeFWzE/GDfMWHgPpQwxcmwNMv3cvUFt04T2W4jg3hsvtoNfl5jjKsWIJ6lym5Yw8Iz5nlU8kPkscYqOB7gHtvwTFZiUPXApD8KpblsWHoh34FNDdttkKiVqW/4qlf4i1ai4j95pN7D6mxSnpMY0t8T+eMB0p9msdnqP7hVyJhrd3Wj3laiw0ibYoyt3X3oHNAXPzlu4acNZp+4U1xq53Ghdokb7wpRsrBSwuIe9In8zHLO7LEJSvHT4aE0dW00ALW7RdT2Q4nM4iwHXD5RLi7nHNmQt161HRDJMLV7YzmT52YizPhzQLKwWr1NlxGZpb701uziI5bztrx8b84w== Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) by PR3P250MB0257.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:17c::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7270.35; Mon, 12 Feb 2024 00:02:46 +0000 Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::db07:87f6:7c75:38cc]) by DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::db07:87f6:7c75:38cc%3]) with mapi id 15.20.7270.036; Mon, 12 Feb 2024 00:02:46 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Mon, 12 Feb 2024 01:03:26 +0100 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [7x9U0U9FCOTB/JzbyuPZNG4Hm4wtGH9YpenpvA0Infg=] X-ClientProxiedBy: FR3P281CA0146.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:95::18) To DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) X-Microsoft-Original-Message-ID: <20240212000335.27984-13-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU0P250MB0747:EE_|PR3P250MB0257:EE_ X-MS-Office365-Filtering-Correlation-Id: 4d6a39a2-387f-4cd1-7a1c-08dc2b5df1ae X-MS-Exchange-SLBlob-MailProps: Cq7lScuPrnqwZHg0HfSFdl5Zlrnc8YAZMqDtUbVXlqasn6cfMx/BBt5oep8ZNiPVMHpY1yNbAfkVH9PMLikUi2QoeRpSSJKcuWRm8eJ+YL+MhFddrndNFQfFDIH0CoV2dRADxGFyUjouZqPOPTe1YaQY2CNGX8oDgqFxDD5v7aF17LjJMCEJanCtyAkACt28kn1rqvLqfhaUy9D2zBNzwubn8JpBfkggOndPDCIUyQ6cYsS8aznkLZx+PVz0mCkx3sLmoX4w813elknlISfHyz2sXg6EIW9HWxVw5s3AM54umIxkrhZ6U1Zaw8m8TsmPuPgVMT8OiCYymmuMuaFckNgcokbLbCJEPUO7bdhV7YO57G8LOg/EtHo9bnWFBEFg8x+V0dqQkHzNpNnv+ToclkUXMzQxe9LJrUtKz25pnd7Aa1PXBm14kMLy5IDKePN32FHlK/4t9+3BM5IW6CpuN7FHLGogd1/EkOURw+RBcWhVq7dov6is0qBchNXIl+RHKU3gIfnnIyca3kO0EpdqFVFd4CLHJLzr9jBMaKqRZDDHuVkCwSrvwIN9s4evQBSfkyxTo7N92lP59QA/J8A1ThhsrLgkYsJweD21VTIZ0+56O6dJyCId9usf51yN1C404/jgmwsRpwaabwFWS3CRHyirvXisn5XtABXYD+Zb2Ou+fHCenvL3rBSWEkjLu0PP7YuTqbXYTznom6u/t6Y/wphHC4HbXEIUIEUCr8Xu/A3rySzX7v8fjPwIRh7BkLOD9T2uGBxeIGM= X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: FAWnlw0e3LhkT2/JctukxvZWvt3ZX+RSN6yWZGXv7Gl8Qs6+yfmAuFu9p5q28lGcybKNS2DVDmQSwdITcxVSYN0DteKxXhsLov0OHfTboCOAHPVFuAj92nuGIxWDlJHhHG2AqG+Oipxk4NeqqROCSu3O5FF0zH3AKbyTTZAL4FpX+iP5kCSK0yT+nxDFsYf0uqrEkO6JEKN6a1qIpyjQTDSPhNsHBjn5NDmPHG9wCEvvELm/N4GqavhjMhUGs48KLaZsMIgWWhl0i7DIFciE26wietRQEO2ASnmyVcrUYxo/MVP3bwaa7JnImmz3z+0TkT6RzHfJ6C4SF+rFH8fBoqi+Qp+MUzSFm3KNgaWBi+aOC3ioVrUIEq0+/V59nudUCUNqPQu8NTL8gRE9j97UCdHxaBW8dGBWUtonJi4V26/PdYQQQD5KJGpqrD7Sp8wKYFc3Ikq7LzuINDiltNj8QBajjg3wVgRBpk95cZXLCfRvLBALrIA/cU2a5fTusgA/pUGNP87HPbZgw69SrgFG1i9SPlPS+QxtoAwgF8oRaG6hAP9C3bHcUJBy+gXKSSo1hnxprISwBDlBVSx4uQHnwfQ0xCXAzA/x+2fiaLOz0V0CID4azNJLbO2DzbrQB6ku X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xG/1BHY91UL/b9p8wok2LSmpMaWMtF6aqeDdYWRVDbqjcw8SU+IUFqMw731o?= =?us-ascii?Q?HaEfaK9UnRpVhePttHJm6KMsTw1C+wycEALDuUb22TL+IYVcWtIOEODH1RIS?= =?us-ascii?Q?IdHoJbfNNcYHCD2RMtEJmdxbJOPt5ZljGwKVZhj1CEykex+M8IT4F53lUwhS?= =?us-ascii?Q?3PGyjJgUT18ve8xWk0CNGkq2gPC2eADScSkdZdMrweq/MHOaq72h5pbHxnKd?= =?us-ascii?Q?43pZ7Psyg4/ML0+VZEZztafWcoG2FtSj4LRTwFGve1MDzccT1oziCRflwLw9?= =?us-ascii?Q?gZDHIGXqmo/iyBewJkayA4vvNuibX61EjyPkvJAAS4iFZYn4COS2YYTrXjW7?= =?us-ascii?Q?LPRrJsRCaZa/opm1hqnrOjMTjvGkphkFwQpZvNuweU4rhaGArB30dDnM089y?= =?us-ascii?Q?Q3dDK3Iarza7EdFN9ndLHDnBducushiHa+8y+sG4DZiXD9hmlmjBoLm2uqqb?= =?us-ascii?Q?AHsAsTbQdf45bLGsSpXg5g3aFe+uge4XnPBMmpjR/HY8UytnrmrcVmCsfGqL?= =?us-ascii?Q?/wHSj8ivHJHmfBtseQFRuL0LkoMkb+KsEPf51uUMZA09kKgqm3tSixBbIuFP?= =?us-ascii?Q?iV0FAtcquPcEQReXMR3+QMiqzO4zaqCNOKZwePRuQoYymSgbFwECjSFQ5o2g?= =?us-ascii?Q?cGl/VuHDngs8F0q9HdY/WUGlbjAtZRFsDBjebqVtCqKQR6bdf2o2bXS/hwja?= =?us-ascii?Q?RHqKbAj99De9Pt4OeDfuhkJmmz/anks9GQsBvxYwokW3jEj95xiEjZsACvvk?= =?us-ascii?Q?TNcUCTYhrQq4OwYBae0qPGt1PYGT54+EvcedNdtvQJ9SSKMm7W6o8RvA2myB?= =?us-ascii?Q?cdphIa4y03Lw/HZbyT1T49gIgD1/i7WvoFF8AGE2ZdgCswB0Ww9bIU/UYVgn?= =?us-ascii?Q?mgPDTrF70O/dqQ/otN6DHJxoi5CDq2sN/ZoiCrMgDs9ZukDV7BnuxfogW8n4?= =?us-ascii?Q?E0ckC5agqIumYmR64Z+f77YoPDhuMp+Y3fhDylMhvzufehz7Ax6P8G9e7Alg?= =?us-ascii?Q?ZVzr5cFf54IfY54snEOokGnfQxa4hpc9fFxEZyPE0RK2N0zyoU9Hw71NCrKc?= =?us-ascii?Q?cEwoo2Qt1RfF7O1UBDuxFnO3C9/alW382O/IZU7UjiPequUhgl5EGzAgTlLZ?= =?us-ascii?Q?pKerPTFo5fYUV0hQRV5VOvT0NOp9uTu8xPwPK0nBJxAO2FzXsAVG6OHpOIdc?= =?us-ascii?Q?klA74dn5Ci1GW6lkar8aj6OJHCn+q1nE4fMqbFIgRAmWon2zblCxZ44WG5/M?= =?us-ascii?Q?p+7RkQTfxxgcqdlWqjrU0ZlcNhvEMQjldcjse8jSeA=3D=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4d6a39a2-387f-4cd1-7a1c-08dc2b5df1ae X-MS-Exchange-CrossTenant-AuthSource: DU0P250MB0747.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Feb 2024 00:02:46.8003 (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: PR3P250MB0257 Subject: [FFmpeg-devel] [PATCH 14/23] avutil/hwcontext_opencl: Allocate pub and priv frames hwctx together 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 Cc: Andreas Rheinhardt 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: This is possible because the lifetime of both coincide. Besides reducing the number of allocations this also simplifies access to OpenCLFramesContext as one no longer has to go through AVHWFramesInternal. Signed-off-by: Andreas Rheinhardt --- libavutil/hwcontext_opencl.c | 38 ++++++++++++++++++++---------------- 1 file changed, 21 insertions(+), 17 deletions(-) diff --git a/libavutil/hwcontext_opencl.c b/libavutil/hwcontext_opencl.c index 54c050ab40..c5af1aec74 100644 --- a/libavutil/hwcontext_opencl.c +++ b/libavutil/hwcontext_opencl.c @@ -138,6 +138,11 @@ typedef struct OpenCLDeviceContext { } OpenCLDeviceContext; typedef struct OpenCLFramesContext { + /** + * The public AVOpenCLFramesContext. See hwcontext_opencl.h for it. + */ + AVOpenCLFramesContext p; + // Command queue used for transfer/mapping operations on this frames // context. If the user supplies one, this is a reference to it. // Otherwise, it is a reference to the default command queue for the @@ -1689,9 +1694,9 @@ fail: static int opencl_frames_init_command_queue(AVHWFramesContext *hwfc) { - AVOpenCLFramesContext *hwctx = hwfc->hwctx; + OpenCLFramesContext *priv = hwfc->hwctx; + AVOpenCLFramesContext *hwctx = &priv->p; OpenCLDeviceContext *devpriv = hwfc->device_ctx->hwctx; - OpenCLFramesContext *priv = hwfc->internal->priv; cl_int cle; priv->command_queue = hwctx->command_queue ? hwctx->command_queue @@ -1721,7 +1726,7 @@ static int opencl_frames_init(AVHWFramesContext *hwfc) static void opencl_frames_uninit(AVHWFramesContext *hwfc) { - OpenCLFramesContext *priv = hwfc->internal->priv; + OpenCLFramesContext *priv = hwfc->hwctx; cl_int cle; #if HAVE_OPENCL_DXVA2 || HAVE_OPENCL_D3D11 @@ -1815,7 +1820,7 @@ static int opencl_wait_events(AVHWFramesContext *hwfc, static int opencl_transfer_data_from(AVHWFramesContext *hwfc, AVFrame *dst, const AVFrame *src) { - OpenCLFramesContext *priv = hwfc->internal->priv; + OpenCLFramesContext *priv = hwfc->hwctx; cl_image_format image_format; cl_image_desc image_desc; cl_int cle; @@ -1870,7 +1875,7 @@ static int opencl_transfer_data_from(AVHWFramesContext *hwfc, static int opencl_transfer_data_to(AVHWFramesContext *hwfc, AVFrame *dst, const AVFrame *src) { - OpenCLFramesContext *priv = hwfc->internal->priv; + OpenCLFramesContext *priv = hwfc->hwctx; cl_image_format image_format; cl_image_desc image_desc; cl_int cle; @@ -1932,7 +1937,7 @@ typedef struct OpenCLMapping { static void opencl_unmap_frame(AVHWFramesContext *hwfc, HWMapDescriptor *hwmap) { - OpenCLFramesContext *priv = hwfc->internal->priv; + OpenCLFramesContext *priv = hwfc->hwctx; OpenCLMapping *map = hwmap->priv; cl_event events[AV_NUM_DATA_POINTERS]; int p, e; @@ -1961,7 +1966,7 @@ static void opencl_unmap_frame(AVHWFramesContext *hwfc, static int opencl_map_frame(AVHWFramesContext *hwfc, AVFrame *dst, const AVFrame *src, int flags) { - OpenCLFramesContext *priv = hwfc->internal->priv; + OpenCLFramesContext *priv = hwfc->hwctx; cl_map_flags map_flags; cl_image_format image_format; cl_image_desc image_desc; @@ -2217,7 +2222,7 @@ static void opencl_unmap_from_qsv(AVHWFramesContext *dst_fc, { AVOpenCLFrameDescriptor *desc = hwmap->priv; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; cl_event event; cl_int cle; int p; @@ -2253,7 +2258,7 @@ static int opencl_map_from_qsv(AVHWFramesContext *dst_fc, AVFrame *dst, (AVHWFramesContext*)src->hw_frames_ctx->data; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; AVOpenCLDeviceContext *dst_dev = &device_priv->p; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; AVOpenCLFrameDescriptor *desc; VASurfaceID va_surface; cl_mem_flags cl_flags; @@ -2351,7 +2356,7 @@ static void opencl_unmap_from_dxva2(AVHWFramesContext *dst_fc, { AVOpenCLFrameDescriptor *desc = hwmap->priv; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; cl_event event; cl_int cle; @@ -2376,7 +2381,7 @@ static int opencl_map_from_dxva2(AVHWFramesContext *dst_fc, AVFrame *dst, (AVHWFramesContext*)src->hw_frames_ctx->data; AVDXVA2FramesContext *src_hwctx = src_fc->hwctx; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; AVOpenCLFrameDescriptor *desc; cl_event event; cl_int cle; @@ -2439,7 +2444,7 @@ static int opencl_frames_derive_from_dxva2(AVHWFramesContext *dst_fc, AVDXVA2FramesContext *src_hwctx = src_fc->hwctx; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; AVOpenCLDeviceContext *dst_dev = &device_priv->p; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; cl_mem_flags cl_flags; cl_int cle; int err, i, p, nb_planes; @@ -2516,7 +2521,7 @@ static void opencl_unmap_from_d3d11(AVHWFramesContext *dst_fc, { AVOpenCLFrameDescriptor *desc = hwmap->priv; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; cl_event event; cl_int cle; @@ -2535,7 +2540,7 @@ static int opencl_map_from_d3d11(AVHWFramesContext *dst_fc, AVFrame *dst, const AVFrame *src, int flags) { OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; AVOpenCLFrameDescriptor *desc; cl_event event; cl_int cle; @@ -2595,7 +2600,7 @@ static int opencl_frames_derive_from_d3d11(AVHWFramesContext *dst_fc, AVD3D11VAFramesContext *src_hwctx = src_fc->hwctx; OpenCLDeviceContext *device_priv = dst_fc->device_ctx->hwctx; AVOpenCLDeviceContext *dst_dev = &device_priv->p; - OpenCLFramesContext *frames_priv = dst_fc->internal->priv; + OpenCLFramesContext *frames_priv = dst_fc->hwctx; cl_mem_flags cl_flags; cl_int cle; int err, i, p, nb_planes; @@ -2938,8 +2943,7 @@ const HWContextType ff_hwcontext_type_opencl = { .name = "OpenCL", .device_hwctx_size = sizeof(OpenCLDeviceContext), - .frames_hwctx_size = sizeof(AVOpenCLFramesContext), - .frames_priv_size = sizeof(OpenCLFramesContext), + .frames_hwctx_size = sizeof(OpenCLFramesContext), .device_create = &opencl_device_create, .device_derive = &opencl_device_derive, -- 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".