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 812D54B72A for ; Wed, 12 Jun 2024 13:54:32 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 197FD68D9FF; Wed, 12 Jun 2024 16:53:52 +0300 (EEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2024.outbound.protection.outlook.com [40.92.90.24]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 89EC868D9E9 for ; Wed, 12 Jun 2024 16:53:50 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Tr4g63FvaazSUgXNGggEbluqJfAW8fXzu06DMGicirpgKP0/vLNMZa1d01S9/IoI9i9A8xruXYSu43Kjk/5yvmoWyO/2bn3wVGMpoDE1PpUau9OmMSfiZzmigGoHmIgBinh4YRP/pfE7DYUvbQWH6SOtyAKbTx65UvNZPA7ziEhGsfz+XTjEt36kETq9Ano5COMdpfZvPMFMKkDv1spJ52DUQ/+eJRwx3q+j/X6LO2aH6I6eSB4BWk+00sShRVBUPTVNap2IH+oXiwlWrj+N9n77RB+hstVbDtWD8CkKSwiTFZWmGZXtu8ny97VWdc5wVsxYsezSKGpGLDnuWIU3PA== 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=FUUqs+HomfvFCHW0lIo7RIPajWrxM1v7C565WjpSN8c=; b=Rf2UCtmspFatNNRd7E++fhU9uSOLl8gMRTsODV2YJeIa95FD68X4xxEqgD5SEcf6TN9VxblonGLqU/SlLdZf0PawOwmSJlhzTiBh6tlBDRgdaX5xd7bcZJEPmXiqGETjn7PQ82dO80bFWEKPM8W8Fjle/Fvcpr+rCzLUbc8IdFozXvNk4lRAMqzoQyRxsZttZ5zUEUw9FkfQhgJBLmx448HUjLvzJpikVKeZbSvIN2ZnPszZq0iMWz/QB1IB2GOFOTFIikxW3bnz7nWr/wsecUE8q6M151veSYrZkG0E6Ws/YLXiLJ962kbHZvJnW2nXUowe1ez8aBEPk2NogKNXqA== 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=FUUqs+HomfvFCHW0lIo7RIPajWrxM1v7C565WjpSN8c=; b=Jd/AV+ShnQ32J1zIBkav3huSKdbnAFlzEYqtMRYxeanpzp7gtc09wMpZRcTQio7jMlq+FvpOJH/7ypMg4LOQS1eNGTh7J/4xGAChyEAXuSHmjAlmt1qdSYD3EuSTa/kcE5PbphxgT9sW9HVrMb7wX5Fd/64lheq7oHKmVMsO50Pcu9g+21JxrryD4b8CPbFttjDuH4Ls2dN0Shww9mRo8ocG5RG4mQjVUhe2S5YZ2h/VnPWXFItf/c287INJkE4aRjKrfcZu26/Nhvh5F/ZKwX6dB+vWlLJkP9AXQ6SOLKdVw5dK27srWp14uZqm1cV5jsCNSxhaqVT+u4zX14SyIg== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU2P250MB0272.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:27d::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7633.39; Wed, 12 Jun 2024 13:53:49 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614%3]) with mapi id 15.20.7633.036; Wed, 12 Jun 2024 13:53:49 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Wed, 12 Jun 2024 15:48:28 +0200 Message-ID: X-Mailer: git-send-email 2.40.1 In-Reply-To: References: X-TMN: [OH9sUlvgftoUnAdZNA9E+jjo5YDBPuLW6X+/ZHAc/k4=] X-ClientProxiedBy: ZR0P278CA0189.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:44::6) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <20240612134853.2102377-31-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DU2P250MB0272:EE_ X-MS-Office365-Filtering-Correlation-Id: a2fd1d00-9cdc-4bc3-850f-08dc8ae715d5 X-Microsoft-Antispam: BCL:0; ARA:14566002|461199020|440099020|3412199017|1710799020; X-Microsoft-Antispam-Message-Info: 0DQCD3cHwtth5SOvza3gK/ZczBJWlKc18Bq4fTZN4UOecYXZAitQJtvSkmiFbXtSzJm6GaTHR3tnwWVynQG2+HneuSqODinzYRXf46F8LOcVmJdHhtd0YJcq+1Sy7Pn2AHUZ1Mz5DsHMtmYiKKPfOxgDONJpF8cZ3phTCD4j63H2NDt+Mr8oKUcxj6CzQuIRP9wMh5FZoUpsEFeQOAJSnGF1MtYsWV1qoGWYNm9zlxj+CrWyToluNQr0w0pB2GSUpg8hjEYBWm2CUC9eEg9cXMSIjQCcXw3Iq1FEt7loNXE3cuGzeydpM/8Ut6nhR2wV0+4KA8s/RVENDUuQR1tRFp3Z9+43ed++mgSR9r+gjyo4q4ULsntxpcMD8QA+XFtOn6SWPSMI2hAho3zGd4jXP5HBlvJ9dtksCwU7IOZu5bJdOHf72lnVMiOKobg2Mpw++M9G8d9YEg/fOIY13pQ5Wn/TdCrAmsb8QcuhQIVIR7yF3fGWqs5ws3U73fS1YaqwXv/EU7Offbmgpx1/KV5L0zeyRCxXoG9V4iIY9DL1occmmHrIytMPS7Q234pOg5UMQngxBsr3dsKHAuDAECWvSK/8T8qk6Liya1iFrNtOAr6QIf5YQNZDCGgvOTHHWyXV X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tMSUQRAKCKPQk2dM92RRASw2J4L8tODIM9SLtFgo3W0iad7udQHaXiYuggyo?= =?us-ascii?Q?bGihz2AL+5kPRDl8s97VVpTqXZKmso2UqnUHHay9mXwZZEcGbxkUYlf++NXa?= =?us-ascii?Q?gyB1I7cI66t8t2p/eUShjGSEnOce3x7j9TDcc9nt8e33oFoNpbaiwQdqeSJd?= =?us-ascii?Q?w0q2mbPmbDrpu1/lASgCrKFCiGy5VM5zXc2fak8lPowECij8vXQ3RqPX5qYZ?= =?us-ascii?Q?hEobnYb6KrkjaLhqtfXihI1JKQ+RUwxRLHjncVZDPPHKX+0H1ea6zNyG+eVe?= =?us-ascii?Q?cnF192Evh2WQd8bhFJx59udSd2ac8tpfSqVRBcq1bivfSoleKFgFaKiej0ly?= =?us-ascii?Q?JRqFR7uDtI5pplFD7qNY0L4VXA69mF5C6t4/VoXQ/1b0J1H3C701d2IkAhY9?= =?us-ascii?Q?IhqIVsrGeNUI+7RGtQYfWKnWIorivGxX/CjZjFV0xhh8hWqSFQbZeVTjT9TR?= =?us-ascii?Q?ALWbxGfDpFVy0ouJ4VNnqHh4z4NJ2Do2ujcB7PpORhT77oehk7h44H2QUuV1?= =?us-ascii?Q?8l+vxRG1jF2FX0nGyEd2C5cGA4EeIqYVVjZ4fwvk4+FbX+86YiHoFQ/pXkZX?= =?us-ascii?Q?vQL6Nb9K7ls0hrHaLYwglrzBb0zIJiybz/WmFipB318K8NmQWc85gso+6J8n?= =?us-ascii?Q?Ibsh7CpVPjdN2SMDyWGOIojp1ENz2f6P73y2f6nVCAhBVAkCkSbMtIEK1fqy?= =?us-ascii?Q?2NZ99XZunJB8Cj7bKAwb2KbbBmrconiqhGRjFKpxiGE0llU8oytZ89/z6U6t?= =?us-ascii?Q?zg1JXuntbXK5gGZUSa/oHmAp4ydIkXoHTnemsJx5p4BkKxu9TuzundOfpxAB?= =?us-ascii?Q?59NPtbKd+ZAkN4BqrDGWNpcxqJR19rIyDA/ETyBzm8JU0vjzF7/FLiSY2Nwg?= =?us-ascii?Q?WZVd3E4UkyGhDv3UylDDYI8FGWyge7nV0NJpXzjZ/IzUb4dnIc2rGQMh4dTu?= =?us-ascii?Q?qK7MHO4IRFsBgmFfTC7vP5mHJkWUnnej06Uq4OkY1dQqPGxRQquiWMCVfwTP?= =?us-ascii?Q?q3SfMAJct06TJJZYxU+ebNvA0DtfP5iy2Ta9oijHMqFKe+Z2CBf8mODFB8RQ?= =?us-ascii?Q?10dw/Ob4KIfxGTLInCyc+kspXvoqQJmXnAttaXANg+pURMv1zI4g+DQOsXaB?= =?us-ascii?Q?k+QdjBWWiRzr+BEkKXRYIgMOhbnITqZMhB3wDyds0RWPT+K1TWV+4jwfJfs/?= =?us-ascii?Q?OKM+/s5ZnskAVh852t+4k9BuuXrKjzgDNiRoQOKb0SnnJglzMVi50CUDXOvP?= =?us-ascii?Q?uaYNulSwswxnCJS7r3usW3MEUJascvTIxLTrBhfdBA=3D=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a2fd1d00-9cdc-4bc3-850f-08dc8ae715d5 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2024 13:53:48.9977 (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: DU2P250MB0272 Subject: [FFmpeg-devel] [PATCH 32/57] avcodec/mpegutils: Fix ff_draw_horiz_band() 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: Broken in 5ecf5b93dda9d0c69875b80d28929f0d97dd7d06. More precisely, 3994623df2efd2749631c3492184dd8d4ffa9d1b changed the precursor of ff_mpv_reconstruct_mb() to always decode to the first row of macroblocks for B pictures when a draw_horiz_band callback is set and to (they are exported to the caller via said callback and each row overwrites the previously decoded row; this was probably intended as a cache-optimization). This first macroblock row was used as source for the draw_horiz_band callback. This of course means that the ordinary output B-frame was not decoded correctly at all. Therefore the first aforementioned commit removed this special handling of draw_horiz_band; yet it did not remove the special handling for B-frames in ff_draw_horiz_band(), which broke draw_horiz_band for B-frames. This commit fixes this. (Actually, draw_horiz_band was already broken before 5ecf5b93dda9d0c69875b80d28929f0d97dd7d06 when using slice-threading: All slice-threads would write to the first row of macroblocks for B-frames, leading to data races. It seems no one has ever complained about this, just as no one has ever complained about the breakage caused by 5ecf5b93dda9d0c69875b80d28929f0d97dd7d06. Probably no one uses draw_horiz_band.) Signed-off-by: Andreas Rheinhardt --- libavcodec/mpegutils.c | 23 ++++++++--------------- 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/libavcodec/mpegutils.c b/libavcodec/mpegutils.c index a567165fd9..a53996852f 100644 --- a/libavcodec/mpegutils.c +++ b/libavcodec/mpegutils.c @@ -57,6 +57,7 @@ void ff_draw_horiz_band(AVCodecContext *avctx, int first_field, int low_delay) { const int field_pic = picture_structure != PICT_FRAME; + const AVPixFmtDescriptor *desc; const AVFrame *src; int offset[AV_NUM_DATA_POINTERS]; @@ -82,21 +83,13 @@ void ff_draw_horiz_band(AVCodecContext *avctx, else return; - if (cur->pict_type == AV_PICTURE_TYPE_B && - picture_structure == PICT_FRAME && - avctx->codec_id != AV_CODEC_ID_SVQ3) { - for (int i = 0; i < AV_NUM_DATA_POINTERS; i++) - offset[i] = 0; - } else { - const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->pix_fmt); - int vshift = desc->log2_chroma_h; - - offset[0] = y * src->linesize[0]; - offset[1] = - offset[2] = (y >> vshift) * src->linesize[1]; - for (int i = 3; i < AV_NUM_DATA_POINTERS; i++) - offset[i] = 0; - } + desc = av_pix_fmt_desc_get(avctx->pix_fmt); + + offset[0] = y * src->linesize[0]; + offset[1] = + offset[2] = (y >> desc->log2_chroma_h) * src->linesize[1]; + for (int i = 3; i < AV_NUM_DATA_POINTERS; i++) + offset[i] = 0; emms_c(); -- 2.40.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".