Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Feb 2009 00:31:34 +0000 (UTC)
From:      Rui Paulo <rpaulo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r189096 - head/sbin/ifconfig
Message-ID:  <200902270031.n1R0VY8E043693@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rpaulo
Date: Fri Feb 27 00:31:34 2009
New Revision: 189096
URL: http://svn.freebsd.org/changeset/base/189096

Log:
  Replace clone_setcallback() with a new function clone_setdefcallback()
  that selects a callback from an interface prefix name. This allows us to
  report a meaningful error when the user types 'ifconfig wlan0 create',
  for example, and also kills some redundant code.
  
  Reviewed by:    sam (earlier version)

Modified:
  head/sbin/ifconfig/ifclone.c
  head/sbin/ifconfig/ifconfig.h
  head/sbin/ifconfig/ifieee80211.c
  head/sbin/ifconfig/ifvlan.c

Modified: head/sbin/ifconfig/ifclone.c
==============================================================================
--- head/sbin/ifconfig/ifclone.c	Thu Feb 26 23:21:32 2009	(r189095)
+++ head/sbin/ifconfig/ifclone.c	Fri Feb 27 00:31:34 2009	(r189096)
@@ -32,6 +32,7 @@ static const char rcsid[] =
   "$FreeBSD$";
 #endif /* not lint */
 
+#include <sys/queue.h>
 #include <sys/types.h>
 #include <sys/ioctl.h>
 #include <sys/socket.h>
@@ -88,14 +89,24 @@ list_cloners(void)
 	free(buf);
 }
 
-static clone_callback_func *clone_cb = NULL;
+struct clone_defcb {
+	char ifprefix[IFNAMSIZ];
+	clone_callback_func *clone_cb;
+	SLIST_ENTRY(clone_defcb) next;
+};
+
+static SLIST_HEAD(, clone_defcb) clone_defcbh =
+   SLIST_HEAD_INITIALIZER(clone_defcbh);
 
 void
