gmane.linux.nfs

[patch 1/1] lockd: convert reclaimer thread to kthread interface

2008-12-01T22:27:48

From: Jeff Layton public.gmane.org> My understanding is that there is a push to turn the kernel_thread interface into a non-exported symbol and move all kernel threads to use the kthread API. This patch changes lockd to use kthread_run to spawn the reclaimer thread. I've made the assumption here that the extra module references taken when we spawn this thread are unnecessary and removed them. I've also added a KERN_ERR printk that pops if the thread can't be spawned to warn the admin that the locks won't be reclaimed. In the future, it would be nice to be able to notify userspace that locks have been lost (probably by implementing SIGLOST), and adding some good policies about how long we should reattempt to reclaim the locks. Finally, I removed a comment about memory leaks that I believe is obsolete and added a new one to clarify the result of sending a SIGKILL to the reclaimer thread. As best I can tell, doing so doesn't actually cause a memory leak. Signed-off-by: Jeff

[NFS] D entists Contact List for the USA

Strong O Karin — 2008-12-01T22:04:01

Comes with unlimited use license and at a very resonable price: --> 164,574 |>entists --> 158,242 Physical Addresses --> 163,185 Phone Numbers --> 77,878 Office Fax Numbers --> 45,954 e mails only during this week you won't pay the regular $593 price, you get it for $296 Write an email to Monroe-YiFMxoQ3Mhodw2Fo+rOw49BPR1lH4CV8< at >public.gmane.org for additional info to get off please email exclude-YiFMxoQ3Mhodw2Fo+rOw49BPR1lH4CV8< at >public.gmane.org ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ NFS maillist - NFS-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f< at >public.gmane.org https://lists.sourceforge.net/lists/listinfo/nfs ___________________________

[PATCH 0/6] First set of patches for 2.6.29

Chuck Lever — 2008-12-01T18:57:25

These patches add some IPv6 support to the Linux kernel's NSM upcall implementation, and address a couple of shortcomings in the existing NLM/NSM implementation. I'd like you to consider these for 2.6.29. There are 40 more patches I have queued for 2.6.29 that finish IPv6 support in the NSM upcall implementation, enable IPv6 for lockd, and finish and enable IPv6 support in NFSD. --- Chuck Lever (6): NSM: Serialize SM_MON and SM_UNMON upcalls NLM: Clean up nsm_monitor() call NLM: Clean up nsm_monitor() call NSM: Support IPv6 version of mon_name NLM: Support IPv6 scope IDs in nlm_display_address() NLM: Remove address eye-catcher buffers from nlm_host fs/lockd/host.c | 53 ++++++++++-------- fs/lockd/mon.c | 120 +++++++++++++++++++++------------------- include/linux/lockd/lockd.h | 21 ++++++- include/linux/lockd/sm_inter.h | 2 - 4 files changed, 109 insertions(+), 87 deletions(-)

[PATCH 0/4] Make mount.nfs use new getport() function

Chuck Lever — 2008-12-01T18:48:33

Hi Steve- Four new IPv6-related patches for mount.nfs. These replace the existing PMAP_GETPORT implementation in utils/mount/network.c with a call to the new library functions we just added. The probe_statd() function is used by both the legacy and the text- based paths, but replacing the mount.nfs command's getport() function affects only the *legacy* NFS mount path, not text-based mounts. For anyone testing this code, please try it on pre-2.6.23 kernels, or patch mount.nfs to use legacy binary style mounts. This will exercise the new PMAP_GETPORT implementation using all the special cases needed by mount.nfs. The new RPCB_GETADDR implementation will not be used until mount.nfs can pass in IPv6 addresses. --- Chuck Lever (4): mount command: remove local getport() implementation mount command: Replace clnt_ping() and getport() calls in probe_port() mount command: Use nfs_error() instead of perror() mount command: Use nfs_pmap_getport() in probe_statd() utils/mount/network.c

