From patchwork Mon Aug 15 16:31:21 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37289 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:3d0d:b0:8d:a68e:8a0e with SMTP id y13csp1374821pzi; Mon, 15 Aug 2022 09:31:46 -0700 (PDT) X-Google-Smtp-Source: AA6agR5zJodri30LCdyKt1KgUnKTg3SOyNQ3NWIXgsH6Cr/9vIBW5NdoUXsU5mvWBecxFxtGTvZ6 X-Received: by 2002:a17:907:12d7:b0:730:6d62:4ef7 with SMTP id vp23-20020a17090712d700b007306d624ef7mr10520073ejb.133.1660581106668; Mon, 15 Aug 2022 09:31:46 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id s27-20020a05640217db00b0043d800168absi7349819edy.346.2022.08.15.09.31.44; Mon, 15 Aug 2022 09:31:46 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b="RT9/xPJW"; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id ACF7A68B956; Mon, 15 Aug 2022 19:31:40 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR03-DBA-obe.outbound.protection.outlook.com (mail-dbaeur03olkn2075.outbound.protection.outlook.com [40.92.58.75]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id F27EC68B201 for ; Mon, 15 Aug 2022 19:31:33 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mLcsNFHZ41SRukocF2c9tdIHHGKtndrpaq41zhs7VHj2dVL6oCBEdn6B9rDMv9XD7K3y5cmmZosnJ+nB5/yvQw3qY+q1qwTJr1XPBrZP62m8YghP8H0w2zjTuyuyo2HX9u8Uc8Ousrxd2KQUpxQM7G2MjKkrsfWwJunNGjAIggHAu4ZKlcOjQx54EU2d8lnW8Ypqa1cD4LzL5dMAKUt5I8qzAoPv7Yez77XafpP4yviVXqPrx28I6MBTZCa6jqlFaeGGO5yhJ8xyl0zQkLkTfoDAhI1MfSGdxl2kj1LfNF1oFs/t88yk8VqsIBPjm4rysi8Hnxdt8/Dg+SKZhWBHig== 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=03eCDnHswZCQqy/+wOgJQxYhklP45aOuejfeH9Uv31Y=; b=nECmSE6HopqVSG4jSYg/q6LQcdSGm1gbdur5/+GdCNjc4oceTpKISiGWNqrdyxwj7zL3xHk8cOEWrH2oVVDEUyyo2KeNCtRYfysTZiCSUswGoYQRvAI474hSUwuygg9fmtp8T9Ps+Rh3tSzMlMDUo7/eAVvtxLzt7QoRpRjNCtsFcMSuF+/dqe6tA1RjpTv37Z56QFsSPRWQERH2wdkrhGoxP2tXrwRiBPIC7LQEJhyVXEYwdYuCQdrM3bH/l48/mn2pOWdae6q8BoO+EXqeW4sDwVu1h7NOVV6rNWTKbfTEnBytAp8Qi3FIZ3koTw130031AvSin9GWTTZexZDw7A== 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=03eCDnHswZCQqy/+wOgJQxYhklP45aOuejfeH9Uv31Y=; b=RT9/xPJWIJM6TTDi+M6SicR8W09YDcgujQ0L2psoeqcZmkW+sbILxE9evp+4HtKDxZbsz1Vmv+FqMlhuwAGtiIw2wKwpT9S/uQCbZZ17nGnJX4QcwLEvKrVbYmbvioIaYuRcZp4LxMzI7cGcxsq0hi1TM2J03ir4iC2DzD5rae1rjJMVROhOEJ9hPYkO73+ecL0euZutQFFiuJt1FbFS9Qfz/Cf6a52tFN7sUCIz12B+9xWH4EvspzCgLrugiHrvaDAAFzD2xVhKiVqqzPnTNz6sT3cDVz+b4Sde7fkKE2AVBmUFmyuQ59Ouk8yeVZveJFdGdIKA1XruZTwvcr9JJA== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by AM6PR01MB5894.eurprd01.prod.exchangelabs.com (2603:10a6:20b:e8::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5525.11; Mon, 15 Aug 2022 16:31:32 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46%11]) with mapi id 15.20.5504.027; Mon, 15 Aug 2022 16:31:31 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Mon, 15 Aug 2022 18:31:21 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 X-TMN: [p8NvQqfCq3NdU9eacig213pY5mJlv/Sp] X-ClientProxiedBy: ZR0P278CA0168.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:45::23) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <20220815163121.2881203-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 9d98bb99-7488-4cbb-fb0f-08da7edb9b8f X-MS-Exchange-SLBlob-MailProps: +LiGfBxqLEszruPA3NiRbbLMwB8eVehy4d4VxsU0GH5rNwepAHkkDxge7crIcv/VEeUTWGPGbV8Qt/I1Q8NA7//CxtkY3Ke87ltytzpsqqrwbqtfmcCUqzogNTyXLOL6BkVU5TQO/DSdsj4FOYA6M5K1b/KNZ/q1j8SsjCzyL/va4KFJZG7iOH4wPqGybNnREiNtQNVwh7uxqQ0/Ngpq7kQ6kpc1x1msHulj+BHuxbUwHuI6s883QDf0KZu7RbCHrjBV3KXyzGkhfGfBl9Omhdu61O3J/afmTM7MmSpKWxWRJfn6B66o64oLNpqIu2LZ1og6oBm8eQQMVRxhJkF2vKlMk3OICoWfkP8WjLTglg6wIdLTWpy7CGhQrgOUbhPaOcPr/rYLxsTGX0TcpBVKOD/0t7MhDOIvkBPrSdX4y7CtdfQV5jUFlijLFPC7SedVjU12HE7KkJPpiBFTUYyiLv4yhKdy2Sq+zqid3UocqkFDDStpZZngaIFH5WXq/hi2ZjebmXySpCYy26pSLu1ggW7wLQXwIkKnlMDkpoBWtjPHiugOS7TaFAi6D6d+QGdm94KynOBaC8ehV9xYnjh3YMtPjA2NMjBI4jsgCZRi0FIySGkEjidutUynhvyA+mme1VwIP+sav2KHTnPmJ4+ZRhlL0TXPl3Ty1yZKBhAQXcnGln2NU96Mk5b8mJEcp062jvsOMxFAmxs1i0wMVbNRNdwvphMLttCxdfCuOFoW0Emo1/kZVNO8qNAYGM9gja/LUJRy2Iev8GENQyzM/4X0v6LLt+UXDv/c X-MS-TrafficTypeDiagnostic: AM6PR01MB5894:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: lsRGYUDaMMIxkHaAf3CPvwIpIX4KcDK7T7DhnAQNWlxodzSp6TztcIRIqETXbKzrpwNF5ltIc+TbgJ6fJ0D3YWm/tvdXXnH7QbVNiYHyzEpwg0/XsQAUhYk9IrUfP8em18ScLUQjBKBF6BMYlU3WWNZ4XcWl2vhWcYjoPHnjicN38PZf1In6ttT5eg+zvUq/WtsOeneVENbaSXJpJWBQaxoSOwzSch7W6YPlvlWU+ZXWA+Gflt3lVM/JBfz5WEmDYGpZ1ftFOtT7JHn/2Q9WvPgAZQtjCCRs+6M9NnUOuf+4Jo8LW1ngUGBNXCiPTK6rL7b466R6J4l+UCP6JnspWi9ZF+SwKG0kp7qWpADidHbmblKyZkybwUm9I082XmEIiz9CcmQwrxcwvcESO3wyS0H5LB5TBl7MV1FI2S0O6en9p05yZcBmkvEkVjOB/oYFFsVaOqzGpU/2tROe50ftiTxTqSyA64acQa+sZoGUeaigNQ3284oOpcAIrdxNa2gLUIrcEpRnH7CRtRhY6yDZ/h2xAsjlsQtP2nvkKHvCoUhhCzxZyngM+FHXbfbsM/EF+iu0Y8Qj5s6GKrwJx0UlVZmVD+ZTjTuDWUHnbRhM9dN45z5wN5pdKXMXVF8mnFrtJU2GaTa1Cz/SmvtLTJdTdA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: LA036+XOQc+ILjbbBSO02Ibv+fJ4M6wObR4C4dVg9UcRD6zdmbqbwUNZ3maXKE+YmCpMNIxxv1zTTKUvmXMytVCW6/jydeihonx3hN8Nq7idD9sEpxUwjv+SMa2f0+7AgGuiFpyW4V85vE0vZcozcIrWLJ1QSuVupWhf3pHDOG7yhoRWUu1CdaalLFlaZGY9qGottd4j8AMlPtC0qjCG5YrrB4xy7PG7Qn2Ds4+Rj13RYjtg5YakHj630Fb/st8xqn2Dl7gT5u8Ak0QPHXnEvthrlX/d2890fYo26U10hfJ1RzP3JtGeDYoFUJOoZdjvyWHK9kXrtLUG3XQbgHeJTWwj3GUe3lLMfM2jZMDyPcZXIDxzpmuNNIIfIdhXIWGWpJZzZqs+8jdDIafahlXQkAH6txLiVj7qtSU4pdRFukIk1UMvHVk2ED19k1vCpcyGf7H8Gw0/6oLqBWMQUadkJXp4XZOnC1kX+TUKY347eNw4nLTrIRC852a/+OpKeFQvtwEFInGhrEIhUiT+HeIuBKpHPsWoaXMTGDYAeBu+3oLJWgXbPNM0TkwdYNoC7qDmlu7uRNPjRWuEKN/PXJ5wfjxqPWMjH/n+nm9W1lhD/TfxgATgKQ/tp3TRE1ZkBrIIZYkZ+Fy9HfvKMbSvlllclkLGt6niT0ABmRkTIMugIh0naeS4cVRMNkDb+D2JTQKyGJ8D1oxXf8pMQGqWO6DLp1yswLl3nvoxA6+ogSPE0Xh5JOzoT/vxOmJqeqCQmiA4pcerVf0xgFQzPhCVxfphmmRiawt0wh89IscEeH7dqPF2s5z7sxIfaV7hyVME4cqzxqNZC0JPzpo/P4ClhiZGsJK7znOvCBuO++4iiqoHffr76qbSlSZaKzkOPWJ1v/V2+FUULQXBm3SUdwEznvHw8dk8d8Jtl6uD/Zu432lNiAjIkgX2/Dm2LAI1kBieBGELJu7HBAHA8V7xJszWYA8jAI1OSG6myJFIIUzF2acQB6t+NydEktgWHypYhCLZmVfr9OXutCzZ5pjQjD5WzUtwflXGefdPqv7ZAd4qQXMtBcu57gV7l9SL4pyMlVZqvqOl/CMiTE7FWUpqxjSFbSvJfJfF8YfTBEk+PH87V/wy0C7NgfREzdWkMURNBbH7ZFYWx5LkVTpxmhSPOPPGaXa2KEEkoV5GOwOrQ4Zwf93KZvWOxksTZq0ESX1m+PdzXCCq4p552aQWk9ekyrvJQbKc/4xZ0QzXYmxVBAbGpGPUSzX88aZJ33+ZenjgMm+50p470CMMBu3oiJheB8xU017z9yJ3Q7u04HXHhfrSb1nZBJg= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9d98bb99-7488-4cbb-fb0f-08da7edb9b8f X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Aug 2022 16:31:31.2937 (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: AM6PR01MB5894 Subject: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_dec: Don't sync AVCodecContext fields manually 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: eEXQAPd32OXl They are already synced generically in update_context_from_thread() in pthread_frame.c. Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo_dec.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/libavcodec/mpegvideo_dec.c b/libavcodec/mpegvideo_dec.c index 7566fe69f9..fd706070b2 100644 --- a/libavcodec/mpegvideo_dec.c +++ b/libavcodec/mpegvideo_dec.c @@ -92,11 +92,6 @@ int ff_mpeg_update_thread_context(AVCodecContext *dst, return ret; } - s->avctx->coded_height = s1->avctx->coded_height; - s->avctx->coded_width = s1->avctx->coded_width; - s->avctx->width = s1->avctx->width; - s->avctx->height = s1->avctx->height; - s->quarter_sample = s1->quarter_sample; s->coded_picture_number = s1->coded_picture_number; From patchwork Wed Aug 17 13:29:18 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37327 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp84236pzh; Wed, 17 Aug 2022 06:29:38 -0700 (PDT) X-Google-Smtp-Source: AA6agR4KwKjD3EOVZjt3WP+aMDu6rU0hjbl7LB+jdXwfeNVqi+5xd9timz4z559DdiQITUP7aCoB X-Received: by 2002:a17:907:d8d:b0:730:a937:fb01 with SMTP id go13-20020a1709070d8d00b00730a937fb01mr16294601ejc.246.1660742978242; Wed, 17 Aug 2022 06:29:38 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id eb5-20020a0564020d0500b0043ab664c9e7si14547680edb.282.2022.08.17.06.29.37; Wed, 17 Aug 2022 06:29:38 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=r0ZP20PA; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A39E868B8E8; Wed, 17 Aug 2022 16:29:33 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2087.outbound.protection.outlook.com [40.92.90.87]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id F071668B7E6 for ; Wed, 17 Aug 2022 16:29:26 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jUl1TR8ONz5bZ4Cnjql53Oi2yMf3Amn0KW179Qn9HSXbB/GzuGhSNZTG2IXXYG7Q3YZwPTpSpH4G4nCnbtO0soBS+jnc+Os/k7eNGyjgJnDhiTMw4JBQ1uw3fx8zj3BfP5VkY1OREF7BwLRBlWiPQJFzWET4jlD/uYWDoXqd6D7hb3bAIJMnstaeJo9MSyJrg8qDP6Swgr7Ebg/MSrsaypvMmVxuoMPkgAjtj1D24iIXzCfWyrlq99xnmlYo4aiYrof+nvthwNk9v4qtVDgJ2CSo/L5V3Xaie6Bv8g/zPyEfpCXz2/QTgxIlMvdmGZGoJ+MQ3w9GQubWzkJNBmKRcA== 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=EildedH4JHMiGRJtG8zeWcptQ5kLs/ieAQkJcOUImvc=; b=L9tsAs6oK49MAYMK5Ypyyc+J3V+MjuUi0TPHX7UKNMOUBvoE8UKk/VkIWo2gyTRBeykmOu3jBEZjQo1DV7R9Z5ZBaMMvW/s0I0YWHvVrhegxkZG8VbyP6z8NZrsWL7uTkqIlewGakElenGdwd0nJ5G3jZMm8cxnRD+MUJ+VePnddZWJlHBzkioS3x21wciLQ9jtsnB/s0C5j2iCG9n+6xZyQehE8ghw8G20KiMU1aw7aQuNC5dyUYkTrQJWUp4UgoDgYh2Yl64wKJ0PQxRUOv86K+NMrTgJgi21ApzinfiXW1QrwbvTIt9rBA3PQp9ANyr9t8rA1oNtdkbn18NNgKg== 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=EildedH4JHMiGRJtG8zeWcptQ5kLs/ieAQkJcOUImvc=; b=r0ZP20PADcR0WJUaEN2/hPymLFUGC4BXupX2HkFR4o5QT+SzPBjyJtvWoTakPQg+vvBF9gV+LDzrEt184RnjITKdxfWQcG3y3ufYLKWeWxGSyrcnUgaTaT3J+9AXCPbEOXzd+c4iLQhIbnP0s4rjetWvjiKBJZSEk2ofNDpfMy6a17u1QtHXySwikJmSTWM2nvzlQgIiUpEZtAS6Vf9sbETbYuoeETiNZ/jwzZrLLak5C937AyNksJxZPMe7byr/0yQIWKBtA0dodBOTVXTnXl3Ef0hGbP68AS8EUIVoNpy1zTFx6pbIxjAwoMQwYYk+IxS1cLkSaNgSONZMQNe/cg== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by DU2PR01MB8509.eurprd01.prod.exchangelabs.com (2603:10a6:10:2f4::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5504.14; Wed, 17 Aug 2022 13:29:25 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46%11]) with mapi id 15.20.5504.028; Wed, 17 Aug 2022 13:29:25 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Wed, 17 Aug 2022 15:29:18 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [rCITnaRj+HzeiEOwcdN8zvwUukmr1skh] X-ClientProxiedBy: ZR0P278CA0010.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:16::20) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <20220817132918.3783073-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 56ea7aa7-e779-4a57-1f7f-08da8054808e X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiTISFcQEGF4mJSjl423AJ2My2EhLw+JwiU99LaNnbRYeTiMrEvDdL5BJhx01G27hgXstm3cbq2PanILLeH4jqp2yVXuAFT4CtfVVKDUOw8q9uEzPkON1+PIZtqR78w33bY1mtq964lKlVOxQP9hcS/36ETJqKnWp0KZYTvWilqzCjuMsV5UgnyjP5LRi0apA+c0GaZewPm2AtUlSrb3xWgTAi+303d/tuLiJjDP+PmwqUZPRkRPLsGlq7zZ69fzz2SmRJQ5gNOOIWQFqjKMVG56RaFxIQtEO1EievQtokub2LkWxFLhZvTFGXf0+RL0KH5tqoyxsI4ksfPyeGKKwNOiE+E4t6vLF4HF6PxmOODYPGby9liHiPEPtjcdF1HhVV8jaSaLtOhVWDBTl1cDYPifmCRTNgfrJ+H9FQa3/HqWyKEoV9RMzc+ym7CuPulPTGhqTSrkyYhIH7LhfHoq9lfqqLbtKyBmviQAJ+uIgYV0nkJXYozaJQ9GnXG10lFU3JoYwmKtfDE1XZvIhN8PLtQZ17tuELm5/V0wjfYry431R1sRTzSLyxvjyEAVIGNcorPV4Uhv0Z93Morz6uy29D0yMmRacFHKk4IbfGOSSUrrn+aWfqzwSrXkpbBeBBa9hYVE/R9Bumga0XediDQzQuIqP7eYyvApoDWAfS/IGXuI4RE+1x1kuOMrDPW7QYdwn3DElVVhpLcGDcA49KG0tkFt608piVlb3drJKR6pIaRowrwmauI+ahfImsAnYjYStO0= X-MS-TrafficTypeDiagnostic: DU2PR01MB8509:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: U5Q8RWUMYU0Et5dXsStk7Cy7IccM9EszAgVPNLvJ6hK7QKUxQ7EsrwSZkXz1BD1CRYQxOwdt1uvs3FunLUPMVWgxGx4L9xAPBWpPPXaql6BWd5o/Cb1H0zRJxIfHyynha+0QcSkau9HnZJQyznzqpEsPSfewoC5ujlLyIBIFsg9wjBQj2Qj1q1HeRSoxg7i0n9l+NV7DjFPFe9uE4fFjFHxhsLRp0GevNDuT6yLlbRBOqFoDa0xka2yzzEJCd6J+tNbmH6NYog3EwygYnPpL4ZawtyRcWFKirGsvHk0W3c/SDpnv6bDuIekw5ejwxGVr/tRa3EmabRPDxOik9lNENrux/hOXhMDrO1Ay2EBgFaWu136NDOmkg0CFia88I+h0u6UzGvV5RBzXk/MrQ69vnNn/uU3BmpIp/MQNm3N0628+1XU2gb/iHx9hav8repQ3A5G/Y9Qa7na6VK+UzEb6ZgSmEt5FFArbz4RDUDNRBAmacEJ2qSsGkKCqsh9MPsBg86cX7SPqzfVhJ9wnzzQNFILjnSRtU9a9ysOzAFnVkH20rkuR4mBMsfdMNbnMvVbturoQ6EU5W4TOUwYOhExNz8IKrdLvFm1hg96As61qfmp9DtIevwa8t7936v/5RkbryPPaXXQdPAptLx34FI1RgQ== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: DNB+EyMwOXQugPCva2u+bJ1+3/9eulpPQMVntv2qVh3hwfXxcnqgmUWkmRZuGj/SpSmxPEqN/ZtZG3zqPiDFbSMbTUT3XdMQGlcBDxvwv6uPhrfT3TAjNMysIpwlJwmjxbrIuvdYWyUCJaXAYBbRa81IG8wMPD8Y3XObfphFUSvk/FxPSPhsv935VfKOrpl93/FoqlGAonOFqy8H7fZBaFiGXVYAZPmZC2Jd4hiJZd3D8qjWCcWbNhMSPedt9QBpHaIZZImjFih4E2ZSpEkoMBwPTjM56kSS3enKtUj8+78egXdEkvJEzwsvLyOfaDALY3p580ositxZz4iif900y8UdA/llRJJb66OrupVRYonuDJZhjWhdpHPsFlMg8Y3VJvg7+hqfJyqQ2Wiy6utOKD1JedZ3Vp+XiWpMg++WvqsSQVo4NFuMLtSBOT/E8gdnGPD21CPCFaOrT8iUAtqm74/C9dnDAu3UfwFnCbfxxAVexCW+g/7HmExRldYXS4Thb4OP9HzKG0acttveMmxFhJ5SRyoSLOTSgq/p4fiWjbgPKeQBhi7wqCsa/+Ji3Mc2GMGucyLOahsB9xMnvCqtLKp+fwHfC+f7yhI2KjCIejj9c00GmMgXrIixuY+DlWEPt134xOHxbW7KGK8/KaiQKF3yH2KKcv4ReOKNDDUNBr/n1CmxbhYRb8onVO9CksRZQdBjJdpWj3aioivYCQG84kP5EGlSI5YlqxM0ueF3qpeL/YbYV1HK/jzIR0p8cc/TiBQevjBQIbEcyqdqy38oO0fJJfz5c81mYz7QA6nPAUxoVcGSKMR3R2Mtp+KIHWF6xaqEh66vZLkTZOo7zRhqZPgSWUKzNhu4u83t5fiD9PJgLE5KxmDq3q0gMZsSZrS9spGmmXVuIOpv7W6O8REaxn9Rv6uJJrcre2q+oyOFHsfZIzTQMCQSPcjUmjjoEVHwsUtQL40rQFRst0ggJ74lUuQ54E5t4TgfGKR4KWRwydoHidP/gdedBX7HFBQqxDXnIRjWv55Am9ndGgJewPWLaK8ur/c/mJKXiPgo64t4GDtgHOXUx1DSs8deaY5a6JEpOuudf3gtjTpevK47hOjnVB9H9x63b39NQC8EgEFUyzC+4BSsBQY8PaEEfEw3Umx8q3+eRF/IXRheWTu2m91L6VkNsq/OXebwfcrpH9AH99X5m3RPLkZ2bBMqFvTXXPmcofQ9lpDDXcoyuXTJoc9Bo263NhgaTSLwLQFrRGdZUjBFCoDyvsqIS9iZGMudXVP/hVrLFjUF2/PYS9BSqqvnIMv4YCT919MPWK+Hevw7VkI= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 56ea7aa7-e779-4a57-1f7f-08da8054808e X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Aug 2022 13:29:25.3752 (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: DU2PR01MB8509 Subject: [FFmpeg-devel] [PATCH 2/2] avcodec/mpeg4videodec: Keep data_partitioning in sync between threads 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: 5abuWzzRBCK7 Fixes frame-threaded decoding of samples created by concatting a video with data partitioning and a video not using it. (Only the MPEG-4 decoder sets this, so it is synced in mpeg4_update_thread_context() despite being a MpegEncContext-field.) Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg4videodec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index bfebc3806c..d89adf8d63 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -3534,6 +3534,7 @@ static int mpeg4_update_thread_context(AVCodecContext *dst, s->vol_sprite_usage = s1->vol_sprite_usage; s->sprite_brightness_change = s1->sprite_brightness_change; s->num_sprite_warping_points = s1->num_sprite_warping_points; + s->m.data_partitioning = s1->m.data_partitioning; s->rvlc = s1->rvlc; s->resync_marker = s1->resync_marker; s->t_frame = s1->t_frame; From patchwork Wed Aug 17 15:49:54 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37328 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp167466pzh; Wed, 17 Aug 2022 08:50:14 -0700 (PDT) X-Google-Smtp-Source: AA6agR7mhVx4NynQHW7oHidYPIayWLRIXNR4oj6TbofU2clvzY2ayEgx4irMNv/+2H8IzJPOkRdT X-Received: by 2002:a17:907:75ec:b0:738:4c20:4f73 with SMTP id jz12-20020a17090775ec00b007384c204f73mr10190932ejc.98.1660751414502; Wed, 17 Aug 2022 08:50:14 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id dr20-20020a170907721400b007308e7cd167si14363473ejc.846.2022.08.17.08.50.13; Wed, 17 Aug 2022 08:50:14 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=Ar5ILSsO; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 45C3768B8DE; Wed, 17 Aug 2022 18:50:10 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR02-HE1-obe.outbound.protection.outlook.com (mail-oln040092068031.outbound.protection.outlook.com [40.92.68.31]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 67DC868AE59 for ; Wed, 17 Aug 2022 18:50:03 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dyN5wuh80Qq/raa0Tx/CTEKVcJ4A7RoWCf4YpF7ZUkfWCYxpNNl29ciY1xLiAuqYwQnbVehIhWqhle4N5wpCIBPHeKrBWUzBe0htHFHs31d/TtddFqv2bZnvn0ptF/S5HJmTZjcdDHh97azT3QrTiNReOLabTenKlwRoreqtBI8rUMmsTo4unDL8is9EYpSwZrLrmbDQsk20CYXrZprCSezgSO/Gq3353DI8Z+oHuBvdRifWuyQNzm/Tg6FwEEATrIMfX8x1f/PBxtv6iTKURpb3L7BwOQc3MBfWtD7DxsodNDLZ+xL25g7S76G+JyKEjxOXtxbT+0T2sBgQc3aDvw== 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=JwENMYk7wWgtO35o3PAP7Zn6d+N7Dnq8bHHXdTjD6nM=; b=bnXQFhum3QmynitFW0R8r5dIsahqmTzqvVYv8a7rWyw6k2/HDmU7ImCJJsO79siA2ccrsYGX2+x9V1KKHe6tKgr6YV6huMR4atmqOOsW2HopZgmJDh8QuG5PS/VmzOFF/wam0vceIrv5QgNHQQmOmmAkmQTm8aj34Rqm3rTnb2qe5KdWUa+M1rnQgXo9doAg8wQwUiX3plzgC6W5cb9u5wwHS0wh99XxBVuM5NsWLExxcVjW6xAgZnMsl6R+0v1rNeSoxQiUKCLa31MFiIjSTtSOS3eiyPArwwHh5cDtX5yOtvmgeAQ3AqcB2mWAS9yDfm8KKzXUMmXvkT8m/BCd2w== 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=JwENMYk7wWgtO35o3PAP7Zn6d+N7Dnq8bHHXdTjD6nM=; b=Ar5ILSsO4D0sOurs8h2Cwd8eR0r0tXkGoFshOGr2FSBFrTNCFd2IfZA+tzO5lJqQutKWdnfKMKCheNOgOOWrGyaezbvTjNxKbXcSvrP381xZ/ITLYuz7TiFnJoP9B/voPkLAJz86rebCHOHw7qnAww/RnI1U+a+X5uMM9T/OrWaQK0f6h1YSbbAua8+kXr6smPt2wGFEhzqeX3S5wKYIWsDPDJv9sAUUQvxOKgZ5QBGP/DvkYp6uWda90xusFwN5xBJ1Bg63fVSRqFUm0tGtIwbmYkMuj0u3Od91PYdvOwkUUyouXZKFY5XqZJkVhsoMCtvDTSvr5lvtRLXlVJQbdQ== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by PA4PR01MB8923.eurprd01.prod.exchangelabs.com (2603:10a6:102:2a3::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5504.28; Wed, 17 Aug 2022 15:50:01 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46%11]) with mapi id 15.20.5504.028; Wed, 17 Aug 2022 15:50:01 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Wed, 17 Aug 2022 17:49:54 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [0Qz2mcwgkVpMeavf6v22g/LOhPBcwmz0] X-ClientProxiedBy: ZR0P278CA0120.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:20::17) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <20220817154954.4108071-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d4897be7-5e80-4af2-5420-08da806824c4 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiTISFcQEGF4mJSjl423AJ2MYIC1d7xVH/z+8IrPUWZuFePcYaLDCIqvLfnv9xG1Va/l6fZ763yduuMBbZnxNuUYNgj1wIcQEHGD6G+vb83qJqoxYJuJP72Guqwi7+st3U8uzFaTg1kcnJ3W7croJX1nnAsTbRNnfNPdign650NTXLDBpajsADxiXpiuFWta++ih57g3f4sQ3WbWBrZbDGqWFx6rUskGCmLoNvT/wmrKibDOdL3UTahVkoMunwhmO/bDAi4MnWM0RvhBtv71ux1zvBT+k7xSKBYT7l6mvQ4CEg82ciO1+xlhq+1aFRIPkkCasaBFE/QCLu0WO72sGmE80aAEGjkbpnWUqp81n4xRyAz0vGyeQWypaDbHWrn2azu14HJhMl7nnhQcRwIcMnCFSJaEiMEWs50eQRVyPi9bIyJqeDi1yU8kouP0H5VMas0SfDlQFIkWHe8+64OLk7ZIWucYk5rs4uFJy7Nxlq2vUIAtVBwQsFVRGcly806w3zRuS+GQx0ieVxXKUAFarY6X4X4CxxlvVOUlTwkvpPhOXTJWCZ6+ecXxRPaCjfK4QbjpoKRQRVt2K9b/XT3d6+AcFrzJ4OcB6V7QK+RwcjBNLqvEmTP8j1TMhEEB14U4IBpYj8rY/yMPCz55X5hTSFHnjykcjX7xTClUVxTlKZZjjCvRUAORBKMgQ27jYkk3lr93v2LLT8yUXx+uJihIPmhq4MTbwO/hodT/ovzZ6NKC0/zwYitm2f2kJT2qVppONdw= X-MS-TrafficTypeDiagnostic: PA4PR01MB8923:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: rN2oPbmN6s1q09yqNzrG8y6gyk2xUJCLa6OvDJaEBfr2/i48S/sk7bpPMj6OOsKZeYdcOiFqUyVg3ludH93AHSL+eJFvGsnQkkt7rHBmOYeQu6FFVxSxeLWSDpLumqgpb1c5kJ2U94yauNopz+40oFTG8CDDptVawUfQ4dMcpsEax+h2HDEHXYGBS9jagviDDsjA+JVpLSjlALoCyhXIC+5CUNoQFCAAe+FtyxzV0WtrBSRbt7l0Bacs0ruauAy3GiL8hm9Ky/skqCFBL8gUpwFs/fTjBhN5mzayS9b2bojE1rE4bAzzsiP91URHbVa0nxmI9Du0KdjyAHymV18ysMMUA48U97U7wCp6kMXIWHqStXxhvr9vNmgGkN9Xs9U9iDJ0xpmHz/lSLQXDHsHfq12DACDqduNEi0+fHZVWF/Ky3TugC2ejIBwX2n+jKcmPLdd5ThDv7u2q7eNKVBgIB67TObSf7i1WcAThLmUuRUcVmWW9umejur1kVUGswKgxyQn6qr7T/+85n9sqga+tsn/eGQ77HxUxUqWN3crHnDWGJOPq8A6WqUE88we92/oPY9p0wnsuchDW2R591Vk/XpTTYuXNMPDTOvp+I+HDOT+WT1jUQKtzPaiL5RK0vNhT+jX3O/PUo4uI4iSkb+WJ4w== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: DvEdN8fJgQWAJN4kC5JsY17W9GrSULn5qSZQk8L9VICJmbIpCypFZ5KYWERkBJiayWB4CwH/nrt2O+JLrjJaipd+5F84n8iXUsL4N6NJ1Z7CHDx2z4RptyLOHwjGsk1os1pObtdBsYwpqwYn+YXhBk0Z6hQlnXDmIQShOOjRhnagH7lLpgvlJx03GkgBN6835dm8yXNZDb2xE0aKAVXN6EIfyDtNNLAGI5GVntan/09y8kLhxUOkNPaTTyJ/gtHtdAxWh1mGIk2fCOjvcARdFYmBHa2iXa10LwuAodphmKFdlt4ULnO+SCze86oZSex+XNSkQHZBwfUr86IRHT9UKbN4PEORoo97Q3eahas9a3dHXM+Y7H3UYTwLmRC+X0+CUvdhkxgKe5FY5D0L5R/2ccI/mGs/sviQL1x0pEH00eyiKfV2EiOfHAgvwGFIjr7KOOnhl7eB3fiT4nRHZ2MxGMFt/5ocMLWdQgLg3lD5Nxyn+SUeUPzG4pvOqSpRqEtq3xkKaAXUxLzmRxvXU/MF6cOZ0X3FxI3JWdgZKCVZXpqm25taZF65Nph21UztOnT2ju+O24ET9lEkvFMcZXBMQL7o8zVTUhEodML3DV9Ck8pMtjRlPROsP0TiUUESO9wPAyweyKT/xYNVFRBBT4m/+dTYxAkMhL83Hp8YLbYdc8gBH4sAOvEH5INcoki+UH7XVD+U7vTZFqCx1TSPet/xp49GTN6xJ+1PtLP+U2/fAituoXMm9DQtTl+jDph6RSsoJE/6WX1f/u0fF8+3uZe5zQQjpCOuEbdzOi+qwYkE6fyREcnq/iaLlgmYe7gZxZqVePDwXFDVJJiIycKOvmpi+AcUtBovFmJYB7Jo492W4muFCFZcuPK5BSvcVayd3ozMzo46o96LRj4YUuZ9a0YbWTwWl3+QQ3SYaYobJmhwognDuh0RAv1VcI5cRaKE1sr956wGJ28PS0KqYxB5x0aB3xph6GawknaxSr3BuNr5rqt6JBrIG+VR59ExqjkBAjkDempamj+pqrOl6EAIXcWD7hCtdzmEdQdKLwzzuP2ci813J/0K5u1Aq4OId2ne+RYLnF4VtKxlRtiAfr0X3OlTViv7KMZXJcagIR3B3uaeIKWYD1MbwfnYM1pEyX7HKWUTUQ2z8EV3YlQTS1ZYVjJg7DPs4ZVbz3jmb9F+/H1x5p9EmKcYV/VGe1CZIRLJTXaLbSSWp1F/0aKZZCzOZUfXmT48PT+4vxBQFvsyX7+XQGBV8tnNm01q03pwvObqEKOEoGUJMco2cFTXTNASjPoGNT3T0F4PEFEnlsYReSFlzcQ= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: d4897be7-5e80-4af2-5420-08da806824c4 X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Aug 2022 15:50:01.1584 (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: PA4PR01MB8923 Subject: [FFmpeg-devel] [PATCH 3/3] avcodec/mpegvideo: Allocate encoder-only tables in mpegvideo_enc.c 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: bMAJGzmQZf9F This commit moves the encoder-only allocations of the tables owned solely by the main encoder context to mpegvideo_enc.c. This avoids checks in mpegvideo.c for whether we are dealing with an encoder; it also improves modularity (if encoders are disabled, this code will no longer be included in the binary). And it also avoids having to reset these pointers at the beginning of ff_mpv_common_init() (in case the dst context is uninitialized, ff_mpeg_update_thread_context() simply copies the src context into dst which therefore may contain pointers not owned by it, but this does not happen for encoders at all). Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegvideo.c | 118 +++---------------------------------- libavcodec/mpegvideo_enc.c | 67 ++++++++++++++++++++- 2 files changed, 72 insertions(+), 113 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 1190f29954..86fe5e4022 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -566,61 +566,13 @@ int ff_mpv_init_context_frame(MpegEncContext *s) s->mb_index2xy[s->mb_height * s->mb_width] = (s->mb_height - 1) * s->mb_stride + s->mb_width; // FIXME really needed? - if (s->encoding) { - /* Allocate MV tables */ - if (!FF_ALLOCZ_TYPED_ARRAY(s->p_mv_table_base, mv_table_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->b_forw_mv_table_base, mv_table_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->b_back_mv_table_base, mv_table_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->b_bidir_forw_mv_table_base, mv_table_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->b_bidir_back_mv_table_base, mv_table_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->b_direct_mv_table_base, mv_table_size)) - return AVERROR(ENOMEM); - s->p_mv_table = s->p_mv_table_base + s->mb_stride + 1; - s->b_forw_mv_table = s->b_forw_mv_table_base + s->mb_stride + 1; - s->b_back_mv_table = s->b_back_mv_table_base + s->mb_stride + 1; - s->b_bidir_forw_mv_table = s->b_bidir_forw_mv_table_base + s->mb_stride + 1; - s->b_bidir_back_mv_table = s->b_bidir_back_mv_table_base + s->mb_stride + 1; - s->b_direct_mv_table = s->b_direct_mv_table_base + s->mb_stride + 1; - - /* Allocate MB type table */ - if (!FF_ALLOCZ_TYPED_ARRAY(s->mb_type, mb_array_size) || - !FF_ALLOCZ_TYPED_ARRAY(s->lambda_table, mb_array_size) || - !FF_ALLOC_TYPED_ARRAY (s->cplx_tab, mb_array_size) || - !FF_ALLOC_TYPED_ARRAY (s->bits_tab, mb_array_size)) - return AVERROR(ENOMEM); - -#define ALLOCZ_ARRAYS(p, mult, numb) ((p) = av_calloc(numb, mult * sizeof(*(p)))) - if (s->codec_id == AV_CODEC_ID_MPEG4 || - (s->avctx->flags & AV_CODEC_FLAG_INTERLACED_ME)) { - int16_t (*tmp1)[2]; - uint8_t *tmp2; - if (!(tmp1 = ALLOCZ_ARRAYS(s->b_field_mv_table_base, 8, mv_table_size)) || - !(tmp2 = ALLOCZ_ARRAYS(s->b_field_select_table[0][0], 2 * 4, mv_table_size)) || - !ALLOCZ_ARRAYS(s->p_field_select_table[0], 2 * 2, mv_table_size)) - return AVERROR(ENOMEM); - - s->p_field_select_table[1] = s->p_field_select_table[0] + 2 * mv_table_size; - tmp1 += s->mb_stride + 1; - - for (int i = 0; i < 2; i++) { - for (int j = 0; j < 2; j++) { - for (int k = 0; k < 2; k++) { - s->b_field_mv_table[i][j][k] = tmp1; - tmp1 += mv_table_size; - } - s->b_field_select_table[i][j] = tmp2; - tmp2 += 2 * mv_table_size; - } - } - } - } - if (s->codec_id == AV_CODEC_ID_MPEG4 || (s->avctx->flags & AV_CODEC_FLAG_INTERLACED_ME)) { - int16_t (*tmp)[2]; /* interlaced direct mode decoding tables */ - if (!(tmp = ALLOCZ_ARRAYS(s->p_field_mv_table_base, 4, mv_table_size))) + int16_t (*tmp)[2] = av_calloc(mv_table_size, 4 * sizeof(*tmp)); + if (!tmp) return AVERROR(ENOMEM); + s->p_field_mv_table_base = tmp; tmp += s->mb_stride + 1; for (int i = 0; i < 2; i++) { for (int j = 0; j < 2; j++) { @@ -663,8 +615,6 @@ int ff_mpv_init_context_frame(MpegEncContext *s) static void clear_context(MpegEncContext *s) { - int i, j, k; - memset(&s->next_picture, 0, sizeof(s->next_picture)); memset(&s->last_picture, 0, sizeof(s->last_picture)); memset(&s->current_picture, 0, sizeof(s->current_picture)); @@ -693,31 +643,10 @@ static void clear_context(MpegEncContext *s) s->bitstream_buffer = NULL; s->allocated_bitstream_buffer_size = 0; s->picture = NULL; - s->mb_type = NULL; - s->p_mv_table_base = NULL; - s->b_forw_mv_table_base = NULL; - s->b_back_mv_table_base = NULL; - s->b_bidir_forw_mv_table_base = NULL; - s->b_bidir_back_mv_table_base = NULL; - s->b_direct_mv_table_base = NULL; - s->p_mv_table = NULL; - s->b_forw_mv_table = NULL; - s->b_back_mv_table = NULL; - s->b_bidir_forw_mv_table = NULL; - s->b_bidir_back_mv_table = NULL; - s->b_direct_mv_table = NULL; - s->b_field_mv_table_base = NULL; s->p_field_mv_table_base = NULL; - for (i = 0; i < 2; i++) { - for (j = 0; j < 2; j++) { - for (k = 0; k < 2; k++) { - s->b_field_mv_table[i][j][k] = NULL; - } - s->b_field_select_table[i][j] = NULL; + for (int i = 0; i < 2; i++) + for (int j = 0; j < 2; j++) s->p_field_mv_table[i][j] = NULL; - } - s->p_field_select_table[i] = NULL; - } s->dc_val_base = NULL; s->coded_block_base = NULL; @@ -730,10 +659,6 @@ static void clear_context(MpegEncContext *s) s->er.error_status_table = NULL; s->er.er_temp_buffer = NULL; s->mb_index2xy = NULL; - s->lambda_table = NULL; - - s->cplx_tab = NULL; - s->bits_tab = NULL; } /** @@ -824,37 +749,12 @@ av_cold int ff_mpv_common_init(MpegEncContext *s) void ff_mpv_free_context_frame(MpegEncContext *s) { - int i, j, k; - free_duplicate_contexts(s); - av_freep(&s->mb_type); - av_freep(&s->p_mv_table_base); - av_freep(&s->b_forw_mv_table_base); - av_freep(&s->b_back_mv_table_base); - av_freep(&s->b_bidir_forw_mv_table_base); - av_freep(&s->b_bidir_back_mv_table_base); - av_freep(&s->b_direct_mv_table_base); - s->p_mv_table = NULL; - s->b_forw_mv_table = NULL; - s->b_back_mv_table = NULL; - s->b_bidir_forw_mv_table = NULL; - s->b_bidir_back_mv_table = NULL; - s->b_direct_mv_table = NULL; - av_freep(&s->b_field_mv_table_base); - av_freep(&s->b_field_select_table[0][0]); av_freep(&s->p_field_mv_table_base); - av_freep(&s->p_field_select_table[0]); - for (i = 0; i < 2; i++) { - for (j = 0; j < 2; j++) { - for (k = 0; k < 2; k++) { - s->b_field_mv_table[i][j][k] = NULL; - } - s->b_field_select_table[i][j] = NULL; + for (int i = 0; i < 2; i++) + for (int j = 0; j < 2; j++) s->p_field_mv_table[i][j] = NULL; - } - s->p_field_select_table[i] = NULL; - } av_freep(&s->dc_val_base); av_freep(&s->coded_block_base); @@ -867,10 +767,6 @@ void ff_mpv_free_context_frame(MpegEncContext *s) av_freep(&s->er.error_status_table); av_freep(&s->er.er_temp_buffer); av_freep(&s->mb_index2xy); - av_freep(&s->lambda_table); - - av_freep(&s->cplx_tab); - av_freep(&s->bits_tab); s->linesize = s->uvlinesize = 0; } diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index 63fa0663d3..0b398c56ab 100644 --- a/libavcodec/mpegvideo_enc.c +++ b/libavcodec/mpegvideo_enc.c @@ -313,7 +313,7 @@ av_cold int ff_mpv_encode_init(AVCodecContext *avctx) MpegEncContext *s = avctx->priv_data; AVCPBProperties *cpb_props; int i, ret; - int mb_array_size; + int mb_array_size, mv_table_size; mpv_encode_defaults(s); @@ -824,12 +824,59 @@ av_cold int ff_mpv_encode_init(AVCodecContext *avctx) !FF_ALLOCZ_TYPED_ARRAY(s->reordered_input_picture, MAX_PICTURE_COUNT)) return AVERROR(ENOMEM); + /* Allocate MV tables; the MV and MB tables will be copied + * to slice contexts by ff_update_duplicate_context(). */ + mv_table_size = (s->mb_height + 2) * s->mb_stride + 1; + if (!FF_ALLOCZ_TYPED_ARRAY(s->p_mv_table_base, mv_table_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->b_forw_mv_table_base, mv_table_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->b_back_mv_table_base, mv_table_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->b_bidir_forw_mv_table_base, mv_table_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->b_bidir_back_mv_table_base, mv_table_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->b_direct_mv_table_base, mv_table_size)) + return AVERROR(ENOMEM); + s->p_mv_table = s->p_mv_table_base + s->mb_stride + 1; + s->b_forw_mv_table = s->b_forw_mv_table_base + s->mb_stride + 1; + s->b_back_mv_table = s->b_back_mv_table_base + s->mb_stride + 1; + s->b_bidir_forw_mv_table = s->b_bidir_forw_mv_table_base + s->mb_stride + 1; + s->b_bidir_back_mv_table = s->b_bidir_back_mv_table_base + s->mb_stride + 1; + s->b_direct_mv_table = s->b_direct_mv_table_base + s->mb_stride + 1; + + /* Allocate MB type table */ mb_array_size = s->mb_stride * s->mb_height; - if (!FF_ALLOCZ_TYPED_ARRAY(s->mc_mb_var, mb_array_size) || + if (!FF_ALLOCZ_TYPED_ARRAY(s->mb_type, mb_array_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->lambda_table, mb_array_size) || + !FF_ALLOC_TYPED_ARRAY (s->cplx_tab, mb_array_size) || + !FF_ALLOC_TYPED_ARRAY (s->bits_tab, mb_array_size) || + !FF_ALLOCZ_TYPED_ARRAY(s->mc_mb_var, mb_array_size) || !FF_ALLOCZ_TYPED_ARRAY(s->mb_var, mb_array_size) || !(s->mb_mean = av_mallocz(mb_array_size))) return AVERROR(ENOMEM); +#define ALLOCZ_ARRAYS(p, mult, numb) ((p) = av_calloc(numb, mult * sizeof(*(p)))) + if (s->codec_id == AV_CODEC_ID_MPEG4 || + (s->avctx->flags & AV_CODEC_FLAG_INTERLACED_ME)) { + int16_t (*tmp1)[2]; + uint8_t *tmp2; + if (!(tmp1 = ALLOCZ_ARRAYS(s->b_field_mv_table_base, 8, mv_table_size)) || + !(tmp2 = ALLOCZ_ARRAYS(s->b_field_select_table[0][0], 2 * 4, mv_table_size)) || + !ALLOCZ_ARRAYS(s->p_field_select_table[0], 2 * 2, mv_table_size)) + return AVERROR(ENOMEM); + + s->p_field_select_table[1] = s->p_field_select_table[0] + 2 * mv_table_size; + tmp1 += s->mb_stride + 1; + + for (int i = 0; i < 2; i++) { + for (int j = 0; j < 2; j++) { + for (int k = 0; k < 2; k++) { + s->b_field_mv_table[i][j][k] = tmp1; + tmp1 += mv_table_size; + } + s->b_field_select_table[i][j] = tmp2; + tmp2 += 2 * mv_table_size; + } + } + } + if (s->noise_reduction) { if (!FF_ALLOCZ_TYPED_ARRAY(s->dct_offset, 2)) return AVERROR(ENOMEM); @@ -945,6 +992,22 @@ av_cold int ff_mpv_encode_end(AVCodecContext *avctx) av_freep(&avctx->stats_out); + av_freep(&s->p_mv_table_base); + av_freep(&s->b_forw_mv_table_base); + av_freep(&s->b_back_mv_table_base); + av_freep(&s->b_bidir_forw_mv_table_base); + av_freep(&s->b_bidir_back_mv_table_base); + av_freep(&s->b_direct_mv_table_base); + av_freep(&s->b_field_mv_table_base); + av_freep(&s->b_field_select_table[0][0]); + av_freep(&s->p_field_select_table[0]); + + av_freep(&s->mb_type); + av_freep(&s->lambda_table); + + av_freep(&s->cplx_tab); + av_freep(&s->bits_tab); + if(s->q_chroma_intra_matrix != s->q_intra_matrix ) av_freep(&s->q_chroma_intra_matrix); if(s->q_chroma_intra_matrix16 != s->q_intra_matrix16) av_freep(&s->q_chroma_intra_matrix16); s->q_chroma_intra_matrix= NULL;