Feature #22105: provide a way to look up snapshotted inodes by vinodeno_t - CephFS - Ceph

Actions

Copy link

Feature #22105

closed

provide a way to look up snapshotted inodes by vinodeno_t

Added by Jeff Layton over 6 years ago. Updated about 5 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Category:

Target version:

% Done:

Source:

Tags:

Backport:

Reviewed:

Affected Versions:

Component(FS):

Client, Ganesha FSAL

Labels (FS):

Pull request ID:

Description

An NFS client could conceivably present a filehandle that refers to a snapshot inode after ganesha has been stopped and restarted. Currently we have no interface for looking up a snapshot inode based on a inode number + snapid tuple.

I imagine that people will expect to be able to access snapshots via NFS, so we will likely need this ability. Basically, we need an analogue to ceph_ll_lookup_inode that takes a vinodeno_t instead of just an inode number.

Files

snap_export.patch (9.82 KB) snap_export.patch

Zheng Yan, 11/14/2017 02:46 PM

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Jeff Layton over 6 years ago

Subject changed from provide a way to look up snapshotted inodes by ino+snapid tuple to provide a way to look up snapshotted inodes by vinodeno_t

Actions

Copy link

Updated by Zheng Yan over 6 years ago

For directory inode, snapped inode are always stored together with head inode. The hard part is non-directory inode, because there can multiple snapped inodes stored in different directories.

Is it possible to make nfs client remember snapped inode's parent directory (and hash of corresponding dentry). If we can, implementing this function should be easy.

Actions

Copy link

Updated by Jeff Layton over 6 years ago

Zheng Yan wrote:

Is it possible to make nfs client remember snapped inode's parent directory (and hash of corresponding dentry). If we can, implementing this function should be easy.

That may be possible. NFSv4 filehandles are 128 bits. We currently just encode vinodeno_t as the filehandle, but we probably could mix in some info about the parent inode if that would be useful.

Is it possible though that a file like that could be renamed into a different directory? If so, then would its filehandle have to change?

Actions

Copy link