[PATCH] nfs: remove redundant tests on reading new pages

Wu Fengguang — 2008-12-01T03:19:22

aops->readpages() and its NFS helper readpage_async_filler() will only be called to do readahead I/O for newly allocated pages. So it's not necessary to test for the always 0 dirty/uptodate page flags. The removal of nfs_wb_page() call also fixes a readahead bug: the NFS readahead has been synchronous since 2.6.23, because that call will clear PG_readahead, which is the reminder for asynchronous readahead. More background: the PG_readahead page flag is shared with PG_reclaim, one for read path and the other for write path. clear_page_dirty_for_io() unconditionally clears PG_readahead to prevent possible readahead residuals, assuming itself to be always called in the write path. However, NFS is one and the only exception in that it _always_ calls clear_page_dirty_for_io() in the read path, i.e. for readpages()/readpage(). Cc: Trond Myklebust public.gmane.org> Signed-off-by: Wu Fengguang public.gmane.org> --- fs/nfs/read.c | 6 ------ 1

ipv6 status

Brian J. Murrell — 2008-12-01T00:40:16

Hi All, So what exactly is the status of NFS and IPv6 in the most recent (say, 2.6.24-28) kernels? Is it present and if so, working? Feel free to simply point me to a roadmap that shows where we are if you like. Cheers and thanx, b.

No write access with recent versions of nfs-utils in Debian

David Liontooth — 2008-11-30T22:13:49

This appears to be a Debian-specific issue, so I'm just looking for a pointer from someone who may know about it. I installed Debian sid on a new machine back in July, including the current nfs-common and nfs-kernel-server. NFS mounts were successful, but did not allow write -- I lost some serious data on one occasion before I noticed the problem. mount and nfs-utils were upgraded several times since, and never gave the requested write access. I've been assuming the problem was firewall-related and lived with it. On the other machines in the group, NFS writes worked fine. I banged my head against the wall for a while today, after finally finding time to look into the problem, and discovered a completely different version of nfs-utils was running on the new machine. After a painful downgrade things work fine again. Here's the verbose output of the version that didn't work, nfs-common 1:1.1.3-2: mount -vvv /mnt/cortona1 mount: fstab path: "/etc/fstab" mount: lock path: "/etc/mtab~" mount:

Is order for "exportfs -r" and rpc.mountd important?

Dr. Tilmann Bubeck — 2008-11-28T14:41:38

According to manual page "exportfs -r" has two modes: legacy and new. Fedora 10 seems to use the new mode and mounts /proc/fs/nfsd. But they use the following order in /etc/init.d/nfs: exportfs -r rpc.mountd This results in /proc/fs/nfsd/exports having all obsolete entries from /var/lib/nfs/rmtab which causes a lot of trouble with stale NFS file handles. This seems to be wrong according to "man exportfs" in new mode. If I change the order to rpc.mountd exportfs -r then everything is OK and /proc/fs/nfsd/exports only contains the info from /etc/exports and not all stale clients. This seems to be correct according to "man exportfs" in new mode. Questions: is this behaviour a bug or at least bad? Shouldn't exportfs being changed, so that when it detects "new" mode it will never feed rmtab into the kernel and rely upon rpc.mountd answering the kernels questions as stated in "man exportfs"? If yes, then the order or the programs is irrelevant. If not, should the order of the programs be reve

[PATCH 1/1] NFS: fix ERR_PTR with un mapped NFSv4 error

2008-11-26T22:49:00

