GNU bug report logs - #6600
[PATCH] sort: add --threads option to parallelize internal sort.

Previous Next

Full log

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

From: Paul Eggert <eggert <at> CS.UCLA.EDU>
To: Pádraig Brady <P <at> draigBrady.com>
Cc: Bug Coreutils <bug-coreutils <at> gnu.org>, Chen Guo <chenguo4 <at> yahoo.com>,
	Glen Lenker <glen.lenker <at> gmail.com>, Mike Nichols <mykphyre <at> gmail.com>,
	Gene Auyeung <quaker4lyf <at> gmail.com>, Chris Dickens <cdickens <at> ucla.edu>
Subject: Re: [PATCH] sort: add --threads option to parallelize internal sort.
Date: Fri, 09 Jul 2010 19:54:26 -0700

On 07/09/10 18:07, Pádraig Brady wrote:
> Chen Guo wrote:
>> That happened when more than one instance of memcoll is called on the same
>> line at once, since memcoll replaces the eolchar with '\0'. Under our approach,
>> the same line shouldn't ever be compared at the same time, so we're fine.

Ah, sorry, I wasn't aware of that.

> I'm thinking of dropping
> the whole xmemcoll0() thing altogether assuming your
> statement above is correct, that a particular line will
> not be used at the same time by multiple threads.

Yes, that makes sense.  We can revert that change from gnulib, since it
makes gnulib bigger unnecessarily.

Previous Next

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