Do not grab the NET_LOCK() for routing sockets operations.

The only function that need the lock is rtm_output() as it messes with
the routing table.  So grab the lock there since it is safe to sleep
in a process context.

ok bluhm@

3 files changed, 14 insertions, 10 deletions

diff --git a/sys/net/raw_usrreq.c b/sys/net/raw_usrreq.c
index 1d777cec7e0..de4b7836a92 100644
--- a/sys/net/raw_usrreq.c
+++ b/sys/net/raw_usrreq.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: raw_usrreq.c,v 1.29 2017/03/03 15:48:02 bluhm Exp $	*/
+/*	$OpenBSD: raw_usrreq.c,v 1.30 2017/03/07 09:23:27 mpi Exp $	*/
 /*	$NetBSD: raw_usrreq.c,v 1.11 1996/02/13 22:00:43 christos Exp $	*/
 
 /*
@@ -63,7 +63,7 @@ raw_usrreq(struct socket *so, int req, struct mbuf *m, struct mbuf *nam,
 	int error = 0;
 	int len;
 
-	NET_ASSERT_LOCKED();
+	soassertlocked(so);
 
 	if (req == PRU_CONTROL)
 		return (EOPNOTSUPP);
diff --git a/sys/net/route.h b/sys/net/route.h
index 376d086994e..6ad78b38379 100644
--- a/sys/net/route.h
+++ b/sys/net/route.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: route.h,v 1.159 2017/03/06 08:56:39 mpi Exp $	*/
+/*	$OpenBSD: route.h,v 1.160 2017/03/07 09:23:27 mpi Exp $	*/
 /*	$NetBSD: route.h,v 1.9 1996/02/13 22:00:49 christos Exp $	*/
 
 /*
@@ -417,10 +417,6 @@ struct sockaddr_in6;
 struct bfd_config;
 
 void	 route_init(void);
-int	 route_output(struct mbuf *, struct socket *, struct sockaddr *,
-	    struct mbuf *);
-int	 route_usrreq(struct socket *, int, struct mbuf *,
-			   struct mbuf *, struct mbuf *, struct proc *);
 void	 rtm_ifchg(struct ifnet *);
 void	 rtm_ifannounce(struct ifnet *, int);
 void	 rtm_bfd(struct bfd_config *);
diff --git a/sys/net/rtsock.c b/sys/net/rtsock.c
index 67aca043704..250f411ca56 100644
--- a/sys/net/rtsock.c
+++ b/sys/net/rtsock.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: rtsock.c,v 1.231 2017/03/07 06:58:55 claudio Exp $	*/
+/*	$OpenBSD: rtsock.c,v 1.232 2017/03/07 09:23:27 mpi Exp $	*/
 /*	$NetBSD: rtsock.c,v 1.18 1996/03/29 00:32:10 cgd Exp $	*/
 
 /*
@@ -98,7 +98,11 @@ struct walkarg {
 	caddr_t	w_where, w_tmem;
 };
 
+int	route_output(struct mbuf *, struct socket *, struct sockaddr *,
+	    struct mbuf *);
 int	route_ctloutput(int, struct socket *, int, int, struct mbuf *);
+int	route_usrreq(struct socket *, int, struct mbuf *, struct mbuf *,
+	    struct mbuf *, struct proc *);
 void	route_input(struct mbuf *m0, struct socket *, sa_family_t);
 int	route_arp_conflict(struct rtentry *, struct rt_addrinfo *);
 int	route_cleargateway(struct rtentry *, void *, unsigned int);
@@ -159,8 +163,6 @@ route_usrreq(struct socket *so, int req, struct mbuf *m, struct mbuf *nam,
 	int		 af;
 	int		 error = 0;
 
-	NET_ASSERT_LOCKED();
-
 	rp = sotorawcb(so);
 
 	switch (req) {
@@ -345,6 +347,8 @@ route_input(struct mbuf *m0, struct socket *so, sa_family_t sa_family)
 	struct socket *last = NULL;
 	struct sockaddr *sosrc, *sodst;
 
+	KERNEL_ASSERT_LOCKED();
+
 	sosrc = &route_src;
 	sodst = &route_dst;
 
@@ -734,7 +738,9 @@ rtm_output(struct rt_msghdr *rtm, struct rtentry **prt,
 	struct sockaddr_mpls	*psa_mpls;
 #endif
 	int			 plen, newgate = 0, error = 0;
+	int			 s;
 
+	NET_LOCK(s);
 	switch (rtm->rtm_type) {
 	case RTM_ADD:
 		if (info->rti_info[RTAX_GATEWAY] == NULL) {
@@ -979,6 +985,8 @@ change:
 			error = ESRCH;
 		break;
 	}
+	NET_UNLOCK(s);
+
 	*prt = rt;
 	return (error);
 }