From: Andy Adamson public.gmane.org> NFSv4 errors need to be mapped prior to ERR_PTR. Signed-off-by: Andy Adamson public.gmane.org> --- fs/nfs/client.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/nfs/client.c b/fs/nfs/client.c index 7547600..49f06ce 100644 --- a/fs/nfs/client.c +++ b/fs/nfs/client.c < at >< at > -1154,7 +1154,7 < at >< at > struct nfs_server *nfs4_create_server(const struct nfs_parsed_mount_data *data, error: nfs_free_server(server); dprintk("<-- nfs4_create_server() = error %d\n", error); -return ERR_PTR(error); +return ERR_PTR(nfs4_map_errors(error)); } /* < at >< at > -1231,7 +1231,7 < at >< at > struct nfs_server *nfs4_create_referral_server(struct nfs_clone_mount *data, error: nfs_free_server(server); dprintk("<-- nfs4_create_referral_server() = error %d\n", error); -return ERR_PTR(error); +return ERR_PTR(nfs4_map_errors(error)); } #endif /* CONFIG_NFS_V4 */ < at >< at > -1299,7 +1299,7 < at >< at > struct nfs_server *nfs_c

[NFS] Comprehensive Healthcare Marketing List

Elizabeth N Ritter — 2008-11-26T17:36:41

Certified Medical Doctors in the US Medical Doctors in many different specialties you can sort by many different fields like city, state or zip Regular price is $494 but this week you only pay: $399 ~~~~~ These lists come at no charge when you order: ~~~~~ --> Dentists --> Veterinarians --> Physical Therapists --> Visiting Nurses & RN's Email us at:: Benitez-Px9Zx9/iZEOakBO8gow8eQ< at >public.gmane.org this offer is only valid until Nov 28, 2008 ============== To invoke no further correspondence status please send an email to gone-Px9Zx9/iZEOakBO8gow8eQ< at >public.gmane.org ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ NFS maillist - NFS

Patch to prevent rpc.gssd spamming logs when Kerberos credentials are expired

Andrew Pollock — 2008-11-25T21:22:21

Hi, We've seen excessive (disk filling) log spam from rpc.gssd where users mount their home directory via Kerberised NFS, and their Kerberos TGT expires. You get lots of noise like described at https://bugs.launchpad.net/ubuntu/+source/nfs-utils/+bug/293705 Attached is the patch that we've been using to address this. regards Andrew -- Andrew Pollock Google Systems Administrator

[NFS] Database of geriatric specialists and more

Harvey spiky — 2008-11-25T15:56:27

Fully Licensed MDs in the US Coverage in many different areas of medicine such as Endocrinology, Pathology, Urology, Neurology, Plastic Surgery, Psychiatry, Cardiology and much more you can sort by many different fields like city, state or zip This week's special price = $394 ***** Order this week and you will get at no charge: ***** <> Chiropractors <> Physical Therapists <> Massage Therapists <> Acupuncturists contact your rep:: Calhoun-Px9Zx9/iZEOakBO8gow8eQ< at >public.gmane.org from now until Friday . have a look at this if you don't like to recieve these kinds of emails please send an email to gone-Px9Zx9/iZEOakBO8gow8eQ< at >public.gmane.org ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=1

[PATCH] NLM: client-side nlm_lookup_host() should avoid matching onsrcaddr

Chuck Lever — 2008-11-24T17:51:55

Since commit c98451bd, the loop in nlm_lookup_host() unconditionally compares the host's h_srcaddr field to the incoming source address. For client-side nlm_host entries, both are always AF_UNSPEC, so this check is unnecessary. Since commit 781b61a6, which added support for AF_INET6 addresses to nlm_cmp_addr(), nlm_cmp_addr() now returns FALSE for AF_UNSPEC addresses, which causes nlm_lookup_host() to create a fresh nlm_host entry every time it is called on the client. These extra entries will eventually expire once the server is unmounted, so the impact of this regression, introduced with lockd IPv6 support in 2.6.28, should be minor. We could fix this by adding an arm in nlm_cmp_addr() for AF_UNSPEC addresses, but really, nlm_lookup_host() shouldn't be matching on the srcaddr field for client-side nlm_host lookups. Signed-off-by: Chuck Lever public.gmane.org> --- fs/lockd/host.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/fs/lockd/hos

