When I want to monitor what's happening with nfs, I want a second-by-second view of what's happening and while that usually involved many of the counters in nfsstat, they're not particularly useful in their present form. When I wrote collectl (see http://collectl.sourceforge.net/), which shows just about any performance counter you can think of all nicely formatted, I couldn't help but add nfs counters as well. Now you can see them all on a single line (I'm currently talking V2 and V3 ...

Should we delete the toclose checks from svc_sock_names(), then, under the assumption it's always called with toclose non-NULL now? And why can't we just completely replace svc_sock_names() at this point? -

-

This is the BUG_ON() in svc_xprt_enqueue(), here: process: if (!list_empty(&pool->sp_threads)) { rqstp = list_entry(pool->sp_threads.next, struct svc_rqst, rq_list); dprintk("svc: transport %p served by daemon %p\n", xprt, rqstp); svc_thread_dequeue(pool, rqstp); if (rqstp->rq_xprt) printk(KERN_ERR ...