From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Bavier Subject: Re: 01/01: gnu: fftw: Build SIMD codelets. Date: Wed, 9 May 2018 02:29:48 +0000 Message-ID: References: <20180417212549.1283.62731@vcs0.savannah.gnu.org> <20180417212551.BB270208E4@vcs0.savannah.gnu.org> <87efisjp91.fsf@netris.org> <87r2mlkgru.fsf@netris.org> , <87h8nhk9q7.fsf@netris.org> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:46948) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fGErd-00047E-Ce for guix-devel@gnu.org; Tue, 08 May 2018 22:29:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fGEra-0006hF-6S for guix-devel@gnu.org; Tue, 08 May 2018 22:29:57 -0400 Received: from esa2.cray.iphmx.com ([68.232.143.164]:50426) by eggs.gnu.org with esmtps (TLS1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.71) (envelope-from ) id 1fGErZ-0006fK-O0 for guix-devel@gnu.org; Tue, 08 May 2018 22:29:54 -0400 In-Reply-To: <87h8nhk9q7.fsf@netris.org> Content-Language: en-US List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Mark H Weaver Cc: "guix-devel@gnu.org" Hello Mark, No problem. I don't mind some parallel work that results in more confidenc= e in the changes. I saw your commit 69d5909, which looks good to me (you even untangled my na= sty quoting, which I often can't seem to get right). Sorry for the slow response, and thanks for finding the issue in the first = place. Eric Bavier, Scientific Libraries, Cray Inc. ________________________________________ From: Mark H Weaver Sent: Tuesday, May 8, 2018 18:29 To: Eric Bavier Cc: guix-devel@gnu.org Subject: Re: 01/01: gnu: fftw: Build SIMD codelets. Hi Eric, Eric Bavier writes: > Ok, if you feel confident, that's fine with me. > > Otherwise, I have a patch that I've tested on x86-64, and in the > middle of testing for armhf that I can push in a few hours. I could > easily add a commit that remove the "--enable-armv7a-cntvct" flag. Sorry for the duplicate work, but I already pushed my fixes in commit 69d5909e032e2fba57814ea9db52389d384d9341 to core-updates. I tested them on x86_64, but nowhere else. It's not ideal, and I certainly wouldn't do such a thing on 'master', but given the large number of affected builds and my uncertainty about when you would fix it, I didn't want to wait any longer. FYI, I also removed "--enable-armv8-cntvct-el0" on 64-bit ARM, because suggests that as on 32-bit ARM, the relevant instruction is not normally available from user mode. I removed "--enable-mips-zbus-timer" on MIPS as well, because I suspect that it's not supported on Loongson devices, although I'm not certain. If you see any problems with my commit, please let me know. Thanks! Mark