From: Boaz Harrosh <bharrosh@panasas.com>
To: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>
Cc: trivial@kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, bhalevy@panasas.com,
open-osd mailing-list <osd-dev@open-osd.org>
Subject: Re: [PATCH] trivial: some small fixes in exofs documentation
Date: Tue, 28 Jul 2009 15:34:54 +0300 [thread overview]
Message-ID: <4A6EF06E.40101@panasas.com> (raw)
In-Reply-To: <1248711992-11183-1-git-send-email-cascardo@holoscopio.com>
On 07/27/2009 07:26 PM, Thadeu Lima de Souza Cascardo wrote:
> Add exofs.txt to filesystems Documentation index and fix some typos,
> identation and grammar.
>
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>
Thank you very much Thadeu!
I appreciate you taking the time, going over this.
As my English is lacking, I know I need help in this department.
I have included this for the next Kernel.
(It will appear on linux-next in the next day or two)
Boaz
> ---
> Documentation/filesystems/00-INDEX | 2 ++
> Documentation/filesystems/exofs.txt | 23 ++++++++++++-----------
> 2 files changed, 14 insertions(+), 11 deletions(-)
>
> diff --git a/Documentation/filesystems/00-INDEX b/Documentation/filesystems/00-INDEX
> index f15621e..7001782 100644
> --- a/Documentation/filesystems/00-INDEX
> +++ b/Documentation/filesystems/00-INDEX
> @@ -36,6 +36,8 @@ dnotify.txt
> - info about directory notification in Linux.
> ecryptfs.txt
> - docs on eCryptfs: stacked cryptographic filesystem for Linux.
> +exofs.txt
> + - info, usage, mount options, design about EXOFS.
> ext2.txt
> - info, mount options and specifications for the Ext2 filesystem.
> ext3.txt
> diff --git a/Documentation/filesystems/exofs.txt b/Documentation/filesystems/exofs.txt
> index 0ced74c..abd2a9b 100644
> --- a/Documentation/filesystems/exofs.txt
> +++ b/Documentation/filesystems/exofs.txt
> @@ -60,13 +60,13 @@ USAGE
>
> mkfs.exofs --pid=65536 --format /dev/osd0
>
> - The --format is optional if not specified no OSD_FORMAT will be
> - preformed and a clean file system will be created in the specified pid,
> + The --format is optional. If not specified, no OSD_FORMAT will be
> + performed and a clean file system will be created in the specified pid,
> in the available space of the target. (Use --format=size_in_meg to limit
> the total LUN space available)
>
> - If pid already exist it will be deleted and a new one will be created in it's
> - place. Be careful.
> + If pid already exists, it will be deleted and a new one will be created in
> + its place. Be careful.
>
> An exofs lives inside a single OSD partition. You can create multiple exofs
> filesystems on the same device using multiple pids.
> @@ -81,7 +81,7 @@ USAGE
>
> 7. For reference (See do-exofs example script):
> do-exofs start - an example of how to perform the above steps.
> - do-exofs stop - an example of how to unmount the file system.
> + do-exofs stop - an example of how to unmount the file system.
> do-exofs format - an example of how to format and mkfs a new exofs.
>
> 8. Extra compilation flags (uncomment in fs/exofs/Kbuild):
> @@ -104,8 +104,8 @@ Where:
> exofs specific options: Options are separated by commas (,)
> pid=<integer> - The partition number to mount/create as
> container of the filesystem.
> - This option is mandatory
> - to=<integer> - Timeout in ticks for a single command
> + This option is mandatory.
> + to=<integer> - Timeout in ticks for a single command.
> default is (60 * HZ) [for debugging only]
>
> ===============================================================================
> @@ -116,7 +116,7 @@ DESIGN
> with a special ID (defined in common.h).
> Information included in the file system control block is used to fill the
> in-memory superblock structure at mount time. This object is created before
> - the file system is used by mkexofs.c It contains information such as:
> + the file system is used by mkexofs.c. It contains information such as:
> - The file system's magic number
> - The next inode number to be allocated
>
> @@ -134,8 +134,8 @@ DESIGN
> attributes. This applies to both regular files and other types (directories,
> device files, symlinks, etc.).
>
> -* Credentials are generated per object (inode and superblock) when they is
> - created in memory (read off disk or created). The credential works for all
> +* Credentials are generated per object (inode and superblock) when they are
> + created in memory (read from disk or created). The credential works for all
> operations and is used as long as the object remains in memory.
>
> * Async OSD operations are used whenever possible, but the target may execute
> @@ -145,7 +145,8 @@ DESIGN
> from executing in reverse order:
> - The following are handled with the OBJ_CREATED and OBJ_2BCREATED
> flags. OBJ_CREATED is set when we know the object exists on the OSD -
> - in create's callback function, and when we successfully do a read_inode.
> + in create's callback function, and when we successfully do a
> + read_inode.
> OBJ_2BCREATED is set in the beginning of the create function, so we
> know that we should wait.
> - create/delete: delete should wait until the object is created
prev parent reply other threads:[~2009-07-28 12:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-27 16:26 [PATCH] trivial: some small fixes in exofs documentation Thadeu Lima de Souza Cascardo
2009-07-28 12:34 ` Boaz Harrosh [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A6EF06E.40101@panasas.com \
--to=bharrosh@panasas.com \
--cc=bhalevy@panasas.com \
--cc=cascardo@holoscopio.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=osd-dev@open-osd.org \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox