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 AE5D140681 for ; Thu, 28 Apr 2022 13:24:59 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 0C68868B3D7; Thu, 28 Apr 2022 16:24:58 +0300 (EEST) Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10olkn2093.outbound.protection.outlook.com [40.92.40.93]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id DA97368B3CA for ; Thu, 28 Apr 2022 16:24:55 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NtJ2I5qumLuEwPaPqiAyKfav7OT60KdznrWnw1aWJMqLbMHAWtSf12RJnFD3LzDjFh8Ayf2dF3We100A36HrsCOL4/AYor29+NXpNBUfIXlPwJbVzTQa7Vopc+k8v7SuAJ/wnpxkshxB1NsAENC5elOijTHqOGBHgCe1WOq3wp3VynJw6D4N/VkPssW2Z4x1/nm2V90DTzkhuBhMKi4fM3l2HlSyqQ+786mDKd86jnqgKZnzDrTLUKTcFB46fXb3COvb5P9v/3ECajkK32RHf3qPcdWzjVoQWjqqpOskT4hUZCZ1HYTpniPPIh3K4rIDAKJ44n8LwhrtgMpTQL8XCg== 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=VLLF2qwKgS+91yOzs98PwlFKoQZzOIY6QVYQu4eK9I0=; b=Kj0+7d7tEpWLXp59lq8BSKyfOnI8+apgRIUNbKdRRmJZr19c1Sbap7lBAi7iVHpCwwCtI87gXXaZylob0U4N/hRu8RsXT1iLtCjG8ZMyrtMBHEeXwRYqPFkJJ6wwx8XfDBd3zynaG0+P9gD2gLKZ24xHMFIElczXeg3JRJjDl532o7+SFcCD1y5VLxX2cH+u4Jo37he3M4dopO97webQSGa9tNbF/HVO2Tcyz36FjaJoHrBaZA05yImEbBUOTKv2xT9TN1HFCmHDx5ht0uxx1BJiCwWrcR2GY5HxsJWoW8CrRzOvFfRBNnPKMdbJ9JvAARAp66Ks2/zg7Xx2Z9wrlg== 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=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VLLF2qwKgS+91yOzs98PwlFKoQZzOIY6QVYQu4eK9I0=; b=F8nURD9k/p+m1vUSxEo5iCyNJcyio9FwSH32TKD97KMfI2qZQ8psETuUmGT7fNm/Qp7LIAAHEbw59jNQ/CR3xl/6oM2hTA6PWHV0Dy5vpbW1c3dQG2AeO3POs9Gd20i8w3hEsnIH6ezoAqmbJHOvMwyRUpThdGzRTvM5ehCeniL/eMGPib7e3fbXZ0qfcJZPBzC/tBcefvEe6L9mZpZnNwYE9/CyWYpeS/Gwv8TPtY/GqXtJh3Ch+Pp8wKOrbqmqgU2gF5BnE7KE7kK3vzu96s5ZySlA/f7YO/FaEr4CObWAQ33wUOS8xgOrqjnlLK/7h9XOMz95Ah5qU1AezaHTbg== Received: from MN2PR18MB3328.namprd18.prod.outlook.com (2603:10b6:208:165::16) by PH0PR18MB4104.namprd18.prod.outlook.com (2603:10b6:510:2d::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5206.13; Thu, 28 Apr 2022 13:24:53 +0000 Received: from MN2PR18MB3328.namprd18.prod.outlook.com ([fe80::46e:9af4:d749:f44]) by MN2PR18MB3328.namprd18.prod.outlook.com ([fe80::46e:9af4:d749:f44%5]) with mapi id 15.20.5206.013; Thu, 28 Apr 2022 13:24:53 +0000 Message-ID: Date: Thu, 28 Apr 2022 09:24:51 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 From: Felix LeClair To: ffmpeg-devel@ffmpeg.org Content-Language: en-US X-TMN: [/eCaTKCbuQ8AOLvrB+E90r8uIdY/ITJ3] X-ClientProxiedBy: YT3PR01CA0063.CANPRD01.PROD.OUTLOOK.COM (2603:10b6:b01:84::28) To MN2PR18MB3328.namprd18.prod.outlook.com (2603:10b6:208:165::16) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 597026d3-a812-492a-7d67-08da291a7a6d X-MS-TrafficTypeDiagnostic: PH0PR18MB4104:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Ucc3gTFEfZtTKbcG+vQsZKtbM7DkMMLIKiMoVIiuQUwIk8bcaXSe/6zYu5u+6u+130h7rEInD6EiC2DquJw4ZhelqjPuxRZiKu5AlXgawKl0s+7InRE+UORRjEuNTnqEYxUz0NlRlGJdgvxucD+1c8u6xrJ0XWbfi6UEP5nSMglnVrz6hWrvhcPfVO19vuZQGLQzuPv9XtMUBVQNpXQSMwLn8/dl0H+6RgOn/1WO9QTBVAhlzanut48D2ilU20o4I+QBaNade2f8uViLft/AikVoBt7GHWoCRW30HyVkOdWZ7ByAGTr5CEeIOsywrY8Le7yCkkEWipKKWGBHQWFH8nGC3zO8n3ZNdTFzJDjIGqvpZsYbFxYRnZeMITgabgatNebHJCzSwQwCuUeMhZSd7ww6gYumsV00S8WpvdfshHdTx3Q+A0CWzZo3aSNJmRd/8FuUoXu869vd1y9a7NF6UPLZWDa1LJoelSjyUGZ76TBkWOkUzU7VBviA3HBmUiMfCMu4582Lbz7dp7B/NvAUnQ7iPTPAudHnrDSGdO3JTE8DpQlyY6Jfe16L6SZHXDjSTmcZMi8vgKjB/wq11Z5r3Q== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S0VIc0dEcUtmKzNRbHlQSDBLY29lNjZISGZ6MEV2R2xlcUhPVDlKUG5GSEVt?= =?utf-8?B?WGt4VTBpRzU0UnUwaEhuK0FYcWVRcDQ4ZFkzRklNTjZVV3NsNG5FTkorRXFD?= =?utf-8?B?TmxaRTF2dDFXayt6aTJPWEU3NGpuanVIOEZNd0V0QmUxRkFFQXlVY0dqb0dO?= =?utf-8?B?QmVCODdqcVJJZ21CZlFUdTdmWitkeWlLT1RCaVQ5RWNvcDQ4a1FqamxJUnJJ?= =?utf-8?B?VHJyaEROUFBaK0lTQTAzSE9LZUwxSGRSVnh0UkRCeTNxNWdBMWFlMDhKcXo2?= =?utf-8?B?alBzU1doVWlqWmIvZWpwazFuaHlWbWd3cmd1cDYwQ0hsRGhlVitpaFRoZWxi?= =?utf-8?B?SjlScXBqWVNlR2Y3VkJMbnF1enV6bi9TQk94NTZYeXVheTROaDBsYkpJTjVL?= =?utf-8?B?ZTNiUVZDUkpnTDBFUlA4d0dxUG40UmttSUpaQk1NYUpwZ0Z0VTE2aHVTbHZX?= =?utf-8?B?Z2lPazhJMGtkVDlid3QzSDVPdEZ2REFuMWdXQTVxSFROV3ArbWxoRDBXN1l6?= =?utf-8?B?R1Jac3ZnbXlmYWVZdWZsUTNSTnZ3WTB5dmlGb1ZBa1JnL0FCS3M0RGxEeEph?= =?utf-8?B?Wm11NWd5T3VpTXF1NU0xVjBoVFNwK1FiWmF1cnY2Zkh1MVRkYWtSRDEzV1Jz?= =?utf-8?B?b2EzbWFHTnNya1BNOVNSM2t3clh6TXNTV0MxUkhpNUFTbnBXa051eW0zYkZC?= =?utf-8?B?ekhKRFRsOWxXdGRTV3JrVWg1RkhTMXpNUnBsZHdkbllMcUk4cHhwQjVsRXdu?= =?utf-8?B?WlkwamR4WG4wU3pYVDA4TSs1WFlDV3VJb3oyTjBsWXFmR25aNjZVVGh6TXJZ?= =?utf-8?B?WWNiWHhkUFIvaEV4Rm9VVGdEb1Juc1ZBM0s4dXlicmlqTEpDbytxM0k3eFcx?= =?utf-8?B?OTcwdmFTemtCVHlJTEZJakpKTGJQTW1tT0Y1K0dUdDZ1MEU1VzB3eGgwaTN0?= =?utf-8?B?SjNLQ1lsWmdCeDRHMUIzVGQvZmVSNTJ4ZDY5aHErZFdHbkxxK0lVbmNrd2lW?= =?utf-8?B?aU9zbm1Ub1ZiNHFxVmpGMkVtcVBYZkFYckxFQ2RReXJEekRUVlpVYmlrV0xQ?= =?utf-8?B?d2h6T2dFcUpQY0VsN1V3NlNwdWwrUW0rajY5YkhJMUhEUDU4d21ZbHJ2USs2?= =?utf-8?B?RTJVTkpLTHVSVlEreVd2UUZZNmxYTExTU1hjRzRUbTdhVEJuQXdBNEljYU44?= =?utf-8?B?ZjRZeS92amkzMWdzYmdWa1lnM3U1U0trVXJ1YjB2MFVQeStDSjNTKy9KSzl3?= =?utf-8?B?R1ZKMmZNUDJEZ3FQRnIvdUd2emJrZzkrQU9hNFVOcm12OEtGRTRtKzhTSmNu?= =?utf-8?B?MUVQSTFHN3REL1Zveis3MG8wVFI0WEpQeFFnWHhtU3ZvUXNRS1RkYkdXRWl5?= =?utf-8?B?UjlCbzJNOEdDc0JTUWpHOGc3V1JYOGdJVG1xa1RQWGFiY1Y2Z2huNlhsdENr?= =?utf-8?B?UCtCTUk0WHNBUnFlSHA2eFhENUZ3eE1GUGZEQTd6eGNEMXJiemR4Z0s1RlB0?= =?utf-8?B?cWNQRVdLU0lSb0dXbU9ya3VYc3MrVkk4YjFQaFFnZ2JZTWpRTmJKNVhrMEZu?= =?utf-8?B?YUMvVkg2Y3k0Mmt3d0ZXT3dMQkNBbVZaemwvQnZuTmpibFpWbzdNWlM4NGhp?= =?utf-8?B?WnlnMURjalUwOHB5K003YWNKYmxxUU9RRzNYTCtITWRvbnVzcjhEbDJWcCtx?= =?utf-8?B?VzVZbzdVMzNuMFE2Q01xbXNBT0hpL09CK3pqQ2V2b2ZmUzRnK2dqSHpFT3Er?= =?utf-8?B?QkxyVWs0N3ovOHpCamFhRlcxYzNtVnNVTHlKMlZNazBvOUdza2NSRHZndVJR?= =?utf-8?B?WXZFdmFNa0E5VllZQlZ4YXhNaHRiWDZvVHNpaEs0Ym1NYjZCQlE0SHVocSs0?= =?utf-8?B?TjJScW13a0ZjR1JKL3YyS2lwbWVRcDVpV2JDMWl5TVhDeURxa1pINSt3Vkc1?= =?utf-8?Q?g24yNBPgKt6CRIkSf7LbaXTH/C++AoWi?= X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-db494.templateTenant X-MS-Exchange-CrossTenant-Network-Message-Id: 597026d3-a812-492a-7d67-08da291a7a6d X-MS-Exchange-CrossTenant-AuthSource: MN2PR18MB3328.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Apr 2022 13:24:53.1699 (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: PH0PR18MB4104 Subject: [FFmpeg-devel] [PATCH-v2] Fix AVX-512-VNNI__hevc_qpel_filters_avx512icl_v_%_2 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 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Issue: On extremely new hardware using either IceLake or super sets of Intel IceLakes avx512 instructions, commit d4cd8830bdac3e26c8e75cd92e574c159fecc4f7 causes build issues. Specifically a NASM macro expansion of qpel_filter_v is never properly defined/initialized. The issue is the definition was erroneously placed inside a conditional which will not trigger unless the original definition failed (has to do with if PIC is defined, becomes a bit of a catch 22) Specifically the error is ``` X86ASM libavcodec/x86/hevc_mc.o libavcodec/x86/hevc_mc.asm:1854: error: symbol `..@88472.table' not defined libavcodec/x86/hevc_mc.asm:1806: ... from macro `HEVC_PUT_HEVC_QPEL_HV_AVX512ICL' defined here libavcodec/x86/hevc_mc.asm:1730: ... from macro `QPEL_FILTER_V' defined here ... repeats a few times... ... make: *** [ffbuild/common.mak:100: libavcodec/x86/hevc_mc.o] Error 1 ``` Specific error was discussed by kurosu and myself (fclc) on the ffmpeg-devel irc. This commit fixes the above by swapping lines 1796 and 1795, moving the define out of the conditional Side note: It seems fate didn't pick up on this, may merit looking into (as mentioned by nevcairiel). Signed-off-by: FelixCLC (FCLC) --- libavcodec/x86/hevc_mc.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/hevc_mc.asm b/libavcodec/x86/hevc_mc.asm index c07948dd9b..e159adccc7 100644 --- a/libavcodec/x86/hevc_mc.asm +++ b/libavcodec/x86/hevc_mc.asm @@ -1792,8 +1792,8 @@ cglobal hevc_put_hevc_qpel_hv%1_%2, 6, 7, 27, dst, src, srcstride, height, mx, m sub srcq, tmpq sub myq, 1 shl myq, 5 -%ifdef PIC %define %%table hevc_qpel_filters_avx512icl_v_%1 +%ifdef PIC lea tmpq, [%%table] %define FILTER tmpq %else -- 2.32.0 _______________________________________________ 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".