From owner-freebsd-stable@FreeBSD.ORG Tue Oct 4 16:55:31 2011 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CDB24106564A for ; Tue, 4 Oct 2011 16:55:31 +0000 (UTC) (envelope-from boland37@xs4all.nl) Received: from smtp-vbr2.xs4all.nl (smtp-vbr2.xs4all.nl [194.109.24.22]) by mx1.freebsd.org (Postfix) with ESMTP id 63DBD8FC08 for ; Tue, 4 Oct 2011 16:55:31 +0000 (UTC) Received: from charlemagne.boland.org (59-36-215.ftth.xms.internl.net [82.215.36.59]) (authenticated bits=0) by smtp-vbr2.xs4all.nl (8.13.8/8.13.8) with ESMTP id p94GY7Pp021085 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO) for ; Tue, 4 Oct 2011 18:34:07 +0200 (CEST) (envelope-from boland37@xs4all.nl) Message-ID: <4E8B357F.3050504@xs4all.nl> Date: Tue, 04 Oct 2011 18:34:07 +0200 From: Michiel Boland User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US; rv:1.9.2.23) Gecko/20110930 Thunderbird/3.1.15 MIME-Version: 1.0 To: freebsd-stable@freebsd.org References: <20110918045413.GA63773@DataIX.net> <20110918053901.GA31617@icarus.home.lan> <86d3eydsmf.fsf@kopusha.home.net> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: by XS4ALL Virus Scanner Subject: Re: /usr/bin/script eating 100% cpu with portupgrade and xargs X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Oct 2011 16:55:31 -0000 On 10/04/2011 13:15, Mikolaj Golub wrote: > On Sun, Sep 18, 2011 at 1:58 PM, Mikolaj Golub wrote: [...] >> >> I believe the behaviour is after this commit: >> >> http://svnweb.freebsd.org/base?view=revision&revision=125848 >> >> I think we should skip select on STDIN after reading EOF from it, like in the >> patch below. > > For the record. The issue has been fixed in CURRENT and the fix has > been merged to STABLE. > > Thanks Kostik and Chris for their comments and suggestions. > Does this mean that bin/72501 can be closed? Cheers Michiel