[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87fx1ppy21.fsf@linux.vnet.ibm.com>
Date: Tue, 18 May 2010 11:10:38 +0530
From: "Aneesh Kumar K. V" <aneesh.kumar@...ux.vnet.ibm.com>
To: "J. R. Okajima" <hooanon05@...oo.co.jp>
Cc: hch@...radead.org, viro@...iv.linux.org.uk, adilger@....com,
corbet@....net, serue@...ibm.com, neilb@...e.de,
linux-fsdevel@...r.kernel.org, sfrench@...ibm.com,
philippe.deniel@....FR, linux-kernel@...r.kernel.org
Subject: Re: [PATCH -V8 2/9] vfs: Add name to file handle conversion support
On Tue, 18 May 2010 11:33:50 +0900, "J. R. Okajima" <hooanon05@...oo.co.jp> wrote:
>
> "Aneesh Kumar K.V":
> > This patch add a new superblock operations get_fsid that returns the
> > UUID mapping for the file system. The UUID returned is used to
> > identify the file system apart of file_handle
>
> I am afraid get_fsid in s_op may conflict with "fsid=" option in /etc/exports.
> Generally all FSs have UUID or device number and they can return "fsid"
> correctly. But some of them may not have such id, or users may assign
> different fsid for them.
> Is "fsid=" value passed to superblock and can FS return it? Otherwise
> they cannot implement ->get_fsid().
The file_handle I mentioned above is the file handle returned by
sys_name_to_handle_at syscall. NFS kernel server won't be using the
interface.
If file system doesn't support a unique identifier then they can leave
->get_fsid callback NULL. The UUID part of the file_handle will be zero
filled. For a NFS userspace server, the server can update handle.fsid with
what ever value is configured for fsid. The only requirement is later
when provided with the same handle NFS server should be able to map the
handle.fsid to a mountdirfd which can be used with sys_open_by_handle_at.
-aneesh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists