Revert "vfs: add flags to d_real()"
authorMiklos Szeredi <mszeredi@redhat.com>
Wed, 18 Jul 2018 13:44:43 +0000 (15:44 +0200)
committerMiklos Szeredi <mszeredi@redhat.com>
Wed, 18 Jul 2018 13:44:43 +0000 (15:44 +0200)
This reverts commit 495e642939114478a5237a7d91661ba93b76f15a.

No user of "flags" argument of d_real() remain.

Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Documentation/filesystems/Locking
Documentation/filesystems/vfs.txt
fs/open.c
fs/overlayfs/super.c
include/linux/dcache.h
include/linux/fs.h

index 2c391338c6757f505eac6dfcbe98a169452ad305..24e1a4f37c83099f612f854ad1b80d28144edf1b 100644 (file)
@@ -22,7 +22,7 @@ prototypes:
        struct vfsmount *(*d_automount)(struct path *path);
        int (*d_manage)(const struct path *, bool);
        struct dentry *(*d_real)(struct dentry *, const struct inode *,
-                                unsigned int, unsigned int);
+                                unsigned int);
 
 locking rules:
                rename_lock     ->d_lock        may block       rcu-walk
index 829a7b7857a46904cfb7f02646212504a3a7f259..6417b161f88df59e95c40dd4667530ec5c962fef 100644 (file)
@@ -1001,7 +1001,7 @@ struct dentry_operations {
        struct vfsmount *(*d_automount)(struct path *);
        int (*d_manage)(const struct path *, bool);
        struct dentry *(*d_real)(struct dentry *, const struct inode *,
-                                unsigned int, unsigned int);
+                                unsigned int);
 };
 
   d_revalidate: called when the VFS needs to revalidate a dentry. This
index a3c03a4a9078cb119af3c10441762eedec2dde66..a973ca07489680f3e6ea6cc09b5d66546eea3da7 100644 (file)
--- a/fs/open.c
+++ b/fs/open.c
@@ -96,7 +96,7 @@ long vfs_truncate(const struct path *path, loff_t length)
         * write access on the upper inode, not on the overlay inode.  For
         * non-overlay filesystems d_real() is an identity function.
         */
-       upperdentry = d_real(path->dentry, NULL, O_WRONLY, 0);
+       upperdentry = d_real(path->dentry, NULL, O_WRONLY);
        error = PTR_ERR(upperdentry);
        if (IS_ERR(upperdentry))
                goto mnt_drop_write_and_out;
index a7e2287e4b8f6c7ca3180015521dac544dde13d4..5bc261de5041bb9927ea06c7cd8961a68c58be00 100644 (file)
@@ -92,7 +92,7 @@ static int ovl_check_append_only(struct inode *inode, int flag)
 
 static struct dentry *ovl_d_real(struct dentry *dentry,
                                 const struct inode *inode,
-                                unsigned int open_flags, unsigned int flags)
+                                unsigned int open_flags)
 {
        struct dentry *real;
        int err;
@@ -128,7 +128,7 @@ static struct dentry *ovl_d_real(struct dentry *dentry,
                goto bug;
 
        /* Handle recursion */
-       real = d_real(real, inode, open_flags, 0);
+       real = d_real(real, inode, open_flags);
 
        if (!inode || inode == d_inode(real))
                return real;
index ddae4103d3247524e6616b482bb1fd8c4c6c1876..8fe4efa94af6c9a4e42d41a8f5fda68291ab917f 100644 (file)
@@ -146,7 +146,7 @@ struct dentry_operations {
        struct vfsmount *(*d_automount)(struct path *);
        int (*d_manage)(const struct path *, bool);
        struct dentry *(*d_real)(struct dentry *, const struct inode *,
-                                unsigned int, unsigned int);
+                                unsigned int);
 } ____cacheline_aligned;
 
 /*
@@ -568,8 +568,7 @@ static inline struct dentry *d_backing_dentry(struct dentry *upper)
  * d_real - Return the real dentry
  * @dentry: the dentry to query
  * @inode: inode to select the dentry from multiple layers (can be NULL)
- * @open_flags: open flags to control copy-up behavior
- * @flags: flags to control what is returned by this function
+ * @flags: open flags to control copy-up behavior
  *
  * If dentry is on a union/overlay, then return the underlying, real dentry.
  * Otherwise return the dentry itself.
@@ -578,10 +577,10 @@ static inline struct dentry *d_backing_dentry(struct dentry *upper)
  */
 static inline struct dentry *d_real(struct dentry *dentry,
                                    const struct inode *inode,
-                                   unsigned int open_flags, unsigned int flags)
+                                   unsigned int flags)
 {
        if (unlikely(dentry->d_flags & DCACHE_OP_REAL))
-               return dentry->d_op->d_real(dentry, inode, open_flags, flags);
+               return dentry->d_op->d_real(dentry, inode, flags);
        else
                return dentry;
 }
@@ -596,7 +595,7 @@ static inline struct dentry *d_real(struct dentry *dentry,
 static inline struct inode *d_real_inode(const struct dentry *dentry)
 {
        /* This usage of d_real() results in const dentry */
-       return d_backing_inode(d_real((struct dentry *) dentry, NULL, 0, 0));
+       return d_backing_inode(d_real((struct dentry *) dentry, NULL, 0));
 }
 
 struct name_snapshot {
index 8f8c9ac1c9d5655f0c5f99a4665a0f3447d2685f..16e2741cec3c2fa736255ad5d243ad2ec3062095 100644 (file)
@@ -1249,7 +1249,7 @@ static inline struct inode *file_inode(const struct file *f)
 
 static inline struct dentry *file_dentry(const struct file *file)
 {
-       return d_real(file->f_path.dentry, file_inode(file), 0, 0);
+       return d_real(file->f_path.dentry, file_inode(file), 0);
 }
 
 static inline int locks_lock_file_wait(struct file *filp, struct file_lock *fl)