GNU bug report logs - #29810
gnu: maths: Fix cache size detected by openblas on some

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 29810 in the body.
You can then email your comments to 29810 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Dave Love <fx <at> gnu.org>
To: guix-patches <at> gnu.org
Subject: gnu: maths: Fix cache size detected by openblas on some
Date: Fri, 22 Dec 2017 13:13:29 +0000

[Message part 1 (text/plain, inline)]

This addresses a potential performance problem, fixed in the post-0.2.20
source.  It's intended for application to a package definition updated
to 0.2.20, which Ludo said is in the pipeline.  Apologies that I don't
seem to have converged on an acceptable style for changes.

[0001-gnu-maths-Fix-cache-size-detected-by-openblas-on-som.patch (text/x-diff, attachment)]

Message #8 received at 29810 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Dave Love <fx <at> gnu.org>
Cc: 29810 <at> debbugs.gnu.org
Subject: Re: [bug#29810] gnu: maths: Fix cache size detected by openblas on
 some
Date: Mon, 08 Jan 2018 10:43:45 +0100

Hi Dave,

Dave Love <fx <at> gnu.org> skribis:

> This addresses a potential performance problem, fixed in the post-0.2.20
> source.  It's intended for application to a package definition updated
> to 0.2.20, which Ludo said is in the pipeline.  Apologies that I don't
> seem to have converged on an acceptable style for changes.
>
>>From 23ad3a438ef7bcd34e2354f6cbdede634f0188d4 Mon Sep 17 00:00:00 2001
> From: Dave Love <fx <at> gnu.org>
> Date: Fri, 22 Dec 2017 12:48:29 +0000
> Subject: [PATCH 1/2] gnu: maths: Fix cache size detected by openblas on some
>  x86_64.
>
> * gnu/packages/patches/openblas-Add-dummy-implementation-of-cpuid_count-for-the-CPUI.patch,
> gnu/packages/patches/openblas-Use-cpuid-4-with-subleafs-to-query-L1-cache-size-on-.patch:
> New files.
> * gnu/packages/maths.scm (openblas)[source]: Use them.
> * gnu/local.mk: Register them.

Thanks for the patch.  Given the number of dependents, we would not push
it in master (info "(guix) Submitting Patches").  At the same time,
since 0.2.20 is in core-updates and well on its way, do you think we
should keep those patches?

Perhaps in core-updates we could keep both 0.2.19 with these patches and
0.2.20 (ISTR you said there were incompatibilities between these two
versions)?  Would it make sense?

Thanks,
Ludo’.

Message #15 received at 29810-done <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
Cc: Dave Love <fx <at> gnu.org>, 29810-done <at> debbugs.gnu.org
Subject: Re: [bug#29810] gnu: maths: Fix cache size detected by openblas on
 some
Date: Tue, 12 Feb 2019 18:21:48 -0500

[Message part 1 (text/plain, inline)]

On Mon, Jan 08, 2018 at 10:43:45AM +0100, Ludovic Courtès wrote:
> Thanks for the patch.  Given the number of dependents, we would not push
> it in master (info "(guix) Submitting Patches").  At the same time,
> since 0.2.20 is in core-updates and well on its way, do you think we
> should keep those patches?
> 
> Perhaps in core-updates we could keep both 0.2.19 with these patches and
> 0.2.20 (ISTR you said there were incompatibilities between these two
> versions)?  Would it make sense?

Our openblas package is currently at version 0.3.4. Due to lack of
response from the submitter and the "staleness" of these patches I'm
closing the bug ticket. Please re-open the bug if it is still relevant
to you.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.