* [PATCH 00/12] xfs: another chunk of comment fixes
@ 2013-08-12 3:14 zwu.kernel
2013-08-12 3:14 ` [PATCH 01/12] xfs: fix the comment of xfs_bmap_count_tree() zwu.kernel
` (13 more replies)
0 siblings, 14 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Submit another chunk of xfs comment fixes.
Zhi Yong Wu (12):
xfs: fix the comment of xfs_bmap_count_tree()
xfs: fix the comment of xfs_bmap_validate_ret()
xfs: fix the comment of xfs_bmap_last_before()
xfs: fix the comment of xfs_bmap_punch_delalloc_range()
xfs: fix the comment of xfs_setsize_buftarg_early()
xfs: fix the comment of xfs_extent_busy_update_extent()
xfs: fix the comment of xfs_ialloc_ag_select()
xfs: fix the comment of xfs_ifree_cluster()
xfs: fix the comment of xfs_log_unmount_write()
xfs: fix the comment of xlog_recover_do_dquot_buffer()
xfs: fix the comment of xfs_sb_quiet_read_verify()
xfs: fix the comment of xfs_mountfs()
fs/xfs/xfs_bmap.c | 10 +++++-----
fs/xfs/xfs_buf.c | 2 +-
fs/xfs/xfs_extent_busy.c | 2 +-
fs/xfs/xfs_ialloc.c | 2 +-
fs/xfs/xfs_inode.c | 2 +-
fs/xfs/xfs_log.c | 2 +-
fs/xfs/xfs_log_recover.c | 2 +-
fs/xfs/xfs_mount.c | 4 ++--
8 files changed, 13 insertions(+), 13 deletions(-)
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 01/12] xfs: fix the comment of xfs_bmap_count_tree()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 02/12] xfs: fix the comment of xfs_bmap_validate_ret() zwu.kernel
` (12 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_bmap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index 05c698c..75bb841 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -306,7 +306,7 @@ xfs_bmap_disk_count_leaves(
/*
* Recursively walks each level of a btree
- * to count total fsblocks is use.
+ * to count total fsblocks in use.
*/
STATIC int /* error */
xfs_bmap_count_tree(
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 02/12] xfs: fix the comment of xfs_bmap_validate_ret()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
2013-08-12 3:14 ` [PATCH 01/12] xfs: fix the comment of xfs_bmap_count_tree() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 03/12] xfs: fix the comment of xfs_bmap_last_before() zwu.kernel
` (11 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_bmap.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index 75bb841..b8a0537 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -724,8 +724,8 @@ xfs_bmap_trace_exlist(
/*
* Validate that the bmbt_irecs being returned from bmapi are valid
- * given the callers original parameters. Specifically check the
- * ranges of the returned irecs to ensure that they only extent beyond
+ * given the caller's original parameters. Specifically check the
+ * ranges of the returned irecs to ensure that they only extend beyond
* the given parameters if the XFS_BMAPI_ENTIRE flag was set.
*/
STATIC void
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 03/12] xfs: fix the comment of xfs_bmap_last_before()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
2013-08-12 3:14 ` [PATCH 01/12] xfs: fix the comment of xfs_bmap_count_tree() zwu.kernel
2013-08-12 3:14 ` [PATCH 02/12] xfs: fix the comment of xfs_bmap_validate_ret() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 04/12] xfs: fix the comment of xfs_bmap_punch_delalloc_range() zwu.kernel
` (10 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_bmap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index b8a0537..0a838a3 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -1815,7 +1815,7 @@ xfs_bmap_first_unused(
}
/*
- * Returns the file-relative block number of the last block + 1 before
+ * Returns the file-relative block number of the last block - 1 before
* last_block (input value) in the file.
* This is not based on i_size, it is based on the extent records.
* Returns 0 for local files, as they do not have extent records.
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 04/12] xfs: fix the comment of xfs_bmap_punch_delalloc_range()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (2 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 03/12] xfs: fix the comment of xfs_bmap_last_before() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 05/12] xfs: fix the comment of xfs_setsize_buftarg_early() zwu.kernel
` (9 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_bmap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index 0a838a3..1d077c2 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -6074,7 +6074,7 @@ xfs_getbmap(
/*
* dead simple method of punching delalyed allocation blocks from a range in
* the inode. Walks a block at a time so will be slow, but is only executed in
- * rare error cases so the overhead is not critical. This will alays punch out
+ * rare error cases so the overhead is not critical. This will always punch out
* both the start and end blocks, even if the ranges only partially overlap
* them, so it is up to the caller to ensure that partial blocks are not
* passed in.
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 05/12] xfs: fix the comment of xfs_setsize_buftarg_early()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (3 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 04/12] xfs: fix the comment of xfs_bmap_punch_delalloc_range() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent() zwu.kernel
` (8 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_buf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c
index 3997571..e2e82d9 100644
--- a/fs/xfs/xfs_buf.c
+++ b/fs/xfs/xfs_buf.c
@@ -1621,7 +1621,7 @@ xfs_setsize_buftarg_flags(
/*
* When allocating the initial buffer target we have not yet
* read in the superblock, so don't know what sized sectors
- * are being used is at this early stage. Play safe.
+ * are being used at this early stage. Play safe.
*/
STATIC int
xfs_setsize_buftarg_early(
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (4 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 05/12] xfs: fix the comment of xfs_setsize_buftarg_early() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-19 18:27 ` Mark Tinguely
2013-08-12 3:14 ` [PATCH 07/12] xfs: fix the comment of xfs_ialloc_ag_select() zwu.kernel
` (7 subsequent siblings)
13 siblings, 1 reply; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_extent_busy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_extent_busy.c b/fs/xfs/xfs_extent_busy.c
index 85e9f87..4ad8fee 100644
--- a/fs/xfs/xfs_extent_busy.c
+++ b/fs/xfs/xfs_extent_busy.c
@@ -147,7 +147,7 @@ xfs_extent_busy_search(
* extent. If the overlap covers the beginning, the end, or all of the busy
* extent, the overlapping portion can be made unbusy and used for the
* allocation. We can't split a busy extent because we can't modify a
- * transaction/CIL context busy list, but we can update an entries block
+ * transaction/CIL context busy list, but we can update an entrie's block
* number or length.
*
* Returns true if the extent can safely be reused, or false if the search
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 07/12] xfs: fix the comment of xfs_ialloc_ag_select()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (5 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:14 ` [PATCH 08/12] xfs: fix the comment of xfs_ifree_cluster() zwu.kernel
` (6 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_ialloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_ialloc.c b/fs/xfs/xfs_ialloc.c
index 7a0c17d..cb13f68 100644
--- a/fs/xfs/xfs_ialloc.c
+++ b/fs/xfs/xfs_ialloc.c
@@ -506,7 +506,7 @@ xfs_ialloc_next_ag(
/*
* Select an allocation group to look for a free inode in, based on the parent
- * inode and then mode. Return the allocation group buffer.
+ * inode and the mode. Return the allocation group buffer.
*/
STATIC xfs_agnumber_t
xfs_ialloc_ag_select(
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 08/12] xfs: fix the comment of xfs_ifree_cluster()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (6 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 07/12] xfs: fix the comment of xfs_ialloc_ag_select() zwu.kernel
@ 2013-08-12 3:14 ` zwu.kernel
2013-08-12 3:15 ` [PATCH 09/12] xfs: fix the comment of xfs_log_unmount_write() zwu.kernel
` (5 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:14 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_inode.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
index ccca9cd..520e200 100644
--- a/fs/xfs/xfs_inode.c
+++ b/fs/xfs/xfs_inode.c
@@ -1861,7 +1861,7 @@ xfs_iunlink_remove(
}
/*
- * A big issue when freeing the inode cluster is is that we _cannot_ skip any
+ * A big issue when freeing the inode cluster is that we _cannot_ skip any
* inodes that are in memory - they all must be marked stale and attached to
* the cluster buffer.
*/
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 09/12] xfs: fix the comment of xfs_log_unmount_write()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (7 preceding siblings ...)
2013-08-12 3:14 ` [PATCH 08/12] xfs: fix the comment of xfs_ifree_cluster() zwu.kernel
@ 2013-08-12 3:15 ` zwu.kernel
2013-08-12 3:15 ` [PATCH 10/12] xfs: fix the comment of xlog_recover_do_dquot_buffer() zwu.kernel
` (4 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:15 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_log.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_log.c b/fs/xfs/xfs_log.c
index 0fd27dc..6fd274c 100644
--- a/fs/xfs/xfs_log.c
+++ b/fs/xfs/xfs_log.c
@@ -720,7 +720,7 @@ xfs_log_mount_finish(xfs_mount_t *mp)
* Unmount record used to have a string "Unmount filesystem--" in the
* data section where the "Un" was really a magic number (XLOG_UNMOUNT_TYPE).
* We just write the magic number now since that particular field isn't
- * currently architecture converted and "nUmount" is a bit foo.
+ * currently architecture converted and "Unmount" is a bit foo.
* As far as I know, there weren't any dependencies on the old behaviour.
*/
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 10/12] xfs: fix the comment of xlog_recover_do_dquot_buffer()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (8 preceding siblings ...)
2013-08-12 3:15 ` [PATCH 09/12] xfs: fix the comment of xfs_log_unmount_write() zwu.kernel
@ 2013-08-12 3:15 ` zwu.kernel
2013-08-12 3:15 ` [PATCH 11/12] xfs: fix the comment of xfs_sb_quiet_read_verify() zwu.kernel
` (3 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:15 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_log_recover.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c
index a5e1bf6..a65ed50 100644
--- a/fs/xfs/xfs_log_recover.c
+++ b/fs/xfs/xfs_log_recover.c
@@ -2366,7 +2366,7 @@ xfs_qm_dqcheck(
/*
* Perform a dquot buffer recovery.
- * Simple algorithm: if we have found a QUOTAOFF logitem of the same type
+ * Simple algorithm: if we have found a QUOTAOFF log item of the same type
* (ie. USR or GRP), then just toss this buffer away; don't recover it.
* Else, treat it as a regular buffer and do recovery.
*/
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 11/12] xfs: fix the comment of xfs_sb_quiet_read_verify()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (9 preceding siblings ...)
2013-08-12 3:15 ` [PATCH 10/12] xfs: fix the comment of xlog_recover_do_dquot_buffer() zwu.kernel
@ 2013-08-12 3:15 ` zwu.kernel
2013-08-12 3:15 ` [PATCH 12/12] xfs: fix the comment of xfs_mountfs() zwu.kernel
` (2 subsequent siblings)
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:15 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_mount.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c
index 5642be3..1981be9 100644
--- a/fs/xfs/xfs_mount.c
+++ b/fs/xfs/xfs_mount.c
@@ -785,7 +785,7 @@ out_error:
/*
* We may be probed for a filesystem match, so we may not want to emit
* messages when the superblock buffer is not actually an XFS superblock.
- * If we find an XFS superblock, the run a normal, noisy mount because we are
+ * If we find an XFS superblock, then run a normal, noisy mount because we are
* really going to mount it and want to know about errors.
*/
static void
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 12/12] xfs: fix the comment of xfs_mountfs()
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (10 preceding siblings ...)
2013-08-12 3:15 ` [PATCH 11/12] xfs: fix the comment of xfs_sb_quiet_read_verify() zwu.kernel
@ 2013-08-12 3:15 ` zwu.kernel
2013-08-19 18:39 ` [PATCH 00/12] xfs: another chunk of comment fixes Mark Tinguely
2013-08-20 21:17 ` Ben Myers
13 siblings, 0 replies; 17+ messages in thread
From: zwu.kernel @ 2013-08-12 3:15 UTC (permalink / raw)
To: xfs; +Cc: Zhi Yong Wu
From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Signed-off-by: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
---
fs/xfs/xfs_mount.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c
index 1981be9..d202238 100644
--- a/fs/xfs/xfs_mount.c
+++ b/fs/xfs/xfs_mount.c
@@ -1400,7 +1400,7 @@ xfs_mountfs(
xfs_set_inoalignment(mp);
/*
- * Check that the data (and log if separate) are an ok size.
+ * Check that the data (and log if separate) is an ok size.
*/
error = xfs_check_sizes(mp);
if (error)
--
1.7.11.7
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent()
2013-08-12 3:14 ` [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent() zwu.kernel
@ 2013-08-19 18:27 ` Mark Tinguely
2013-08-19 19:36 ` Zhi Yong Wu
0 siblings, 1 reply; 17+ messages in thread
From: Mark Tinguely @ 2013-08-19 18:27 UTC (permalink / raw)
To: zwu.kernel; +Cc: Zhi Yong Wu, xfs
On 08/11/13 22:14, zwu.kernel@gmail.com wrote:
> From: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com>
>
> Signed-off-by: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com>
> ---
> fs/xfs/xfs_extent_busy.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/xfs/xfs_extent_busy.c b/fs/xfs/xfs_extent_busy.c
> index 85e9f87..4ad8fee 100644
> --- a/fs/xfs/xfs_extent_busy.c
> +++ b/fs/xfs/xfs_extent_busy.c
> @@ -147,7 +147,7 @@ xfs_extent_busy_search(
> * extent. If the overlap covers the beginning, the end, or all of the busy
> * extent, the overlapping portion can be made unbusy and used for the
> * allocation. We can't split a busy extent because we can't modify a
> - * transaction/CIL context busy list, but we can update an entries block
> + * transaction/CIL context busy list, but we can update an entrie's block
^^^^^^
changed that to entry's
> * number or length.
> *
> * Returns true if the extent can safely be reused, or false if the search
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 00/12] xfs: another chunk of comment fixes
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (11 preceding siblings ...)
2013-08-12 3:15 ` [PATCH 12/12] xfs: fix the comment of xfs_mountfs() zwu.kernel
@ 2013-08-19 18:39 ` Mark Tinguely
2013-08-20 21:17 ` Ben Myers
13 siblings, 0 replies; 17+ messages in thread
From: Mark Tinguely @ 2013-08-19 18:39 UTC (permalink / raw)
To: zwu.kernel; +Cc: Zhi Yong Wu, xfs
On 08/11/13 22:14, zwu.kernel@gmail.com wrote:
> From: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com>
>
> Submit another chunk of xfs comment fixes.
>
> Zhi Yong Wu (12):
> xfs: fix the comment of xfs_bmap_count_tree()
> xfs: fix the comment of xfs_bmap_validate_ret()
> xfs: fix the comment of xfs_bmap_last_before()
> xfs: fix the comment of xfs_bmap_punch_delalloc_range()
> xfs: fix the comment of xfs_setsize_buftarg_early()
> xfs: fix the comment of xfs_extent_busy_update_extent()
> xfs: fix the comment of xfs_ialloc_ag_select()
> xfs: fix the comment of xfs_ifree_cluster()
> xfs: fix the comment of xfs_log_unmount_write()
> xfs: fix the comment of xlog_recover_do_dquot_buffer()
> xfs: fix the comment of xfs_sb_quiet_read_verify()
> xfs: fix the comment of xfs_mountfs()
I fixed up the files:
xfs_inode.c -> xfs_inode_fork.c
xfs_bmap.c -> xfs_bmap_util.c
The mentioned patch 6/12 in this series.
I will send the changes to Ben
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent()
2013-08-19 18:27 ` Mark Tinguely
@ 2013-08-19 19:36 ` Zhi Yong Wu
0 siblings, 0 replies; 17+ messages in thread
From: Zhi Yong Wu @ 2013-08-19 19:36 UTC (permalink / raw)
To: Mark Tinguely; +Cc: Zhi Yong Wu, xfstests
On Tue, Aug 20, 2013 at 2:27 AM, Mark Tinguely <tinguely@sgi.com> wrote:
> On 08/11/13 22:14, zwu.kernel@gmail.com wrote:
>>
>> From: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com>
>>
>> Signed-off-by: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com>
>> ---
>> fs/xfs/xfs_extent_busy.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/fs/xfs/xfs_extent_busy.c b/fs/xfs/xfs_extent_busy.c
>> index 85e9f87..4ad8fee 100644
>> --- a/fs/xfs/xfs_extent_busy.c
>> +++ b/fs/xfs/xfs_extent_busy.c
>> @@ -147,7 +147,7 @@ xfs_extent_busy_search(
>> * extent. If the overlap covers the beginning, the end, or all of the
>> busy
>> * extent, the overlapping portion can be made unbusy and used for the
>> * allocation. We can't split a busy extent because we can't modify a
>> - * transaction/CIL context busy list, but we can update an entries block
>> + * transaction/CIL context busy list, but we can update an entrie's block
>
>
> ^^^^^^
> changed that to entry's
Good catch, thanks.
>
>> * number or length.
>> *
>> * Returns true if the extent can safely be reused, or false if the
>> search
>
>
--
Regards,
Zhi Yong Wu
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 00/12] xfs: another chunk of comment fixes
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
` (12 preceding siblings ...)
2013-08-19 18:39 ` [PATCH 00/12] xfs: another chunk of comment fixes Mark Tinguely
@ 2013-08-20 21:17 ` Ben Myers
13 siblings, 0 replies; 17+ messages in thread
From: Ben Myers @ 2013-08-20 21:17 UTC (permalink / raw)
To: zwu.kernel; +Cc: Zhi Yong Wu, xfs
On Mon, Aug 12, 2013 at 11:14:51AM +0800, zwu.kernel@gmail.com wrote:
> From: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
>
> Submit another chunk of xfs comment fixes.
Applied this series. Thanks.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2013-08-20 21:17 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-12 3:14 [PATCH 00/12] xfs: another chunk of comment fixes zwu.kernel
2013-08-12 3:14 ` [PATCH 01/12] xfs: fix the comment of xfs_bmap_count_tree() zwu.kernel
2013-08-12 3:14 ` [PATCH 02/12] xfs: fix the comment of xfs_bmap_validate_ret() zwu.kernel
2013-08-12 3:14 ` [PATCH 03/12] xfs: fix the comment of xfs_bmap_last_before() zwu.kernel
2013-08-12 3:14 ` [PATCH 04/12] xfs: fix the comment of xfs_bmap_punch_delalloc_range() zwu.kernel
2013-08-12 3:14 ` [PATCH 05/12] xfs: fix the comment of xfs_setsize_buftarg_early() zwu.kernel
2013-08-12 3:14 ` [PATCH 06/12] xfs: fix the comment of xfs_extent_busy_update_extent() zwu.kernel
2013-08-19 18:27 ` Mark Tinguely
2013-08-19 19:36 ` Zhi Yong Wu
2013-08-12 3:14 ` [PATCH 07/12] xfs: fix the comment of xfs_ialloc_ag_select() zwu.kernel
2013-08-12 3:14 ` [PATCH 08/12] xfs: fix the comment of xfs_ifree_cluster() zwu.kernel
2013-08-12 3:15 ` [PATCH 09/12] xfs: fix the comment of xfs_log_unmount_write() zwu.kernel
2013-08-12 3:15 ` [PATCH 10/12] xfs: fix the comment of xlog_recover_do_dquot_buffer() zwu.kernel
2013-08-12 3:15 ` [PATCH 11/12] xfs: fix the comment of xfs_sb_quiet_read_verify() zwu.kernel
2013-08-12 3:15 ` [PATCH 12/12] xfs: fix the comment of xfs_mountfs() zwu.kernel
2013-08-19 18:39 ` [PATCH 00/12] xfs: another chunk of comment fixes Mark Tinguely
2013-08-20 21:17 ` Ben Myers
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.