On 11/26/2010 10:45 AM, Bruce Dubbs wrote:
> Matthew Burgess wrote:
>> On Wed, 24 Nov 2010 23:47:41 -0600, DJ Lucas <d...@linuxfromscratch.org>
>> wrote:
>>
>>> Can somebody else on a mult-core CPU, on 32-bit, confirm before I
>>> troubleshoot further?
>>
>> This is also being tracked upstream and at RedHat:
>>
>> http://lists.gnu.org/archive/html/coreutils/2010-11/msg00083.html
>> https://bugzilla.redhat.com/show_bug.cgi?id=655096
>
Cool..so I'm not insane.
> Should we revert to .6 until this is fixed?
>
Either that or apply a really cheezy hack to disable multi-threading.
:-) I'm sure there is a better test, but this was quick and easy.
--- coreutils-8.7/src/sort.c 2010-10-25 10:07:57.000000000 +0000
+++ coreutils-8.7-new/src/sort.c 2010-11-26 17:38:53.000000000 +0000
@@ -4534,6 +4534,11 @@
if (!nthreads || nthreads > np2)
nthreads = np2;
+ /* Threading is currently broken in multibyte locales, so disable it
+ temporarily if not C. I'm sure there is a better test for
this... */
+ if (getenv ("LC_COLLATE") != "C")
+ nthreads = 1;
+
sort (files, nfiles, outfile, nthreads);
}
-- DJ Lucas
--
This message has been scanned for viruses and
dangerous content, and is believed to be clean.
--
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page
