On Thu, 2009-08-06 at 14:55 -0400, Chuck Lever wrote:
> After a bind completes, update the transport instance's address
> strings so debugging messages display the current port the transport
> is connected to.
>
> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx>
> ---
>
> net/sunrpc/xprtsock.c | 82 +++++++++++++++++++++++++------------------------
> 1 files changed, 41 insertions(+), 41 deletions(-)
>
> diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c
> index eee5ac9..8aaf900 100644
> --- a/net/sunrpc/xprtsock.c
> +++ b/net/sunrpc/xprtsock.c
> @@ -299,11 +299,34 @@ static inline struct sockaddr_in6 *xs_addr_in6(struct rpc_xprt *xprt)
> static void xs_format_common_peer_addresses(struct rpc_xprt *xprt)
> {
> struct sockaddr *sap = xs_addr(xprt);
> + struct sockaddr_in6 *sin6;
> + struct sockaddr_in *sin;
> char buf[128];
>
> (void)rpc_ntop(sap, buf, sizeof(buf));
> xprt->address_strings[RPC_DISPLAY_ADDR] = kstrdup(buf, GFP_KERNEL);
>
> + switch (sap->sa_family) {
> + case AF_INET:
> + sin = xs_addr_in(xprt);
> + (void)snprintf(buf, sizeof(buf), "%02x%02x%02x%02x",
> + NIPQUAD(sin->sin_addr.s_addr));
> + break;
> + case AF_INET6:
> + sin6 = xs_addr_in6(xprt);
> + (void)snprintf(buf, sizeof(buf), "%pi6", &sin6->sin6_addr);
> + break;
Hmm... Doesn't it make sense to convert this to use rpc_ntop() now that
I've applied those patches?
Trond
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
