diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2016-07-31 16:37:25 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2016-07-31 16:37:25 -0400 |
commit | 6fa67e707559303e086303aeecc9e8b91ef497d5 (patch) | |
tree | e5d034b0e9c533bd4e14fc0755432b50d69da2f2 /Documentation/filesystems | |
parent | d3fe19852e96aabe3084c82ec2f3fb4918ab6d1e (diff) |
get rid of 'parent' argument of ->d_compare()
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'Documentation/filesystems')
-rw-r--r-- | Documentation/filesystems/Locking | 2 | ||||
-rw-r--r-- | Documentation/filesystems/porting | 7 | ||||
-rw-r--r-- | Documentation/filesystems/vfs.txt | 2 |
3 files changed, 9 insertions, 2 deletions
diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking index 1b3c39a7de62..d30fb2cb5066 100644 --- a/Documentation/filesystems/Locking +++ b/Documentation/filesystems/Locking | |||
@@ -12,7 +12,7 @@ prototypes: | |||
12 | int (*d_revalidate)(struct dentry *, unsigned int); | 12 | int (*d_revalidate)(struct dentry *, unsigned int); |
13 | int (*d_weak_revalidate)(struct dentry *, unsigned int); | 13 | int (*d_weak_revalidate)(struct dentry *, unsigned int); |
14 | int (*d_hash)(const struct dentry *, struct qstr *); | 14 | int (*d_hash)(const struct dentry *, struct qstr *); |
15 | int (*d_compare)(const struct dentry *, const struct dentry *, | 15 | int (*d_compare)(const struct dentry *, |
16 | unsigned int, const char *, const struct qstr *); | 16 | unsigned int, const char *, const struct qstr *); |
17 | int (*d_delete)(struct dentry *); | 17 | int (*d_delete)(struct dentry *); |
18 | int (*d_init)(struct dentry *); | 18 | int (*d_init)(struct dentry *); |
diff --git a/Documentation/filesystems/porting b/Documentation/filesystems/porting index a5fb89cac615..b1bd05ea66b2 100644 --- a/Documentation/filesystems/porting +++ b/Documentation/filesystems/porting | |||
@@ -585,3 +585,10 @@ in your dentry operations instead. | |||
585 | in the instances. Rationale: !@#!@# security_d_instantiate() needs to be | 585 | in the instances. Rationale: !@#!@# security_d_instantiate() needs to be |
586 | called before we attach dentry to inode and !@#!@##!@$!$#!@#$!@$!@$ smack | 586 | called before we attach dentry to inode and !@#!@##!@$!$#!@#$!@$!@$ smack |
587 | ->d_instantiate() uses not just ->getxattr() but ->setxattr() as well. | 587 | ->d_instantiate() uses not just ->getxattr() but ->setxattr() as well. |
588 | -- | ||
589 | [mandatory] | ||
590 | ->d_compare() doesn't get parent as a separate argument anymore. If you | ||
591 | used it for finding the struct super_block involved, dentry->d_sb will | ||
592 | work just as well; if it's something more complicated, use dentry->d_parent. | ||
593 | Just be careful not to assume that fetching it more than once will yield | ||
594 | the same value - in RCU mode it could change under you. | ||
diff --git a/Documentation/filesystems/vfs.txt b/Documentation/filesystems/vfs.txt index 8a196851f01d..9ace359d6cc5 100644 --- a/Documentation/filesystems/vfs.txt +++ b/Documentation/filesystems/vfs.txt | |||
@@ -931,7 +931,7 @@ struct dentry_operations { | |||
931 | int (*d_revalidate)(struct dentry *, unsigned int); | 931 | int (*d_revalidate)(struct dentry *, unsigned int); |
932 | int (*d_weak_revalidate)(struct dentry *, unsigned int); | 932 | int (*d_weak_revalidate)(struct dentry *, unsigned int); |
933 | int (*d_hash)(const struct dentry *, struct qstr *); | 933 | int (*d_hash)(const struct dentry *, struct qstr *); |
934 | int (*d_compare)(const struct dentry *, const struct dentry *, | 934 | int (*d_compare)(const struct dentry *, |
935 | unsigned int, const char *, const struct qstr *); | 935 | unsigned int, const char *, const struct qstr *); |
936 | int (*d_delete)(const struct dentry *); | 936 | int (*d_delete)(const struct dentry *); |
937 | int (*d_init)(struct dentry *); | 937 | int (*d_init)(struct dentry *); |