guenther@ said, that sys/protosw.h hunk on top of the diff makes it
easier to review. Is it really helpful?

ok?

Index: sys/sys/protosw.h
===================================================================
RCS file: /cvs/src/sys/sys/protosw.h,v
retrieving revision 1.40
diff -u -p -r1.40 protosw.h
--- sys/sys/protosw.h   21 Aug 2022 22:45:55 -0000      1.40
+++ sys/sys/protosw.h   21 Aug 2022 23:19:23 -0000
@@ -69,6 +69,7 @@ struct pr_usrreqs {
        int     (*pru_bind)(struct socket *, struct mbuf *, struct proc *);
        int     (*pru_listen)(struct socket *);
        int     (*pru_connect)(struct socket *, struct mbuf *);
+       int     (*pru_accept)(struct socket *, struct mbuf *);
 };
 
 struct protosw {
@@ -291,8 +292,9 @@ pru_connect(struct socket *so, struct mb
 static inline int
 pru_accept(struct socket *so, struct mbuf *nam)
 {
-       return (*so->so_proto->pr_usrreqs->pru_usrreq)(so,
-           PRU_ACCEPT, NULL, nam, NULL, curproc);
+       if (so->so_proto->pr_usrreqs->pru_accept)
+               return (*so->so_proto->pr_usrreqs->pru_accept)(so, nam);
+       return (EOPNOTSUPP);
 }
 
 static inline int
Index: sys/kern/uipc_usrreq.c
===================================================================
RCS file: /cvs/src/sys/kern/uipc_usrreq.c,v
retrieving revision 1.171
diff -u -p -r1.171 uipc_usrreq.c
--- sys/kern/uipc_usrreq.c      21 Aug 2022 22:45:55 -0000      1.171
+++ sys/kern/uipc_usrreq.c      21 Aug 2022 23:19:23 -0000
@@ -133,6 +133,7 @@ const struct pr_usrreqs uipc_usrreqs = {
        .pru_bind       = uipc_bind,
        .pru_listen     = uipc_listen,
        .pru_connect    = uipc_connect,
+       .pru_accept     = uipc_accept,
 };
 
 void
@@ -244,18 +245,6 @@ uipc_usrreq(struct socket *so, int req, 
                unp_disconnect(unp);
                break;
 
-       case PRU_ACCEPT:
-               /*
-                * Pass back name of connected socket,
-                * if it was bound and we are still connected
-                * (our peer may have closed already!).
-                */
-               so2 = unp_solock_peer(so);
-               uipc_setaddr(unp->unp_conn, nam);
-               if (so2 != NULL && so2 != so)
-                       sounlock(so2);
-               break;
-
        case PRU_SHUTDOWN:
                socantsendmore(so);
                unp_shutdown(unp);
@@ -549,6 +538,24 @@ int
 uipc_connect(struct socket *so, struct mbuf *nam)
 {
        return unp_connect(so, nam, curproc);
+}
+
+int
+uipc_accept(struct socket *so, struct mbuf *nam)
+{
+       struct socket *so2;
+       struct unpcb *unp = sotounpcb(so);
+
+       /*
+        * Pass back name of connected socket, if it was bound and
+        * we are still connected (our peer may have closed already!).
+        */
+       so2 = unp_solock_peer(so);
+       uipc_setaddr(unp->unp_conn, nam);
+
+       if (so2 != NULL && so2 != so)
+               sounlock(so2);
+       return (0);
 }
 
 int
Index: sys/net/pfkeyv2.c
===================================================================
RCS file: /cvs/src/sys/net/pfkeyv2.c,v
retrieving revision 1.238
diff -u -p -r1.238 pfkeyv2.c
--- sys/net/pfkeyv2.c   21 Aug 2022 22:45:55 -0000      1.238
+++ sys/net/pfkeyv2.c   21 Aug 2022 23:19:23 -0000
@@ -358,7 +358,6 @@ pfkeyv2_usrreq(struct socket *so, int re
        switch (req) {
        /* no connect, bind, accept. Socket is connected from the start */
        case PRU_CONNECT2:
-       case PRU_ACCEPT:
                error = EOPNOTSUPP;
                break;
 
Index: sys/net/rtsock.c
===================================================================
RCS file: /cvs/src/sys/net/rtsock.c,v
retrieving revision 1.338
diff -u -p -r1.338 rtsock.c
--- sys/net/rtsock.c    21 Aug 2022 22:45:55 -0000      1.338
+++ sys/net/rtsock.c    21 Aug 2022 23:19:23 -0000
@@ -234,7 +234,6 @@ route_usrreq(struct socket *so, int req,
        switch (req) {
        /* no connect, bind, accept. Socket is connected from the start */
        case PRU_CONNECT2:
-       case PRU_ACCEPT:
                error = EOPNOTSUPP;
                break;
 
Index: sys/netinet/ip_divert.c
===================================================================
RCS file: /cvs/src/sys/netinet/ip_divert.c,v
retrieving revision 1.73
diff -u -p -r1.73 ip_divert.c
--- sys/netinet/ip_divert.c     21 Aug 2022 22:45:55 -0000      1.73
+++ sys/netinet/ip_divert.c     21 Aug 2022 23:19:23 -0000
@@ -292,7 +292,6 @@ divert_usrreq(struct socket *so, int req
                break;
 
        case PRU_CONNECT2:
-       case PRU_ACCEPT:
        case PRU_DISCONNECT:
        case PRU_SENDOOB:
        case PRU_FASTTIMO:
Index: sys/netinet/raw_ip.c
===================================================================
RCS file: /cvs/src/sys/netinet/raw_ip.c,v
retrieving revision 1.133
diff -u -p -r1.133 raw_ip.c
--- sys/netinet/raw_ip.c        21 Aug 2022 22:45:55 -0000      1.133
+++ sys/netinet/raw_ip.c        21 Aug 2022 23:19:23 -0000
@@ -543,7 +543,6 @@ rip_usrreq(struct socket *so, int req, s
        /*
         * Not supported.
         */
-       case PRU_ACCEPT:
        case PRU_SENDOOB:
        case PRU_RCVD:
        case PRU_RCVOOB:
Index: sys/netinet/tcp_usrreq.c
===================================================================
RCS file: /cvs/src/sys/netinet/tcp_usrreq.c,v
retrieving revision 1.191
diff -u -p -r1.191 tcp_usrreq.c
--- sys/netinet/tcp_usrreq.c    21 Aug 2022 22:45:55 -0000      1.191
+++ sys/netinet/tcp_usrreq.c    21 Aug 2022 23:19:23 -0000
@@ -118,6 +118,7 @@ const struct pr_usrreqs tcp_usrreqs = {
        .pru_bind       = tcp_bind,
        .pru_listen     = tcp_listen,
        .pru_connect    = tcp_connect,
+       .pru_accept     = tcp_accept,
 };
 
 static int pr_slowhz = PR_SLOWHZ;
@@ -237,20 +238,6 @@ tcp_usrreq(struct socket *so, int req, s
                break;
 
        /*
-        * Accept a connection.  Essentially all the work is
-        * done at higher levels; just return the address
-        * of the peer, storing through addr.
-        */
-       case PRU_ACCEPT:
-#ifdef INET6
-               if (inp->inp_flags & INP_IPV6)
-                       in6_setpeeraddr(inp, nam);
-               else
-#endif
-                       in_setpeeraddr(inp, nam);
-               break;
-
-       /*
         * Mark the connection as being incapable of further output.
         */
        case PRU_SHUTDOWN:
@@ -849,6 +836,37 @@ out:
        return (error);
 }
 
+/*
+ * Accept a connection.  Essentially all the work is done at higher
+ * levels; just return the address of the peer, storing through addr.
+ */
+int
+tcp_accept(struct socket *so, struct mbuf *nam)
+{
+       struct inpcb *inp;
+       struct tcpcb *tp;
+       int error;
+       short ostate;
+
+       soassertlocked(so);
+
+       if ((error = tcp_sogetpcb(so, &inp, &tp)))
+               return (error);
+
+       if (so->so_options & SO_DEBUG)
+               ostate = tp->t_state;
+
+#ifdef INET6
+       if (inp->inp_flags & INP_IPV6)
+               in6_setpeeraddr(inp, nam);
+       else
+#endif
+               in_setpeeraddr(inp, nam);
+
+       if (so->so_options & SO_DEBUG)
+               tcp_trace(TA_USER, ostate, tp, tp, NULL, PRU_ACCEPT, 0);
+       return (error);
+}
 
 /*
  * Initiate (or continue) disconnect.
Index: sys/netinet/tcp_var.h
===================================================================
RCS file: /cvs/src/sys/netinet/tcp_var.h,v
retrieving revision 1.144
diff -u -p -r1.144 tcp_var.h
--- sys/netinet/tcp_var.h       21 Aug 2022 22:45:55 -0000      1.144
+++ sys/netinet/tcp_var.h       21 Aug 2022 23:19:23 -0000
@@ -717,6 +717,7 @@ int  tcp_detach(struct socket *);
 int     tcp_bind(struct socket *, struct mbuf *, struct proc *);
 int     tcp_listen(struct socket *);
 int     tcp_connect(struct socket *, struct mbuf *);
+int     tcp_accept(struct socket *, struct mbuf *);
 void    tcp_xmit_timer(struct tcpcb *, int);
 void    tcpdropoldhalfopen(struct tcpcb *, u_int16_t);
 void    tcp_sack_option(struct tcpcb *,struct tcphdr *,u_char *,int);
Index: sys/netinet/udp_usrreq.c
===================================================================
RCS file: /cvs/src/sys/netinet/udp_usrreq.c,v
retrieving revision 1.285
diff -u -p -r1.285 udp_usrreq.c
--- sys/netinet/udp_usrreq.c    21 Aug 2022 22:45:55 -0000      1.285
+++ sys/netinet/udp_usrreq.c    21 Aug 2022 23:19:23 -0000
@@ -1080,10 +1080,6 @@ udp_usrreq(struct socket *so, int req, s
                error = EOPNOTSUPP;
                break;
 
-       case PRU_ACCEPT:
-               error = EOPNOTSUPP;
-               break;
-
        case PRU_DISCONNECT:
 #ifdef INET6
                if (inp->inp_flags & INP_IPV6) {
Index: sys/netinet6/ip6_divert.c
===================================================================
RCS file: /cvs/src/sys/netinet6/ip6_divert.c,v
retrieving revision 1.72
diff -u -p -r1.72 ip6_divert.c
--- sys/netinet6/ip6_divert.c   21 Aug 2022 22:45:55 -0000      1.72
+++ sys/netinet6/ip6_divert.c   21 Aug 2022 23:19:23 -0000
@@ -298,7 +298,6 @@ divert6_usrreq(struct socket *so, int re
                break;
 
        case PRU_CONNECT2:
-       case PRU_ACCEPT:
        case PRU_DISCONNECT:
        case PRU_SENDOOB:
        case PRU_FASTTIMO:
Index: sys/netinet6/raw_ip6.c
===================================================================
RCS file: /cvs/src/sys/netinet6/raw_ip6.c,v
retrieving revision 1.153
diff -u -p -r1.153 raw_ip6.c
--- sys/netinet6/raw_ip6.c      21 Aug 2022 22:45:55 -0000      1.153
+++ sys/netinet6/raw_ip6.c      21 Aug 2022 23:19:23 -0000
@@ -660,7 +660,6 @@ rip6_usrreq(struct socket *so, int req, 
        /*
         * Not supported.
         */
-       case PRU_ACCEPT:
        case PRU_SENDOOB:
        case PRU_RCVD:
        case PRU_RCVOOB:
Index: sys/sys/unpcb.h
===================================================================
RCS file: /cvs/src/sys/sys/unpcb.h,v
retrieving revision 1.30
diff -u -p -r1.30 unpcb.h
--- sys/sys/unpcb.h     21 Aug 2022 22:45:55 -0000      1.30
+++ sys/sys/unpcb.h     21 Aug 2022 23:19:23 -0000
@@ -116,6 +116,7 @@ int uipc_detach(struct socket *);
 int    uipc_bind(struct socket *, struct mbuf *, struct proc *);
 int    uipc_listen(struct socket *);
 int    uipc_connect(struct socket *, struct mbuf *);
+int    uipc_accept(struct socket *, struct mbuf *);
 
 void   unp_init(void);
 int    unp_bind(struct unpcb *, struct mbuf *, struct proc *);

Reply via email to