2023-06-28 08:33:55

by Immad Mir

[permalink] [raw]
Subject: [PATCH] FS: JFS: (trivial) Fix grammatical error in extAlloc

From: Immad Mir <[email protected]>

There is a grammatical error in one the commnents
in extAlloc function.

Signed-off-by: Immad Mir <[email protected]>
---
fs/jfs/jfs_extent.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/jfs/jfs_extent.c b/fs/jfs/jfs_extent.c
index ae99a7e23..399492308 100644
--- a/fs/jfs/jfs_extent.c
+++ b/fs/jfs/jfs_extent.c
@@ -166,7 +166,7 @@ extAlloc(struct inode *ip, s64 xlen, s64 pno, xad_t * xp, bool abnr)
/*
* COMMIT_SyncList flags an anonymous tlock on page that is on
* sync list.
- * We need to commit the inode to get the page written disk.
+ * We need to commit the inode to get the page written to the disk.
*/
if (test_and_clear_cflag(COMMIT_Synclist,ip))
jfs_commit_inode(ip, 0);
--
2.40.0



2023-07-12 09:21:08

by Immad Mir

[permalink] [raw]
Subject: Re: [PATCH] FS: JFS: (trivial) Fix grammatical error in extAlloc


Hi, May I please request a review on this patch.


On 28/06/23 1:14 pm, [email protected] wrote:
> From: Immad Mir <[email protected]>
>
> There is a grammatical error in one the commnents
> in extAlloc function.
>
> Signed-off-by: Immad Mir <[email protected]>
> ---
> fs/jfs/jfs_extent.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/jfs/jfs_extent.c b/fs/jfs/jfs_extent.c
> index ae99a7e23..399492308 100644
> --- a/fs/jfs/jfs_extent.c
> +++ b/fs/jfs/jfs_extent.c
> @@ -166,7 +166,7 @@ extAlloc(struct inode *ip, s64 xlen, s64 pno, xad_t * xp, bool abnr)
> /*
> * COMMIT_SyncList flags an anonymous tlock on page that is on
> * sync list.
> - * We need to commit the inode to get the page written disk.
> + * We need to commit the inode to get the page written to the disk.
> */
> if (test_and_clear_cflag(COMMIT_Synclist,ip))
> jfs_commit_inode(ip, 0);
> --
> 2.40.0
>

2023-07-18 20:44:23

by Dave Kleikamp

[permalink] [raw]
Subject: Re: [PATCH] FS: JFS: (trivial) Fix grammatical error in extAlloc

On 6/28/23 2:44AM, [email protected] wrote:
> From: Immad Mir <[email protected]>
>
> There is a grammatical error in one the commnents
> in extAlloc function.
>
> Signed-off-by: Immad Mir <[email protected]>

Applied.

Thanks,
Shaggy

> ---
> fs/jfs/jfs_extent.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/jfs/jfs_extent.c b/fs/jfs/jfs_extent.c
> index ae99a7e23..399492308 100644
> --- a/fs/jfs/jfs_extent.c
> +++ b/fs/jfs/jfs_extent.c
> @@ -166,7 +166,7 @@ extAlloc(struct inode *ip, s64 xlen, s64 pno, xad_t * xp, bool abnr)
> /*
> * COMMIT_SyncList flags an anonymous tlock on page that is on
> * sync list.
> - * We need to commit the inode to get the page written disk.
> + * We need to commit the inode to get the page written to the disk.
> */
> if (test_and_clear_cflag(COMMIT_Synclist,ip))
> jfs_commit_inode(ip, 0);
> --
> 2.40.0
>