-clone_setcallback(clone_callback_func *p)
+clone_setdefcallback(const char *ifprefix, clone_callback_func *p)
 {
-	if (clone_cb != NULL && clone_cb != p)
-		errx(1, "conflicting device create parameters");
-	clone_cb = p;
+	struct clone_defcb *dcp;
+
+	dcp = malloc(sizeof(*dcp));
+	strlcpy(dcp->ifprefix, ifprefix, IFNAMSIZ-1);
+	dcp->clone_cb = p;
+	SLIST_INSERT_HEAD(&clone_defcbh, dcp, next);
 }
 
 /*
@@ -108,9 +119,22 @@ static void
 ifclonecreate(int s, void *arg)
 {
 	struct ifreq ifr;
+	struct clone_defcb *dcp;
+	clone_callback_func *clone_cb = NULL;
 
 	memset(&ifr, 0, sizeof(ifr));
 	(void) strlcpy(ifr.ifr_name, name, sizeof(ifr.ifr_name));
+
+	if (clone_cb == NULL) {
+		/* Try to find a default callback */
+		SLIST_FOREACH(dcp, &clone_defcbh, next) {
+			if (strncmp(dcp->ifprefix, ifr.ifr_name,
+			    strlen(dcp->ifprefix)) == 0) {
+				clone_cb = dcp->clone_cb;
+				break;
+			}
+		}
+	}
 	if (clone_cb == NULL) {
 		/* NB: no parameters */
 		if (ioctl(s, SIOCIFCREATE2, &ifr) < 0)

Modified: head/sbin/ifconfig/ifconfig.h
==============================================================================
--- head/sbin/ifconfig/ifconfig.h	Thu Feb 26 23:21:32 2009	(r189095)
+++ head/sbin/ifconfig/ifconfig.h	Fri Feb 27 00:31:34 2009	(r189096)
@@ -141,7 +141,7 @@ void	printb(const char *s, unsigned valu
 void	ifmaybeload(const char *name);
 
 typedef void clone_callback_func(int, struct ifreq *);
-void	clone_setcallback(clone_callback_func *);
+void	clone_setdefcallback(const char *, clone_callback_func *);
 
 /*
  * XXX expose this so modules that neeed to know of any pending

Modified: head/sbin/ifconfig/ifieee80211.c
==============================================================================
--- head/sbin/ifconfig/ifieee80211.c	Thu Feb 26 23:21:32 2009	(r189095)
+++ head/sbin/ifconfig/ifieee80211.c	Fri Feb 27 00:31:34 2009	(r189096)
@@ -4667,7 +4667,8 @@ wlan_create(int s, struct ifreq *ifr)
 	static const uint8_t zerobssid[IEEE80211_ADDR_LEN];
 
 	if (params.icp_parent[0] == '\0')
-		errx(1, "must specify a parent when creating a wlan device");
+		errx(1, "must specify a parent device (wlandev) when creating "
+		    "a wlan device");
 	if (params.icp_opmode == IEEE80211_M_WDS &&
 	    memcmp(params.icp_bssid, zerobssid, sizeof(zerobssid)) == 0)
 		errx(1, "no bssid specified for WDS (use wlanbssid)");
@@ -4680,7 +4681,6 @@ static
 DECL_CMD_FUNC(set80211clone_wlandev, arg, d)
 {
 	strlcpy(params.icp_parent, arg, IFNAMSIZ);
-	clone_setcallback(wlan_create);
 }
 
 static
@@ -4692,7 +4692,6 @@ DECL_CMD_FUNC(set80211clone_wlanbssid, a
 	if (ea == NULL)
 		errx(1, "%s: cannot parse bssid", arg);
 	memcpy(params.icp_bssid, ea->octet, IEEE80211_ADDR_LEN);
-	clone_setcallback(wlan_create);
 }
 
 static
@@ -4705,7 +4704,6 @@ DECL_CMD_FUNC(set80211clone_wlanaddr, ar
 		errx(1, "%s: cannot parse addres", arg);
 	memcpy(params.icp_macaddr, ea->octet, IEEE80211_ADDR_LEN);
 	params.icp_flags |= IEEE80211_CLONE_MACADDR;
-	clone_setcallback(wlan_create);
 }
 
 static
@@ -4729,7 +4727,6 @@ DECL_CMD_FUNC(set80211clone_wlanmode, ar
 		params.icp_flags |= IEEE80211_CLONE_TDMA;
 	} else
 		errx(1, "Don't know to create %s for %s", arg, name);
-	clone_setcallback(wlan_create);
 #undef iseq
 }
 
@@ -4741,7 +4738,6 @@ set80211clone_beacons(const char *val, i
 		params.icp_flags &= ~IEEE80211_CLONE_NOBEACONS;
 	else
 		params.icp_flags |= IEEE80211_CLONE_NOBEACONS;
-	clone_setcallback(wlan_create);
 }
 
 static void
@@ -4751,7 +4747,6 @@ set80211clone_bssid(const char *val, int
 		params.icp_flags |= IEEE80211_CLONE_BSSID;
 	else
 		params.icp_flags &= ~IEEE80211_CLONE_BSSID;
-	clone_setcallback(wlan_create);
 }
 
 static void
@@ -4761,7 +4756,6 @@ set80211clone_wdslegacy(const char *val,
 		params.icp_flags |= IEEE80211_CLONE_WDSLEGACY;
 	else
 		params.icp_flags &= ~IEEE80211_CLONE_WDSLEGACY;
-	clone_setcallback(wlan_create);
 }
 
 static struct cmd ieee80211_cmds[] = {
@@ -4935,5 +4929,6 @@ ieee80211_ctor(void)
 	for (i = 0; i < N(ieee80211_cmds);  i++)
 		cmd_register(&ieee80211_cmds[i]);
 	af_register(&af_ieee80211);
+	clone_setdefcallback("wlan", wlan_create);
 #undef N
 }

Modified: head/sbin/ifconfig/ifvlan.c
==============================================================================
--- head/sbin/ifconfig/ifvlan.c	Thu Feb 26 23:21:32 2009	(r189095)
+++ head/sbin/ifconfig/ifvlan.c	Fri Feb 27 00:31:34 2009	(r189096)
@@ -136,8 +136,6 @@ DECL_CMD_FUNC(setvlantag, val, d)
 
 	if (getvlan(s, &ifr, &vreq) != -1)
 		vlan_set(s, &ifr);
-	else
-		clone_setcallback(vlan_create);
 }
 
 static
@@ -149,8 +147,6 @@ DECL_CMD_FUNC(setvlandev, val, d)
 
 	if (getvlan(s, &ifr, &vreq) != -1)
 		vlan_set(s, &ifr);
-	else
-		clone_setcallback(vlan_create);
 }
 
 static
@@ -202,5 +198,6 @@ vlan_ctor(void)
 		cmd_register(&vlan_cmds[i]);
 	af_register(&af_vlan);
 	callback_register(vlan_cb, NULL);
+	clone_setdefcallback("vlan", vlan_create);
 #undef N
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200902270031.n1R0VY8E043693>