The commit is pushed to "branch-rh7-3.10.0-1160.21.1.vz7.174.x-ovz" and will 
appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-1160.21.1.vz7.174.10
------>
commit 0674b8f918f86e4e5666aacc366c50e6afe386b9
Author: Stefano Garzarella <sgarz...@redhat.com>
Date:   Wed Apr 21 17:52:47 2021 +0300

    ms/vsock: add vsock_create_connected() called by transports
    
    All transports call __vsock_create() with the same parameters,
    most of them depending on the parent socket. In order to simplify
    the VSOCK core APIs exposed to the transports, this patch adds
    the vsock_create_connected() callable from transports to create
    a new socket when a connection request is received.
    We also unexported the __vsock_create().
    
    Suggested-by: Stefan Hajnoczi <stefa...@redhat.com>
    Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
    Reviewed-by: Jorgen Hansen <jhan...@vmware.com>
    Signed-off-by: Stefano Garzarella <sgarz...@redhat.com>
    Signed-off-by: David S. Miller <da...@davemloft.net>
    
    https://jira.sw.ru/browse/PSBM-128702
    (cherry picked from commit b9ca2f5ff7784d46285a8f1b14419ac4645096f7)
    Signed-off-by: Konstantin Khorenko <khore...@virtuozzo.com>
---
 include/net/af_vsock.h                  |  5 +----
 net/vmw_vsock/af_vsock.c                | 18 ++++++++++++------
 net/vmw_vsock/hyperv_transport.c        |  3 +--
 net/vmw_vsock/virtio_transport_common.c |  3 +--
 net/vmw_vsock/vmci_transport.c          |  3 +--
 5 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/include/net/af_vsock.h b/include/net/af_vsock.h
index f29c343..ec1fefb 100644
--- a/include/net/af_vsock.h
+++ b/include/net/af_vsock.h
@@ -84,10 +84,7 @@ struct vsock_sock {
 
 s64 vsock_stream_has_data(struct vsock_sock *vsk);
 s64 vsock_stream_has_space(struct vsock_sock *vsk);
-struct sock *__vsock_create(struct net *net,
-                           struct socket *sock,
-                           struct sock *parent,
-                           gfp_t priority, unsigned short type);
+struct sock *vsock_create_connected(struct sock *parent);
 
 /**** TRANSPORT ****/
 
diff --git a/net/vmw_vsock/af_vsock.c b/net/vmw_vsock/af_vsock.c
index 41141eb..9d14bd7 100644
--- a/net/vmw_vsock/af_vsock.c
+++ b/net/vmw_vsock/af_vsock.c
@@ -593,11 +593,11 @@ static int __vsock_bind(struct sock *sk, struct 
sockaddr_vm *addr)
 
 static void vsock_connect_timeout(struct work_struct *work);
 
-struct sock *__vsock_create(struct net *net,
-                           struct socket *sock,
-                           struct sock *parent,
-                           gfp_t priority,
-                           unsigned short type)
+static struct sock *__vsock_create(struct net *net,
+                                  struct socket *sock,
+                                  struct sock *parent,
+                                  gfp_t priority,
+                                  unsigned short type)
 {
        struct sock *sk;
        struct vsock_sock *psk;
@@ -664,7 +664,6 @@ struct sock *__vsock_create(struct net *net,
 
        return sk;
 }
-EXPORT_SYMBOL_GPL(__vsock_create);
 
 static void __vsock_release(struct sock *sk)
 {
@@ -725,6 +724,13 @@ static int vsock_queue_rcv_skb(struct sock *sk, struct 
sk_buff *skb)
        return err;
 }
 
+struct sock *vsock_create_connected(struct sock *parent)
+{
+       return __vsock_create(sock_net(parent), NULL, parent, GFP_KERNEL,
+                             parent->sk_type);
+}
+EXPORT_SYMBOL_GPL(vsock_create_connected);
+
 s64 vsock_stream_has_data(struct vsock_sock *vsk)
 {
        return vsk->transport->stream_has_data(vsk);
diff --git a/net/vmw_vsock/hyperv_transport.c b/net/vmw_vsock/hyperv_transport.c
index 1550b31..7a5c532 100644
--- a/net/vmw_vsock/hyperv_transport.c
+++ b/net/vmw_vsock/hyperv_transport.c
@@ -358,8 +358,7 @@ static void hvs_open_connection(struct vmbus_channel *chan)
                if (sk->sk_ack_backlog >= sk->sk_max_ack_backlog)
                        goto out;
 
-               new = __vsock_create(sock_net(sk), NULL, sk, GFP_KERNEL,
-                                    sk->sk_type);
+               new = vsock_create_connected(sk);
                if (!new)
                        goto out;
 
diff --git a/net/vmw_vsock/virtio_transport_common.c 
b/net/vmw_vsock/virtio_transport_common.c
index ea9823a..b30289f 100644
--- a/net/vmw_vsock/virtio_transport_common.c
+++ b/net/vmw_vsock/virtio_transport_common.c
@@ -872,8 +872,7 @@ virtio_transport_recv_listen(struct sock *sk, struct 
virtio_vsock_pkt *pkt)
                return -ENOMEM;
        }
 
-       child = __vsock_create(sock_net(sk), NULL, sk, GFP_KERNEL,
-                              sk->sk_type);
+       child = vsock_create_connected(sk);
        if (!child) {
                virtio_transport_reset(vsk, pkt);
                return -ENOMEM;
diff --git a/net/vmw_vsock/vmci_transport.c b/net/vmw_vsock/vmci_transport.c
index aa533b5..2925295 100644
--- a/net/vmw_vsock/vmci_transport.c
+++ b/net/vmw_vsock/vmci_transport.c
@@ -984,8 +984,7 @@ static int vmci_transport_recv_listen(struct sock *sk,
                return -ECONNREFUSED;
        }
 
-       pending = __vsock_create(sock_net(sk), NULL, sk, GFP_KERNEL,
-                                sk->sk_type);
+       pending = vsock_create_connected(sk);
        if (!pending) {
                vmci_transport_send_reset(sk, pkt);
                return -ENOMEM;
_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to