Module Name:    src
Committed By:   riastradh
Date:           Thu Aug 20 21:33:53 UTC 2020

Modified Files:
        src/sys/net: if_wg.c

Log Message:
Use SYSCTL_SETUP for net.wireguard subtree.


To generate a diff of this commit:
cvs rdiff -u -r1.7 -r1.8 src/sys/net/if_wg.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/net/if_wg.c
diff -u src/sys/net/if_wg.c:1.7 src/sys/net/if_wg.c:1.8
--- src/sys/net/if_wg.c:1.7	Thu Aug 20 21:31:47 2020
+++ src/sys/net/if_wg.c	Thu Aug 20 21:33:52 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_wg.c,v 1.7 2020/08/20 21:31:47 riastradh Exp $	*/
+/*	$NetBSD: if_wg.c,v 1.8 2020/08/20 21:33:52 riastradh Exp $	*/
 
 /*
  * Copyright (C) Ryota Ozaki <ozaki.ry...@gmail.com>
@@ -43,7 +43,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_wg.c,v 1.7 2020/08/20 21:31:47 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_wg.c,v 1.8 2020/08/20 21:33:52 riastradh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_inet.h"
@@ -676,8 +676,6 @@ static void	wg_stop(struct ifnet *, int)
 static int	wg_clone_create(struct if_clone *, int);
 static int	wg_clone_destroy(struct ifnet *);
 
-static void	wg_setup_sysctl(void);
-
 struct wg_ops {
 	int (*send_hs_msg)(struct wg_peer *, struct mbuf *);
 	int (*send_data_msg)(struct wg_peer *, struct mbuf *);
@@ -771,8 +769,6 @@ static void
 wginit(void)
 {
 
-	wg_setup_sysctl();
-
 	wg_psref_class = psref_class_create("wg", IPL_SOFTNET);
 
 	mutex_init(&wg_softcs.lock, MUTEX_DEFAULT, IPL_NONE);
@@ -4366,60 +4362,49 @@ wg_stop(struct ifnet *ifp, int disable)
 	/* Need to do something? */
 }
 
-static struct sysctllog *wg_sysctllog;
-static void
-wg_setup_sysctl(void)
-{
 #ifdef WG_DEBUG_PARAMS
+SYSCTL_SETUP(sysctl_net_wireguard_setup, "sysctl net.wireguard setup")
+{
 	const struct sysctlnode *node = NULL;
 
-	sysctl_createv(&wg_sysctllog, 0, NULL, &node,
-		CTLFLAG_PERMANENT, CTLTYPE_NODE, "wireguard",
-		SYSCTL_DESCR("WireGuard"),
-		NULL, 0, NULL, 0, CTL_NET, CTL_CREATE, CTL_EOL);
-	if (node != NULL) {
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_LONG, "rekey_after_messages",
-			SYSCTL_DESCR("session liftime by messages"),
-			NULL, 0, &wg_rekey_after_messages, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_LONG, "rekey_after_time",
-			SYSCTL_DESCR("session liftime"),
-			NULL, 0, &wg_rekey_after_time, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_LONG, "rekey_timeout",
-			SYSCTL_DESCR("session handshake retry time"),
-			NULL, 0, &wg_rekey_timeout, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_LONG, "rekey_attempt_time",
-			SYSCTL_DESCR("session handshake timeout"),
-			NULL, 0, &wg_rekey_attempt_time, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_LONG, "keepalive_timeout",
-			SYSCTL_DESCR("keepalive timeout"),
-			NULL, 0, &wg_keepalive_timeout, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-
-		sysctl_createv(&wg_sysctllog, 0, NULL, NULL,
-			CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
-			CTLTYPE_BOOL, "force_underload",
-			SYSCTL_DESCR("force to detemine under load"),
-			NULL, 0, &wg_force_underload, 0,
-			CTL_NET, node->sysctl_num, CTL_CREATE, CTL_EOL);
-	}
-#else
-	(void)wg_sysctllog;
-#endif
+	sysctl_createv(clog, 0, NULL, &node,
+	    CTLFLAG_PERMANENT,
+	    CTLTYPE_NODE, "wireguard",
+	    SYSCTL_DESCR("WireGuard"),
+	    NULL, 0, NULL, 0,
+	    CTL_NET, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_LONG, "rekey_after_messages",
+	    SYSCTL_DESCR("session liftime by messages"),
+	    NULL, 0, &wg_rekey_after_messages, 0, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_LONG, "rekey_after_time",
+	    SYSCTL_DESCR("session liftime"),
+	    NULL, 0, &wg_rekey_after_time, 0, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_LONG, "rekey_timeout",
+	    SYSCTL_DESCR("session handshake retry time"),
+	    NULL, 0, &wg_rekey_timeout, 0, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_LONG, "rekey_attempt_time",
+	    SYSCTL_DESCR("session handshake timeout"),
+	    NULL, 0, &wg_rekey_attempt_time, 0, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_LONG, "keepalive_timeout",
+	    SYSCTL_DESCR("keepalive timeout"),
+	    NULL, 0, &wg_keepalive_timeout, 0, CTL_CREATE, CTL_EOL);
+	sysctl_createv(clog, 0, &node, NULL,
+	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE,
+	    CTLTYPE_BOOL, "force_underload",
+	    SYSCTL_DESCR("force to detemine under load"),
+	    NULL, 0, &wg_force_underload, 0, CTL_CREATE, CTL_EOL);
 }
+#endif
 
 #ifdef WG_RUMPKERNEL
 static bool

Reply via email to