All: run codespell on the code and fix issues.

Please review, it's not always just the comments that were fixed.
I've had to revert of course all calls to creat() that were changed
to create() ...

Only compile-tested!

Change-Id: I7d02e82d9766e272a7fd9cc68e51901d69e5aab5
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>

27 files changed, 80 insertions, 80 deletions

diff --git a/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c b/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c
index e393ae0f64e..8e432aee645 100644
--- a/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c
+++ b/xlators/features/bit-rot/src/bitd/bit-rot-scrub.c
@@ -158,7 +158,7 @@ bitd_signature_staleness (xlator_t *this,
         }
 
         /**
-         * save verison for validation in post compute stage
+         * save version for validation in post compute stage
          * c.f. bitd_scrub_post_compute_check()
          */
         *stale = signptr->stale ? 1 : 0;
@@ -358,7 +358,7 @@ br_scrubber_scrub_begin (xlator_t *this, struct br_fsscan_entry *fsentry)
                 skip_stat = _gf_true;
 
         /**
-         * open() an fd for subsequent opertaions
+         * open() an fd for subsequent operations
          */
         fd = fd_create (linked_inode, 0);
         if (!fd) {
@@ -419,7 +419,7 @@ br_scrubber_scrub_begin (xlator_t *this, struct br_fsscan_entry *fsentry)
         if (!skip_stat)
                 br_inc_scrubbed_file (&priv->scrub_stat);
 
-        GF_FREE (sign); /* alloced on post-compute */
+        GF_FREE (sign); /* allocated on post-compute */
 
         /** fd_unref() takes care of closing fd.. like syncop_close() */
 
@@ -1910,7 +1910,7 @@ br_monitor_thread (void *arg)
 
         /*
          * Since, this is the topmost xlator, THIS has to be set by bit-rot
-         * xlator itself (STACK_WIND wont help in this case). Also it has
+         * xlator itself (STACK_WIND won't help in this case). Also it has
          * to be done for each thread that gets spawned. Otherwise, a new
          * thread will get global_xlator's pointer when it does "THIS".
          */
diff --git a/xlators/features/bit-rot/src/bitd/bit-rot.c b/xlators/features/bit-rot/src/bitd/bit-rot.c
index a9b70b61f8a..66e58948fa9 100644
--- a/xlators/features/bit-rot/src/bitd/bit-rot.c
+++ b/xlators/features/bit-rot/src/bitd/bit-rot.c
@@ -947,7 +947,7 @@ out:
  * last run for whatever reason (node crashes, reboots, etc..) become
  * candidates for signing. This allows the signature to "catch up" with
  * the current state of the object. Triggering signing is easy: perform
- * an open() followed by a close() therby resulting in call boomerang.
+ * an open() followed by a close() thereby resulting in call boomerang.
  * (though not back to itself :))
  */
 int
@@ -999,7 +999,7 @@ bitd_oneshot_crawl (xlator_t *subvol,
          *    In this case syncop_getxattr fails with ENODATA and the object
          *    is signed. (In real, when crawler sends lookup, bit-rot-stub
          *    creates the xattrs before returning lookup reply)
-         * 2) Bit-rot was not enabled or BitD was dows for some reasons, during
+         * 2) Bit-rot was not enabled or BitD was does for some reasons, during
          *    which some files were created, but since BitD was down, were not
          *    signed.
          * If the file was just created and was being written some data when
@@ -1457,7 +1457,7 @@ br_handle_events (void *arg)
 
         /*
          * Since, this is the topmost xlator, THIS has to be set by bit-rot
-         * xlator itself (STACK_WIND wont help in this case). Also it has
+         * xlator itself (STACK_WIND won't help in this case). Also it has
          * to be done for each thread that gets spawned. Otherwise, a new
          * thread will get global_xlator's pointer when it does "THIS".
          */
