Commit 22f79326 authored by Pavel Emelyanov's avatar Pavel Emelyanov Committed by J. Bruce Fields

sunrpc: Factor out v4 sockets creation

The UDPv4 and TCPv4 socket creation callbacks now look very similar.
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 b65c0310
...@@ -1631,6 +1631,30 @@ static inline void xs_reclassify_socket6(struct socket *sock) ...@@ -1631,6 +1631,30 @@ static inline void xs_reclassify_socket6(struct socket *sock)
} }
#endif #endif
static struct socket *xs_create_sock4(struct rpc_xprt *xprt,
struct sock_xprt *transport, int type, int protocol)
{
struct socket *sock;
int err;
err = __sock_create(xprt->xprt_net, PF_INET, type, protocol, &sock, 1);
if (err < 0) {
dprintk("RPC: can't create %d transport socket (%d).\n",
protocol, -err);
goto out;
}
xs_reclassify_socket4(sock);
if (xs_bind4(transport, sock)) {
sock_release(sock);
goto out;
}
return sock;
out:
return ERR_PTR(err);
}
static void xs_udp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock) static void xs_udp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock)
{ {
struct sock_xprt *transport = container_of(xprt, struct sock_xprt, xprt); struct sock_xprt *transport = container_of(xprt, struct sock_xprt, xprt);
...@@ -1700,24 +1724,7 @@ static void xs_udp_setup_socket(struct sock_xprt *transport, ...@@ -1700,24 +1724,7 @@ static void xs_udp_setup_socket(struct sock_xprt *transport,
static struct socket *xs_create_udp_sock4(struct rpc_xprt *xprt, static struct socket *xs_create_udp_sock4(struct rpc_xprt *xprt,
struct sock_xprt *transport) struct sock_xprt *transport)
{ {
struct socket *sock; return xs_create_sock4(xprt, transport, SOCK_DGRAM, IPPROTO_UDP);
int err;
err = __sock_create(xprt->xprt_net, PF_INET, SOCK_DGRAM, IPPROTO_UDP, &sock, 1);
if (err < 0) {
dprintk("RPC: can't create UDP transport socket (%d).\n", -err);
goto out;
}
xs_reclassify_socket4(sock);
if (xs_bind4(transport, sock)) {
sock_release(sock);
goto out;
}
return sock;
out:
return ERR_PTR(err);
} }
static void xs_udp_connect_worker4(struct work_struct *work) static void xs_udp_connect_worker4(struct work_struct *work)
...@@ -1943,25 +1950,7 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport, ...@@ -1943,25 +1950,7 @@ static void xs_tcp_setup_socket(struct sock_xprt *transport,
static struct socket *xs_create_tcp_sock4(struct rpc_xprt *xprt, static struct socket *xs_create_tcp_sock4(struct rpc_xprt *xprt,
struct sock_xprt *transport) struct sock_xprt *transport)
{ {
struct socket *sock; return xs_create_sock4(xprt, transport, SOCK_STREAM, IPPROTO_TCP);
int err;
/* start from scratch */
err = __sock_create(xprt->xprt_net, PF_INET, SOCK_STREAM, IPPROTO_TCP, &sock, 1);
if (err < 0) {
dprintk("RPC: can't create TCP transport socket (%d).\n",
-err);
goto out_err;
}
xs_reclassify_socket4(sock);
if (xs_bind4(transport, sock) < 0) {
sock_release(sock);
goto out_err;
}
return sock;
out_err:
return ERR_PTR(-EIO);
} }
/** /**
......
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