linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Documentation: filesystems: btrfs: Fixed typos and whitespace
@ 2015-07-08 15:22 Daniel Grimshaw
  2015-07-08 16:38 ` Jonathan Corbet
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel Grimshaw @ 2015-07-08 15:22 UTC (permalink / raw)
  To: clm, jbacik, dsterba, corbet; +Cc: linux-btrfs, linux-doc, linux-kernel

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 <grimshaw@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
 ============


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

* Re: Documentation: filesystems: btrfs: Fixed typos and whitespace
  2015-07-08 15:22 Documentation: filesystems: btrfs: Fixed typos and whitespace Daniel Grimshaw
@ 2015-07-08 16:38 ` Jonathan Corbet
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Corbet @ 2015-07-08 16:38 UTC (permalink / raw)
  To: Daniel Grimshaw
  Cc: clm, jbacik, dsterba, linux-btrfs, linux-doc, linux-kernel

On Wed, 08 Jul 2015 08:22:20 -0700
Daniel Grimshaw <grimshaw@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

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

end of thread, other threads:[~2015-07-08 16:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-08 15:22 Documentation: filesystems: btrfs: Fixed typos and whitespace Daniel Grimshaw
2015-07-08 16:38 ` Jonathan Corbet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).