Yes, that's basically it.

Greg Banks wrote:
> On Mon, May 21, 2007 at 10:29:46AM -0400, Chuck Lever wrote:
>> Neil Brown wrote:
>>> On Monday May 21, gnb@sgi.com wrote:
>>>> In any case I'm happy to do whatever is necessary in the transport
>>>> patchset,  but I'd appreciate Neil's guidance on which approach he'd
>>>> prefer before I go in with the scalpel.  Neil?
>>> I cannot immediately see the value in having different debug bits for
>>> the socket transport code and the RDMA transport code.  The key
>>> question is "When would you want to see the debugging from one, but
>>> not from the other", and I cannot see when that would be.  Almost
>>> certainly if you are testing one transport, you will not be using the
>>> other, so it will not contribute any noise.
>> That's how I did it in the client transport switch.  One bit enables 
>> debug messages for all transports.
>>
>>> So I suspect we should simply
>>> #define RPCDBG_FACILITY	RPCDBG_SVCSOCK
>> That's expedient, but I suggest you rename the flag to something less 
>> specific to sockets.
> 
> Aah, I see now.  That's very easy.  How's this?
> 
> 
> 
> Rename RPCDBG_SVCSOCK to RPCDBG_SVCXPRT to reflect its
> new more generic nature.
> 
> Signed-off-by: Greg Banks <gnb@melbourne.sgi.com>
> ---
> 
>  include/linux/sunrpc/debug.h |    2 +-
>  net/sunrpc/svcsock.c         |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> Index: linux/include/linux/sunrpc/debug.h
> ===================================================================
> --- linux.orig/include/linux/sunrpc/debug.h	2007-04-26 13:08:32.000000000 +1000
> +++ linux/include/linux/sunrpc/debug.h	2007-05-24 01:09:11.427405573 +1000
> @@ -20,7 +20,7 @@
>  #define RPCDBG_PMAP		0x0020
>  #define RPCDBG_SCHED		0x0040
>  #define RPCDBG_TRANS		0x0080
> -#define RPCDBG_SVCSOCK		0x0100
> +#define RPCDBG_SVCXPRT		0x0100
>  #define RPCDBG_SVCDSP		0x0200
>  #define RPCDBG_MISC		0x0400
>  #define RPCDBG_CACHE		0x0800
> Index: linux/net/sunrpc/svcsock.c
> ===================================================================
> --- linux.orig/net/sunrpc/svcsock.c	2007-05-23 22:39:53.000000000 +1000
> +++ linux/net/sunrpc/svcsock.c	2007-05-24 01:09:15.574878967 +1000
> @@ -73,7 +73,7 @@
>   *
>   */
>  
> -#define RPCDBG_FACILITY	RPCDBG_SVCSOCK
> +#define RPCDBG_FACILITY	RPCDBG_SVCXPRT
>  
>  
>  static struct svc_sock *svc_setup_socket(struct svc_serv *, struct socket *,
> 
> Greg.