From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Dreier Subject: Re: [PATCH/RFC 1/2] anon-inodes: Remove fd_install() from anon_inode_getfd() Date: Wed, 27 Feb 2008 12:14:24 -0800 Message-ID: References: <20080225191043.GA32342@lst.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org, Linux Kernel Mailing List , Avi Kivity , kvm-devel@lists.sourceforge.net, Andrew Morton , Al Viro To: Davide Libenzi Return-path: Received: from sj-iport-2.cisco.com ([171.71.176.71]:27381 "EHLO sj-iport-2.cisco.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756913AbYB0UOc (ORCPT ); Wed, 27 Feb 2008 15:14:32 -0500 In-Reply-To: (Davide Libenzi's message of "Wed, 27 Feb 2008 11:41:24 -0800 (PST)") Sender: linux-fsdevel-owner@vger.kernel.org List-ID: > > The anonymous inodes interface anon_inode_getfd() calls fd_install() > > for the newly created fd, which does not work for some use cases where > > the caller must do futher initialization before exposing the file to > > userspace. This is also probably not the safest interface, since the > > caller must be sure that it is OK if userspace closes the fd before > > anon_inode_getfd() even returns. > > I believe Al changed the interface to not give out inode* and file*, *and* > call fd_install() inside it. I'd slightly prefer Al version, although I > don't see any major problems in this one too. Any pointer to that patch? A web search for "viro" and "anon_inode_getfd" doesn't turn up anything likely looking. - R.