[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191107155353.GB319242@redhat.com>
Date: Thu, 7 Nov 2019 09:53:53 -0600
From: Bill O'Donnell <billodo@...hat.com>
To: Joe Perches <joe@...ches.com>
Cc: "Darrick J. Wong" <darrick.wong@...cle.com>,
linux-xfs@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] xfs: Correct comment tyops -> typos
On Wed, Nov 06, 2019 at 10:01:15PM -0800, Joe Perches wrote:
> Just fix the typos checkpatch notices...
>
> Signed-off-by: Joe Perches <joe@...ches.com>
Thanks!
Reviewed-by: Bill O'Donnell <billodo@...hat.com>
> ---
> fs/xfs/kmem.c | 2 +-
> fs/xfs/libxfs/xfs_alloc.c | 2 +-
> fs/xfs/libxfs/xfs_attr_leaf.c | 2 +-
> fs/xfs/libxfs/xfs_da_format.h | 2 +-
> fs/xfs/libxfs/xfs_fs.h | 2 +-
> fs/xfs/libxfs/xfs_log_format.h | 4 ++--
> fs/xfs/xfs_buf.c | 2 +-
> fs/xfs/xfs_log_cil.c | 4 ++--
> fs/xfs/xfs_symlink.h | 2 +-
> fs/xfs/xfs_trans_ail.c | 8 ++++----
> 10 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/fs/xfs/kmem.c b/fs/xfs/kmem.c
> index da031b9..1da942 100644
> --- a/fs/xfs/kmem.c
> +++ b/fs/xfs/kmem.c
> @@ -32,7 +32,7 @@ kmem_alloc(size_t size, xfs_km_flags_t flags)
>
>
> /*
> - * __vmalloc() will allocate data pages and auxillary structures (e.g.
> + * __vmalloc() will allocate data pages and auxiliary structures (e.g.
> * pagetables) with GFP_KERNEL, yet we may be under GFP_NOFS context here. Hence
> * we need to tell memory reclaim that we are in such a context via
> * PF_MEMALLOC_NOFS to prevent memory reclaim re-entering the filesystem here
> diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c
> index f7a4b5..b39bd8 100644
> --- a/fs/xfs/libxfs/xfs_alloc.c
> +++ b/fs/xfs/libxfs/xfs_alloc.c
> @@ -1488,7 +1488,7 @@ xfs_alloc_ag_vextent_near(
> dofirst = prandom_u32() & 1;
> #endif
>
> - /* handle unitialized agbno range so caller doesn't have to */
> + /* handle uninitialized agbno range so caller doesn't have to */
> if (!args->min_agbno && !args->max_agbno)
> args->max_agbno = args->mp->m_sb.sb_agblocks - 1;
> ASSERT(args->min_agbno <= args->max_agbno);
> diff --git a/fs/xfs/libxfs/xfs_attr_leaf.c b/fs/xfs/libxfs/xfs_attr_leaf.c
> index dca884..8ba3ae8 100644
> --- a/fs/xfs/libxfs/xfs_attr_leaf.c
> +++ b/fs/xfs/libxfs/xfs_attr_leaf.c
> @@ -829,7 +829,7 @@ xfs_attr_shortform_lookup(xfs_da_args_t *args)
> }
>
> /*
> - * Retreive the attribute value and length.
> + * Retrieve the attribute value and length.
> *
> * If ATTR_KERNOVAL is specified, only the length needs to be returned.
> * Unlike a lookup, we only return an error if the attribute does not
> diff --git a/fs/xfs/libxfs/xfs_da_format.h b/fs/xfs/libxfs/xfs_da_format.h
> index ae654e0..6702a08 100644
> --- a/fs/xfs/libxfs/xfs_da_format.h
> +++ b/fs/xfs/libxfs/xfs_da_format.h
> @@ -482,7 +482,7 @@ xfs_dir2_leaf_bests_p(struct xfs_dir2_leaf_tail *ltp)
> }
>
> /*
> - * Free space block defintions for the node format.
> + * Free space block definitions for the node format.
> */
>
> /*
> diff --git a/fs/xfs/libxfs/xfs_fs.h b/fs/xfs/libxfs/xfs_fs.h
> index e9371a..038a16a 100644
> --- a/fs/xfs/libxfs/xfs_fs.h
> +++ b/fs/xfs/libxfs/xfs_fs.h
> @@ -416,7 +416,7 @@ struct xfs_bulkstat {
>
> /*
> * Project quota id helpers (previously projid was 16bit only
> - * and using two 16bit values to hold new 32bit projid was choosen
> + * and using two 16bit values to hold new 32bit projid was chosen
> * to retain compatibility with "old" filesystems).
> */
> static inline uint32_t
> diff --git a/fs/xfs/libxfs/xfs_log_format.h b/fs/xfs/libxfs/xfs_log_format.h
> index e5f97c6..8ef31d7 100644
> --- a/fs/xfs/libxfs/xfs_log_format.h
> +++ b/fs/xfs/libxfs/xfs_log_format.h
> @@ -432,9 +432,9 @@ static inline uint xfs_log_dinode_size(int version)
> }
>
> /*
> - * Buffer Log Format defintions
> + * Buffer Log Format definitions
> *
> - * These are the physical dirty bitmap defintions for the log format structure.
> + * These are the physical dirty bitmap definitions for the log format structure.
> */
> #define XFS_BLF_CHUNK 128
> #define XFS_BLF_SHIFT 7
> diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c
> index 1e63dd3..2ed3c65 100644
> --- a/fs/xfs/xfs_buf.c
> +++ b/fs/xfs/xfs_buf.c
> @@ -461,7 +461,7 @@ _xfs_buf_map_pages(
> unsigned nofs_flag;
>
> /*
> - * vm_map_ram() will allocate auxillary structures (e.g.
> + * vm_map_ram() will allocate auxiliary structures (e.g.
> * pagetables) with GFP_KERNEL, yet we are likely to be under
> * GFP_NOFS context here. Hence we need to tell memory reclaim
> * that we are in such a context via PF_MEMALLOC_NOFS to prevent
> diff --git a/fs/xfs/xfs_log_cil.c b/fs/xfs/xfs_log_cil.c
> index a120442..48435c 100644
> --- a/fs/xfs/xfs_log_cil.c
> +++ b/fs/xfs/xfs_log_cil.c
> @@ -179,7 +179,7 @@ xlog_cil_alloc_shadow_bufs(
>
> /*
> * We free and allocate here as a realloc would copy
> - * unecessary data. We don't use kmem_zalloc() for the
> + * unnecessary data. We don't use kmem_zalloc() for the
> * same reason - we don't need to zero the data area in
> * the buffer, only the log vector header and the iovec
> * storage.
> @@ -682,7 +682,7 @@ xlog_cil_push(
> }
>
>
> - /* check for a previously pushed seqeunce */
> + /* check for a previously pushed sequence */
> if (push_seq < cil->xc_ctx->sequence) {
> spin_unlock(&cil->xc_push_lock);
> goto out_skip;
> diff --git a/fs/xfs/xfs_symlink.h b/fs/xfs/xfs_symlink.h
> index 9743d8c..b1fa09 100644
> --- a/fs/xfs/xfs_symlink.h
> +++ b/fs/xfs/xfs_symlink.h
> @@ -5,7 +5,7 @@
> #ifndef __XFS_SYMLINK_H
> #define __XFS_SYMLINK_H 1
>
> -/* Kernel only symlink defintions */
> +/* Kernel only symlink definitions */
>
> int xfs_symlink(struct xfs_inode *dp, struct xfs_name *link_name,
> const char *target_path, umode_t mode, struct xfs_inode **ipp);
> diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c
> index aea71e..00cc5b 100644
> --- a/fs/xfs/xfs_trans_ail.c
> +++ b/fs/xfs/xfs_trans_ail.c
> @@ -427,15 +427,15 @@ xfsaild_push(
>
> case XFS_ITEM_FLUSHING:
> /*
> - * The item or its backing buffer is already beeing
> + * The item or its backing buffer is already being
> * flushed. The typical reason for that is that an
> * inode buffer is locked because we already pushed the
> * updates to it as part of inode clustering.
> *
> * We do not want to to stop flushing just because lots
> - * of items are already beeing flushed, but we need to
> + * of items are already being flushed, but we need to
> * re-try the flushing relatively soon if most of the
> - * AIL is beeing flushed.
> + * AIL is being flushed.
> */
> XFS_STATS_INC(mp, xs_push_ail_flushing);
> trace_xfs_ail_flushing(lip);
> @@ -612,7 +612,7 @@ xfsaild(
> * The push is run asynchronously in a workqueue, which means the caller needs
> * to handle waiting on the async flush for space to become available.
> * We don't want to interrupt any push that is in progress, hence we only queue
> - * work if we set the pushing bit approriately.
> + * work if we set the pushing bit appropriately.
> *
> * We do this unlocked - we only need to know whether there is anything in the
> * AIL at the time we are called. We don't need to access the contents of
>
>
Powered by blists - more mailing lists