All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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 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.