Linux-ext4 Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] ext4: fix spelling mistakes in extents.c
@ 2020-06-11  3:19 Keyur Patel
  2020-07-08 13:44 ` Jan Kara
  2020-07-29 20:12 ` tytso
  0 siblings, 2 replies; 3+ messages in thread
From: Keyur Patel @ 2020-06-11  3:19 UTC (permalink / raw)
  To: Theodore Ts'o, Andreas Dilger, linux-ext4, linux-kernel; +Cc: Keyur Patel

Fix spelling issues over the comments in the code.

requsted ==> requested
deterimined ==> determined
insde ==> inside
neet ==> need
somthing ==> something

Signed-off-by: Keyur Patel <iamkeyur96@gmail.com>
---
 fs/ext4/extents.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
index 7d088ff1e902..8028e1aae8de 100644
--- a/fs/ext4/extents.c
+++ b/fs/ext4/extents.c
@@ -1915,7 +1915,7 @@ static unsigned int ext4_ext_check_overlap(struct ext4_sb_info *sbi,
 
 /*
  * ext4_ext_insert_extent:
- * tries to merge requsted extent into the existing extent or
+ * tries to merge requested extent into the existing extent or
  * inserts requested extent as new one into the tree,
  * creating new leaf in the no-space case.
  */
@@ -3125,7 +3125,7 @@ static int ext4_ext_zeroout(struct inode *inode, struct ext4_extent *ex)
  *
  *
  * Splits extent [a, b] into two extents [a, @split) and [@split, b], states
- * of which are deterimined by split_flag.
+ * of which are determined by split_flag.
  *
  * There are two cases:
  *  a> the extent are splitted into two extent.
@@ -3650,7 +3650,7 @@ static int ext4_split_convert_extents(handle_t *handle,
 		eof_block = map->m_lblk + map->m_len;
 	/*
 	 * It is safe to convert extent to initialized via explicit
-	 * zeroout only if extent is fully insde i_size or new_size.
+	 * zeroout only if extent is fully inside i_size or new_size.
 	 */
 	depth = ext_depth(inode);
 	ex = path[depth].p_ext;
@@ -4495,7 +4495,7 @@ static long ext4_zero_range(struct file *file, loff_t offset,
 	}
 
 	/*
-	 * Round up offset. This is not fallocate, we neet to zero out
+	 * Round up offset. This is not fallocate, we need to zero out
 	 * blocks, so convert interior block aligned part of the range to
 	 * unwritten and possibly manually zero out unaligned parts of the
 	 * range.
@@ -5579,7 +5579,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
 		}
 		ex1 = path1[path1->p_depth].p_ext;
 		ex2 = path2[path2->p_depth].p_ext;
-		/* Do we have somthing to swap ? */
+		/* Do we have something to swap ? */
 		if (unlikely(!ex2 || !ex1))
 			goto finish;
 
-- 
2.26.2


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ext4: fix spelling mistakes in extents.c
  2020-06-11  3:19 [PATCH] ext4: fix spelling mistakes in extents.c Keyur Patel
@ 2020-07-08 13:44 ` Jan Kara
  2020-07-29 20:12 ` tytso
  1 sibling, 0 replies; 3+ messages in thread
From: Jan Kara @ 2020-07-08 13:44 UTC (permalink / raw)
  To: Keyur Patel; +Cc: Theodore Ts'o, Andreas Dilger, linux-ext4, linux-kernel

On Wed 10-06-20 23:19:46, Keyur Patel wrote:
> Fix spelling issues over the comments in the code.
> 
> requsted ==> requested
> deterimined ==> determined
> insde ==> inside
> neet ==> need
> somthing ==> something
> 
> Signed-off-by: Keyur Patel <iamkeyur96@gmail.com>

Thanks. The fixes look good. You can add:

Reviewed-by: Jan Kara <jack@suse.cz>

Ted, I think this patch has fallen through the cracks...

								Honza
> ---
>  fs/ext4/extents.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
> index 7d088ff1e902..8028e1aae8de 100644
> --- a/fs/ext4/extents.c
> +++ b/fs/ext4/extents.c
> @@ -1915,7 +1915,7 @@ static unsigned int ext4_ext_check_overlap(struct ext4_sb_info *sbi,
>  
>  /*
>   * ext4_ext_insert_extent:
> - * tries to merge requsted extent into the existing extent or
> + * tries to merge requested extent into the existing extent or
>   * inserts requested extent as new one into the tree,
>   * creating new leaf in the no-space case.
>   */
> @@ -3125,7 +3125,7 @@ static int ext4_ext_zeroout(struct inode *inode, struct ext4_extent *ex)
>   *
>   *
>   * Splits extent [a, b] into two extents [a, @split) and [@split, b], states
> - * of which are deterimined by split_flag.
> + * of which are determined by split_flag.
>   *
>   * There are two cases:
>   *  a> the extent are splitted into two extent.
> @@ -3650,7 +3650,7 @@ static int ext4_split_convert_extents(handle_t *handle,
>  		eof_block = map->m_lblk + map->m_len;
>  	/*
>  	 * It is safe to convert extent to initialized via explicit
> -	 * zeroout only if extent is fully insde i_size or new_size.
> +	 * zeroout only if extent is fully inside i_size or new_size.
>  	 */
>  	depth = ext_depth(inode);
>  	ex = path[depth].p_ext;
> @@ -4495,7 +4495,7 @@ static long ext4_zero_range(struct file *file, loff_t offset,
>  	}
>  
>  	/*
> -	 * Round up offset. This is not fallocate, we neet to zero out
> +	 * Round up offset. This is not fallocate, we need to zero out
>  	 * blocks, so convert interior block aligned part of the range to
>  	 * unwritten and possibly manually zero out unaligned parts of the
>  	 * range.
> @@ -5579,7 +5579,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
>  		}
>  		ex1 = path1[path1->p_depth].p_ext;
>  		ex2 = path2[path2->p_depth].p_ext;
> -		/* Do we have somthing to swap ? */
> +		/* Do we have something to swap ? */
>  		if (unlikely(!ex2 || !ex1))
>  			goto finish;
>  
> -- 
> 2.26.2
> 
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ext4: fix spelling mistakes in extents.c
  2020-06-11  3:19 [PATCH] ext4: fix spelling mistakes in extents.c Keyur Patel
  2020-07-08 13:44 ` Jan Kara
@ 2020-07-29 20:12 ` tytso
  1 sibling, 0 replies; 3+ messages in thread
From: tytso @ 2020-07-29 20:12 UTC (permalink / raw)
  To: Keyur Patel; +Cc: Andreas Dilger, linux-ext4, linux-kernel

On Wed, Jun 10, 2020 at 11:19:46PM -0400, Keyur Patel wrote:
> Fix spelling issues over the comments in the code.
> 
> requsted ==> requested
> deterimined ==> determined
> insde ==> inside
> neet ==> need
> somthing ==> something
> 
> Signed-off-by: Keyur Patel <iamkeyur96@gmail.com>

Applied, thanks.

						- Ted

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-11  3:19 [PATCH] ext4: fix spelling mistakes in extents.c Keyur Patel
2020-07-08 13:44 ` Jan Kara
2020-07-29 20:12 ` tytso

Linux-ext4 Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-ext4/0 linux-ext4/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-ext4 linux-ext4/ https://lore.kernel.org/linux-ext4 \
		linux-ext4@vger.kernel.org
	public-inbox-index linux-ext4

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-ext4


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git