[PATCH nfs-utils] Ensure statd gets started if required when non-root user mounts an NFS filesystem.

Neil Brown — 2008-11-24T00:07:19

openSUSE-11.1 betas rely in mount.nfs to start statd when required. This helped me discover that it doesn't work when a non-root user is mounting a filesystem as the setuid status gets lots. This patch fixes it. Thanks, NeilBrown From 78d2001f031553ef90db708d5a7b3fbb9b31a873 Mon Sep 17 00:00:00 2001 From: Neil Brown public.gmane.org> Date: Mon, 24 Nov 2008 11:00:19 +1100 Subject: [PATCH] Ensure statd gets started if required when non-root user mounts an NFS filesystem. The first time an NFS filesystem is mounted, we start statd from /sbin/mount.nfs. If this first time is a non-root user doing the mount, (thanks to e.g. the 'users' option in /etc/fstab) then we need to be sure that the 'setuid' status from mount.nfs is inherited through to rpc.statd so that it runs as root. There are two places where we loose our setuid status due to the shell (/bin/sh) discarding. 1/ mount.nfs uses "system" to run /usr/sbin/start-statd. This runs a shell which is likely to drop privileges. So c

[PATCH] svcsock: Add a reference to sunrpc in svc_addsock

Tom Tucker — 2008-11-23T14:08:36

