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 3D21649653 for ; Sat, 15 Jun 2024 17:18:05 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id CBA4F68D6A5; Sat, 15 Jun 2024 20:17:57 +0300 (EEST) Received: from EUR02-DB5-obe.outbound.protection.outlook.com (mail-db5eur02olkn2021.outbound.protection.outlook.com [40.92.50.21]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id A6C6A68D674 for ; Sat, 15 Jun 2024 20:17:55 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jBkaWuPjm5V0TqYoFcw02c7yWdTLr+8+X9tHzAWAkg2n0Jk3vuqH+XS9xQPsCib9+iB+MOvGslIL9mAyvH0rZkHe2WfcBRU3Mi3UBtyzhhy8cz+3paa1da2NM+qy7dkYNDElE1ZVjaUAeMXUtx/OY6KY+AF/RouP3aVb50cKxjIOE3ShybH4Mbv3yZQ7cnQ20njmkSY8PbE4KeMpmdII4csB6rkZIROaDT8rweU29q3+ubdApNcVDFvRLyd4I8MduWjiVeUG4doTcqXiBO8kVmPUklTz/uPO/aAnjHYLbdxLr8nBA1WZ9/w0Lgk4Q+JcX6tOU02Qag6EiyaYlNbvDw== 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=t/IH1vx4BX0l/PTwp0R8t5qw7MAqtMI0cm/6hNdojac=; b=oXojDdwNpPOpzh+YI9vMQ/g966TYwgaN8/Ia9QQgnIXZRxGLeoxwEYowBDQmteuEeYwloGfTM1i83RJIDeKpMdmZeXrzfNZ43+uc9BC4MoBLQSU6DsttxtBZBPQgCDUjztPh31xX5foWq1KM7P8fvNMmhXezGg//QuY8u9CJHyQbhDzJII1rCSmotbZzgWr1PE/di0xVPPdmqyAc+7tzhbqSo0+AFE5L/PWh3keS/4FwsTDPiIQdPG/1T22hSDne5YBaPKCtzQAlDy6hXPl0T3Sh138s8RGrmovgMmLjEIcRgEZWzPonphDaAKM7cA4vRHkzy/zLLIaZ4krGftb3ew== 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=t/IH1vx4BX0l/PTwp0R8t5qw7MAqtMI0cm/6hNdojac=; b=HEOoN/LNmrHS+KPuT/5MAh8JbC4pw5zxil3jrgrE936gImZyUvpaFckySWkxfbARrOTYSt3ocACrqj+udaTyJTEQ15ufSO8pupDEt0nTlv5RdmOVpx0mRDlViqIsMpUFzFO0LpTFiZ2vWXFe9lVcpRRv4MsvDBfstohr7wKU+oV7DIyXRciclU+NA69J4oibCdOHoLmfeSjZqKhFiAsQRue9astQd2vN2N2AqzdYxUArDonoXrHElzgOKYHSJ/l14ZQ310CqkOnPImSZxs2g7985aOEneMNLHplD9XHWoDj+tL4/7U0rZ6Hz2nSWbvR2FZVsZuXnLA9z6y2IocqfZQ== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by PR3P250MB0131.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:176::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.28; Sat, 15 Jun 2024 17:17:53 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614%5]) with mapi id 15.20.7677.026; Sat, 15 Jun 2024 17:17:53 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 15 Jun 2024 19:16:50 +0200 Message-ID: X-Mailer: git-send-email 2.40.1 In-Reply-To: References: X-TMN: [cOtMG48c1WmdY4Btgu1yU6x/pPS6B3zZnHlo3IiToAo=] X-ClientProxiedBy: ZR0P278CA0099.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:23::14) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <20240615171650.3721642-6-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|PR3P250MB0131:EE_ X-MS-Office365-Filtering-Correlation-Id: 24593065-a050-4923-2411-08dc8d5f178c X-Microsoft-Antispam: BCL:0; ARA:14566002|461199025|440099025|3412199022|1710799023; X-Microsoft-Antispam-Message-Info: zxPv2ImRJWdvftg0alB+Dq6VigxRvFl2N6k4Yy/I8rN2PzyvR/fldfbkoi8BwGiTQT2akpOe3O7mUA4Grd8f7FcU64IouErFtejtC7ly0oHI6Qc7xxZ3igJzitqL5vmymsGDAQERcvPnfgXRhCH8cbffGdqEmmmwR2Cfi4eZ+573bUb/7doakcBVJcZ6c9JtROpQG+ypqWbnABkyicgAUfKhy0r0ZHLyMo3ApCwfjR6zvVAb79P0Yg1yt3voS8zc/+H41lhL3axMchyxBvpthz8n58TY4NfqecIe/FalxFwufcniNZSO7D8M7kOUT5sZLDa8I0BBYjzGC3PUnmBgHWCm/lYpH7MihCqxeOMTuQfl2eGaM3zO1bfbegjdM8ZUKQ0n8dviePqVgUZIAQmklKKe6RBJC9mu2AoT+eSHfXcKBu9PaaFi9OG+KqXA8eq2U9TW6azFAgs3V6ceL30Uwrxncm0DLwDQRjKGDp1IUXcDOvA9tMZGkQ2QRCSnlKOviVXW5E2q3zSU1tpiXB3ia8Be19+AQI2nBZGhR8vwerSgmJzufC7A5ynjltDr9CtQWa87Ih17Vx4+zwECWADBiof93WZXDlCeI3AnBMsQctf37vv9Z/KnDDY/s2V0pT4z X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?vaFRfs7BnQqOBvj9uedwWdSou1Ny2r61H1YN6H7I1nMUtATITgglA6F5zqNh?= =?us-ascii?Q?79n3YbOdm7eKHlnLfTLihZDHZziPcLRXaYPTsNWVw0gOmiZIkT+5T+ZFHCd/?= =?us-ascii?Q?shJwcY2ONyDnvFAXG4IcnxZxBBGeuoTOSA7zWBLG6D/fCTd9/qAOfZvMaM5a?= =?us-ascii?Q?QF6+6BiQ2FC2vQbWThIK1+2GG2E8kIs9yuJhJeTv6LhRRVFIVFtBOf1hFk2z?= =?us-ascii?Q?rcU8PFxf4XVwRwLlp50Hwi52AtQ63ymH+VgtwLQu5C5kikAbIsBeJ3CcOIek?= =?us-ascii?Q?0oDrTXZdbrjKoHf9yTygw/y3NLGrJcJ4U5aa1VfiOyh0SRMW6KhUqSxY8S34?= =?us-ascii?Q?qShPaLPPobAemJ1zrHUkyLBzDelIJc0+HlIx+qB0yRN323GLycpAIdwapKgl?= =?us-ascii?Q?VbnurWjBSR6k3TZZOGZiQyrGGoSTxBOORWGNUQAmZhefUC1qHXqxMb3lU6q5?= =?us-ascii?Q?BRB4QM19KzFGCw1k9GemRKzPJjOX1yk1qp2Ykuu1GNELcOGAFkwkcy1W/YeK?= =?us-ascii?Q?oqdVkGoldYXs9vIHUJyDXRHKnmkA4lEmsHwoWRotQm+Rl+Cw50gqgMhNCjv0?= =?us-ascii?Q?WIhKl1NKXJbTHzv9ZB3m8dF5tR8BE6Q+OGYW4oG5+1AudGhQZOWdrMBD6nQ9?= =?us-ascii?Q?3ONxWjqnMiNcYqUrDMDTgYH5eVLHzuhzMZ8qmE79lu77rzJv4jhn0+jJA8aD?= =?us-ascii?Q?2PnEKI7YPpW6dhnl3Sw55okze19DtZO4aSpF7oZP2egjgSBTzy+ZUwG/nXxa?= =?us-ascii?Q?6GnBX6dyBbbwkm+Al+XDqFv6WCZ+B07I5W/V9aSuz2cgaJfjGc1WsAC/jabC?= =?us-ascii?Q?spkDVYoiamUYGSvZoD0wKQum9BkNldVE7vaE0jcPBeOJceNwhQCfb/PkHh1H?= =?us-ascii?Q?IyHqjCzEJcBOhnhg1uul8ZBcqHfIczTqNkB+6W/97Ce3ebE/PT+d61NVdgJS?= =?us-ascii?Q?EmG3tM27NAw15SHFrDGpMyDus4inCsiHTkIRm2g1AJ981ff8KS1+eG1qHGZj?= =?us-ascii?Q?rMsKkDc9jJtd+ORkX2NJpsjeetxb9tBZ2QtsqPwOaFzKEtZwcqq1w07GxAZV?= =?us-ascii?Q?K8smSw/N/xYWSNryyDl7FD9F7QPSPVA1orycO3EBJHOgiQhVPXgVj4zOHsNk?= =?us-ascii?Q?15XCpWQbMuJPNOJyX0mJlQGI4oEFINH7bVhvWxSSO/DMRYxwMdC/z31Erz7/?= =?us-ascii?Q?e/7KTms+U9mIfJVH0WyUASuAFXs0mfOFe3Nlcr640deCy1kXE3T5I7tcH0pu?= =?us-ascii?Q?W4f3M11jwXAW1F6q6Ae5a29kPVAUVp8U87rOJ2k9qA=3D=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 24593065-a050-4923-2411-08dc8d5f178c X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jun 2024 17:17:53.8350 (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: PR3P250MB0131 Subject: [FFmpeg-devel] [PATCH 73/73] avcodec/h261enc: Fix ac_vlc_length tables 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: These tables are supposed to contain the number of bits needed to encode a given (run, level) pair. Yet the number of bits for pairs needing the escape code was wrong (it only contained the escape code and not the bits needed for run and level). Furthermore, H.261 (a format with explicit end-of-block codes) does not work well together with the RLTable API from rl.c: The EOB code is the first one in ff_h261_rl_tcoeff's VLC table and has a run value of zero. Therefore the result of get_rl_index() is off by one for run == 0 and level values with explicit (run, level) pair. Fixing this necessitated changing the ref files of the vsynth*-h261-trellis tests. Both filesizes as well as PSNR decreased. If one used a qscale value of 11 for this test, one would have received files with about the same size as before this patch (with qscale 12), but with better PSNR. Signed-off-by: Andreas Rheinhardt --- libavcodec/h261enc.c | 59 +++++------------------ tests/ref/vsynth/vsynth1-h261-trellis | 8 +-- tests/ref/vsynth/vsynth2-h261-trellis | 8 +-- tests/ref/vsynth/vsynth_lena-h261-trellis | 8 +-- 4 files changed, 24 insertions(+), 59 deletions(-) diff --git a/libavcodec/h261enc.c b/libavcodec/h261enc.c index b19830d578..a901c32e42 100644 --- a/libavcodec/h261enc.c +++ b/libavcodec/h261enc.c @@ -38,14 +38,15 @@ #define H261_MAX_RUN 26 #define H261_MAX_LEVEL 15 +#define H261_ESC_LEN (6 + 6 + 8) static struct VLCLUT { uint8_t len; uint16_t code; } vlc_lut[H261_MAX_RUN + 1][32 /* 0..2 * H261_MAX_LEN are used */]; -static uint8_t uni_h261_rl_len [64*64*2*2]; -#define UNI_ENC_INDEX(last,run,level) ((last)*128*64 + (run)*128 + (level)) +static uint8_t uni_h261_rl_len [64 * 128]; +static uint8_t uni_h261_rl_len_last[64 * 128]; typedef struct H261EncContext { MpegEncContext s; @@ -320,51 +321,10 @@ void ff_h261_encode_mb(MpegEncContext *s, int16_t block[6][64], } } -static av_cold void init_uni_h261_rl_tab(const RLTable *rl, uint8_t *len_tab) -{ - int slevel, run, last; - - av_assert0(MAX_LEVEL >= 64); - av_assert0(MAX_RUN >= 63); - - for(slevel=-64; slevel<64; slevel++){ - if(slevel==0) continue; - for(run=0; run<64; run++){ - for(last=0; last<=1; last++){ - const int index= UNI_ENC_INDEX(last, run, slevel+64); - int level= slevel < 0 ? -slevel : slevel; - int len, code; - - len_tab[index]= 100; - - /* ESC0 */ - code= get_rl_index(rl, 0, run, level); - len= rl->table_vlc[code][1] + 1; - if(last) - len += 2; - - if(code!=rl->n && len < len_tab[index]){ - len_tab [index]= len; - } - /* ESC */ - len = rl->table_vlc[rl->n][1]; - if(last) - len += 2; - - if(len < len_tab[index]){ - len_tab [index]= len; - } - } - } - } -} - static av_cold void h261_encode_init_static(void) { - static uint8_t h261_rl_table_store[2][2 * MAX_RUN + MAX_LEVEL + 3]; - - ff_rl_init(&ff_h261_rl_tcoeff, h261_rl_table_store); - init_uni_h261_rl_tab(&ff_h261_rl_tcoeff, uni_h261_rl_len); + memset(uni_h261_rl_len, H261_ESC_LEN, sizeof(uni_h261_rl_len)); + memset(uni_h261_rl_len_last, H261_ESC_LEN + 2 /* EOB */, sizeof(uni_h261_rl_len_last)); // The following loop is over the ordinary elements, not EOB or escape. for (size_t i = 1; i < FF_ARRAY_ELEMS(ff_h261_tcoeff_vlc) - 1; i++) { @@ -375,6 +335,11 @@ static av_cold void h261_encode_init_static(void) vlc_lut[run][H261_MAX_LEVEL + level] = (struct VLCLUT){ len, code << 1 }; vlc_lut[run][H261_MAX_LEVEL - level] = (struct VLCLUT){ len, (code << 1) | 1 }; + + uni_h261_rl_len [UNI_AC_ENC_INDEX(run, 64 + level)] = len; + uni_h261_rl_len [UNI_AC_ENC_INDEX(run, 64 - level)] = len; + uni_h261_rl_len_last[UNI_AC_ENC_INDEX(run, 64 + level)] = len + 2; + uni_h261_rl_len_last[UNI_AC_ENC_INDEX(run, 64 - level)] = len + 2; } } @@ -398,10 +363,10 @@ av_cold int ff_h261_encode_init(MpegEncContext *s) s->min_qcoeff = -127; s->max_qcoeff = 127; - s->ac_esc_length = 6+6+8; + s->ac_esc_length = H261_ESC_LEN; s->intra_ac_vlc_length = s->inter_ac_vlc_length = uni_h261_rl_len; - s->intra_ac_vlc_last_length = s->inter_ac_vlc_last_length = uni_h261_rl_len + 128*64; + s->intra_ac_vlc_last_length = s->inter_ac_vlc_last_length = uni_h261_rl_len_last; ff_thread_once(&init_static_once, h261_encode_init_static); return 0; diff --git a/tests/ref/vsynth/vsynth1-h261-trellis b/tests/ref/vsynth/vsynth1-h261-trellis index 87b078b0d5..0cbb9b0e18 100644 --- a/tests/ref/vsynth/vsynth1-h261-trellis +++ b/tests/ref/vsynth/vsynth1-h261-trellis @@ -1,4 +1,4 @@ -02b4109ce5343b7ef24fb11c2635498a *tests/data/fate/vsynth1-h261-trellis.avi -655416 tests/data/fate/vsynth1-h261-trellis.avi -70ceba944548ba680b1101c91707ea25 *tests/data/fate/vsynth1-h261-trellis.out.rawvideo -stddev: 8.75 PSNR: 29.28 MAXDIFF: 90 bytes: 7603200/ 7603200 +9980463214cb744a7780737a16408540 *tests/data/fate/vsynth1-h261-trellis.avi +590384 tests/data/fate/vsynth1-h261-trellis.avi +7bbb0520a5ca26cdcea148f9997eae27 *tests/data/fate/vsynth1-h261-trellis.out.rawvideo +stddev: 9.09 PSNR: 28.96 MAXDIFF: 93 bytes: 7603200/ 7603200 diff --git a/tests/ref/vsynth/vsynth2-h261-trellis b/tests/ref/vsynth/vsynth2-h261-trellis index b9c694f016..03d17e194d 100644 --- a/tests/ref/vsynth/vsynth2-h261-trellis +++ b/tests/ref/vsynth/vsynth2-h261-trellis @@ -1,4 +1,4 @@ -f5e0cfc70bbe4f4048c15be88dea4378 *tests/data/fate/vsynth2-h261-trellis.avi -249856 tests/data/fate/vsynth2-h261-trellis.avi -15452237f6c333690d3e05f354f63196 *tests/data/fate/vsynth2-h261-trellis.out.rawvideo -stddev: 7.10 PSNR: 31.10 MAXDIFF: 96 bytes: 7603200/ 7603200 +cdf6013013b8de3fe56476178b5adc89 *tests/data/fate/vsynth2-h261-trellis.avi +227156 tests/data/fate/vsynth2-h261-trellis.avi +d1107f161a7d923e0ffb6aeb9c713633 *tests/data/fate/vsynth2-h261-trellis.out.rawvideo +stddev: 7.24 PSNR: 30.93 MAXDIFF: 87 bytes: 7603200/ 7603200 diff --git a/tests/ref/vsynth/vsynth_lena-h261-trellis b/tests/ref/vsynth/vsynth_lena-h261-trellis index a306508032..75558679ec 100644 --- a/tests/ref/vsynth/vsynth_lena-h261-trellis +++ b/tests/ref/vsynth/vsynth_lena-h261-trellis @@ -1,4 +1,4 @@ -41522be50f14b7fff6b1fb8d10b1ff00 *tests/data/fate/vsynth_lena-h261-trellis.avi -184586 tests/data/fate/vsynth_lena-h261-trellis.avi -f9df8cd110a2f3d9706dd2f29a1d0a89 *tests/data/fate/vsynth_lena-h261-trellis.out.rawvideo -stddev: 6.32 PSNR: 32.11 MAXDIFF: 89 bytes: 7603200/ 7603200 +aa3e00ffeb48a23452bb1c5b0dde9e77 *tests/data/fate/vsynth_lena-h261-trellis.avi +167808 tests/data/fate/vsynth_lena-h261-trellis.avi +055f8805df2bb21ef6752dc944e7828d *tests/data/fate/vsynth_lena-h261-trellis.out.rawvideo +stddev: 6.43 PSNR: 31.96 MAXDIFF: 87 bytes: 7603200/ 7603200 -- 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".