Unionfs: minor coding style updates
authorErez Zadok <ezk@cs.sunysb.edu>
Tue, 27 Nov 2007 00:54:52 +0000 (19:54 -0500)
committerErez Zadok <ezk@cs.sunysb.edu>
Tue, 27 Nov 2007 00:54:52 +0000 (19:54 -0500)
Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
fs/unionfs/debug.c
fs/unionfs/dentry.c
fs/unionfs/inode.c
fs/unionfs/main.c
fs/unionfs/union.h

index 0351376b0c26256febdfcd741a75f0da6a3f33e4..b7240a900bfc75bce37f10cb648ad251ce3528eb 100644 (file)
@@ -467,7 +467,8 @@ void __show_dinode_times(const struct dentry *dentry,
                lower_inode = unionfs_lower_inode_idx(inode, bindex);
                if (!lower_inode)
                        continue;
-               printk("DT(%s:%lu:%d): ", dentry->d_name.name, inode->i_ino, bindex);
+               printk("DT(%s:%lu:%d): ", dentry->d_name.name, inode->i_ino,
+                      bindex);
                printk("%s:%s:%d ",file,fxn,line);
                printk("um=%lu/%lu lm=%lu/%lu ",
                       inode->i_mtime.tv_sec, inode->i_mtime.tv_nsec,
@@ -490,7 +491,8 @@ void __show_inode_counts(const struct inode *inode,
                printk("SiC: Null inode\n");
                return;
        }
-       for (bindex=sbstart(inode->i_sb); bindex <= sbend(inode->i_sb); bindex++) {
+       for (bindex=sbstart(inode->i_sb); bindex <= sbend(inode->i_sb);
+            bindex++) {
                lower_inode = unionfs_lower_inode_idx(inode, bindex);
                if (!lower_inode)
                        continue;
index 08b572267dbdf1a2c0ed854491f0bad65e288ba1..d9bb199617232e768ccd7db203f67f7959a1c677 100644 (file)
@@ -26,7 +26,7 @@
  * Returns true if valid, false otherwise.
  */
 static bool __unionfs_d_revalidate_one(struct dentry *dentry,
-                                     struct nameidata *nd)
+                                      struct nameidata *nd)
 {
        bool valid = true;      /* default is valid */
        struct dentry *lower_dentry;
index c5ea4957153a2324cc1598d5204a6dba1f5a0db6..dd9aaea22d5a6da97eb3a3faae132937378f29ac 100644 (file)
@@ -99,7 +99,8 @@ static int unionfs_create(struct inode *parent, struct dentry *dentry,
                 * if lower_dentry is NULL, create the entire
                 * dentry directory structure in branch 0.
                 */
-               lower_dentry = create_parents(parent, dentry, dentry->d_name.name, 0);
+               lower_dentry = create_parents(parent, dentry,
+                                             dentry->d_name.name, 0);
                if (IS_ERR(lower_dentry)) {
                        err = PTR_ERR(lower_dentry);
                        goto out;
@@ -447,9 +448,8 @@ static int unionfs_symlink(struct inode *dir, struct dentry *dentry,
 
                if (!(err = is_robranch_super(dentry->d_sb, bindex))) {
                        mode = S_IALLUGO;
-                       err =
-                               vfs_symlink(lower_dir_dentry->d_inode,
-                                           lower_dentry, symname, mode);
+                       err = vfs_symlink(lower_dir_dentry->d_inode,
+                                         lower_dentry, symname, mode);
                }
                unlock_dir(lower_dir_dentry);
 
@@ -883,9 +883,11 @@ static void unionfs_put_link(struct dentry *dentry, struct nameidata *nd)
  *       readonly, to allow copyup to work.
  *   (3) we do call security_inode_permission, and therefore security inside
  *       SELinux, etc. are performed.
+ *
+ * @inode: the lower inode we're checking permission on
  */
-static int inode_permission(struct super_block *sb, struct inode *inode, int mask,
-                           struct nameidata *nd, int bindex)
+static int inode_permission(struct super_block *sb, struct inode *inode,
+                           int mask, struct nameidata *nd, int bindex)
 {
        int retval, submask;
 
index f5b5be9a1e303a6b4f8d42eae5dd9f2d595c9921..0d96f150c61b18eef2a6e7d4b2d3039128bdf401 100644 (file)
@@ -275,14 +275,14 @@ int __parse_branch_mode(const char *name)
  */
 int parse_branch_mode(const char *name)
 {
-       int perms =  __parse_branch_mode(name);
+       int perms = __parse_branch_mode(name);
 
        if (perms == 0)
                perms = MAY_READ | MAY_WRITE;
        return perms;
 }
 
-/* 
+/*
  * parse the dirs= mount argument
  *
  * We don't need to lock the superblock private data's rwsem, as we get
index 8256e8337e8bb4a90663e3ffc6a3a1d4c4e908b9..929daacc5bb1df815d591bdb8624202770723c9f 100644 (file)
@@ -553,7 +553,7 @@ extern void __show_inode_counts(const struct inode *inode,
 
 /* we leave useful hooks for these check functions throughout the code */
 #define unionfs_check_inode(i)         do { } while(0)
-#define unionfs_check_dentry(d)        do { } while(0)
+#define unionfs_check_dentry(d)                do { } while(0)
 #define unionfs_check_file(f)          do { } while(0)
 #define show_branch_counts(sb)         do { } while(0)
 #define show_inode_times(i)            do { } while(0)