Re: [PATCH] Documentation: filesystems: btrfs: Fixed typos and whitespace

2015-07-09 Thread Jonathan Corbet
On Wed, 08 Jul 2015 10:44:51 -0700
Daniel Grimshaw grims...@linux.vnet.ibm.com wrote:

 I am a high school student trying to become familiar with
 Linux kernel development. The btrfs documentation in
 Documentation/filesystems had a few typos and errors in
 whitespace. This patch corrects both of these.
 
 This is a resend of an earlier patch with corrected patchfile.

Applied to the docs tree, thanks.

Just FYI, if you put lines like the last one above after the '---' line,
they won't find their way into the commit changelog, which is
preferable.  I edited it out.

jon
--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Documentation: filesystems: btrfs: Fixed typos and whitespace

2015-07-08 Thread Daniel Grimshaw
I am a high school student trying to become familiar with Linux kernel 
development. The btrfs documentation in Documentation/filesystems had a few 
typos and errors in whitespace. This patch corrects both of these.

Signed-off-by: Daniel Grimshaw grims...@linux.vnet.ibm.com

---
diff --git a/Documentation/filesystems/btrfs.txt 
b/Documentation/filesystems/btrfs.txt
index d11cc2f..57d9d54 100644
--- a/Documentation/filesystems/btrfs.txt
+++ b/Documentation/filesystems/btrfs.txt
@@ -61,7 +61,7 @@ Options with (*) are default options and will not show in the 
mount options.
 
 check_int enables the integrity checker module, which examines all
 block write requests to ensure on-disk consistency, at a large
-memory and CPU cost. 
+memory and CPU cost.
 
 check_int_data includes extent data in the integrity checks, and
 implies the check_int option.
@@ -113,7 +113,7 @@ Options with (*) are default options and will not show in 
the mount options.
 Disable/enable debugging option to be more verbose in some ENOSPC 
conditions.
 
   fatal_errors=action
-Action to take when encountering a fatal error:
+Action to take when encountering a fatal error:
   bug - BUG() on a fatal error.  This is the default.
   panic - panic() on a fatal error.
 
@@ -132,10 +132,10 @@ Options with (*) are default options and will not show in 
the mount options.
 
   max_inline=bytes
 Specify the maximum amount of space, in bytes, that can be inlined in
-a metadata B-tree leaf.  The value is specified in bytes, optionally
+a metadata B-tree leaf.  The value is specified in bytes, optionally
 with a K, M, or G suffix, case insensitive.  In practice, this value
 is limited by the root sector size, with some space unavailable due
-to leaf headers.  For a 4k sectorsize, max inline data is ~3900 bytes.
+to leaf headers.  For a 4k sector size, max inline data is ~3900 bytes.
 
   metadata_ratio=value
 Specify that 1 metadata chunk should be allocated after every value
@@ -161,7 +161,7 @@ Options with (*) are default options and will not show in 
the mount options.
 
   datasum(*)
   nodatasum
-Enable/disable data checksumming for newly created files.
+Enable/disable data check-summing for newly created files.
 Datasum implies datacow.
 
   treelog(*)
@@ -170,7 +170,7 @@ Options with (*) are default options and will not show in 
the mount options.
 
   recovery
 Enable autorecovery attempts if a bad tree root is found at mount time.
-Currently this scans a list of several previous tree roots and tries to
+Currently this scans a list of several previous tree roots and tries to
 use the first readable.
 
   rescan_uuid_tree
@@ -194,7 +194,7 @@ Options with (*) are default options and will not show in 
the mount options.
   ssd_spread
 Options to control ssd allocation schemes.  By default, BTRFS will
 enable or disable ssd allocation heuristics depending on whether a
-rotational or nonrotational disk is in use.  The ssd and nossd options
+rotational or non-rotational disk is in use.  The ssd and nossd options
 can override this autodetection.
 
 The ssd_spread mount option attempts to allocate into big chunks
@@ -216,13 +216,13 @@ Options with (*) are default options and will not show in 
the mount options.
 This allows mounting of subvolumes which are not in the root of the mounted
 filesystem.
 You can use btrfs subvolume show  to see the object ID for a subvolume.
-   
+
   thread_pool=number
 The number of worker threads to allocate.  The default number is equal
 to the number of CPUs + 2, or 8, whichever is smaller.
 
   user_subvol_rm_allowed
-Allow subvolumes to be deleted by a non-root user. Use with caution.
+Allow subvolumes to be deleted by a non-root user. Use with caution.
 
 MAILING LIST
 

--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Documentation: filesystems: btrfs: Fixed typos and whitespace

