Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 31 Oct 2019 21:01:34 +0000 (UTC)
From:      Vincenzo Maffione <vmaffione@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r354229 - in head: share/man/man4 sys/dev/netmap sys/net tools/tools/netmap usr.sbin usr.sbin/valectl
Message-ID:  <201910312101.x9VL1YhC043270@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: vmaffione
Date: Thu Oct 31 21:01:34 2019
New Revision: 354229
URL: https://svnweb.freebsd.org/changeset/base/354229

Log:
  add valectl to the system commands
  
  The valectl(4) program is used to manage vale(4) switches.
  Add it to the system commands so that it can be used right away.
  This program was previously called vale-ctl, and stored in
  tools/tools/netmap
  
  Reviewed by:	hrs, bcr, lwhsu, kevans
  MFC after:	1 week
  Differential Revision:	https://reviews.freebsd.org/D22146

Added:
  head/usr.sbin/valectl/
  head/usr.sbin/valectl/Makefile   (contents, props changed)
  head/usr.sbin/valectl/valectl.8
     - copied, changed from r354228, head/tools/tools/netmap/vale-ctl.4
  head/usr.sbin/valectl/valectl.c
     - copied, changed from r354228, head/tools/tools/netmap/vale-ctl.c
Deleted:
  head/tools/tools/netmap/vale-ctl.4
  head/tools/tools/netmap/vale-ctl.c
Modified:
  head/share/man/man4/netmap.4
  head/sys/dev/netmap/netmap_bdg.c
  head/sys/net/netmap_legacy.h
  head/tools/tools/netmap/Makefile
  head/tools/tools/netmap/README
  head/tools/tools/netmap/lb.8
  head/usr.sbin/Makefile

Modified: head/share/man/man4/netmap.4
==============================================================================
--- head/share/man/man4/netmap.4	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/share/man/man4/netmap.4	Thu Oct 31 21:01:34 2019	(r354229)
@@ -27,7 +27,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd November 20, 2018
+.Dd October 26, 2019
 .Dt NETMAP 4
 .Os
 .Sh NAME
@@ -1087,14 +1087,14 @@ changing port names, e.g.,
 .Pp
 The following command attaches an interface and the host stack
 to a switch:
-.Dl vale-ctl -h vale2:em0
+.Dl valectl -h vale2:em0
 Other
 .Nm
 clients attached to the same switch can now communicate
 with the network card or the host.
 .Sh SEE ALSO
 .Xr vale 4 ,
-.Xr vale-ctl 4 ,
+.Xr valectl 8 ,
 .Xr bridge 8 ,
 .Xr lb 8 ,
 .Xr nmreplay 8 ,