The svc_addsock function adds transport instances without taking a reference on the sunrpc.ko module, however, the generic transport destruction code drops a reference when a transport instance is destroyed. Add a try_module_get call to the svc_addsock function for transport instances added by this function. Signed-off-by: Tom Tucker public.gmane.org> --- net/sunrpc/svcsock.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c index 95293f5..a1951dc 100644 --- a/net/sunrpc/svcsock.c +++ b/net/sunrpc/svcsock.c < at >< at > -1183,7 +1183,11 < at >< at > int svc_addsock(struct svc_serv *serv, else if (so->state > SS_UNCONNECTED) err = -EISCONN; else { -svsk = svc_setup_socket(serv, so, &err, SVC_SOCK_DEFAULTS); +if (!try_module_get(THIS_MODULE)) +err = -ENOENT; +else +svsk = svc_setup_socket(serv, so, &err, +SVC_SOCK_DEFAULTS); if (svsk) { struct sockaddr_storage addr; struct sockad

[PATCH] NLM: AF_UNSPEC addresses are all equal

Chuck Lever — 2008-11-22T16:58:08

The nlm_cmp_addr() function, which compares two IP addresses, is sometimes used to check AF_UNSPEC addresses. If both addresses are AF_UNSPEC, then it should consider them equal -- the contents of an AF_UNSPEC address, other than the address family, are inconsequential. This is important because the loop in nlm_lookup_host() always compares the h_srcaddr field to the incoming source address. For client nlm_host entries, both are always AF_UNSPEC, and should allow the entry match to succeed. If it always fails, then we create a fresh nlm_host entry every time we mount the same server, even if all other items in the existing entry (protocol, NLM version, host address) match. These extra entries will eventually expire once the server is unmounted, so the impact of this bug, introduced in 2.6.28 with commit 781b61a6, should be minor. Signed-off-by: Chuck Lever public.gmane.org> --- Bruce- Another minor lockd IPv6 bug fix for 2.6.28. Alternate fix for this problem is to

[NFS] General Dentists List in America

Roman Dalton — 2008-11-21T04:34:07

Just Released: for the USA --> 164,341 D entists --> 158,048 Postal Addresses --> 163,958 Tel #'s --> 77,093 Fax Numbers --> 45,870 E-Mail Addresses for only this week you can take the above with unlimited use rights for $297 - a $594 value Write an email to Dalton-W3yqWAQfaE6Bik42HM7KXg< at >public.gmane.org for additional info By emailing release-W3yqWAQfaE6Bik42HM7KXg< at >public.gmane.org you will have your email taken off ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ NFS maillist - NFS-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f< at >public.gmane.org https://lists.sourceforge.net/lists/listinfo/nfs _______________________________________________ Please n

[PATCH] SUNRPC: Fix a performance regression in the RPC authentication code

Trond Myklebust — 2008-11-20T21:06:21

Fix a regression reported by Max Kellermann whereby kernel profiling showed that his clients were spending 45% of their time in rpcauth_lookup_credcache. It turns out that although his processes had identical uid/gid/groups, generic_match() was failing to detect this, because the task->group_info pointers were not shared. This again lead to the creation of a huge number of identical credentials at the RPC layer. The regression is fixed by comparing the contents of task->group_info if the actual pointers are not identical. Signed-off-by: Trond Myklebust public.gmane.org> --- net/sunrpc/auth_generic.c | 20 ++++++++++++++++++-- 1 files changed, 18 insertions(+), 2 deletions(-) diff --git a/net/sunrpc/auth_generic.c b/net/sunrpc/auth_generic.c index 744b79f..4028502 100644 --- a/net/sunrpc/auth_generic.c +++ b/net/sunrpc/auth_generic.c < at >< at > -133,13 +133,29 < at >< at > static int generic_match(struct auth_cred *acred, struct rpc_cred *cred, int flags) { struct generic_cr

nfs-utils, umount -l, and unmount requests

David Mathog — 2008-11-19T23:54:42

[I hope this is the right place to send this. I tried agud-H+wXaHxf7aLQT0dZR+AlfA< at >public.gmane.org, who is listed on the relevant file's copyright, but that mailbox was disabled.] This concerns nfsmount.c in nfs-utils-1.1.4. I spent most of today trying to figure out why after upgrading an NFS client machine from Mandriva 2007.1 to 2008.1 suddenly the NFS server stopped logging "authenticated unmount request" messages when the client rebooted. It turned out this was a consequence of the umount "-l" flag, which is used in the shutdown scripts and the many recent changes in the organization of the umount commands. Long story short, under Mandriva 2007.1 umount looked for umount.nfs, which was not present in that release, and then went on to send the unmount request itself. On Mandriva 2008.1, where umount.nfs is present, the ball was passed to this program, which promptly dropped it if "-l" had been specified. The NFS directory was unmounted correctly, but the "unmount request" was never sent. I tr

[PATCH] lockd: handle fl_grant callbacks with coalesced locks (RFC)

Jeff Layton — 2008-11-19T21:37:03

Dave Teigland opened a bug stating that he was having some problems with DLM and lockd. Essentially, the problem boiled down to the fact that when you do a posix lock of a region that touches another lock, the VFS will coalesce the locks and return a lock that encompasses the whole range. The problem here is that DLM then does a fl_grant callback to lockd with the new coalesced lock. The fl_grant callback then looks through all of the blocks and eventually returns -ENOENT since none match the coalesced lock. I'm having a very hard time tracking down info about how the fl_grant callback is supposed to work. Is it OK to send an fl_grant callback with a lock that's larger than the one requested? If so, then lockd needs to account for that possibility. Also, what exactly is the purpose of the second arg on fl_grant ("conf" in nlmsvc_grant_deferred)? What follows is a patch that changes nlmsvc_grant_deferred to account for the possibility that it may receive an fl_grant that has already been coalesced. It chan

[PATCH 0/4] Support for IPv6 in the showmount command

Chuck Lever — 2008-11-18T19:29:28

Hi Steve- These patches implement support for rpcbind v3/v4 queries and IPv6 NFS servers in the showmount command. Please consider these for the next release of nfs-utils. Thanks! --- Chuck Lever (4): showmount command: support querying IPv6 servers showmount command: move logic to acquire RPC client handle out of main() showmount command: Remove unused local getport() implementation showmount command: call nfs_getport instead of local getport utils/showmount/showmount.c | 237 +++++++++++++++++-------------------------- 1 files changed, 91 insertions(+), 146 deletions(-)