2015-07-08 Thread Jonathan Corbet
On Wed, 08 Jul 2015 08:22:20 -0700
Daniel Grimshaw grims...@linux.vnet.ibm.com wrote:

 I am a high school student trying to become familiar with Linux kernel
 development.

That's great!

 The btrfs documentation in Documentation/filesystems had a
 few typos and errors in whitespace. This patch corrects both of these.

So I went to apply this patch to the docs tree, but it fails to apply due
to whitespace problems of its own; something has corrupted your patch on
its way out to the mailing list.

Can I ask you to fix up your email client so that it doesn't do this?
Please have a look at Documentation/email-clients.txt for some hints; in
particular, there are things that can be done to bludgeon Thunderbird
(which you appear to be using) into a more cooperative state of mind.
Once you've done that, send the patch to yourself and try to apply the
result; after that works, please resubmit your patch here.

One comment, though: checksum (or checksumming) is pretty standard
usage and shouldn't really be fixed.

Thanks,

jon
--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] Documentation: filesystems: btrfs: Fixed typos and whitespace

2015-07-08 Thread Daniel Grimshaw
I am a high school student trying to become familiar with
Linux kernel development. The btrfs documentation in
Documentation/filesystems had a few typos and errors in
whitespace. This patch corrects both of these.

This is a resend of an earlier patch with corrected patchfile.

Signed-off-by: Daniel Grimshaw grims...@linux.vnet.ibm.com
---
 Documentation/filesystems/btrfs.txt |   16 
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Documentation/filesystems/btrfs.txt 
b/Documentation/filesystems/btrfs.txt
index d11cc2f..c772b47 100644
--- a/Documentation/filesystems/btrfs.txt
+++ b/Documentation/filesystems/btrfs.txt
@@ -61,7 +61,7 @@ Options with (*) are default options and will not show in the 
mount options.
 
check_int enables the integrity checker module, which examines all
block write requests to ensure on-disk consistency, at a large
-   memory and CPU cost.  
+   memory and CPU cost.
 
check_int_data includes extent data in the integrity checks, and
implies the check_int option.
@@ -113,7 +113,7 @@ Options with (*) are default options and will not show in 
the mount options.
Disable/enable debugging option to be more verbose in some ENOSPC 
conditions.
 
   fatal_errors=action
-   Action to take when encountering a fatal error: 
+   Action to take when encountering a fatal error:
  bug - BUG() on a fatal error.  This is the default.
  panic - panic() on a fatal error.
 
@@ -132,10 +132,10 @@ Options with (*) are default options and will not show in 
the mount options.
 
   max_inline=bytes
Specify the maximum amount of space, in bytes, that can be inlined in
-   a metadata B-tree leaf.  The value is specified in bytes, optionally 
+   a metadata B-tree leaf.  The value is specified in bytes, optionally
with a K, M, or G suffix, case insensitive.  In practice, this value
is limited by the root sector size, with some space unavailable due
-   to leaf headers.  For a 4k sectorsize, max inline data is ~3900 bytes.
+   to leaf headers.  For a 4k sector size, max inline data is ~3900 bytes.
 
   metadata_ratio=value
Specify that 1 metadata chunk should be allocated after every value
@@ -170,7 +170,7 @@ Options with (*) are default options and will not show in 
the mount options.
 
   recovery
Enable autorecovery attempts if a bad tree root is found at mount time.
-   Currently this scans a list of several previous tree roots and tries to 
+   Currently this scans a list of several previous tree roots and tries to
use the first readable.
 
   rescan_uuid_tree
@@ -194,7 +194,7 @@ Options with (*) are default options and will not show in 
the mount options.
   ssd_spread
Options to control ssd allocation schemes.  By default, BTRFS will
enable or disable ssd allocation heuristics depending on whether a
-   rotational or nonrotational disk is in use.  The ssd and nossd options
+   rotational or non-rotational disk is in use.  The ssd and nossd options
can override this autodetection.
 
The ssd_spread mount option attempts to allocate into big chunks
@@ -216,13 +216,13 @@ Options with (*) are default options and will not show in 
the mount options.
This allows mounting of subvolumes which are not in the root of the 
mounted
filesystem.
You can use btrfs subvolume show  to see the object ID for a 
subvolume.
-   
+
   thread_pool=number
The number of worker threads to allocate.  The default number is equal
to the number of CPUs + 2, or 8, whichever is smaller.
 
   user_subvol_rm_allowed
-   Allow subvolumes to be deleted by a non-root user. Use with caution. 
+   Allow subvolumes to be deleted by a non-root user. Use with caution.
 
 MAILING LIST
 
-- 
1.7.1

--
To unsubscribe from this list: send the line unsubscribe linux-btrfs in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html