Commit 573018c0 authored by Pavel Emelyanov's avatar Pavel Emelyanov Committed by J. Bruce Fields

sunrpc: Call xs_create_sockX directly from setup_socket

Remove now unneeded wrappers that just add type and protocol
to socket creation callback.
Signed-off-by: default avatarPavel Emelyanov <xemul@openvz.org>
Reviewed-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 22d44a7d
...@@ -1710,7 +1710,7 @@ static void xs_udp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock) ...@@ -1710,7 +1710,7 @@ static void xs_udp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock)
static void xs_udp_setup_socket(struct sock_xprt *transport, static void xs_udp_setup_socket(struct sock_xprt *transport,
struct socket *(*create_sock)(struct rpc_xprt *, struct socket *(*create_sock)(struct rpc_xprt *,
struct sock_xprt *)) struct sock_xprt *, int type, int protocol))
{ {
struct rpc_xprt *xprt = &transport->xprt; struct rpc_xprt *xprt = &transport->xprt;
struct socket *sock = transport->sock; struct socket *sock = transport->sock;
...@@ -1721,7 +1721,7 @@ static void xs_udp_setup_socket(struct sock_xprt *transport, ...@@ -1721,7 +1721,7 @@ static void xs_udp_setup_socket(struct sock_xprt *transport,
/* Start by resetting any existing state */ /* Start by resetting any existing state */
xs_reset_transport(transport); xs_reset_transport(transport);
sock = create_sock(xprt, transport); sock = create_sock(xprt, transport, SOCK_DGRAM, IPPROTO_UDP);
if (IS_ERR(sock)) if (IS_ERR(sock))
goto out; goto out;
...@@ -1745,18 +1745,12 @@ static void xs_udp_setup_socket(struct sock_xprt *transport, ...@@ -1745,18 +1745,12 @@ static void xs_udp_setup_socket(struct sock_xprt *transport,
* Invoked by a work queue tasklet. * Invoked by a work queue tasklet.
*/ */
static struct socket *xs_create_udp_sock4(struct rpc_xprt *xprt,
struct sock_xprt *transport)
{
return xs_create_sock4(xprt, transport, SOCK_DGRAM, IPPROTO_UDP);
}
static void xs_udp_connect_worker4(struct work_struct *work) static void xs_udp_connect_worker4(struct work_struct *work)
{ {
struct sock_xprt *transport = struct sock_xprt *transport =
container_of(work, struct sock_xprt, connect_worker.work); container_of(work, struct sock_xprt, connect_worker.work);
xs_udp_setup_socket(transport, xs_create_udp_sock4); xs_udp_setup_socket(transport, xs_create_sock4);
} }
/** /**
...@@ -1766,18 +1760,12 @@ static void xs_udp_connect_worker4(struct work_struct *work) ...@@ -1766,18 +1760,12 @@ static void xs_udp_connect_worker4(struct work_struct *work)
* Invoked by a work queue tasklet. * Invoked by a work queue tasklet.
*/ */
static struct socket *xs_create_udp_sock6(struct rpc_xprt *xprt,
struct sock_xprt *transport)
{
return xs_create_sock6(xprt, transport, SOCK_DGRAM, IPPROTO_UDP);
}
static void xs_udp_connect_worker6(struct work_struct *work) static void xs_udp_connect_worker6(struct work_struct *work)
{ {
struct sock_xprt *transport = struct sock_xprt *transport =
container_of(work, struct sock_xprt, connect_worker.work); container_of(work, struct sock_xprt, connect_worker.work);
xs_udp_setup_socket(transport, xs_create_udp_sock6); xs_udp_setup_socket(transport, xs_create_sock6);
} }
/* /*
...@@ -1883,7 +1871,7 @@ static int xs_tcp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock) ...@@ -1883,7 +1871,7 @@ static int xs_tcp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock)
*/ */
static void xs_tcp_setup_socket(struct sock_xprt *transport, static void xs_tcp_setup_socket(struct sock_xprt *transport,
struct socket *(*create_sock)(struct rpc_xprt *, struct socket *(*create_sock)(struct rpc_xprt *,
struct sock_xprt *)) struct sock_xprt *, int type, int protocol))
{ {
struct socket *sock = transport->sock; struct socket *sock = transport->sock;
struct rpc_xprt *xprt = &transport->xprt; struct rpc_xprt *xprt = &transport->xprt;
...@@ -1894,7 +1882,7 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport, ...@@ -1894,7 +1882,7 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport,
if (!sock) { if (!sock) {
clear_bit(XPRT_CONNECTION_ABORT, &xprt->state); clear_bit(XPRT_CONNECTION_ABORT, &xprt->state);
sock = create_sock(xprt, transport); sock = create_sock(xprt, transport, SOCK_STREAM, IPPROTO_TCP);
if (IS_ERR(sock)) { if (IS_ERR(sock)) {
status = PTR_ERR(sock); status = PTR_ERR(sock);
goto out; goto out;
...@@ -1954,12 +1942,6 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport, ...@@ -1954,12 +1942,6 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport,
xprt_wake_pending_tasks(xprt, status); xprt_wake_pending_tasks(xprt, status);
} }
static struct socket *xs_create_tcp_sock4(struct rpc_xprt *xprt,
struct sock_xprt *transport)
{
return xs_create_sock4(xprt, transport, SOCK_STREAM, IPPROTO_TCP);
}
/** /**
* xs_tcp_connect_worker4 - connect a TCP socket to a remote endpoint * xs_tcp_connect_worker4 - connect a TCP socket to a remote endpoint
* @work: RPC transport to connect * @work: RPC transport to connect
...@@ -1971,13 +1953,7 @@ static void xs_tcp_connect_worker4(struct work_struct *work) ...@@ -1971,13 +1953,7 @@ static void xs_tcp_connect_worker4(struct work_struct *work)
struct sock_xprt *transport = struct sock_xprt *transport =
container_of(work, struct sock_xprt, connect_worker.work); container_of(work, struct sock_xprt, connect_worker.work);
xs_tcp_setup_socket(transport, xs_create_tcp_sock4); xs_tcp_setup_socket(transport, xs_create_sock4);
}
static struct socket *xs_create_tcp_sock6(struct rpc_xprt *xprt,
struct sock_xprt *transport)
{
return xs_create_sock6(xprt, transport, SOCK_STREAM, IPPROTO_TCP);
} }
/** /**
...@@ -1991,7 +1967,7 @@ static void xs_tcp_connect_worker6(struct work_struct *work) ...@@ -1991,7 +1967,7 @@ static void xs_tcp_connect_worker6(struct work_struct *work)
struct sock_xprt *transport = struct sock_xprt *transport =
container_of(work, struct sock_xprt, connect_worker.work); container_of(work, struct sock_xprt, connect_worker.work);
xs_tcp_setup_socket(transport, xs_create_tcp_sock6); xs_tcp_setup_socket(transport, xs_create_sock6);
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment