From: Jeff Layton <jlayton@redhat.com>
To: linux-nfs@vger.kernel.org
Cc: steved@redhat.com, chuck.lever@oracle.com
Subject: [PATCH 0/6] nfs-utils: add IPv6 support for rpc.nfsd (try #4)
Date: Tue, 2 Jun 2009 07:43:43 -0400 [thread overview]
Message-ID: <1243943029-6400-1-git-send-email-jlayton@redhat.com> (raw)
This is the fourth attempt at a patchset to add IPv6 support to the
rpc.nfsd program. This set is quite a bit different from the earlier
ones. The main differences are:
1) it first walks /etc/netconfig to determine the protocols and address
families to enable. Then disables any specified by command line options.
2) it uses getaddrinfo to generate sockaddrs for sockets that get passed
off to the kernel. This simplifies the code quite a bit and makes for
more robust handling of the -H option.
3) converts nfsd to use xlog logging facility. It also adds a --debug
option that causes it to log to stderr rather than syslog, and makes it
print out all debug messages.
4) in the event that an ipv6-enabled rpc.nfsd is run on a kernel that
doesn't have an ipv6-enabled knfsd, the program doesn't log an error
message if it's able to start nfsd on an IPv4 socket. This should cut
down spurious logging until IPv6 support is more widely available.
There is also a lot of general cleanup and reorganization of the startup
logic. Although this patchset is a bit larger than the prior ones, I
think the result is a much cleaner set of code.
This set should be bisectable, but I've only really tested the final
result. I have tested it on various combinations of build options and
with ipv6.ko blacklisted and it seems to work appropriately in all
cases.
Jeff Layton (6):
nfs-utils: don't link libexport.a and libmisc.a to nfsd
nfs-utils: clean up option parsing in nfsd.c
nfs-utils: break up nfssvc.c into more individually callable
functions
nfs-utils: set IPV6_V6ONLY on nfssvc IPv6 sockets
nfs-utils: add IPv6 support to nfsd
nfs-utils: update the nfsd manpage
support/include/nfs/nfs.h | 15 ++-
support/include/nfslib.h | 7 +-
support/nfs/nfssvc.c | 231 ++++++++++++++++++++++++++++---------------
utils/nfsd/Makefile.am | 4 +-
utils/nfsd/nfsd.c | 243 +++++++++++++++++++++++++++++++++------------
utils/nfsd/nfsd.man | 19 +++-
6 files changed, 365 insertions(+), 154 deletions(-)
next reply other threads:[~2009-06-02 11:43 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-02 11:43 Jeff Layton [this message]
2009-06-02 11:43 ` [PATCH 1/6] nfs-utils: don't link libexport.a and libmisc.a to nfsd Jeff Layton
2009-06-02 11:43 ` [PATCH 2/6] nfs-utils: clean up option parsing in nfsd.c Jeff Layton
2009-06-02 11:43 ` [PATCH 3/6] nfs-utils: break up nfssvc.c into more individually callable functions Jeff Layton
2009-06-02 15:32 ` Chuck Lever
2009-06-02 18:40 ` Jeff Layton
2009-06-02 11:43 ` [PATCH 4/6] nfs-utils: set IPV6_V6ONLY on nfssvc IPv6 sockets Jeff Layton
2009-06-02 11:43 ` [PATCH 5/6] nfs-utils: add IPv6 support to nfsd Jeff Layton
2009-06-02 15:35 ` Chuck Lever
2009-06-02 18:55 ` Jeff Layton
[not found] ` <20090602145537.04fdec57-9yPaYZwiELC+kQycOl6kW4xkIHaj4LzF@public.gmane.org>
2009-06-02 19:13 ` Chuck Lever
2009-06-02 20:04 ` Jeff Layton
2009-06-02 11:43 ` [PATCH 6/6] nfs-utils: update the nfsd manpage Jeff Layton
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=1243943029-6400-1-git-send-email-jlayton@redhat.com \
--to=jlayton@redhat.com \
--cc=chuck.lever@oracle.com \
--cc=linux-nfs@vger.kernel.org \
--cc=steved@redhat.com \
/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