Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757282Ab2HUMMB (ORCPT ); Tue, 21 Aug 2012 08:12:01 -0400 Received: from fieldses.org ([174.143.236.118]:59889 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757229Ab2HUML7 (ORCPT ); Tue, 21 Aug 2012 08:11:59 -0400 Date: Tue, 21 Aug 2012 08:11:55 -0400 From: "J. Bruce Fields" To: Pavel Emelyanov Cc: "Aneesh Kumar K.V" , Al Viro , Cyrill Gorcunov , "linux-kernel@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" , Alexey Dobriyan , Andrew Morton , James Bottomley , Matthew Helsley Subject: Re: [patch 4/8] fs, exportfs: Add export_encode_inode_fh helper Message-ID: <20120821121155.GC9483@fieldses.org> References: <87fw7habo4.fsf@skywalker.in.ibm.com> <20120820163338.GN23596@moon> <20120820183225.GB4911@fieldses.org> <20120820190606.GE27443@moon> <20120820193204.GD5779@fieldses.org> <50335261.5090504@parallels.com> <87wr0sle4v.fsf@skywalker.in.ibm.com> <503367CB.9080609@parallels.com> <20120821105424.GA7670@moon> <50336C51.60501@parallels.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <50336C51.60501@parallels.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1344 Lines: 31 On Tue, Aug 21, 2012 at 03:09:05PM +0400, Pavel Emelyanov wrote: > On 08/21/2012 02:54 PM, Cyrill Gorcunov wrote: > > On Tue, Aug 21, 2012 at 02:49:47PM +0400, Pavel Emelyanov wrote: > >>>> > >>>> However, if you have some better ideas on what information about inode should be exported > >>>> to the userspace please share. > >>>> > >>> > >>> Why not use name_to_handle(fd,...) and open_by_handle(handle,..) ? > >> > >> Because we don't have an fd at hands by the time we need to know the handle. > > > > Yeah, this might be not clear from patchset itself but inotify marks carry > > inodes inside kernel thus it's inodes what we can use when we fetch information > > about targets and put it into fdinfo output. > > Al, Bruce, Aneesh, > > What if we calculate the handle at the time we do have struct path at hands (i.e. > when we create the inotify) and store it on the inotify structure purely to be > shown later in proc. Would that be acceptable? Was it the lack of a dentry that was really the problem? I thought it was just the fact that not all filesystems support filehandles. --b. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/