Date: Tue, 4 Oct 2011 10:09:07 +0200 From: Maxime Henrion <mux@FreeBSD.org> To: Adrian Chadd <adrian@freebsd.org> Cc: freebsd-current@freebsd.org, Alexander Zagrebin <alexz@visp.ru>, Jilles Tjoelker <jilles@stack.nl> Subject: Re: cvsup broken on amd64? Message-ID: <CAM=sVkc0=k7VFXo2-NF6ya4jxu3MTL=_9D5szOKfv2qOofOiqA@mail.gmail.com> In-Reply-To: <CAJ-VmomSjm7xHUvd=Sr7UhA=N=h2_TP7CC1SuRuEoBZ2grE8Bw@mail.gmail.com> References: <20110908221356.Horde.MFEsZ6Qd9PdOaSIEaid2X_A@avocado.salatschuessel.net> <4E692F87.5010708@sentex.net> <20110909073305.Horde.oi-EGaQd9PdOaaURAsTRVJk@avocado.salatschuessel.net> <4E6A076D.7040309@wintek.com> <EA16DA2E96DD411C8C8A7EC81E034162@vosz.local> <CAJ-VmonzwUVEHgzOU1QAY7hGKwLn6Dx2RxTaE4bEn0m4eoee3w@mail.gmail.com> <EBF2A1324C994063B9EAD7090563EC36@vosz.local> <CAJ-VmokwiVYHDymaOVcKQEbJ_ohjN8XOa8Q11e5GQ-mf1Hqt7g@mail.gmail.com> <CAM=sVkdXy7p52Skud1D1MyhcJxPWVkLzMihCegGC72BvH5zq5Q@mail.gmail.com> <20111003213024.GA42715@stack.nl> <CAM=sVkfSJ-KaVwRTjQ1711d=gJ0evDUEz2A6Yy8e4Wrma7zy-A@mail.gmail.com> <CAJ-VmomSjm7xHUvd=Sr7UhA=N=h2_TP7CC1SuRuEoBZ2grE8Bw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Oct 4, 2011 at 2:19 AM, Adrian Chadd <adrian@freebsd.org> wrote: > On 4 October 2011 05:53, Maxime Henrion <mux@freebsd.org> wrote: > >> Great, that's a relief. I knew the pthread library was free to wake a >> thread up even if it hadn't been signaled, which is why one always has >> to call pthread_cond_wait() inside of a while() loop checking for the >> condition, but wasn't sure about that particular scenario, so I'm glad >> to hear a confirmation. Thanks! > > So shall I commit your change, if someone hasn't already? That would be great (I cannot commit it myself anymore). If you could also fix the misleading comment I've been talking about (s/lister thread/detailer thread/), it would be perfect. Thanks, Maxime
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAM=sVkc0=k7VFXo2-NF6ya4jxu3MTL=_9D5szOKfv2qOofOiqA>