From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <ffmpeg-devel-bounces@ffmpeg.org> Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 561ED4D932 for <ffmpegdev@gitmailbox.com>; Mon, 21 Apr 2025 11:19:35 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AFFF4687C19; Mon, 21 Apr 2025 14:19:31 +0300 (EEST) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05olkn2043.outbound.protection.outlook.com [40.92.91.43]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8D802687B57 for <ffmpeg-devel@ffmpeg.org>; Mon, 21 Apr 2025 14:19:25 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ybGgiQlJxbc+ngGGrAkUkMH8QNKflW4+g5AOF/elHN8ILHlWO+ARy9rjCGOszxSiTMcfcndYIBZedsUypq5oDOPo8Ae0e+TZExYYmTcBZEHzsim1kLnDPNCIN+jGl6LJjr/2+A4t+Lte6VFH6dhp3v7Q2uUYqHkb3jP1925p+H2A1X9hk47swRhvxdGu/OL1FLdVcnJdokowxWFyy12534qkhUtR0CaStM16rCHRWn8GeVS9P5t/ba4nTfKEOoN4uujE+ZeLiQwlU225YL9yccdPN2+KR9DfW2vbQrbXveblQYdUijMIYXIKBfZx3EdLTu1xGXchpHEvKRujUObUjA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=jEv9cY3LCwn8tdmkc08BxMK/2dqo481Rrnz3r6+LSxw=; b=DFTGXE6LRwFA/fGw67+S5lNWK9ilWgVID0sPSalda6A59U2x/k2Wa5yiGnm12OOpunCRuqrthK8THXYvws/R33QoVm0iD/NevQNCqlxqUEbDio5Y7nyNnPxaLGxcNyMV8NzYufxmFcMekrR0AHzHI/SzIWJEQsR/6uzb4JPsW2zJqnfyK/NB0aJsXcLdCTYA6NaWy78q9R8cet/RmzjIFaT65XyEK01HBqX3W2kgdSRrx15vB0HDjkMEQhgnUeRGcRAc0JkaFnfwDybdo2GIWdjXQCxTlm2OmpAEahXgUZqanbNQD8sS/D6VJqlmp27J/EDWJ67zG/EWzMGZyzjfJQ== 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=jEv9cY3LCwn8tdmkc08BxMK/2dqo481Rrnz3r6+LSxw=; b=b148gOWHEVFHAQiC6MeIXueLV54Vpr96bnHlw6cFRI3OAOo0zzopVoepDVBsW24lftPnBQUZb2+y92xwVzu9+ASI9hq1JlLfVdnN6s7fqTvBGiFJy6b7HpL5/q40PK5KkkEk3erkxNVdc2eyl6n2RqmsTj3iMOGsCYcz1P8lgdvg8oYy5i03qRzdelki8+eklxy70G4w5vrN4AARUlVyRPhRfwrZQs7eeJh+1dOHkCy3PTdvOHnzEdaaOBZ9K9D9ZLop4b/AjJLC/hnd/VD7xxTEnAcM+vyOHg4NxGLIhJGEMJL5QMNEkXo16oK+2lgTLC8JsDIJTk/nz5dygvvQ0Q== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU0P250MB0818.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3e2::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8655.33; Mon, 21 Apr 2025 11:19:23 +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.8655.031; Mon, 21 Apr 2025 11:19:23 +0000 Message-ID: <GV1P250MB07370626849AC5D7858CC85B8FB82@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> Date: Mon, 21 Apr 2025 13:19:22 +0200 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20250421023345.7558-1-emma@emma.gg> <20250421023345.7558-3-emma@emma.gg> Content-Language: en-US From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> In-Reply-To: <20250421023345.7558-3-emma@emma.gg> X-ClientProxiedBy: FR4P281CA0429.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:d1::10) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <bfc2a5f4-7458-4988-8916-e98d1fc6e6e7@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DU0P250MB0818:EE_ X-MS-Office365-Filtering-Correlation-Id: 82b9f891-f0de-4e7e-1613-08dd80c65e4a X-MS-Exchange-SLBlob-MailProps: AZnQBsB9XmpI8tRyoOXaSUmOkLVCBKas51nXb1vWqv4jv3cpS6BVslhjHOoVPvRF4cuokzBoRHz3MpmfOmSTYG5UvADSmVxgvflhvk5ndJtcozJpzA1h7Le9VEm4Wa32o7VQFuJzrUjlV8XxqAxsDQmfEDM7qKPiYH/cGQR0ZK/gAz4+v+K3/2BGnT4QFksX6Oxa0Q58gHntXKU5/05ObiKxgV4v1WkKONc1+OfAspkt2xStLcu7eyVFYkgae8kxyA9hLQxQilw2BCwlRImN2EIEefnL3vetTqq+g6gO8Qc+qwfZ8Aof8q4wXB5oYEJZ74KO5R97FrRsa+KsI9aMDfKY93HKDb0XObHO3IpvfHroNGCh2kXviwbydbui3wqAXWZsRHVT8HRjcLwlacFFIiTgXoPs/Ctf2yxRiH0kbmgnRogLuGksKGSP4TOxIgYAsUWYCMrbqLzRfQsOivvVCclO96LTmVp+paWOZC3gWMLpY4xa8kFEHoxLEg/HVuTHiv4WGx08YvzBpupzIrS3Lpwq1x9OHJUWcRjZJK1Zqb+vwO/t01m+bFIEMXIwDYeQ99pwaQ5yBrf1xVCV9J+mW1bEAgX5RpcKLxjTz1CCyV2kzxDBCLu7i3rweE1jYa8MNVhI0BpUSlxEsKFZIUGDzVFBE6IFGmnTYk7UPWyc9D+KuRqR5SgYvPc3XHCMniv7bQDGLx9W0gkgRl8YSE+pr098FfHmFQA6JfOW97+rdeIEYbEHL43KecbqsmxfU/LXtkAwoi/CLc4= X-Microsoft-Antispam: BCL:0; ARA:14566002|8060799006|19110799003|6090799003|461199028|5072599009|7092599003|15080799006|3412199025|440099028|41001999003; X-Microsoft-Antispam-Message-Info: =?utf-8?B?RzN0SkNuY3pCUTNsZTVDdmVvdVdrVTlicmE4dWVTU3ZkUWpDWGozakpvUC8r?= =?utf-8?B?dnQ5VUw2RFBsZlRacGdXVVIwYWlMenRtL2ZxRUs0OHNtT1VmNCs2c01FWi9p?= =?utf-8?B?ZEJuK2lYRnhMOTJ3cjUrOWJqeGN5Y2YzWGZCZjBCSnl3amZHOEtEM2FzUkx4?= =?utf-8?B?Y2pHdTdSbGF5eTZ1NzBUTDFJdmZBRDRPUUsxMTJ1SThkWVJ5MVhFUnNVZXBX?= =?utf-8?B?Tk1nWWN3VDF5ZlJSelNRQUUxMXJoQk1CeHF0OXhvdUxySGY1S0YyYzVKQzQv?= =?utf-8?B?d1JTbHFXV0wwZ0JtTjRyaWM3YUFjTnRPTnNPeXM2RzJOaXkwZ3ZMTlVwY0Zj?= =?utf-8?B?ZmhveTRPNWd4b3VUYlUzMi8zbmhGOUU1MWtEakg5MVQ3VUcya0llSTBZcU9o?= =?utf-8?B?dnZickhWWXpWN0tIZHk1L1lPM2JaaHNjelJZUzY4Vm4yNlBTQy9RdVBtd1RY?= =?utf-8?B?b0JZaTdHbUJYcVhFMWFxeFlyMzdnazkzVEVRRUwvVXBJdEtEYzUxRi9zaWgr?= =?utf-8?B?RUNtQ210cExkUUZhWWUwdDNrQ0JlRndVZnBISHdMRnh2Y3gzRFFBNFA1QkRY?= =?utf-8?B?MUp4cnBRSllKVi80TGNCbmpMbHcvS3NDQmtBU08xbTU2VmZuSUZJblA5QWpH?= =?utf-8?B?cElJSlJjNVNUOFN6ekNqbkYwRUlXcmwwbHNBcVlBRitPSEJvQzVhQittSk56?= =?utf-8?B?RUlHQWQ4L0laaXhkWG1lWGtYK0F0ckQ3SHF6Q25VbUFaUDBGdFFmS2djeHZt?= =?utf-8?B?djJBYno4R0hyWlk0M3FGTElKcTJwUkkzc1ZleSs3eGw4MG4xYU5Wd2JXWTRp?= =?utf-8?B?TlBHc01Kc3hObDRXZWpaRDB4OUFHcVd1b2VhdFUzNVVYKy9NeFhwZ0dnYUto?= =?utf-8?B?KzlSWW1OV3VPdCtIWnJEWWNzQ2YzNGtUc3JWTnFSeEt5KzdPY0FzRENUY0xE?= =?utf-8?B?NVk0Uit2cGpUaDArK3VxR2R3aW1aYWM3dk9nRDFGaGpQVFVvVUhDeWhCanlQ?= =?utf-8?B?K3B1c1M4bjRONXFZVk5YRkNjcWJSWjIwZURvNmhTRWNiM2tGOWtyaGFSd0hp?= =?utf-8?B?UmdqdTAya0wzVUxpWnRGRG55RjhPcFY5bFZqZ01adXVWT2JEMHhJK2xNeXQ4?= =?utf-8?B?Q3liN0h4dWdRbTVwQ1J2cDRVREtDeXptaWsrME9xVUNtS2JEWW54NTdhb0V0?= =?utf-8?B?eUpoZkNYMlJaYm96REtwbldzOGJVUEI5ck1CaUJoK0lIdzRHd3A5bHBBUEtH?= =?utf-8?B?UHZYWXBncy9WTmQyTDJuK0w5V2dwaXJmZ3NzQmlERk9VbnNoK3J2S040QmZ2?= =?utf-8?B?UTRYT2R4aVBxQ0JtUTlRZ2p5OS9NRW5DR1BoeFhsNkNMaDJXeHBlUzZseTQ4?= =?utf-8?B?cmZJZ25BeVFSOUl4Rlp3YUpLeHFxbUlhNXFlSktHZ1NQRjlkTnNXY3dVS1ll?= =?utf-8?B?eGxvYnkwSm5zOFV0cS9qQVlGR1AvRjJXYW1QZGZRV2dRZHRmZnA3RVcrTkdG?= =?utf-8?B?emdVTk53MVF1bXV6eEoxTGRySmxrZklSQnF3YnNjRlJSeDlRaVNkUDRrL3B6?= =?utf-8?Q?0iEIURuuch9D/eCKvTqVYBhC0=3D?= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QmsrYVZrc1JyTklyaXFveld2cVRlSzJKbVQ4bmI1QXNPekgvdTh5RlNNOEtz?= =?utf-8?B?Rk1RNVJDUkdpT3IxajF4UW9LaXpkdVdNdjdqbjVNVFQ3RDRJM3M3NmNBekxj?= =?utf-8?B?RHpBL0VKUjBpa2RQNkY3SWt5MDJrdTRkMEVJSExSZVBRR3RyYlRhNmNDOXJy?= =?utf-8?B?djh4UXVhcmlQREJ4UU1vbDJZc0o0eGl3VmlSMU56dHY2ZVZIZkowY2ZQcnY1?= =?utf-8?B?NWEzWVFLdTg0TmZGNU03bHlyaHpJM0QrZ0ZLWUVxV1pmckFNTDlPeXRTVFN6?= =?utf-8?B?MGdWRFRTaUUwQWw1T3J4em1BWU8yVjdxVENXbDBoVmRTK2JBVzJlbEs2NVBT?= =?utf-8?B?T0llUzl6Q2xMaUg0aVJNb2Mzc016RG9TT2FhWDl6SmJvM0VmMExsRXBoTGdk?= =?utf-8?B?UjlZYTExd3Q1ekFYd2NvQ3UwK1U4OGtmcDNTOURRSWVYYVlYQ3NvUUcvMGsx?= =?utf-8?B?dko1bEh4NXB1SC85VWNVeXZlbk1iSEdmZjlqdDlPc2xwOFVvZjVMK0lHMktJ?= =?utf-8?B?MmNDWGNBN2EyL3c0ZFg5YTlvZlNOODNDY1htZ2xYK0Z3bFZKZDczNG1xSEN1?= =?utf-8?B?Z2FRcGQrckt3TlI3c0gxYTMzTnlTM0lxZzczT3kwK2NmR0RiYTBWY0xVY3pm?= =?utf-8?B?UEFTdTNka045ckE4S3MxdWw5cERib01VWkZ0MFZhSGpKVXBheWprR1pWYTQx?= =?utf-8?B?Q3h3T1RDaE9tc2xWMGZmSmtQQ2M0YjVJcTBtKzJjejN0d09XUEVUVzFNMm9S?= =?utf-8?B?TG1hMUQ5dU51WUtzSDc0dlN1SVNSekpoMUM3NnNGTWE2bXEwNEFTZXh2NXNt?= =?utf-8?B?U3Q4bzRkb3JNaUFiZ2ZLZjZxMWFtclJ6dFM4Y1dXQXJac0p5S2dndTBNY2pV?= =?utf-8?B?dHA3d3lOZXRzNUdsYVA0L29Zd0FmbUkrcnpUT2RuaVFiZFBITDYxakxBeWNT?= =?utf-8?B?TENGbHc5NklKTnpSQlNWMnZkTCtkZHFISnN2VkpvSEU1eXB4NkdyMUNDQVYy?= =?utf-8?B?RmJvb293MEhQUUFQZkMyWCswL2QvSk8wY0ZjWkdpZ1AyV3JaQ0NzbWtvOUU1?= =?utf-8?B?bWs3Rk9OMCs3cTg2M3VQTllOem9hRnlOVXU3bE9QS0NNSjFIZzRiRHNnYmZw?= =?utf-8?B?Q0E3TzJERXNHSyt3d3BvRFp1cldLMHJWZFh2MDAzTlpHaFdqaHVrTW1uSUFM?= =?utf-8?B?TTUxalZrbld3QUJqUU1WamVJNmxDR20xb2FnSTh1SzkxaEsrdnBHdWxuMkdI?= =?utf-8?B?Q0gvbTJDeFJzRmw4eWQra0UweGo5QlZ2di85ODBrdVh0UnpteWxHNWlFNFcw?= =?utf-8?B?NDkvZTliNWdITkVzUXJNUUMyUzBMU3d3Z3F5Z3gyU1d2VE9KcEZaUkFhZCtI?= =?utf-8?B?K1pMM2hSYWhCOUhiWmZ5UThnNGJWV0xkVWZRQVlwS2pBbk90ZGNwZHJMOS9J?= =?utf-8?B?dDVqWU52ZkgwN0tOdW5ZNktBMS8yUUM2R1dhMGd1Yk55OG9OZDUwcUJrSjY2?= =?utf-8?B?aFpmQmdrZmhOeGI0THhRazhqbzE2NXJVOE1mZlR3ZGpQa1QxYzY4TDJsWnha?= =?utf-8?B?dDlzZnJtK1VDam1WZHVqTkxKQWswL1R1RHFDajJGTlhNVGprSG9Rbmp4UHBJ?= =?utf-8?B?LzFiQmQ3UzRpNy94ODc0czF0N3pJK3h6aEovN1BnWDN0TEkveFZHaXBUNGJ0?= =?utf-8?B?aWIrVXNYZGxEdmF2S2gvbVFaeDhNcmJBRGtXbmN0V0RTUVl4ZGJlaVJiT2ly?= =?utf-8?B?K2xwd2VhTzEycDVJMjZRNGRjbnQ5blVDYWI4M2N1N01WalNkMXorYUdUTjFB?= =?utf-8?B?UFZndGdIRlN4NFZ0NTdCZz09?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 82b9f891-f0de-4e7e-1613-08dd80c65e4a X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Apr 2025 11:19:23.4296 (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: DU0P250MB0818 Subject: Re: [FFmpeg-devel] [PATCH v3 3/3] lavc/dxvenc: improve compatibility with Resolume products X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org> List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe> List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel> List-Post: <mailto:ffmpeg-devel@ffmpeg.org> List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help> List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe> Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org> Archived-At: <https://master.gitmailbox.com/ffmpegdev/GV1P250MB07370626849AC5D7858CC85B8FB82@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM/> List-Archive: <https://master.gitmailbox.com/ffmpegdev/> List-Post: <mailto:ffmpegdev@gitmailbox.com> Emma Worley: > Improves compatibility with Resolume products by adding an additional hashtable > for DXT color+LUT combinations, and padding the DXT texture to the next largest > 16x16 multiple. Produces identical packets to Resolume Alley in manual tests. > Are the files that this encoder currently produces invalid? > Signed-off-by: Emma Worley <emma@emma.gg> > --- > libavcodec/dxvenc.c | 105 ++++++++++++++++++++---------------- > tests/ref/fate/dxv3enc-dxt1 | 2 +- > 2 files changed, 60 insertions(+), 47 deletions(-) > > diff --git a/libavcodec/dxvenc.c b/libavcodec/dxvenc.c > index c64b4d5f15..737ccfff3e 100644 > --- a/libavcodec/dxvenc.c > +++ b/libavcodec/dxvenc.c > @@ -34,6 +34,11 @@ > > #define DXV_HEADER_LENGTH 12 > > +/* > + * Resolume will refuse to display frames that are not padded to 16x16 pixels. > + */ > +#define DXV_ALIGN(x) FFALIGN(x, 16) > + > /* > * DXV uses LZ-like back-references to avoid copying words that have already > * appeared in the decompressed stream. Using a simple hash table (HT) > @@ -60,6 +65,7 @@ typedef struct DXVEncContext { > > struct AVHashtableContext *color_ht; > struct AVHashtableContext *lut_ht; > + struct AVHashtableContext *combo_ht; > } DXVEncContext; > > /* Converts an index offset value to a 2-bit opcode and pushes it to a stream. > @@ -94,10 +100,14 @@ static int dxv_compress_dxt1(AVCodecContext *avctx) > DXVEncContext *ctx = avctx->priv_data; > PutByteContext *pbc = &ctx->pbc; > void *value; > - uint32_t color, lut, idx, color_idx, lut_idx, prev_pos, state = 16, pos = 0, op = 0; > + uint64_t combo; > + uint32_t color, lut, idx, combo_idx, prev_pos, old_pos, state = 16, pos = 0, op = 0; > > av_hashtable_clear(ctx->color_ht); > av_hashtable_clear(ctx->lut_ht); > + av_hashtable_clear(ctx->combo_ht); > + > + av_hashtable_set(ctx->combo_ht, ctx->tex_data, &pos); > > bytestream2_put_le32(pbc, AV_RL32(ctx->tex_data)); > av_hashtable_set(ctx->color_ht, ctx->tex_data, &pos); > @@ -107,51 +117,46 @@ static int dxv_compress_dxt1(AVCodecContext *avctx) > pos++; > > while (pos + 2 <= ctx->tex_size / 4) { > - idx = 0; > - color_idx = 0; > - lut_idx = 0; > + combo = AV_RL64(ctx->tex_data + pos * 4); > + combo_idx = av_hashtable_get(ctx->combo_ht, &combo, &prev_pos) ? pos - prev_pos : 0; > + idx = combo_idx; > + PUSH_OP(2); > + if (pos >= LOOKBACK_WORDS) { > + old_pos = pos - LOOKBACK_WORDS; > + if (av_hashtable_get(ctx->combo_ht, ctx->tex_data + old_pos * 4, &prev_pos) && prev_pos <= old_pos) > + av_hashtable_delete(ctx->combo_ht, ctx->tex_data + old_pos * 4); > + } > + av_hashtable_set(ctx->combo_ht, &combo, &pos); > > color = AV_RL32(ctx->tex_data + pos * 4); > - if (av_hashtable_get(ctx->color_ht, &color, &prev_pos)) > - color_idx = pos - prev_pos; > - av_hashtable_set(ctx->color_ht, &color, &pos); > - > + if (!combo_idx) { > + idx = av_hashtable_get(ctx->color_ht, &color, &prev_pos) ? pos - prev_pos : 0; > + PUSH_OP(2); > + if (!idx) > + bytestream2_put_le32(pbc, color); > + } > if (pos >= LOOKBACK_WORDS) { > - uint32_t old_pos = pos - LOOKBACK_WORDS; > + old_pos = pos - LOOKBACK_WORDS; > if (av_hashtable_get(ctx->color_ht, ctx->tex_data + old_pos * 4, &prev_pos) && prev_pos <= old_pos) > av_hashtable_delete(ctx->color_ht, ctx->tex_data + old_pos * 4); > } > + av_hashtable_set(ctx->color_ht, &color, &pos); > pos++; > > lut = AV_RL32(ctx->tex_data + pos * 4); > - if (color_idx && lut == AV_RL32(ctx->tex_data + (pos - color_idx) * 4)) { > - idx = color_idx; > - } else { > - idx = 0; > - if (av_hashtable_get(ctx->lut_ht, &lut, &prev_pos)) > - lut_idx = pos - prev_pos; > - av_hashtable_set(ctx->lut_ht, &lut, &pos); > + if (!combo_idx) { > + idx = av_hashtable_get(ctx->lut_ht, &lut, &prev_pos) ? pos - prev_pos : 0; > + PUSH_OP(2); > + if (!idx) > + bytestream2_put_le32(pbc, lut); > } > if (pos >= LOOKBACK_WORDS) { > - uint32_t old_pos = pos - LOOKBACK_WORDS; > + old_pos = pos - LOOKBACK_WORDS; > if (av_hashtable_get(ctx->lut_ht, ctx->tex_data + old_pos * 4, &prev_pos) && prev_pos <= old_pos) > av_hashtable_delete(ctx->lut_ht, ctx->tex_data + old_pos * 4); > } > + av_hashtable_set(ctx->lut_ht, &lut, &pos); > pos++; > - > - PUSH_OP(2); > - > - if (!idx) { > - idx = color_idx; > - PUSH_OP(2); > - if (!idx) > - bytestream2_put_le32(pbc, color); > - > - idx = lut_idx; > - PUSH_OP(2); > - if (!idx) > - bytestream2_put_le32(pbc, lut); > - } > } > > return 0; > @@ -172,11 +177,23 @@ static int dxv_encode(AVCodecContext *avctx, AVPacket *pkt, > return ret; > > if (ctx->enc.tex_funct) { > + uint8_t *dst_data[4] = {av_mallocz(DXV_ALIGN(avctx->width) * DXV_ALIGN(avctx->height) * 4), 0, 0, 0}; > + int dst_linesize[4] = {DXV_ALIGN(avctx->width) * 4, 0, 0, 0}; > + > + av_image_copy2( > + dst_data, > + dst_linesize, > + frame->data, > + frame->linesize, > + frame->format, > + avctx->width, > + avctx->height); 1. Unchecked allocation. 2. The buffer leaks unconditionally. 3. The copy is definitely unnecessary if the input is already properly aligned. 4. Can't we avoid the copy in case the input is not properly aligned? 5. Most of dst_data will be initially zeroed only to be overwritten with the final value again. This is wasteful. > + > ctx->enc.tex_data.out = ctx->tex_data; > - ctx->enc.frame_data.in = frame->data[0]; > - ctx->enc.stride = frame->linesize[0]; > - ctx->enc.width = avctx->width; > - ctx->enc.height = avctx->height; > + ctx->enc.frame_data.in = dst_data[0]; > + ctx->enc.stride = dst_linesize[0]; > + ctx->enc.width = DXV_ALIGN(avctx->width); > + ctx->enc.height = DXV_ALIGN(avctx->height); > ff_texturedsp_exec_compress_threads(avctx, &ctx->enc); > } else { > /* unimplemented: YCoCg formats */ > @@ -216,14 +233,6 @@ static av_cold int dxv_init(AVCodecContext *avctx) > return ret; > } > > - if (avctx->width % TEXTURE_BLOCK_W || avctx->height % TEXTURE_BLOCK_H) { > - av_log(avctx, > - AV_LOG_ERROR, > - "Video size %dx%d is not multiple of "AV_STRINGIFY(TEXTURE_BLOCK_W)"x"AV_STRINGIFY(TEXTURE_BLOCK_H)".\n", > - avctx->width, avctx->height); > - return AVERROR_INVALIDDATA; > - } > - > ff_texturedspenc_init(&texdsp); > > switch (ctx->tex_fmt) { > @@ -237,10 +246,10 @@ static av_cold int dxv_init(AVCodecContext *avctx) > return AVERROR_INVALIDDATA; > } > ctx->enc.raw_ratio = 16; > - ctx->tex_size = avctx->width / TEXTURE_BLOCK_W * > - avctx->height / TEXTURE_BLOCK_H * > + ctx->tex_size = DXV_ALIGN(avctx->width) / TEXTURE_BLOCK_W * > + DXV_ALIGN(avctx->height) / TEXTURE_BLOCK_H * > ctx->enc.tex_ratio; > - ctx->enc.slice_count = av_clip(avctx->thread_count, 1, avctx->height / TEXTURE_BLOCK_H); > + ctx->enc.slice_count = av_clip(avctx->thread_count, 1, DXV_ALIGN(avctx->height) / TEXTURE_BLOCK_H); > > ctx->tex_data = av_malloc(ctx->tex_size); > if (!ctx->tex_data) { > @@ -251,6 +260,9 @@ static av_cold int dxv_init(AVCodecContext *avctx) > if (ret < 0) > return ret; > ret = av_hashtable_alloc(&ctx->lut_ht, sizeof(uint32_t), sizeof(uint32_t), LOOKBACK_HT_ELEMS); > + if (ret < 0) > + return ret; > + ret = av_hashtable_alloc(&ctx->combo_ht, sizeof(uint64_t), sizeof(uint32_t), LOOKBACK_HT_ELEMS); > if (ret < 0) > return ret; > > @@ -265,6 +277,7 @@ static av_cold int dxv_close(AVCodecContext *avctx) > > av_hashtable_freep(&ctx->color_ht); > av_hashtable_freep(&ctx->lut_ht); > + av_hashtable_freep(&ctx->combo_ht); > > return 0; > } > diff --git a/tests/ref/fate/dxv3enc-dxt1 b/tests/ref/fate/dxv3enc-dxt1 > index 74849a8031..e09000e181 100644 > --- a/tests/ref/fate/dxv3enc-dxt1 > +++ b/tests/ref/fate/dxv3enc-dxt1 > @@ -3,4 +3,4 @@ > #codec_id 0: dxv > #dimensions 0: 1920x1080 > #sar 0: 1/1 > -0, 0, 0, 1, 76521, 0xed387a5e > +0, 0, 0, 1, 76190, 0x0e6f0326 Is padding the reason for this change or is it the new hashtable? Or both? - 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".