public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Bill O'Donnell <billodo@redhat.com>
To: Joe Perches <joe@perches.com>
Cc: "Darrick J. Wong" <darrick.wong@oracle.com>,
	linux-xfs@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] xfs: Correct comment tyops -> typos
Date: Thu, 7 Nov 2019 09:53:53 -0600	[thread overview]
Message-ID: <20191107155353.GB319242@redhat.com> (raw)
In-Reply-To: <0ceb6a89da4424a4500789610fae4d05ba45ba86.camel@perches.com>

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@perches.com>

Thanks!
Reviewed-by: Bill O'Donnell <billodo@redhat.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
> 
> 


  reply	other threads:[~2019-11-07 15:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-07  6:01 [PATCH] xfs: Correct comment tyops -> typos Joe Perches
2019-11-07 15:53 ` Bill O'Donnell [this message]
2019-11-07 21:25 ` Darrick J. Wong
2019-11-07 21:56   ` Joe Perches
2019-11-07 22:03     ` Darrick J. Wong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191107155353.GB319242@redhat.com \
    --to=billodo@redhat.com \
    --cc=darrick.wong@oracle.com \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-xfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox