From owner-svn-src-head@FreeBSD.ORG Thu Aug 2 14:17:49 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id A4BFD106566B; Thu, 2 Aug 2012 14:17:49 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail08.syd.optusnet.com.au (mail08.syd.optusnet.com.au [211.29.132.189]) by mx1.freebsd.org (Postfix) with ESMTP id 327F48FC08; Thu, 2 Aug 2012 14:17:48 +0000 (UTC) Received: from c122-106-171-246.carlnfd1.nsw.optusnet.com.au (c122-106-171-246.carlnfd1.nsw.optusnet.com.au [122.106.171.246]) by mail08.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id q72EHkvR008441 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 3 Aug 2012 00:17:47 +1000 Date: Fri, 3 Aug 2012 00:17:46 +1000 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: davidxu@freebsd.org In-Reply-To: <501A72AE.30309@gmail.com> Message-ID: <20120803001044.X2863@besplex.bde.org> References: <201207310548.q6V5mZHf091624@svn.freebsd.org> <50179581.9070805@gmail.com> <20120802051805.P3345@besplex.bde.org> <5019ED93.8060802@gmail.com> <20120802173441.X1291@besplex.bde.org> <501A72AE.30309@gmail.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: src-committers@freebsd.org, Giovanni Trematerra , svn-src-all@freebsd.org, Konstantin Belousov , bde@freebsd.org, Bruce Evans , svn-src-head@freebsd.org Subject: Re: svn commit: r238936 - in head/sys: fs/fifofs kern sys X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Aug 2012 14:17:49 -0000 On Thu, 2 Aug 2012, David Xu wrote: > On 2012/8/2 16:12, Bruce Evans wrote: >> ... >> I made similar patches for sockets (to set POLLHUP on hangup (now in >> -current) and to not set POLLIN on hangup unless there is still data >> to be read). I started killing POLLINIGNEOF for sockets. -current >> added it for nameless pipes instead :-(. With the new fifo >> implementation, POLLINIGNEOF is even more of a mistake for sockets, >> but more needed for pipes since named pipes are fifos. >> ... > I think you can kill POLLINIGNEOF at all, I have grepped, and there is no > external user, > only pipe and socket code use it internally. The POLLINIGNEOF is confusing > because > it has same prefix with POLLIN, POLLOUT and other POLL flags. Did you grep all of google for it :-). All of ports should be enough. Uses of it in the kernel are certainly gone, but it was intentionally put in the user API to previde a workaround for the policy that was hard-coded in the kernel. The policy changed slightly, and you could set POLLINIGNEOF to either go back to the old policy or get the new policy for more cases. Hopefully this was never actually used except for testing and termporary workarounds. But it was expanded to work on nameless pipes as well as fifos and sockets. Bruce