From: Steve Dickson <SteveD@redhat.com>
To: chuck.lever@oracle.com
Cc: nfs@lists.sourceforge.net
Subject: Re: [PATCH] export man page updated.
Date: Tue, 18 Sep 2007 14:01:23 -0400 [thread overview]
Message-ID: <46F01273.2090607@RedHat.com> (raw)
In-Reply-To: <46EFF364.7040003@oracle.com>
Chuck Lever wrote:
>
> When I reply the attachment is discarded so I have to do a number of
> steps to quote your patch that wouldn't be necessary if you had inlined
> the patch. The point of inlining patches rather than attaching them is
> to allow others to quote the patch in their reply with no extra steps.
> And there are tools to import inlined patches -- not sure if they work
> for attachments.
>
> It's a scalability thing when reviewers have to deal with dozens of
> patches a day.
I agree.. Its just nobody mentioned they were having problems
with the way I was sending out patch... but its dully noted.
>
> It's hard to say something like this is "a problem" since it is only a
> man page. But I like to leave things a little cleaner (more correct)
> than when I found them.
True... Here is the updated patch:
commit 1cdab9665cf1a7bc2f9e51851f27eb5121b4be16
Author: Steve Dickson <steved@redhat.com>
Date: Tue Sep 18 13:53:25 2007 -0400
Updated exportfs man to talk about /var/lib/nfs/etab
instead of /var/lib/nfs/xtab
Signed-off-by: Steve Dickson <steved@redhat.com>
diff --git a/utils/exportfs/exportfs.man b/utils/exportfs/exportfs.man
index 2bd8e31..9ca4ba3 100644
--- a/utils/exportfs/exportfs.man
+++ b/utils/exportfs/exportfs.man
@@ -22,14 +22,14 @@ The
.B exportfs
command is used to maintain the current table of exported file systems for
NFS. This list is kept in a separate file named
-.BR /var/lib/nfs/xtab
+.BR /var/lib/nfs/etab
which is read by
.B mountd
when a remote host requests access to mount a file tree, and parts of
the list which are active are kept in the kernel's export table.
.P
Normally this
-.B xtab
+.B etab
file is initialized with the list of all file systems named in
.B /etc/exports
by invoking
@@ -59,7 +59,7 @@ In the new mode,
does not give any information to the kernel but only provides it to
.B mountd
through the
-.B /var/lib/nfs/xtab
+.B /var/lib/nfs/etab
file.
.B mountd
will listen to requests from the kernel and will provide information
@@ -69,7 +69,7 @@ In the legacy mode,
any export requests which identify a specific host (rather than a
subnet or netgroup etc) are entered directly into the kernel's export
table as well as being written to
-.BR /var/lib/nfs/xtab .
+.BR /var/lib/nfs/etab .
Further, any mount points listed in
.B /var/lib/nfs/rmtab
which match a non host-specific export request will cause an
@@ -93,8 +93,10 @@ file, so that only default options and options given
on the command
line are used.
.TP
.B -r
-Reexport all directories. It synchronizes /var/lib/nfs/xtab
-with /etc/exports. It removes entries in /var/lib/nfs/xtab
+Reexport all directories. It synchronizes
+.B /var/lib/nfs/etab
+with /etc/exports. It removes entries in
+.B /var/lib/nfs/etab
which are deleted from /etc/exports, and remove any entries from the
kernel export table which are no longer valid.
.TP
@@ -120,7 +122,7 @@ entries to the export table. When using
all directories in
.B exports(5)
are added to
-.B xtab
+.B etab
and the resulting list is pushed into the kernel.
.P
The
@@ -152,7 +154,7 @@ directory.
Modifications of the kernel export table used by
.B nfsd(8)
take place immediately after parsing the command line and updating the
-.B xtab
+.B etab
file.
.P
The default export options are
@@ -163,14 +165,14 @@ The third synopsis shows how to unexported a
currently exported directory.
When using
.BR "exportfs -ua" ,
all entries listed in
-.B xtab
+.B etab
are removed from the kernel export tables, and the file is cleared. This
effectively shuts down all NFS activity.
.P
To remove individual export entries, one can specify a
.I host:/path
pair. This deletes the specified entry from
-.B xtab
+.B etab
and removes the corresponding kernel entry (if any).
.P
.\" -------------------- Dumping the Export Table -----------------
@@ -186,7 +188,7 @@ option, the list of flags pertaining to each export
are shown in addition.
The following adds all directories listed in
.B /etc/exports
to
-.B /var/lib/nfs/xtab
+.B /var/lib/nfs/etab
and pushes the resulting export entries into the kernel:
.P
.nf
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
NFS maillist - NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs
prev parent reply other threads:[~2007-09-18 18:01 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-17 15:34 [PATCH] export man page updated Steve Dickson
2007-09-17 16:46 ` Chuck Lever
2007-09-18 11:07 ` Steve Dickson
2007-09-18 15:48 ` Chuck Lever
2007-09-18 18:01 ` Steve Dickson [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=46F01273.2090607@RedHat.com \
--to=steved@redhat.com \
--cc=chuck.lever@oracle.com \
--cc=nfs@lists.sourceforge.net \
/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