Skip to contentSkip to conversation

EXT4: Trivial typo fixes

PATCH Applied Bhaskar Chowdhury 3 participants 12 messages Mar 27, 2021 – Apr 10, 2021 ext4 file system

lore
Share & tools
lore
#0 lore raw

cover EXT4: Trivial typo fixes

This patch series fixed few mundane typos in the below specific files.

Bhaskar Chowdhury (8):
EXT4: migrate.c: Fixed few typos
EXT4: namei.c: Fixed a typo
EXT4: inline.c: A typo fixed
Fix a typo
EXT4: indirect.c: A trivial typo fix
EXT4: xattr.c: Mundane typo fix
EXT4: fast_commit.c: A mere typo fix
EXT4: mballoc.h: Single typo fix

 fs/ext4/fast_commit.c | 2 +-
 fs/ext4/indirect.c    | 2 +-
 fs/ext4/inline.c      | 2 +-
 fs/ext4/inode.c       | 2 +-
 fs/ext4/mballoc.h     | 2 +-
 fs/ext4/migrate.c     | 6 +++---
 fs/ext4/namei.c       | 8 ++++----
 fs/ext4/xattr.c       | 2 +-
 8 files changed, 13 insertions(+), 13 deletions(-)

--
2.26.2

#1 lore raw

PATCH 1/8 EXT4: migrate.c: Fixed few typos

s/convinience/convenience/ s/accumalate/accumulate/ .....two differnt places.

---
 fs/ext4/migrate.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
fs/ext4/migrate.c +3 -3
index c5e3fc998211..7e0b4f81c6c0 100644
@@ -32,7 +32,7 @@ static int finish_range(handle_t *handle, struct inode *inode,
 	newext.ee_block = cpu_to_le32(lb->first_block);
 	newext.ee_len   = cpu_to_le16(lb->last_block - lb->first_block + 1);
 	ext4_ext_store_pblock(&newext, lb->first_pblock);
-	/* Locking only for convinience since we are operating on temp inode */
+	/* Locking only for convenience since we are operating on temp inode */
 	down_write(&EXT4_I(inode)->i_data_sem);
 	path = ext4_find_extent(inode, lb->first_block, NULL, 0);
 	if (IS_ERR(path)) {
@@ -43,8 +43,8 @@ static int finish_range(handle_t *handle, struct inode *inode,
 
 	/*
 	 * Calculate the credit needed to inserting this extent
-	 * Since we are doing this in loop we may accumalate extra
-	 * credit. But below we try to not accumalate too much
+	 * Since we are doing this in loop we may accumulate extra
+	 * credit. But below we try to not accumulate too much
 	 * of them by restarting the journal.
 	 */
 	needed = ext4_ext_calc_credits_for_single_extent(inode,

--
2.26.2

#2 lore raw

PATCH 2/8 EXT4: namei.c: Fixed a typo

In reply to Bhaskar Chowdhury

s/contiuation/continuation/

---
 fs/ext4/namei.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
fs/ext4/namei.c +1 -1
index 883e2a7cd4ab..3eb07053ca3c 100644
@@ -956,7 +956,7 @@ static int ext4_htree_next_block(struct inode *dir, __u32 hash,
 	 * If the hash is 1, then continue only if the next page has a
 	 * continuation hash of any value.  This is used for readdir
 	 * handling.  Otherwise, check to see if the hash matches the
-	 * desired contiuation hash.  If it doesn't, return since
+	 * desired continuation hash.  If it doesn't, return since
 	 * there's no point to read in the successive index pages.
 	 */
 	bhash = dx_get_hash(p->at);

--
2.26.2

#3 lore raw

PATCH 3/8 EXT4: inline.c: A typo fixed

In reply to Bhaskar Chowdhury

s/metatdata/metadata/

---
 fs/ext4/inline.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/inline.c +1 -1
index b41512d1badc..fbc5086461e1 100644
@@ -795,7 +795,7 @@ ext4_journalled_write_inline_data(struct inode *inode,
  *    clear the inode state safely.
  * 2. The inode has inline data, then we need to read the data, make it
  *    update and dirty so that ext4_da_writepages can handle it. We don't
- *    need to start the journal since the file's metatdata isn't changed now.
+ *    need to start the journal since the file's metadata isn't changed now.
  */
 static int ext4_da_convert_inline_data_to_extent(struct address_space *mapping,
 						 struct inode *inode,

--
2.26.2

#4 lore raw

PATCH 4/8 Fix a typo

In reply to Bhaskar Chowdhury

s/cooresponds/corresponds/

---
 fs/ext4/inode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/inode.c +1 -1
index 0948a43f1b3d..470d310b9b77 100644
@@ -3824,7 +3824,7 @@ static int __ext4_block_zero_page_range(handle_t *handle,
  * starting from file offset 'from'.  The range to be zero'd must
  * be contained with in one block.  If the specified range exceeds
  * the end of the block it will be shortened to end of the block
- * that cooresponds to 'from'
+ * that corresponds to 'from'
  */
 static int ext4_block_zero_page_range(handle_t *handle,
 		struct address_space *mapping, loff_t from, loff_t length)

--
2.26.2

#5 lore raw

PATCH 5/8 EXT4: indirect.c: A trivial typo fix

In reply to Bhaskar Chowdhury

s/conventient/convenient/

---
 fs/ext4/indirect.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/indirect.c +1 -1
index 1223a18c3ff9..a7bc6ad656a9 100644
@@ -705,7 +705,7 @@ static int ext4_ind_trunc_restart_fn(handle_t *handle, struct inode *inode,
 
 /*
  * Truncate transactions can be complex and absolutely huge.  So we need to
- * be able to restart the transaction at a conventient checkpoint to make
+ * be able to restart the transaction at a convenient checkpoint to make
  * sure we don't overflow the journal.
  *
  * Try to extend this transaction for the purposes of truncation.  If

--
2.26.2

#6 lore raw

PATCH 6/8 EXT4: xattr.c: Mundane typo fix

In reply to Bhaskar Chowdhury

s/futher/further/

---
 fs/ext4/xattr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/xattr.c +1 -1
index 6c1018223c54..10ba4b24a0aa 100644
@@ -1617,7 +1617,7 @@ static int ext4_xattr_set_entry(struct ext4_xattr_info *i,
 		 * If storing the value in an external inode is an option,
 		 * reserve space for xattr entries/names in the external
 		 * attribute block so that a long value does not occupy the
-		 * whole space and prevent futher entries being added.
+		 * whole space and prevent further entries being added.
 		 */
 		if (ext4_has_feature_ea_inode(inode->i_sb) &&
 		    new_size && is_block &&

--
2.26.2

#7 lore raw

PATCH 7/8 EXT4: fast_commit.c: A mere typo fix

In reply to Bhaskar Chowdhury

s/ineligiblity/ineligibility/

---
 fs/ext4/fast_commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/fast_commit.c +1 -1
index 7541d0b5d706..8691634044e3 100644
@@ -66,7 +66,7 @@
  * Fast Commit Ineligibility
  * -------------------------
  * Not all operations are supported by fast commits today (e.g extended
- * attributes). Fast commit ineligiblity is marked by calling one of the
+ * attributes). Fast commit ineligibility is marked by calling one of the
  * two following functions:
  *
  * - ext4_fc_mark_ineligible(): This makes next fast commit operation to fall

--
2.26.2

#8 lore raw

PATCH 8/8 EXT4: mballoc.h: Single typo fix

In reply to Bhaskar Chowdhury

s/parition/partition/

---
 fs/ext4/mballoc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ext4/mballoc.h +1 -1
index e75b4749aa1c..7be6288e48ec 100644
@@ -59,7 +59,7 @@
  * by the stream allocator, which purpose is to pack requests
  * as close each to other as possible to produce smooth I/O traffic
  * We use locality group prealloc space for stream request.
- * We can tune the same via /proc/fs/ext4/<parition>/stream_req
+ * We can tune the same via /proc/fs/ext4/<partition>/stream_req
  */
 #define MB_DEFAULT_STREAM_THRESHOLD	16	/* 64K */
 

--
2.26.2

#9 lore raw
In reply to Bhaskar Chowdhury
Quoted 12 lines · Bhaskar Chowdhury

This patch series fixed few mundane typos in the below specific files.

Bhaskar Chowdhury (8):
EXT4: migrate.c: Fixed few typos
EXT4: namei.c: Fixed a typo
EXT4: inline.c: A typo fixed
Fix a typo
EXT4: indirect.c: A trivial typo fix
EXT4: xattr.c: Mundane typo fix
EXT4: fast_commit.c: A mere typo fix
EXT4: mballoc.h: Single typo fix

Quoted 9 lines
 fs/ext4/fast_commit.c | 2 +-
 fs/ext4/indirect.c    | 2 +-
 fs/ext4/inline.c      | 2 +-
 fs/ext4/inode.c       | 2 +-
 fs/ext4/mballoc.h     | 2 +-
 fs/ext4/migrate.c     | 6 +++---
 fs/ext4/namei.c       | 8 ++++----
 fs/ext4/xattr.c       | 2 +-
 8 files changed, 13 insertions(+), 13 deletions(-)

Thanks,
since these are trivial typo fixes, why not just 1 patch for fs/ext4/ IMO, that way it is easier for sender, reviewer and maintainer.

-ritesh

#10 lore raw
Quoted 23 lines · Ritesh Harjani
Quoted 12 lines · Bhaskar Chowdhury

This patch series fixed few mundane typos in the below specific files.

Bhaskar Chowdhury (8):
EXT4: migrate.c: Fixed few typos
EXT4: namei.c: Fixed a typo
EXT4: inline.c: A typo fixed
Fix a typo
EXT4: indirect.c: A trivial typo fix
EXT4: xattr.c: Mundane typo fix
EXT4: fast_commit.c: A mere typo fix
EXT4: mballoc.h: Single typo fix

Quoted 9 lines
 fs/ext4/fast_commit.c | 2 +-
 fs/ext4/indirect.c    | 2 +-
 fs/ext4/inline.c      | 2 +-
 fs/ext4/inode.c       | 2 +-
 fs/ext4/mballoc.h     | 2 +-
 fs/ext4/migrate.c     | 6 +++---
 fs/ext4/namei.c       | 8 ++++----
 fs/ext4/xattr.c       | 2 +-
 8 files changed, 13 insertions(+), 13 deletions(-)
Quoted 4 lines

Thanks,
since these are trivial typo fixes, why not just 1 patch for fs/ext4/ IMO, that way it is easier for sender, reviewer and maintainer.

I have got the other vibes too ..hence the decision to send it like this way..

~Bhaskar

Quoted 1 line

-ritesh

maintainer Applied
#11 lore raw
In reply to Bhaskar Chowdhury
Quoted 12 lines · Bhaskar Chowdhury

This patch series fixed few mundane typos in the below specific files.

Bhaskar Chowdhury (8):
EXT4: migrate.c: Fixed few typos
EXT4: namei.c: Fixed a typo
EXT4: inline.c: A typo fixed
Fix a typo
EXT4: indirect.c: A trivial typo fix
EXT4: xattr.c: Mundane typo fix
EXT4: fast_commit.c: A mere typo fix
EXT4: mballoc.h: Single typo fix

Quoted 9 lines
 fs/ext4/fast_commit.c | 2 +-
 fs/ext4/indirect.c    | 2 +-
 fs/ext4/inline.c      | 2 +-
 fs/ext4/inode.c       | 2 +-
 fs/ext4/mballoc.h     | 2 +-
 fs/ext4/migrate.c     | 6 +++---
 fs/ext4/namei.c       | 8 ++++----
 fs/ext4/xattr.c       | 2 +-
 8 files changed, 13 insertions(+), 13 deletions(-)

I've applied this patch series folded together into a single patch.

- Ted