commit e2fd4023be765efebc0cf73f8a6c559fc3e2ef85
Author: Erez_Zadok <[EMAIL PROTECTED]>
Date: Thu May 31 23:57:14 2007 -0400
place two spaces after a sentence-ending period in long comments
Signed-off-by: Erez Zadok <[EMAIL PROTECTED]>
diff --git a/fs/unionfs/commonfops.c b/fs/unionfs/commonfops.c
index
88c8c25465316c6de636ac123c03697346490617..f81cd4bf0e457daf820d9ad0ea82e1f372b2e16d
100644
--- a/fs/unionfs/commonfops.c
+++ b/fs/unionfs/commonfops.c
@@ -620,8 +620,8 @@ int unionfs_open(struct inode *inode, st
* FIXME: With nfs exporting we can get a disconnected dentry here.
* This can happen if the dcache is dropped on the server while the
* client is working, and export ops decodefh returns ESTALE to the
- * user. It would best to catch this by implementing decodefh
- * or maybe we could connect the dentry ourselves using getparent?
+ * user. It would best to catch this by implementing decodefh or
+ * maybe we could connect the dentry ourselves using getparent?
*/
if (dentry->d_flags & DCACHE_DISCONNECTED) {
err = -ESTALE;
diff --git a/fs/unionfs/config.h b/fs/unionfs/config.h
index
dfd7043e4451c21a661a76c49767fa214fab8e82..232671a7f76a05e109d2a77234d716770cc23e5f
100644
--- a/fs/unionfs/config.h
+++ b/fs/unionfs/config.h
@@ -24,7 +24,7 @@ #define THRESH_BH_MIN 1
#define THRESH_BH_MAX 99
#define THRESH_BH_DEF 80
#define THRESH_BH_INFO "defaults to %d%%. The threshold of consumed disk " \
- "blocks in\n/odf, above which we begin cleaning. Must be higher " \
+ "blocks in\n/odf, above which we begin cleaning. Must be higher " \
"than the value of\n" THRESH_BL
#define THRESH_BL "block_low_thresh"
#define THRESH_BL_MIN 1
@@ -38,7 +38,7 @@ #define THRESH_IH_MIN 1
#define THRESH_IH_MAX 99
#define THRESH_IH_DEF 80
#define THRESH_IH_INFO "defaults to %d%%. The threshold of consumed inodes" \
- " in /odf,\nabove which we begin cleaning. Must be higher than the " \
+ " in /odf,\nabove which we begin cleaning. Must be higher than the " \
"value of " THRESH_IL
#define THRESH_IL "inode_low_thresh"
#define THRESH_IL_MIN 1
diff --git a/fs/unionfs/dentry.c b/fs/unionfs/dentry.c
index
603cf3c861e5aac4e2efc5c2fa51579bda15e800..6de78192693dfc658edd71d74e1526c689910e73
100644
--- a/fs/unionfs/dentry.c
+++ b/fs/unionfs/dentry.c
@@ -38,12 +38,13 @@ static int __unionfs_d_revalidate_one(st
struct nameidata lowernd; /* TODO: be gentler to the stack */
/*
- * FIXME: with nfs exporting, if export ops fails to connect a dentry
- * and returns ESTALE to the client, we can end up with an inode that
- * has ibstart -1. This fix only stops the oopsing but the dentry will
- * become unusable on the client, it will keep getting ESTALE on any
- * attempt to fix it. The best way to fix this is by implementing
- * decode_fh and making sure no inode gets left with -1 ibstart
+ * FIXME: with nfs exporting, if export ops fails to connect a
+ * dentry and returns ESTALE to the client, we can end up with an
+ * inode that has ibstart -1. This fix only stops the oopsing but
+ * the dentry will become unusable on the client, it will keep
+ * getting ESTALE on any attempt to fix it. The best way to fix
+ * this is by implementing decode_fh and making sure no inode gets
+ * left with -1 ibstart
*/
if (dentry->d_inode && ibstart(dentry->d_inode) < 0) {
valid = 0;
diff --git a/fs/unionfs/dirhelper.c b/fs/unionfs/dirhelper.c
index
2cc4a2c6c12fae788658f940f93fa2cf19af5db1..7b7fac405a313d6e926eeda71b52e64e79e16e49
100644
--- a/fs/unionfs/dirhelper.c
+++ b/fs/unionfs/dirhelper.c
@@ -193,12 +193,12 @@ find:
goto out;
/*
- * Ideally we would want all entries we find here to be in the
- * odf and have a persistent inode. However it is possible for the
+ * Ideally we would want all entries we find here to be in the odf
+ * and have a persistent inode. However it is possible for the
* entries not to be in the odf yet, and thus not have a valid odf
- * inode number. We could have created an entry in the odf at this
+ * inode number. We could have created an entry in the odf at this
* point, but since we do not have a unionfs dentry here we wouldn't
- * be able to check for hardlinks
+ * be able to check for hardlinks.
*/
if (odf_dentry)
ino = odf_dentry->d_inode->i_ino;
@@ -560,7 +560,7 @@ out:
/*
* Cleanup function for the odf cleanup thread.
* First cleans up the dir caches in odf/ic and then everything
- * in odf/reclaim. It stops once the requested blocks/inodes
+ * in odf/reclaim. It stops once the requested blocks/inodes
* were freed.
*
* b_to_free and i_to_free contains the requested amount of blocks
diff --git a/fs/unionfs/export.c b/fs/unionfs/export.c
index
622815a4de416e80756e9c0f3d21febe5f2d0a36..78e68c3ffff7b4f957c85415d78cfe8fade0ff64
100644
--- a/fs/unionfs/export.c
+++ b/fs/unionfs/export.c
@@ -87,8 +87,8 @@ static struct dentry *__get_parent(struc
/*
* If this dentry is disconnected, then it is
* anonymous, so we have to get its name using
- * get_name. However since it is disconnected it
- * doesn't know its parent. Thankfully since we
+ * get_name. However since it is disconnected it
+ * doesn't know its parent. Thankfully since we
* already looked up its parent in this loop we have
* its parent in the union dentry's parent odf_info
* struct
diff --git a/fs/unionfs/fanout.h b/fs/unionfs/fanout.h
index
4176228188b7c202914704e131b51f43c0819bdb..1f4e4cb7e3ba1e356cdd2597a99eae5b204c9ee4
100644
--- a/fs/unionfs/fanout.h
+++ b/fs/unionfs/fanout.h
@@ -72,7 +72,7 @@ static inline int branch_id_to_idx(struc
return i;
}
/*
- * Opaqueness is stored in the odf using branch ids. The odf
+ * Opaqueness is stored in the odf using branch ids. The odf
* uses this function to translate the branch id to branch index.
* Since branch management causes some branch ids to change or be
* removed, this function can legally fail to find the branch id
diff --git a/fs/unionfs/inode.c b/fs/unionfs/inode.c
index
e941065f1ea28d6a4721cbd573ff813832757547..eb858b5e774e0a393b4a6b9937f270d2bd433e43
100644
--- a/fs/unionfs/inode.c
+++ b/fs/unionfs/inode.c
@@ -692,7 +692,7 @@ static void unionfs_put_link(struct dent
* (2) if you set the mount option `mode=nfsro', we assume that -EACCES
* means that the export is read-only and we should check standard Unix
* permissions. This means that NFS ACL checks (or other advanced
- * permission features) are bypassed. Note however, that we do call
+ * permission features) are bypassed. Note, however, that we do call
* security_inode_permission, and therefore security inside SELinux, etc.
* are performed.
*/
diff --git a/fs/unionfs/lookup.c b/fs/unionfs/lookup.c
index
abe51287796c75f19b63bd087931e4975f47f6c3..886f050237c311e9cb27eba0e6fdfed4f05e7e49
100644
--- a/fs/unionfs/lookup.c
+++ b/fs/unionfs/lookup.c
@@ -49,7 +49,7 @@ struct dentry *unionfs_lookup_backend(st
/*
* We should already have a lock on this dentry in the case of a
- * partial lookup, or a revalidation. Otherwise it is returned from
+ * partial lookup, or a revalidation. Otherwise it is returned from
* new_dentry_private_data already locked.
*/
if (lookupmode == INTERPOSE_PARTIAL || lookupmode == INTERPOSE_REVAL ||
diff --git a/fs/unionfs/main.c b/fs/unionfs/main.c
index
35629a409c9c565ebb801b3e5f8cf48e7fab5878..d78df573b54e0287b6fa873f7f835511f88911d5
100644
--- a/fs/unionfs/main.c
+++ b/fs/unionfs/main.c
@@ -167,11 +167,10 @@ skip:
err = PTR_ERR(spliced);
/*
- * d_splice can return a dentry if it was disconnected
- * and had to be moved. We must make sure that the pd
- * of the new dentry is correct and that the inode info
- * was filled properly. Finally we must return this new
- * dentry
+ * d_splice can return a dentry if it was disconnected and
+ * had to be moved. We must make sure that the private data
+ * of the new dentry is correct and that the inode info was
+ * filled properly. Finally we must return this new dentry.
*/
else if (spliced && spliced != dentry) {
spliced->d_op = &unionfs_dops;
@@ -489,8 +488,8 @@ static struct unionfs_dentry_info *union
continue;
/*
- * All of our options take an argument now. Insert ones that
- * don't, above this check.
+ * All of our options take an argument now. Insert ones
+ * that don't, above this check.
*/
if (!optarg) {
printk("unionfs: %s requires an argument.\n", optname);
diff --git a/fs/unionfs/odf.c b/fs/unionfs/odf.c
index
db20ebfb9d301007b12da23f5e61622a48e561f8..4e87eedd33ed7474cb6632601a208f3a9ae7058d
100644
--- a/fs/unionfs/odf.c
+++ b/fs/unionfs/odf.c
@@ -951,7 +951,7 @@ out:
/*
* ODF lookup - There are two wrapper functions for looking up entries
- * in the ODF: odf_lookup and and odf_lookup_name. Apart from looking up
+ * in the ODF: odf_lookup and and odf_lookup_name. Apart from looking up
* entries they both can create dirs, files or whiteouts if the requested
* file does not yet exist in the odf.
*
@@ -973,7 +973,7 @@ out:
/*
* odf_lookup: This function should be used whenever we have a unionfs
- * dentry and want to find its entry in the ODF. Before doing the lookup
+ * dentry and want to find its entry in the ODF. Before doing the lookup
* it checks for hardlinks and tries to lazily build up the links in the
* odf.
*
@@ -1077,7 +1077,7 @@ out:
/*
* odf_lookup_name: This function should be used whenever we don't have a
- * unionfs dentry, but want to lookup a name in the odf. It should never
+ * unionfs dentry, but want to lookup a name in the odf. It should never
* be used to create entries in the odf, apart from whiteouts, since it
* does not handle hardlinks.
*
@@ -1101,14 +1101,14 @@ struct dentry *odf_lookup_name(struct de
* required and handles hardlinks
*
* @parent: unionfs dir dentry
- * @target: unionfs target dentry. If not NULL, lookup will update its
+ * @target: unionfs target dentry. If not NULL, lookup will update its
* odf_dentry_info if an entry is found, or else NULLify it.
* Its name is not used for the lookup, as it can be NULL
* @name: the name to look for
* @len: name length
* @flags: ODF_LOOKUP_ flags (see above)
* @link: an odf dentry that's the odf entry of one the target dentry's
- * links. If this is not NULL, lookup will make sure the looked
+ * links. If this is not NULL, lookup will make sure the looked
* up dentry is linked to the given link in the odf
*
* XXX: whether link should be an odf or unionfs dentry should
@@ -1552,7 +1552,7 @@ out:
* ODF_RMV_ANY - remove entry regardless of state
*
* If the item to be removed has i_count > 1 it is silly renamed (moved
- * to (/odf/sr). XXX: If there's any place where we don't want this
+ * to (/odf/sr). XXX: If there's any place where we don't want this
* behavior we can use an additional flag to this function.
*/
int odf_remove(struct dentry *dentry, int flags)
@@ -1642,7 +1642,7 @@ int odf_get_opaque(struct dentry *dentry
}
/*
- * Sets a dentry as opaque. The branch must be the branch_id
+ * Sets a dentry as opaque. The branch must be the branch_id
* as stored in unionfs sb.
*/
int odf_set_opaque(struct dentry *odf_dentry, int branch)
@@ -2050,8 +2050,8 @@ out:
}
/*
- * Worker function for the cleanup thread. It recursively removes all entries
- * in the reclaim directory
+ * Worker function for the cleanup thread. It recursively removes all
+ * entries in the reclaim directory.
*/
void __odf_cleanup(void *args)
{
diff --git a/fs/unionfs/sioq.c b/fs/unionfs/sioq.c
index
49b585232e9c2f2c0c6c4549a2e1b9ad658916a0..529a71f9d3c6b6abf3fec0cf92c44f22cb55eb7b
100644
--- a/fs/unionfs/sioq.c
+++ b/fs/unionfs/sioq.c
@@ -84,11 +84,11 @@ int __run_sioa(void *args)
}
/*
- * Creates an asynchronous thread. Calling process is responsible
- * for calling wait_for_completion. The thread executes the work function,
- * then sleeps for timeout time. When it wakes up it runs the done function
- * and terminates if done, otherwise repeats the loop.
- * The done function should return 0 if not done.
+ * Creates an asynchronous thread. Calling process is responsible for
+ * calling wait_for_completion. The thread executes the work function, then
+ * sleeps for timeout time. When it wakes up it runs the done function and
+ * terminates if done, otherwise repeats the loop. The done function should
+ * return 0 if not done.
*/
void run_sioa(struct sioa_args *args, void (*work) (void*),
signed long timeout)
diff --git a/fs/unionfs/super.c b/fs/unionfs/super.c
index
e5654dca53a8994ded94dc458dc2c0b3812b5a01..b7d1e41a228e5fc210a01493b4a5bc7e873c55d1
100644
--- a/fs/unionfs/super.c
+++ b/fs/unionfs/super.c
@@ -686,8 +686,8 @@ out_no_change:
/*
- * Update the odf. rewrite the superblock data file with all
- * the new branch information
+ * Update the odf. Rewrite the superblock data file with all the
+ * new branch information.
*/
/* first we need to update the uuids */
diff --git a/fs/unionfs/unlink.c b/fs/unionfs/unlink.c
index
174ba8a8dab064965da3dcf82b90cc27e4e8246f..f6bb691be4a72c46690d060b915e685e3449b18a
100644
--- a/fs/unionfs/unlink.c
+++ b/fs/unionfs/unlink.c
@@ -112,9 +112,9 @@ int unionfs_unlink(struct inode *dir, st
/* call d_drop so the system "forgets" about us */
if (!err) {
/*
- * If there was a whiteout created then the original
- * ODF entry was removed and replaced by a previous call
- * to create_whiteout. If not, the original entry is still
+ * If there was a whiteout created then the original ODF
+ * entry was removed and replaced by a previous call to
+ * create_whiteout. If not, the original entry is still
* here, so we just remove it only if its not a whiteout
*/
/* FIXME: what if this fails? */
_______________________________________________
unionfs-cvs mailing list: http://unionfs.filesystems.org/
[email protected]
http://www.fsl.cs.sunysb.edu/mailman/listinfo/unionfs-cvs