diff --git a/xlators/features/bit-rot/src/bitd/bit-rot.h b/xlators/features/bit-rot/src/bitd/bit-rot.h
index c2c0c225792..670060c8d88 100644
--- a/xlators/features/bit-rot/src/bitd/bit-rot.h
+++ b/xlators/features/bit-rot/src/bitd/bit-rot.h
@@ -77,7 +77,7 @@ typedef enum br_child_state {
 
 struct br_child {
         pthread_mutex_t lock;         /* protects child state */
-        char witnessed;               /* witnessed at least one succesfull
+        char witnessed;               /* witnessed at least one successful
                                          connection */
         br_child_state_t c_state;     /* current state of this child */
 
@@ -123,7 +123,7 @@ struct br_scrubber {
 
         /**
          * frequency of scanning for this subvolume. this should
-         * normally be per-child, but since all childs follow the
+         * normally be per-child, but since all children follow the
          * same frequency for a volume, this option ends up here
          * instead of br_child_t.
          */
@@ -225,7 +225,7 @@ struct br_object {
 
         uuid_t gfid;
 
-        unsigned long signedversion;    /* version aginst which this object will
+        unsigned long signedversion;    /* version against which this object will
                                            be signed */
         br_child_t *child;              /* object's subvolume */
 
diff --git a/xlators/features/bit-rot/src/stub/bit-rot-stub-helpers.c b/xlators/features/bit-rot/src/stub/bit-rot-stub-helpers.c
index fff7a7eedf2..cc1a6e9a66f 100644
--- a/xlators/features/bit-rot/src/stub/bit-rot-stub-helpers.c
+++ b/xlators/features/bit-rot/src/stub/bit-rot-stub-helpers.c
@@ -232,7 +232,7 @@ br_stub_check_stub_file (xlator_t *this, char *path)
                 if (fd < 0)
                         gf_msg (this->name, GF_LOG_ERROR, errno,
                                 BRS_MSG_BAD_OBJECT_DIR_FAIL,
-                                "Failed ot create stub file [%s]", path);
+                                "Failed to create stub file [%s]", path);
         }
 
         if (fd >= 0) {
@@ -244,7 +244,7 @@ br_stub_check_stub_file (xlator_t *this, char *path)
 
 error_return:
         gf_msg (this->name, GF_LOG_ERROR, errno,
-                BRS_MSG_BAD_OBJECT_DIR_FAIL, "Failed ot verify stub file [%s]", path);
+                BRS_MSG_BAD_OBJECT_DIR_FAIL, "Failed to verify stub file [%s]", path);
         return -1;
 }
 
@@ -771,7 +771,7 @@ out:
 
 /**
 * NOTE: If the file has multiple hardlinks (in gluster volume
-* namespace), the path would be one of the hardlinks. Its upto
+* namespace), the path would be one of the hardlinks. Its up to
 * the user to find the remaining hardlinks (using find -samefile)
 * and remove them.
 **/
diff --git a/xlators/features/bit-rot/src/stub/bit-rot-stub.c b/xlators/features/bit-rot/src/stub/bit-rot-stub.c
index 830357070d6..c4853487e80 100644
--- a/xlators/features/bit-rot/src/stub/bit-rot-stub.c
+++ b/xlators/features/bit-rot/src/stub/bit-rot-stub.c
@@ -196,7 +196,7 @@ init (xlator_t *this)
  *    incremented upon modification.
  * So object versioning is tied to bitrot daemon's signing. In future, object
  * versioning might be necessary for other things as well apart from bit-rot
- * detection (well thats the objective of bringing in object-versioning :)).
+ * detection (well that's the objective of bringing in object-versioning :)).
  * In that case, better to make versioning a new option and letting it to be
  * enabled despite bit-rot detection is not needed.
  * Ex: ICAP.
@@ -501,7 +501,7 @@ br_stub_need_versioning (xlator_t *this,
 
         /* Bitrot stub inode context was initialized only in lookup, create
          * and mknod cbk path. Object versioning was enabled by default
-         * irrespective of bitrot enbaled or not. But it's made optional now.
+         * irrespective of bitrot enabled or not. But it's made optional now.
          * As a consequence there could be cases where getting inode ctx would
          * fail because it's not set yet.
          * e.g., If versioning (with bitrot enable) is enabled while I/O is
@@ -738,7 +738,7 @@ br_stub_fd_incversioning_cbk (call_frame_t *frame,
 /**
  * perform full or incremental versioning on an inode pointd by an
  * fd. incremental versioning is done when an inode is dirty and a
- * writeback is trigerred.
+ * writeback is triggered.
  */
 
 int
@@ -1761,7 +1761,7 @@ br_stub_getxattr (call_frame_t *frame, xlator_t *this,
         /**
          * If xattr is node-uuid and the inode is marked bad, return EIO.
          * Returning EIO would result in AFR to choose correct node-uuid
-         * coresponding to the subvolume * where the good copy of the
+         * corresponding to the subvolume * where the good copy of the
          * file resides.
          */
         if (IA_ISREG (loc->inode->ia_type) && XATTR_IS_NODE_UUID (name) &&
@@ -1841,7 +1841,7 @@ br_stub_fgetxattr (call_frame_t *frame, xlator_t *this,
         /**
          * If xattr is node-uuid and the inode is marked bad, return EIO.
          * Returning EIO would result in AFR to choose correct node-uuid
-         * coresponding to the subvolume * where the good copy of the
+         * corresponding to the subvolume * where the good copy of the
          * file resides.
          */
         if (IA_ISREG (fd->inode->ia_type) && XATTR_IS_NODE_UUID (name) &&
@@ -1979,7 +1979,7 @@ br_stub_writev_resume (call_frame_t *frame, xlator_t *this, fd_t *fd,
  * This is probably the most crucial part about the whole versioning thing.
  * There's absolutely no differentiation as such between an anonymous fd
  * and a regular fd except the fd context initialization. Object versioning
- * is perfomed when the inode is dirty. Parallel write operations are no
+ * is performed when the inode is dirty. Parallel write operations are no
  * special with each write performing object versioning followed by marking
  * the inode as non-dirty (synced). This is followed by the actual operation
  * (writev() in this case) which on a success marks the inode as modified.
@@ -2019,7 +2019,7 @@ br_stub_writev (call_frame_t *frame, xlator_t *this, fd_t *fd,
                 goto unwind;
 
         /**
-         * The inode is not dirty and also witnessed atleast one successful
+         * The inode is not dirty and also witnessed at least one successful
          * modification operation. Therefore, subsequent operations need not
          * perform any special tracking.
          */
@@ -2623,7 +2623,7 @@ br_stub_lookup_version (xlator_t *this,
                         ? obuf->ongoingversion : BITROT_DEFAULT_CURRENT_VERSION;
 
         /**
-         * If signature is there, but version is not therem then that status is
+         * If signature is there, but version is not there then that status is
          * is treated as INVALID. So in that case, we should not initialize the
          * inode context with wrong version names etc.
          */
@@ -3305,7 +3305,7 @@ br_stub_send_ipc_fop (xlator_t *this, fd_t *fd, unsigned long releaseversion,
  * 3) BR_SIGN_QUICK => reopen has happened and this release should trigger sign
  * 2 events:
  * 1) GF_FOP_RELEASE
- * 2) GF_FOP_WRITE (actually a dummy write fro BitD)
+ * 2) GF_FOP_WRITE (actually a dummy write for BitD)
  *
  * This is how states are changed based on events:
  * EVENT: GF_FOP_RELEASE:
diff --git a/xlators/features/changelog/lib/src/gf-changelog-helpers.c b/xlators/features/changelog/lib/src/gf-changelog-helpers.c
index 45cf55778e4..9ff1d135933 100644
--- a/xlators/features/changelog/lib/src/gf-changelog-helpers.c
+++ b/xlators/features/changelog/lib/src/gf-changelog-helpers.c
@@ -57,7 +57,7 @@ gf_rfc3986_encode_space_newline (unsigned char *s, char *enc, char *estr)
  * that can be done via @fflush(fp), @ftruncate(fd) and @fseek(fp),
  * but this involves mixing POSIX file descriptors and stream FILE *).
  *
- * NOTE: This implmentation still does work with more than one fd's
+ * NOTE: This implementation still does work with more than one fd's
  *       used to perform gf_readline(). For this very reason it's not
  *       made a part of libglusterfs.
  */
diff --git a/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c b/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c
index 07de76e7cda..9c1a498f655 100644
--- a/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c
+++ b/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c
@@ -524,7 +524,7 @@ gf_changelog_publish (xlator_t *this,
         (void) snprintf (to_path, PATH_MAX, "%s%s",
                          jnl->jnl_current_dir, basename (from_path));
 
-        /* handle zerob file that wont exist in current */
+        /* handle zerob file that won't exist in current */
         ret = sys_stat (to_path, &stbuf);
         if (ret) {
                 if (errno == ENOENT)
diff --git a/xlators/features/changelog/lib/src/gf-changelog-reborp.c b/xlators/features/changelog/lib/src/gf-changelog-reborp.c
index d1b3059a60c..f9fb8fcf01a 100644
--- a/xlators/features/changelog/lib/src/gf-changelog-reborp.c
+++ b/xlators/features/changelog/lib/src/gf-changelog-reborp.c
@@ -61,7 +61,7 @@ gf_changelog_connection_janitor (void *arg)
                          "brick=%s", entry->brick,
                          NULL);
 
-                /* 0x0: disbale rpc-clnt */
+                /* 0x0: disable rpc-clnt */
                 rpc_clnt_disable (RPC_PROBER (entry));
 
                 /* 0x1: cleanup callback invoker thread */
@@ -155,7 +155,7 @@ gf_changelog_reborp_init_rpc_listner (xlator_t *this,
 }
 
 /**
- * This is dirty and painful as of now untill there is event filtering in the
+ * This is dirty and painful as of now until there is event filtering in the
  * server. The entire event buffer is scanned and interested events are picked,
  * whereas we _should_ be notified with the events we were interested in
  * (selected at the time of probe). As of now this is complete BS and needs
@@ -283,7 +283,7 @@ orderfn (struct list_head *pos1, struct list_head *pos2)
 void
 queue_ordered_event (struct gf_event_list *ev, struct gf_event *event)
 {
-        /* add event to the ordered event list and wake up listner(s) */
+        /* add event to the ordered event list and wake up listener(s) */
         pthread_mutex_lock (&ev->lock);
         {
                 list_add_order (&event->list, &ev->events, orderfn);
diff --git a/xlators/features/changelog/lib/src/gf-history-changelog.c b/xlators/features/changelog/lib/src/gf-history-changelog.c
index 2f245a725f6..7b973b674a4 100644
--- a/xlators/features/changelog/lib/src/gf-history-changelog.c
+++ b/xlators/features/changelog/lib/src/gf-history-changelog.c
@@ -476,7 +476,7 @@ out:
 
 /*
  * Description: Checks if the changelog path is usable or not,
- *              which is differenciated by checking for "changelog"
+ *              which is differentiated by checking for "changelog"
  *              in the path and not "CHANGELOG".
  *
  * Returns:
diff --git a/xlators/features/changelog/src/changelog-helpers.c b/xlators/features/changelog/src/changelog-helpers.c
index 8bc994fa417..fb1ee06da2c 100644
--- a/xlators/features/changelog/src/changelog-helpers.c
+++ b/xlators/features/changelog/src/changelog-helpers.c
@@ -1172,7 +1172,7 @@ changelog_local_init (xlator_t *this, inode_t *inode,
 
         /**
          * We relax the presence of inode if @update_flag is true.
-         * The caller (implmentation of the fop) needs to be careful to
+         * The caller (implementation of the fop) needs to be careful to
          * not blindly use local->inode.
          */
         if (!update_flag && !inode) {
@@ -1253,7 +1253,7 @@ changelog_drain_black_fops (xlator_t *this, changelog_priv_t *priv)
                          NULL);
         while (priv->dm.black_fop_cnt > 0) {
                 gf_msg_debug (this->name, 0,
-                              "Condtional wait on black fops: %ld",
+                              "Conditional wait on black fops: %ld",
                               priv->dm.black_fop_cnt);
                 priv->dm.drain_wait_black = _gf_true;
                 ret = pthread_cond_wait (&priv->dm.drain_black_cond,
@@ -1297,7 +1297,7 @@ changelog_drain_white_fops (xlator_t *this, changelog_priv_t *priv)
                          NULL);
         while (priv->dm.white_fop_cnt > 0) {
                 gf_msg_debug (this->name, 0,
-                              "Condtional wait on white fops : %ld",
+                              "Conditional wait on white fops : %ld",
                         priv->dm.white_fop_cnt);
                 priv->dm.drain_wait_white = _gf_true;
                 ret = pthread_cond_wait (&priv->dm.drain_white_cond,
@@ -1403,7 +1403,7 @@ changelog_rollover (void *data)
                 /* Adding delay of 1 second only during explicit rollover:
                  *
                  * Changelog rollover can happen either due to actual
-                 * or the explict rollover during snapshot. Actual
+                 * or the explicit rollover during snapshot. Actual
                  * rollover is controlled by tuneable called 'rollover-time'.
                  * The minimum granularity for rollover-time is 1 second.
                  * Explicit rollover is asynchronous in nature and happens
diff --git a/xlators/features/changelog/src/changelog.c b/xlators/features/changelog/src/changelog.c
index 12997181da4..6d32de63db5 100644
--- a/xlators/features/changelog/src/changelog.c
+++ b/xlators/features/changelog/src/changelog.c
@@ -988,7 +988,7 @@ out:
         return 0;
 }
 
-/* creat */
+/* create */
 
 int32_t
 changelog_create_cbk (call_frame_t *frame,
diff --git a/xlators/features/changetimerecorder/src/changetimerecorder.c b/xlators/features/changetimerecorder/src/changetimerecorder.c
index 40bfcf4a32a..a9d73ed0ac8 100644
--- a/xlators/features/changetimerecorder/src/changetimerecorder.c
+++ b/xlators/features/changetimerecorder/src/changetimerecorder.c
@@ -48,7 +48,7 @@ These are the scenarios when the hardlink list is updated:
 1) Named-Lookup: Whenever a named lookup happens on a file, in the
    wind path we copy all required hardlink and inode information to
    ctr_db_record structure, which resides in the frame->local variable.
-   We dont update the database in wind. During the unwind, we read the
+   We don't update the database in wind. During the unwind, we read the
    information from the ctr_db_record and ,
    Check if the inode context variable is created, if not we create it.
    Check if the hard link is there in the hardlink list.
@@ -74,12 +74,12 @@ These are the scenarios when the hardlink list is updated:
 
 What is pending:
 1) This solution will only work for named lookups.
-2) We dont track afr-self-heal/dht-rebalancer traffic for healing.
+2) We don't track afr-self-heal/dht-rebalancer traffic for healing.
 
 */
 
 
-/* This function doesnot write anything to the db,
+/* This function does not write anything to the db,
  * just created the local variable
  * for the frame and sets values for the ctr_db_record */
 static int
@@ -109,9 +109,9 @@ ctr_lookup_wind(call_frame_t                    *frame,
                         goto out;
                 };
                 ctr_local = frame->local;
-                /*Definately no internal fops will reach here*/
+                /*Definitely no internal fops will reach here*/
                 ctr_local->is_internal_fop = _gf_false;
-                /*Dont record counters*/
+                /*Don't record counters*/
                 CTR_DB_REC(ctr_local).do_record_counters = _gf_false;
                 /*Don't record time at all*/
                 CTR_DB_REC(ctr_local).do_record_times = _gf_false;
@@ -135,7 +135,7 @@ ctr_lookup_wind(call_frame_t                    *frame,
                 /* Since we are in lookup we can ignore errors while
                  * Inserting in the DB, because there may be many
                  * to write to the DB attempts for healing.
-                 * We dont want to log all failed attempts and
+                 * We don't want to log all failed attempts and
                  * bloat the log*/
                  ctr_local->gfdb_db_record.ignore_errors = _gf_true;
         }
@@ -216,7 +216,7 @@ ctr_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
 
         CTR_IS_DISABLED_THEN_GOTO(this, out);
 
-        /* if the lookup failed lookup dont do anything*/
+        /* if the lookup failed lookup don't do anything*/
         if (op_ret == -1) {
                 gf_msg_trace (this->name, 0, "lookup failed with %s",
                               strerror (op_errno));
@@ -229,7 +229,7 @@ ctr_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
         }
 
         /* if frame local was not set by the ctr_lookup()
-         * so dont so anything*/
+         * so don't so anything*/
         if (!frame->local) {
                 goto out;
         }
@@ -329,7 +329,7 @@ ctr_lookup (call_frame_t *frame, xlator_t *this,
         GF_ASSERT(frame);
         GF_ASSERT(frame->root);
 
-        /* Dont handle nameless lookups*/
+        /* Don't handle nameless lookups*/
         if (!loc->parent || !loc->name)
                 goto out;
 
@@ -941,7 +941,7 @@ ctr_rename (call_frame_t *frame, xlator_t *this, loc_t *oldloc,
                         "Failed to insert rename wind");
         } else {
                 /* We are doing updation of hard link in inode context in wind
-                 * As we dont get the "inode" in the call back for rename */
+                 * As we don't get the "inode" in the call back for rename */
                 ret = update_hard_link_ctx (frame, this, oldloc->inode);
                 if (ret) {
                         gf_msg (this->name, GF_LOG_ERROR, 0,
@@ -950,13 +950,13 @@ ctr_rename (call_frame_t *frame, xlator_t *this, loc_t *oldloc,
                         goto out;
                 }
 
-                /* If the newloc has an inode. i.e aquiring hardlink of an
+                /* If the newloc has an inode. i.e acquiring hardlink of an
                  * exisitng file i.e overwritting a file.
                  * */
                 if (newloc->inode) {
 
                         /* Getting the ctr inode context variable for
-                         * inode whose hardlink will be aquired during
+                         * inode whose hardlink will be acquired during
                          * the rename
                          * */
                         ctr_xlator_ctx = get_ctr_xlator_ctx (this,
@@ -1106,7 +1106,7 @@ ctr_unlink (call_frame_t *frame, xlator_t *this,
                         "Failed to insert unlink wind");
         } else {
                 /* We are doing delete of hard link in inode context in wind
-                 * As we dont get the "inode" in the call back for rename */
+                 * As we don't get the "inode" in the call back for rename */
                 ret = delete_hard_link_ctx (frame, this, loc->inode);
                 if (ret) {
                         gf_msg (this->name, GF_LOG_ERROR, 0,
diff --git a/xlators/features/changetimerecorder/src/ctr-helper.c b/xlators/features/changetimerecorder/src/ctr-helper.c
index 822526ca833..5a409c2b65e 100644
--- a/xlators/features/changetimerecorder/src/ctr-helper.c
+++ b/xlators/features/changetimerecorder/src/ctr-helper.c
@@ -198,7 +198,7 @@ extract_sql_params(xlator_t *this, dict_t *params_dict)
         SET_DB_PARAM_TO_DICT(this->name, params_dict, GFDB_SQL_PARAM_DBPATH,
                                 db_full_path, ret, out);
 
-        /*Extact rest of the sql params*/
+        /*Extract rest of the sql params*/
         ret = gfdb_set_sql_params(this->name, this->options, params_dict);
         if (ret) {
                 gf_msg (GFDB_DATA_STORE, GF_LOG_ERROR, 0,
diff --git a/xlators/features/changetimerecorder/src/ctr-helper.h b/xlators/features/changetimerecorder/src/ctr-helper.h
index 4fd4f745f4d..9232c16c5a9 100644
--- a/xlators/features/changetimerecorder/src/ctr-helper.h
+++ b/xlators/features/changetimerecorder/src/ctr-helper.h
@@ -76,7 +76,7 @@ typedef struct gf_ctr_private {
  * but currently we record only file inode information.
  *
  * is_internal_fop in gf_ctr_local will tell us if this is a internal fop and
- * take special/no action. We dont record change/acces times or increement heat
+ * take special/no action. We don't record change/access times or increement heat
  * counter for internal fops from rebalancer.
  * */
 typedef struct gf_ctr_local {
@@ -146,7 +146,7 @@ free_ctr_local (gf_ctr_local_t *ctr_local)
  * ****************************************************************************/
 
 /*
- * Context Carrier structures are used to carry relavent information about
+ * Context Carrier structures are used to carry relevant information about
  * inodes and links from the fops calls to the ctr_insert_wind.
  * These structure just have pointers to the original data and donot
  * do a deep copy of any data. This info is deep copied to
diff --git a/xlators/features/compress/src/cdc-helper.c b/xlators/features/compress/src/cdc-helper.c
index 0a9a0e3d29c..7baccb0cd3d 100644
--- a/xlators/features/compress/src/cdc-helper.c
+++ b/xlators/features/compress/src/cdc-helper.c
@@ -335,7 +335,7 @@ cdc_compress (xlator_t *this, cdc_priv_t *priv, cdc_info_t *ci,
         if (ret) {
                 /* Send uncompressed data if we can't _tell_ the client
                  * that deflated data is on it's way. So, we just log
-                 * the faliure and continue as usual.
+                 * the failure and continue as usual.
                  */
                  gf_log (this->name, GF_LOG_ERROR,
                  "Data deflated, but could not set canary"
diff --git a/xlators/features/gfid-access/src/gfid-access.c b/xlators/features/gfid-access/src/gfid-access.c
index 49d781c59ba..0a82321ed3c 100644
--- a/xlators/features/gfid-access/src/gfid-access.c
+++ b/xlators/features/gfid-access/src/gfid-access.c
@@ -756,7 +756,7 @@ ga_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
 
         priv = this->private;
 
-        /* do we need to copy root stbuf everytime? */
+        /* do we need to copy root stbuf every time? */
         /* mostly yes, as we want to have the 'stat' info show latest
            in every _cbk() */
 
diff --git a/xlators/features/leases/src/leases.c b/xlators/features/leases/src/leases.c
index 34143859c56..cbcbff73022 100644
--- a/xlators/features/leases/src/leases.c
+++ b/xlators/features/leases/src/leases.c
@@ -899,7 +899,7 @@ block:
 out:
         /* *
          * currently release is not called after the close fop from the
-         * application. Hence lease fd ctx is resetted on here.
+         * application. Hence lease fd ctx is reset on here.
          * This is actually not the right way, since flush can be called
          * not only from the close op.
          * TODO :
diff --git a/xlators/features/locks/src/locks.h b/xlators/features/locks/src/locks.h
index c2edfff8f00..e9b75fb3f13 100644
--- a/xlators/features/locks/src/locks.h
+++ b/xlators/features/locks/src/locks.h
@@ -87,7 +87,7 @@ struct __pl_inode_lock {
 
         struct timeval     blkd_time;   /*time at which lock was queued into blkd list*/
         struct timeval     granted_time; /*time at which lock was queued into active list*/
-        /*last time at wich lock contention was detected and notified*/
+        /*last time at which lock contention was detected and notified*/
         struct timespec    contention_time;
 
         /* These two together serve to uniquely identify each process
@@ -137,7 +137,7 @@ struct __entry_lock {
 
         struct timeval     blkd_time;   /*time at which lock was queued into blkd list*/
         struct timeval     granted_time; /*time at which lock was queued into active list*/
-        /*last time at wich lock contention was detected and notified*/
+        /*last time at which lock contention was detected and notified*/
         struct timespec    contention_time;
 
         void             *client;
diff --git a/xlators/features/locks/src/posix.c b/xlators/features/locks/src/posix.c
index dc11ab62d0f..e83a93ded11 100644
--- a/xlators/features/locks/src/posix.c
+++ b/xlators/features/locks/src/posix.c
@@ -4477,7 +4477,7 @@ struct volume_options options[] = {
           .tags  = {"locks"},
           .description = "Specifies the mandatory-locking mode. Valid options "
                          "are 'file' to use linux style mandatory locks, "
-                         "'forced' to use volume striclty under mandatory lock "
+                         "'forced' to use volume strictly under mandatory lock "
                          "semantics only and 'optimal' to treat advisory and "
                          "mandatory locks separately on their own."
         },
diff --git a/xlators/features/marker/src/marker-quota.c b/xlators/features/marker/src/marker-quota.c
index b6bf2fb36a4..06ba0820a54 100644
--- a/xlators/features/marker/src/marker-quota.c
+++ b/xlators/features/marker/src/marker-quota.c
@@ -1325,7 +1325,7 @@ mq_reduce_parent_size_task (void *opaque)
         locked = _gf_true;
 
         if (contri.size >= 0) {
-                /* contri paramater is supplied only for rename operation.
+                /* contri parameter is supplied only for rename operation.
                  * remove xattr is alreday performed, we need to skip
                  * removexattr for rename operation
                  */
@@ -1554,7 +1554,7 @@ mq_initiate_quota_task (void *opaque)
                    Suppose hard link for a file f1 present in a directory d1 is
                    created in the directory d2 (as f2). Now, since d2's
                    contribution is not there in f1's inode ctx, d2's
-                   contribution xattr wont be created and will create problems
+                   contribution xattr won't be created and will create problems
                    for quota operations.
 
                    Don't create contribution if parent has been changed after
diff --git a/xlators/features/marker/src/marker.c b/xlators/features/marker/src/marker.c
index 659ca890066..522fdfb86d1 100644
--- a/xlators/features/marker/src/marker.c
+++ b/xlators/features/marker/src/marker.c
@@ -1364,7 +1364,7 @@ marker_rename_unwind (call_frame_t *frame, void *cookie, xlator_t *this,
         if (local->stub != NULL) {
                 /* Remove contribution node from in-memory even if
                  * remove-xattr has failed as the rename is already performed
-                 * if local->stub is set, which means rename was sucessful
+                 * if local->stub is set, which means rename was successful
                  */
                 (void) mq_inode_ctx_get (oplocal->loc.inode, this, &ctx);
                 if (ctx) {
@@ -1641,7 +1641,7 @@ err:
  *      b) we should subtract from src-parent exactly what we contributed to
  *         src-parent
  *    So, We hold a lock on src-parent to block any parallel transcations on
- *    src-inode (since thats the one which survives rename).
+ *    src-inode (since that's the one which survives rename).
  *
  *    If there are any parallel transactions on dst-inode they keep succeeding
  *    till the association of dst-inode with dst-parent is broken because of an
diff --git a/xlators/features/read-only/src/worm-helper.c b/xlators/features/read-only/src/worm-helper.c
index 0770f9d77c4..b5fa4f84a29 100644
--- a/xlators/features/read-only/src/worm-helper.c
+++ b/xlators/features/read-only/src/worm-helper.c
@@ -288,7 +288,7 @@ out:
 }
 
 
-/*This function checks whether a file's timeout is happend for the state
+/*This function checks whether a file's timeout is happened for the state
  * transition and if yes, then it will do the transition from the current state
  * to the appropriate state. It also decides whether to continue or to block
  * the FOP.
diff --git a/xlators/features/snapview-client/src/snapview-client.c b/xlators/features/snapview-client/src/snapview-client.c
index 63ed160bd71..77be491655c 100644
--- a/xlators/features/snapview-client/src/snapview-client.c
+++ b/xlators/features/snapview-client/src/snapview-client.c
@@ -261,7 +261,7 @@ gf_svc_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
 
         /* There is a possibility that, the client process just came online
            and does not have the inode on which the lookup came. In that case,
-           the fresh inode created from fuse for the lookup fop, wont have
+           the fresh inode created from fuse for the lookup fop, won't have
            the inode context set without which svc cannot decide where to
            STACK_WIND to. So by default it decides to send the fop to the
            regular subvolume (i.e first child of the xlator). If lookup fails
@@ -2129,7 +2129,7 @@ out:
 
 /* Creating hardlinks for the files from the snapshot is not allowed as it
    will be equivalent of creating hardlinks across different filesystems.
-   And so is vise versa.
+   And so is vice versa.
 */
 static int32_t
 gf_svc_link (call_frame_t *frame, xlator_t *this, loc_t *oldloc, loc_t *newloc,
@@ -2196,7 +2196,7 @@ gf_svc_removexattr (call_frame_t *frame, xlator_t *this, loc_t *loc,
 
         ret = svc_inode_ctx_get (this, loc->inode, &inode_type);
         if (ret < 0) {
-                gf_log (this->name, GF_LOG_ERROR, "failed to get te inode "
+                gf_log (this->name, GF_LOG_ERROR, "failed to get the inode "
                         "context for %s (gfid: %s)", loc->path,
                         uuid_utoa (loc->inode->gfid));
                 op_ret = -1;
diff --git a/xlators/features/snapview-server/src/snapview-server.c b/xlators/features/snapview-server/src/snapview-server.c
index 18f00d7d2d8..89684ccac07 100644
--- a/xlators/features/snapview-server/src/snapview-server.c
+++ b/xlators/features/snapview-server/src/snapview-server.c
@@ -822,7 +822,7 @@ svs_getxattr (call_frame_t *frame, xlator_t *this, loc_t *loc, const char *name,
            or the inode is SNAP_VIEW_ENTRY_POINT_INODE. Entry point is
            a virtual directory on which setxattr operations are not
            allowed. If getxattr has to be faked as success, then a value
-           for the name of the xattr has to be sent which we dont have.
+           for the name of the xattr has to be sent which we don't have.
         */
         if (inode_ctx->type == SNAP_VIEW_ENTRY_POINT_INODE) {
                 op_ret = -1;
@@ -953,7 +953,7 @@ svs_fgetxattr (call_frame_t *frame, xlator_t *this, fd_t *fd, const char *name,
            or the inode is SNAP_VIEW_ENTRY_POINT_INODE. Entry point is
            a virtual directory on which setxattr operations are not
            allowed. If getxattr has to be faked as success, then a value
-           for the name of the xattr has to be sent which we dont have.
+           for the name of the xattr has to be sent which we don't have.
         */
         if (inode_ctx->type == SNAP_VIEW_ENTRY_POINT_INODE) {
                 op_ret = -1;
@@ -1185,7 +1185,7 @@ svs_forget  (xlator_t *this, inode_t *inode)
 
         ret = inode_ctx_del (inode, this, &value);
         if (ret) {
-                gf_log (this->name, GF_LOG_ERROR, "failed to delte the inode "
+                gf_log (this->name, GF_LOG_ERROR, "failed to delete the inode "
                         "context of %s", uuid_utoa (inode->gfid));
                 goto out;
         }
@@ -1633,11 +1633,11 @@ unwind:
  * the gfid it got from NFS client, for which it was not able to find the right
  * inode. So snapview-server was able to get the fs instance (glfs_t) of the
  * snapshot volume to which the entry belongs to, and the handle for the entry
- * from the corresponding snapshot volume and fill those informations in the
+ * from the corresponding snapshot volume and fill those information in the
  * inode context.
  *
  * But now, since NFS server is able to find the inode from the inode table for
- * the gfid it got from the NFS client, it wont send lookup. Rather it directly
+ * the gfid it got from the NFS client, it won't send lookup. Rather it directly
  * sends the fop it received from the client. Now this causes problems for
  * snapview-server. Because for each fop snapview-server assumes that lookup has
  * been performed on that entry and the entry's inode context contains the
diff --git a/xlators/features/trash/src/trash.c b/xlators/features/trash/src/trash.c
index 8a92685cf4b..e0bfcd9db43 100644
--- a/xlators/features/trash/src/trash.c
+++ b/xlators/features/trash/src/trash.c
@@ -90,7 +90,7 @@ out:
 }
 
 /**
- * The trash directory path should be append at begining of file path for
+ * The trash directory path should be append at beginning of file path for
  * delete or truncate operations. Normal trashing moves the contents to
  * trash directory and trashing done by internal operations are moved to
  * internal_op directory inside trash.
@@ -114,7 +114,7 @@ copy_trash_path (const char *priv_value, gf_boolean_t internal, char *path,
 /**
  * This function performs the reverse operation of copy_trash_path(). It gives
  * out a pointer, whose starting value will be the path inside trash directory,
- * similar to orginal path.
+ * similar to original path.
  */
 void
 remove_trash_path (const char *path, gf_boolean_t internal, char **rem_path)
@@ -1062,7 +1062,7 @@ trash_unlink_rename_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
         GF_VALIDATE_OR_GOTO ("trash", local, out);
 
         if ((op_ret == -1) && (op_errno == ENOENT)) {
-                /* the file path doesnot exists we want to create path
+                /* the file path does not exist we want to create path
                  * for the file
                  */
                 tmp_str = gf_strdup (local->newpath);
@@ -1925,7 +1925,7 @@ trash_truncate_stat_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
                 goto out;
         }
 
-        /* Retrives the name of file from path */
+        /* Retrieves the name of file from path */
         local->loc.name = gf_strdup (strrchr (local->loc.path, '/'));
         if (!local->loc.name) {
                 gf_log (this->name, GF_LOG_DEBUG, "out of memory");
@@ -2290,7 +2290,7 @@ out:
 }
 
 /**
- * Volume set option is handled by the reconfigure funtion.
+ * Volume set option is handled by the reconfigure function.
  * Here we checks whether each option is set or not ,if it
  * sets then corresponding modifciations will be made
  */
@@ -2412,7 +2412,7 @@ notify (xlator_t *this, int event, void *data, ...)
                         goto out;
                 }
 
-                /* Here there is two possiblities ,if trash directory already
+                /* Here there is two possibilities ,if trash directory already
                  * exist ,then we need to perform a rename operation on the
                  * old one. Otherwise, we need to create the trash directory
                  * For both, we need to pass location variable, gfid of parent
diff --git a/xlators/features/upcall/src/upcall-internal.c b/xlators/features/upcall/src/upcall-internal.c
index d8131a9f2ab..73ade8324b7 100644
--- a/xlators/features/upcall/src/upcall-internal.c
+++ b/xlators/features/upcall/src/upcall-internal.c
@@ -705,7 +705,7 @@ out:
 }
 
 /*
- * This is called during upcall_inode_ctx cleanup incase of 'inode_forget'.
+ * This is called during upcall_inode_ctx cleanup in case of 'inode_forget'.
  * Send "UP_FORGET" to all the clients so that they invalidate their cache
  * entry and do a fresh lookup next time when any I/O comes in.
  */
diff --git a/xlators/features/upcall/src/upcall.c b/xlators/features/upcall/src/upcall.c
index 0bf1f413a94..6dfc8723553 100644
--- a/xlators/features/upcall/src/upcall.c
+++ b/xlators/features/upcall/src/upcall.c
@@ -326,7 +326,7 @@ up_setattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
                 goto out;
         }
         /* XXX: setattr -> UP_SIZE or UP_OWN or UP_MODE or UP_TIMES
-         * or INODE_UPDATE (or UP_PERM esp incase of ACLs -> INODE_INVALIDATE)
+         * or INODE_UPDATE (or UP_PERM esp in case of ACLs -> INODE_INVALIDATE)
          * Need to check what attr is changed and accordingly pass UP_FLAGS.
          * Bug1200271.
          */
@@ -2118,11 +2118,11 @@ err:
  * - Do not filter unregistered xattrs in the cbk, but in the call path.
  *   Else, we will be invalidating on every preop, if the file already has
  *   pending xattr set. Filtering unregistered xattrs on the fop path
- *   ensures we invalidate only in postop, everytime a postop comes with
+ *   ensures we invalidate only in postop, every time a postop comes with
  *   pending xattr value 1.
  * - Consider a brick is down, and the postop sets pending xattrs as long
- *   as the other brick is down. But we do not want to invalidate everytime
- *   a pending xattr is set, but we wan't to inalidate only the first time
+ *   as the other brick is down. But we do not want to invalidate every time
+ *   a pending xattr is set, but we want to invalidate only the first time
  *   a pending xattr is set on any file. Hence, to identify if its the first
  *   time a pending xattr is set, we compare the value of pending xattrs that
  *   came in postop and postop cbk, if its same then its the first time.
@@ -2527,7 +2527,7 @@ struct xlator_fops fops = {
          * notifications are not sent in this case.
          * But however, we need to store/update the
          * client info in the upcall state to be able
-         * to notify them incase of any changes done
+         * to notify them in case of any changes done
          * to the data.
          *
          * Below such fops do not trigger upcall