[PATCH] xfs: Fix comment typo

[PATCH] xfs: Fix comment typo

[Xin Gao]

The double `the' is duplicated in line 552, remove one.

Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
---
 fs/xfs/xfs_dquot.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/xfs_dquot.c b/fs/xfs/xfs_dquot.c
index 5a6c3c3c4de2..8fb90da89787 100644
--- a/fs/xfs/xfs_dquot.c
+++ b/fs/xfs/xfs_dquot.c
@@ -549,7 +549,7 @@ xfs_dquot_check_type(
 	 * at the same time.  The non-user quota file can be switched between
 	 * group and project quota uses depending on the mount options, which
 	 * means that we can encounter the other type when we try to load quota
-	 * defaults.  Quotacheck will soon reset the the entire quota file
+	 * defaults.  Quotacheck will soon reset the entire quota file
 	 * (including the root dquot) anyway, but don't log scary corruption
 	 * reports to dmesg.
 	 */
-- 
2.30.2

Re: [PATCH] xfs: Fix comment typo

[Darrick J. Wong]

On Fri, Jul 22, 2022 at 10:26:26AM +0800, Xin Gao wrote:
> The double `the' is duplicated in line 552, remove one.
> 
> Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>

Applied, thanks.
Reviewed-by: Darrick J. Wong <djwong@kernel.org>

--D

> ---
>  fs/xfs/xfs_dquot.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/xfs/xfs_dquot.c b/fs/xfs/xfs_dquot.c
> index 5a6c3c3c4de2..8fb90da89787 100644
> --- a/fs/xfs/xfs_dquot.c
> +++ b/fs/xfs/xfs_dquot.c
> @@ -549,7 +549,7 @@ xfs_dquot_check_type(
>  	 * at the same time.  The non-user quota file can be switched between
>  	 * group and project quota uses depending on the mount options, which
>  	 * means that we can encounter the other type when we try to load quota
> -	 * defaults.  Quotacheck will soon reset the the entire quota file
> +	 * defaults.  Quotacheck will soon reset the entire quota file
>  	 * (including the root dquot) anyway, but don't log scary corruption
>  	 * reports to dmesg.
>  	 */
> -- 
> 2.30.2
> 

Re: [PATCH] xfs: Fix comment typo

[Eric Sandeen]

On 7/22/22 2:43 PM, Xin Gao wrote:
> The double `that' is duplicated in line 575, remove one.
> 
> Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
> ---
>  fs/xfs/xfs_trans.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c
> index 82cf0189c0db..d055b0938eb9 100644
> --- a/fs/xfs/xfs_trans.c
> +++ b/fs/xfs/xfs_trans.c
> @@ -572,7 +572,7 @@ xfs_trans_apply_sb_deltas(
>   * xfs_trans_unreserve_and_mod_sb() is called to release unused reservations and
>   * apply superblock counter changes to the in-core superblock.  The
>   * t_res_fdblocks_delta and t_res_frextents_delta fields are explicitly NOT
> - * applied to the in-core superblock.  The idea is that that has already been
> + * applied to the in-core superblock.  The idea is that has already been
>   * done.
>   *
>   * If we are not logging superblock counters, then the inode allocated/free and

NAK

The comment is correct

Re: [PATCH] xfs: Fix comment typo

[Eric Sandeen]

On 7/22/22 2:36 PM, Xin Gao wrote:
> The double `for' is duplicated in line 3788, remove one.
> 
> Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
> ---
>  fs/xfs/libxfs/xfs_bmap.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
> index 6833110d1bd4..6f6d87ba29a3 100644
> --- a/fs/xfs/libxfs/xfs_bmap.c
> +++ b/fs/xfs/libxfs/xfs_bmap.c
> @@ -3785,7 +3785,7 @@ xfs_bmapi_trim_map(
>  		mval->br_startblock = got->br_startblock +
>  					(*bno - got->br_startoff);
>  	/*
> -	 * Return the minimum of what we got and what we asked for for
> +	 * Return the minimum of what we got and what we asked for
>  	 * the length.  We can use the len variable here because it is
>  	 * modified below and we could have been there before coming
>  	 * here if the first part of the allocation didn't overlap what

NAK.

The comment is correct.

[PATCH] xfs: Fix comment typo

[Xin Gao]

The double `that' is duplicated in line 575, remove one.

Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
---
 fs/xfs/xfs_trans.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c
index 82cf0189c0db..d055b0938eb9 100644
--- a/fs/xfs/xfs_trans.c
+++ b/fs/xfs/xfs_trans.c
@@ -572,7 +572,7 @@ xfs_trans_apply_sb_deltas(
  * xfs_trans_unreserve_and_mod_sb() is called to release unused reservations and
  * apply superblock counter changes to the in-core superblock.  The
  * t_res_fdblocks_delta and t_res_frextents_delta fields are explicitly NOT
- * applied to the in-core superblock.  The idea is that that has already been
+ * applied to the in-core superblock.  The idea is that has already been
  * done.
  *
  * If we are not logging superblock counters, then the inode allocated/free and
-- 
2.30.2

[PATCH] xfs: Fix comment typo

[Xin Gao]

The double `extent' is duplicated in line 837, remove one.

Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
---
 fs/xfs/libxfs/xfs_alloc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c
index d3f2886fdc08..f6bd9181cb13 100644
--- a/fs/xfs/libxfs/xfs_alloc.c
+++ b/fs/xfs/libxfs/xfs_alloc.c
@@ -833,7 +833,7 @@ xfs_alloc_cur_close(
 /*
  * Check an extent for allocation and track the best available candidate in the
  * allocation structure. The cursor is deactivated if it has entered an out of
- * range state based on allocation arguments. Optionally return the extent
+ * range state based on allocation arguments. Optionally return the
  * extent geometry and allocation status if requested by the caller.
  */
 static int
-- 
2.30.2

[PATCH] xfs: Fix comment typo

[Xin Gao]

The double `for' is duplicated in line 3788, remove one.

Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
---
 fs/xfs/libxfs/xfs_bmap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index 6833110d1bd4..6f6d87ba29a3 100644
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -3785,7 +3785,7 @@ xfs_bmapi_trim_map(
 		mval->br_startblock = got->br_startblock +
 					(*bno - got->br_startoff);
 	/*
-	 * Return the minimum of what we got and what we asked for for
+	 * Return the minimum of what we got and what we asked for
 	 * the length.  We can use the len variable here because it is
 	 * modified below and we could have been there before coming
 	 * here if the first part of the allocation didn't overlap what
-- 
2.30.2

[PATCH] xfs: Fix comment typo

[Xin Gao]

The double `being' is duplicated in line 553, remove one.

Signed-off-by: Xin Gao <gaoxin@cdjrlc.com>
---
 fs/xfs/xfs_inode_item.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/xfs_inode_item.c b/fs/xfs/xfs_inode_item.c
index 721def0639fd..d1c5d0658779 100644
--- a/fs/xfs/xfs_inode_item.c
+++ b/fs/xfs/xfs_inode_item.c
@@ -550,7 +550,7 @@ xfs_inode_item_push(
 
 	if (!bp || (ip->i_flags & XFS_ISTALE)) {
 		/*
-		 * Inode item/buffer is being being aborted due to cluster
+		 * Inode item/buffer is being aborted due to cluster
 		 * buffer deletion. Trigger a log force to have that operation
 		 * completed and items removed from the AIL before the next push
 		 * attempt.
-- 
2.30.2