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 C34ED44994 for ; Wed, 28 Sep 2022 12:52:59 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id E40B668BCC6; Wed, 28 Sep 2022 15:52:56 +0300 (EEST) Received: from hekla.abc.se (hekla.abc.se [158.174.61.227]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 72C4B68BC59 for ; Wed, 28 Sep 2022 15:52:49 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; s=default; bh=ydghQOYE+p Ku31ddZIRFSOZugup4so1zkzodmhiMJjs=; h=references:in-reply-to:subject: to:from:date; d=abc.se; b=Twj6+VuXpWGaZ9sR6xCgqTVEd5tTYSzuFgzwMNAGM9gh ss2x5d/QJ0bK2trcn1q8g9uL8+ztQth7DizR99K41bRhEKG6D6GY0smLeKtWDKyzElNeNG 3A4aSwifYeSbKNYJFZr0OmpEtyc+KnU/m2wQLfVpkVIoM6po869VRNwRe19+br0y0GEK5T Eb9aB9LtHnvGo9dlqbBX2yuLJuWg4s+pgHuGNc7UCpXNAL5anwk8UkYEHeJ3VOxNyPEmWb IrqBY5du08OfSG6u54bsVwPVhda0GWxwa4rW9lompf1tjneFTO/x24Ds9Nczv0P3zlg/9S Xa+jtxaNu1OST1exvw== Received: from perkele.intern.softwolves.pp.se (cm-84.209.5.132.get.no [84.209.5.132]) by hekla.abc.se (OpenSMTPD) with ESMTPSA id c13a0f8e (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO) for ; Wed, 28 Sep 2022 14:52:47 +0200 (CEST) Date: Wed, 28 Sep 2022 13:52:46 +0100 (CET) From: Peter Krefting To: FFmpeg development discussions and patches In-Reply-To: <1EC39695-B7C3-4EF3-A2F9-D60280B14EE5@remlab.net> Message-ID: <6d864071-af72-e562-65c6-be714a4ca3@softwolves.pp.se> References: <74f9b1f3-698d-7a3d-1a5-1b43f88ea9f2@softwolves.pp.se> <1EC39695-B7C3-4EF3-A2F9-D60280B14EE5@remlab.net> X-Warning: Junk / bulk email will be reported X-Rating: This message is not to be eaten by humans Organization: /universe/earth/europe/norway/oslo MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH] configure: Remove dcbzl check for e500v1 and e500v2 architectures 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: Hi! >> The DCBZL instruction is not available for the e500v1 and e500v2 >> architectures, but may still be recognized by the toolchain, so we need to >> remove the test for it explicitly for these architectures. > Isn't this the sort of thing that's supposed ti be guarded by run-time CPU flags rather than in the configure script? Our compiler (powerpc-btech-linux-gnuspe-gcc (crosstool-NG 1.24.0) 8.3.0) recognizes the instruction, so the configure test succeeds, but the CPU (e500v2) crashes if it tries to execute it. I previously had a patch (d5733936d857ce5c7d28c0bc9e89a2e2548f8895) to suppress the instruction, but it broke at some point, this patch tries to fix it in a slightly better way. Having said that, the test is there due to the fix in a4adb60858f1fa0b35b08576ea34e531f0f83459 (from 2003), and disabling the instruction does not bring back the old optimizations as it just expects it not to work at all. But for our purposes this is not as important as h aving a working library. -- \\// Peter - http://www.softwolves.pp.se/ _______________________________________________ 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".