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".