Modified: head/sys/dev/netmap/netmap_bdg.c
==============================================================================
--- head/sys/dev/netmap/netmap_bdg.c	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/sys/dev/netmap/netmap_bdg.c	Thu Oct 31 21:01:34 2019	(r354229)
@@ -1442,7 +1442,7 @@ put_out:
 
 
 /* nm_bdg_ctl callback for the bwrap.
- * Called on bridge-attach and detach, as an effect of vale-ctl -[ahd].
+ * Called on bridge-attach and detach, as an effect of valectl -[ahd].
  * On attach, it needs to provide a fake netmap_priv_d structure and
  * perform a netmap_do_regif() on the bwrap. This will put both the
  * bwrap and the hwna in netmap mode, with the netmap rings shared

Modified: head/sys/net/netmap_legacy.h
==============================================================================
--- head/sys/net/netmap_legacy.h	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/sys/net/netmap_legacy.h	Thu Oct 31 21:01:34 2019	(r354229)
@@ -116,13 +116,13 @@
  * nr_cmd (in)	if non-zero indicates a special command:
  *	NETMAP_BDG_ATTACH	 and nr_name = vale*:ifname
  *		attaches the NIC to the switch; nr_ringid specifies
- *		which rings to use. Used by vale-ctl -a ...
+ *		which rings to use. Used by valectl -a ...
  *	    nr_arg1 = NETMAP_BDG_HOST also attaches the host port
- *		as in vale-ctl -h ...
+ *		as in valectl -h ...
  *
  *	NETMAP_BDG_DETACH	and nr_name = vale*:ifname
  *		disconnects a previously attached NIC.
- *		Used by vale-ctl -d ...
+ *		Used by valectl -d ...
  *
  *	NETMAP_BDG_LIST
  *		list the configuration of VALE switches.
@@ -133,10 +133,10 @@
  *
  *	NETMAP_BDG_NEWIF
  *		create a persistent VALE port with name nr_name.
- *		Used by vale-ctl -n ...
+ *		Used by valectl -n ...
  *
  *	NETMAP_BDG_DELIF
- *		delete a persistent VALE port. Used by vale-ctl -d ...
+ *		delete a persistent VALE port. Used by valectl -d ...
  *
  * nr_arg1, nr_arg2, nr_arg3  (in/out)		command specific
  *

Modified: head/tools/tools/netmap/Makefile
==============================================================================
--- head/tools/tools/netmap/Makefile	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/tools/tools/netmap/Makefile	Thu Oct 31 21:01:34 2019	(r354229)
@@ -3,7 +3,7 @@
 #
 # For multiple programs using a single source file each,
 # we can just define 'progs' and create custom targets.
-PROGS	=	pkt-gen nmreplay bridge vale-ctl lb
+PROGS	=	pkt-gen nmreplay bridge lb
 
 CLEANFILES = $(PROGS) *.o
 MAN=
@@ -31,9 +31,6 @@ bridge: bridge.o
 
 nmreplay: nmreplay.o
 	$(CC) $(CFLAGS) -o nmreplay nmreplay.o $(LDFLAGS)
-
-vale-ctl: vale-ctl.o
-	$(CC) $(CFLAGS) -o vale-ctl vale-ctl.o
 
 lb: lb.o pkt_hash.o
 	$(CC) $(CFLAGS) -o lb lb.o pkt_hash.o $(LDFLAGS)

Modified: head/tools/tools/netmap/README
==============================================================================
--- head/tools/tools/netmap/README	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/tools/tools/netmap/README	Thu Oct 31 21:01:34 2019	(r354229)
@@ -6,8 +6,6 @@ This directory contains applications that use the netm
 
 	bridge		a two-port jumper wire, also using the netmap API
 
-	vale-ctl	the program to control and inspect VALE switches
-
 	lb		an L3/L4 load balancer
 
 	nmreplay	a tool to playback a pcap file to a netmap port

Modified: head/tools/tools/netmap/lb.8
==============================================================================
--- head/tools/tools/netmap/lb.8	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/tools/tools/netmap/lb.8	Thu Oct 31 21:01:34 2019	(r354229)
@@ -24,7 +24,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd October 28, 2018
+.Dd October 26, 2019
 .Dt LB 8
 .Os
 .Sh NAME
@@ -111,7 +111,7 @@ If
 .Nm
 does not exit cleanly the ports will not be removed.
 Please use
-.Xr vale-ctl 4
+.Xr valectl 8
 to remove any stale persistent VALE port.
 .Sh SEE ALSO
 .Xr netmap 4 ,

Modified: head/usr.sbin/Makefile
==============================================================================
--- head/usr.sbin/Makefile	Thu Oct 31 20:42:36 2019	(r354228)
+++ head/usr.sbin/Makefile	Thu Oct 31 21:01:34 2019	(r354229)
@@ -96,6 +96,7 @@ SUBDIR=	adduser \
 	trpt \
 	tzsetup \
 	ugidfw \
+	valectl \
 	vigr \
 	vipw \
 	wake \

Added: head/usr.sbin/valectl/Makefile
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/usr.sbin/valectl/Makefile	Thu Oct 31 21:01:34 2019	(r354229)
@@ -0,0 +1,8 @@
+# $FreeBSD$
+
+PROG=	valectl
+MAN=	valectl.8
+
+WARNS?=	3
+
+.include <bsd.prog.mk>

Copied and modified: head/usr.sbin/valectl/valectl.8 (from r354228, head/tools/tools/netmap/vale-ctl.4)
==============================================================================
--- head/tools/tools/netmap/vale-ctl.4	Thu Oct 31 20:42:36 2019	(r354228, copy source)
+++ head/usr.sbin/valectl/valectl.8	Thu Oct 31 21:01:34 2019	(r354229)
@@ -24,15 +24,15 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd October 24, 2018
-.Dt VALE-CTL 4
+.Dd October 26, 2019
+.Dt VALECTL 8
 .Os
 .Sh NAME
-.Nm vale-ctl
+.Nm valectl
 .Nd manage VALE switches provided by netmap
 .Sh SYNOPSIS
 .Bk -words
-.Bl -tag -width "vale-ctl"
+.Bl -tag -width "valectl"
 .It Nm
 .Op Fl g Ar valeSSS:PPP
 .Op Fl a Ar valeSSS:interface
@@ -158,6 +158,6 @@ harware netmap ports.
 .Sh AUTHORS
 .An -nosplit
 .Nm
-has been written by
+was written by
 .An Michio Honda
 at NetApp.

Copied and modified: head/usr.sbin/valectl/valectl.c (from r354228, head/tools/tools/netmap/vale-ctl.c)
==============================================================================
--- head/tools/tools/netmap/vale-ctl.c	Thu Oct 31 20:42:36 2019	(r354228, copy source)
+++ head/usr.sbin/valectl/valectl.c	Thu Oct 31 21:01:34 2019	(r354229)
@@ -42,10 +42,8 @@
 #include <libgen.h>	/* basename */
 #include <stdlib.h>	/* atoi, free */
 
-/* XXX cut and paste from pkt-gen.c because I'm not sure whether this
- * program may include nm_util.h
- */
-void parse_nmr_config(const char* conf, struct nmreq *nmr)
+static void
+parse_nmr_config(const char* conf, struct nmreq *nmr)
 {
 	char *w, *tok;
 	int i, v;
@@ -201,7 +199,7 @@ usage(int errcode)
 {
 	fprintf(stderr,
 	    "Usage:\n"
-	    "vale-ctl arguments\n"
+	    "valectl arguments\n"
 	    "\t-g interface	interface name to get info\n"
 	    "\t-d interface	interface name to be detached\n"
 	    "\t-a interface	interface name to be attached\n"



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